Re: [Libreoffice] [PATCH] set RPATH correctly in the internal ICU

2011-01-06 Thread Cedric Bosdonnat
Hi Petr,

At first glance it looks OK to me, but I'm no build script expert... I
may have missed some important stuffs.

--
Cedric

On Wed, 2011-01-05 at 20:22 +0100, Petr Mladek wrote:
 Hi,
 
 our build service found broken RPATH in libicudata.so.42.1; it is fixed
 by the attached patch
 
 Cedric, could you please sign it off for libreoffice-3-3.
 
 It works well on Linux with bash. I think that it can't be worse on
 other systems ;-)
 
 
 Best Regards,
 Petr
 ___
 LibreOffice mailing list
 LibreOffice@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/libreoffice

-- 
Cédric Bosdonnat
LibreOffice hacker
http://documentfoundation.org
OOo Eclipse Integration developer
http://cedric.bosdonnat.free.fr

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] [PATCH] [PUSHED] Re: Annoying cursor behavior on sw tables

2011-01-06 Thread Cedric Bosdonnat
Hello Octavio,

I just checked again the problem I remembered and couldn't reproduce it
again... let's agree that it's OK. We could hack that area again if
someone finds it again later.

I just pushed your patch, many thanks for your help and continue with
those great contributions :)

Regards, 

On Mon, 2011-01-03 at 09:00 -0800, Octavio Alvarez wrote:
 On Mon, 03 Jan 2011 01:52:01 -0800, Cedric Bosdonnat  
 cbosdon...@novell.com wrote:
  The thing is that Window::Invalidate() gets called if I move or if I  
  type
  inside a table cell, which almost any key triggers, which is wrong.
  bUpdatedTable gets set to True inside SwCallLink::~SwCallLink() after
  some tests. Another option would be to correct those tests, but that
  would be far beyond my knowledge.
 
  The problem I tried to fix was that the whole row wasn't redrawn in some
  cases... I can't remember those for sure now but it was something like
  that:
 
  Make sure to have tables like this:
  +-+---+
  |+---+|   |
  ||   ||   |
  |+---+|   |
  | |   |
  +-+---+
 
  Place your cursor in the right cell and go to the left: the cursor
  should go in the blank paragraph after the nested cell and show it. In
  that kind of cases some part of the border of the right cell wasn't
  redrawn IIRC.
 
  Did you test with a case similar to the one above?
 
 Well, I did try several different combinations including different column
 sizes and nesting levels. The case you point out was also tested and
 I just retested it now. It works. I would say it performs better.
 
 In the patch included I also remove all uses and declaration of
 bUpdatedTable because it was only used on the removed if().
 
 -- 
 Octavio.
 
 Twitter: @alvarezp2000 -- Identi.ca: @alvarezp

-- 
Cédric Bosdonnat
LibreOffice hacker
http://documentfoundation.org
OOo Eclipse Integration developer
http://cedric.bosdonnat.free.fr

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] BrOffice RC2

2011-01-06 Thread Michael Meeks
Hi Andras,

On Wed, 2011-01-05 at 16:55 +0100, Andras Timar wrote:
 Have you also solved
 https://bugs.freedesktop.org/show_bug.cgi?id=32389 (MSI installer is
 always in English regardless of regional settings in Windows)?

Actually a dup of an existing blocker:

https://bugs.freedesktop.org/show_bug.cgi?id=32435

really, our last real / repeatable blocker. I've outlined an (easy)
suggested solution in there, which requires some more de-branded artwork
- which shouldn't be so hard. I hope to get it closed today.

HTH,

Michael.

-- 
 michael.me...@novell.com  , Pseudo Engineer, itinerant idiot


___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice] [Bug 31865] [Task]: LibreOffice 3.3 release blockers / stoppers

2011-01-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=31865

Petr Mladek pmla...@suse.cz changed:

   What|Removed |Added

 Depends on|31820   |

--- Comment #52 from Petr Mladek pmla...@suse.cz 2011-01-06 02:55:50 PST ---
Removing the bug #31820. The extensions registration fails only in some strange
circumstances. It does not affect the base function...

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] Localizations of UI and help - what languages does LibO support?

2011-01-06 Thread Andras Timar
2010/12/29 Andras Timar tima...@gmail.com

 Hi,

 I tested RC2 in regards of localization and I think it needs to be
 reviewed. 110 languages (including en-US) are selectable as UI language and
 help packs are provided for those. However,

 * Asturian (ast), Catalan (Valencian) (ca-XV), and Indonesian (id) are not
 included. Asturian and Catalan (Valencian) are especially important, because
 they are actively participating in LibreOffice localization.

 * Gaelic (Scots) (gd), Kirghiz (ky), Lao (lo), Malay (ms), Papiamento
 (pap), Pushto (ps), ??? (sc), Tigrinya (ti), and Urdu (ur) are included
 despite the fact that in the source there are no translations at all in
 these languages, so you basically package en-US strings. Gaelic (Scots) (gd)
 translation exists at
 ftp://ftp.linux.cz/pub/localization/OpenOffice.org/devel/build/Files/OOO330/GSI_gd.sdf.bz2-
  maybe it can be added.

 * It makes no sense to provide help pack for those languages which have 0%
 of the help translated. See this table for reference:
 http://wiki.documentfoundation.org/Translation_for_3_3#Language_pack_.2F_Help_pack_coverage
 This table was made from the RC2 sources.

 Best regards,
 Andras

 Nobody answered, maybe you missed my mail between Christmas and New Year.
Missing Asturian language pack is also reported in Bugzilla.
https://bugs.freedesktop.org/show_bug.cgi?id=32193

Best regards,
Andras
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] [PATCH] Removed dependencies on tools/solar.h in drawpage.hxx/.cxx

2011-01-06 Thread Michael Meeks

On Wed, 2011-01-05 at 19:56 +0100, Soeren Moeller wrote:
 It is not an attachment as I used git send-email 

Heh - the kernel guys prefer in-line patches it seems; they find them
easier to review and comment on that way; personally - I don't care, its
easy enough to save a file as an mbox and throw it at either patch or
git-am ;-) but - worth pleasing Kohei of course.

thanks for your nice work !

ATB,

Michael.

-- 
 michael.me...@novell.com  , Pseudo Engineer, itinerant idiot


___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] [PATCH] Remove DECLARE_LIST( EditList, AppWIn*)

2011-01-06 Thread David Tardon
On Wed, Jan 05, 2011 at 11:05:02AM +, Noel Power wrote:
 Hi Joseph
 On -10/01/37 20:59, Joseph Powers wrote:
 The attached patch compiles and stands up to my limited testing; however, 
 it's a large patch and touches a  lot of sensitive code so I want someone 
 with better knowledge of the Basic Macro Editor  environment to review  
 test it before I try pushing it.
 
 Thanks,
 Joe P.
 
 I wouldn't claim to have much knowledge of the Basic Macro editor
 but I suppose at least I have touched it in the past ;-)
 Firstly congratulations on this is great work, getting rid of those
 horrific macro lists and replacing with something more modern surely
 will make things easier for new developers to understand.  As far as
 I can see the changes as they stand shouldn't cause any problems and
 could be committed.
 My only comment would be that mostly  the opportunity to simplify
 the code using the power of the vector has not being taken advantage
 of which is a pity ( and would be great to fix here ) e.g.
 
 @@ -627,17 +627,15 @@ void BasicFrame::LoadIniFile()
  if ( pBasic )
  pBasic-LoadIniFile();
 
 -for ( i = 0 ; i  pList-Count() ; i++ )
 -pList-GetObject( i )-LoadIniFile();
 +for ( i = 0 ; i  pList-size() ; i++ )
 +pList-at( i )-LoadIniFile();
  }

Another nitpick: std::vector::at (as opposed to std::vector::operator[])
checks its argument and throws an exception if it is out of bounds. This
check is absolutely useless inside a loop (well, unless the end
condition is wrong) and only adds to run time.

D.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice] [Bug 31865] [Task]: LibreOffice 3.3 release blockers / stoppers

2011-01-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=31865

Andras Timar tima...@gmail.com changed:

   What|Removed |Added

 Depends on||32563, 32193

--- Comment #54 from Andras Timar tima...@gmail.com 2011-01-06 03:57:53 PST 
---
Added License Dialog
Added missing langpacks

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] [PATCH] set the icons size based on the DPI just like we do on Mac

2011-01-06 Thread Michael Meeks
Hi Phil,

On Wed, 2011-01-05 at 13:19 -0500, Phil Turmel wrote:
  :-) so - as I say; the DPI is a constant of 75 across the Linux
  desktop, so this is not a switch but a hard coded setting :-)
 
 I think you are over-reaching here:

:-) So - I'm talking the Xft.dpi setting which is used by GNOME and
LibreOffice; cf.

$ xrdb | grep Xft

 scorpius pturmel ~
 $ xdpyinfo | egrep (dimensions|dots per inch)
   dimensions:3600x1080 pixels (905x272 millimeters)
   resolution:101x101 dots per inch

Sure; and these numbers have (traditionally) been notorious for their
unreliability - but perhaps that has improved in recent times.

Thanks though :-)

Michael.

-- 
 michael.me...@novell.com  , Pseudo Engineer, itinerant idiot


___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] [PATCH] set RPATH correctly in the internal ICU

2011-01-06 Thread Caolán McNamara
On Thu, 2011-01-06 at 09:47 +, Michael Meeks wrote:
 On Wed, 2011-01-05 at 20:13 +, Caolán McNamara wrote:
  On Wed, 2011-01-05 at 20:22 +0100, Petr Mladek wrote:
   our build service found broken RPATH in libicudata.so.42.1
  
  That libicudata is the biggest nuisance, causes all sorts of problems on
  different platforms. Rather an odd use of an elf file as data storage
  IIRC and manually created by icu rather than by the normal compiler
  +linker chain.
 
   It is also huge; the biggest single files we have in our install (on
 Windows at least):

I *think* icu can be configured to use a different format for the
icudata, I've never tried this, but fiddling configure options for it
might cause some different format to be generated, which at least for
testing/dumping purposes might be easier to work with.

C.

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice] icons - we should use the native theme's icon size by default.

2011-01-06 Thread Michael Meeks
Hi guys,

On Wed, 2011-01-05 at 18:59 +0100, Christian Lohmaier wrote:
 On GNOME it follows the theme, wich in turn follows the gconf-setting
 /desktop/gnome/interface/toolbar_icons_size

Right - which should be propagated via x-settings and magic to avoid
the need for a gconf dependency. Thanks Christian for digging that out.

 so whatever your theme defines in the gtk-icon-sizes property will
 be used.

Right; we should hook out and use this in salnativewidgets-gtk.c if
that is not so, lets add an easy hack for it, or perhaps someone can
jump in with some code of this form to decisively end the discussion :-)

NB. this will still be only for the Automatic / default setting - the
user can override it easily.

Thanks,

Michael.

-- 
 michael.me...@novell.com  , Pseudo Engineer, itinerant idiot


___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] [PATCH] set the icons size based on the DPI just like we do on Mac

2011-01-06 Thread Lubos Lunak
On Wednesday 05 of January 2011, Robert Nagy wrote:
 Oh well then Linux is broken, again. :)

 In gedit the icons are bigger than the small ones but not as huge
 as in LibreOffice.
 Here you can see (LO with small icons):
 http://blade2k.humppa.hu/2011-01-05-182917_1920x1080_scrot.png

 (ws robert 22435)$ xdpyinfo | egrep (dimensions|dots per inch)
   dimensions:1920x1080 pixels (518x291 millimeters)
   resolution:94x94 dots per inch

 It is strange for me tho that linux *hardcodes* 75. Where is taht
 hardcoded?

 I think that is the default value for X.Org if it can't find out the real 
value from the hardware or configuration options. Normally X should find out 
the real DPI, although desktops also have options for various misguided 
attempts at forcing a specific DPI.

-- 
 Lubos Lunak
 l.lu...@suse.cz
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] Localizations of UI and help - what languages does LibO support?

2011-01-06 Thread Michael Meeks
Hi Andras,

On Thu, 2011-01-06 at 12:14 +0100, Andras Timar wrote:
 Nobody answered, maybe you missed my mail between Christmas and New

I read it, and added it to my mail of things for Fridrich to sort out
when he gets back on Monday ;-)

 Missing Asturian language pack is also reported in Bugzilla.
 https://bugs.freedesktop.org/show_bug.cgi?id=32193

Great, the way to get attention best is to add it to the blocker bug,
which I believe you've now done ? :-)

Thanks !

Michael.

-- 
 michael.me...@novell.com  , Pseudo Engineer, itinerant idiot


___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] [PATCH] use libreoffice and lo* wrappers

2011-01-06 Thread Rene Engelhard
On Wed, Jan 05, 2011 at 07:59:06PM +0100, Petr Mladek wrote:
 Rene, could you please review it for libreoffice-3-3 branch?

Looks OK, but why did you leave oo* in? package-ooo still generates it.

Regards,

Rene

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice] [PUSHED] Re: [PATCH] use libreoffice and lo* wrappers

2011-01-06 Thread Rene Engelhard
On Thu, Jan 06, 2011 at 02:13:44PM +0100, Rene Engelhard wrote:
 On Wed, Jan 05, 2011 at 07:59:06PM +0100, Petr Mladek wrote:
  Rene, could you please review it for libreoffice-3-3 branch?
 
 Looks OK, but why did you leave oo* in? package-ooo still generates it.

To answer myself after discussion on IRC, compat with old habits
calling oowriter etc.

Pushed.

Grüße/Regards,

René
-- 
 .''`.  René Engelhard -- Debian GNU/Linux Developer
 : :' : http://www.debian.org | http://people.debian.org/~rene/
 `. `'  r...@debian.org | GnuPG-Key ID: D03E3E70
   `-   Fingerprint: E12D EA46 7506 70CF A960 801D 0AA0 4571 D03E 3E70
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] Oracle wordbook in LibO

2011-01-06 Thread Kálmán „KAMI” Szalai


On 01/05/2011 06:28 PM, Petr Mladek wrote:
 Kálmán „KAMI” Szalai píše v St 05. 01. 2011 v 17:39 +0100:
 Hi!

 2011-01-05 17:01 keltezéssel, Petr Mladek írta: 
 Ah, we use Novell.dic in our build. It includes just
 language-independent names of Novell products, so it should not cause
 any harm to anyone. I attached different patches to the issue. KAMI,
 could you please review them?
 Can we introduce here some other technical, and distibution related
 files here, not just Novell related, words from Ubuntu, Debian, etc. I
 think we should have only one extra dic file, or is is a performance
 killer thing?
 Great idea! I would call it technical.dic or so.

 KAMI, could you please cook up something? 


 Best Regards,
 Petr

May we include vendor specific sfuffs from oracle.dic?
http://cgit.freedesktop.org/libreoffice/extras/commit/?h=libreoffice-3-3id=7fbdfd0a0d2af30af92839ebf8bcf97c465519e8

KAMI



signature.asc
Description: OpenPGP digital signature
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] [PATCH] Fix passing distro specific branding via configure

2011-01-06 Thread Kálmán „KAMI” Szalai
Thank you for the patch :o)

KAMI

On 01/05/2011 07:46 PM, Petr Mladek wrote:
 Hi,

 the configure options --with-intro-bitmaps, --with-about-bitmaps did not
 longer worked. I think that nobody needs more pictures. The attached
 patches rename the above options to pass only one picture and fix the
 stuff to work.

 KAMI, could you sign it off for libreoffice-3-3 branch?


 Best Regards,
 Petr




signature.asc
Description: OpenPGP digital signature
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] Oracle wordbook in LibO

2011-01-06 Thread Petr Mladek
Kálmán „KAMI” Szalai píše v Čt 06. 01. 2011 v 15:40 +0100:
 
 On 01/05/2011 06:28 PM, Petr Mladek wrote:
  Kálmán „KAMI” Szalai píše v St 05. 01. 2011 v 17:39 +0100:
  Hi!
 
  2011-01-05 17:01 keltezéssel, Petr Mladek írta: 
  Ah, we use Novell.dic in our build. It includes just
  language-independent names of Novell products, so it should not cause
  any harm to anyone. I attached different patches to the issue. KAMI,
  could you please review them?
  Can we introduce here some other technical, and distibution related
  files here, not just Novell related, words from Ubuntu, Debian, etc. I
  think we should have only one extra dic file, or is is a performance
  killer thing?
  Great idea! I would call it technical.dic or so.
 
  KAMI, could you please cook up something? 
 
 
  Best Regards,
  Petr
 
 May we include vendor specific sfuffs from oracle.dic?
 http://cgit.freedesktop.org/libreoffice/extras/commit/?h=libreoffice-3-3id=7fbdfd0a0d2af30af92839ebf8bcf97c465519e8

Looks good to me.

Best Regards,
Petr



___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice] Got an error while compiling latest libreoffice code

2011-01-06 Thread Grega Fajdiga
Entering /home/grega/osebno/source/libo/sc/source/core/data

Compiling: sc/source/core/data/conditio.cxx
/home/grega/osebno/source/libo/clone/calc/sc/source/core/data/conditio.cxx:
In function 'BOOL lcl_IsDuplicate(ScDocument*, double, const
String, const ScAddress, const ScRangeListRef)':
/home/grega/osebno/source/libo/clone/calc/sc/source/core/data/conditio.cxx:748:
error: 'class ScRangeList' has no member named 'Count'
/home/grega/osebno/source/libo/clone/calc/sc/source/core/data/conditio.cxx:751:
error: 'class ScRangeList' has no member named 'GetObject'
../../../inc/address.hxx: At global scope:
../../../inc/address.hxx:80: warning: 'SCCOLROW_MAX' defined but
not used
../../../inc/address.hxx:81: warning: 'SCSIZE_MAX' defined but
not used
../../../inc/address.hxx:105: warning: 'SC_TAB_APPEND' defined
but not used
../../../inc/address.hxx:106: warning: 'TABLEID_DOC' defined but
not used
../../../inc/address.hxx:108: warning: 'SCCOL_REPEAT_NONE'
defined but not used
../../../inc/address.hxx:109: warning: 'SCROW_REPEAT_NONE'
defined but not used
dmake:  Error code 1, while making
'../../../unxlngx6.pro/slo/conditio.obj'
Forcing regeneration of dependency info

Retrying /home/grega/osebno/source/libo/sc/source/core/data

Making:all_data.dpslo
Compiling: sc/source/core/data/conditio.cxx
/home/grega/osebno/source/libo/clone/calc/sc/source/core/data/conditio.cxx:
In function 'BOOL lcl_IsDuplicate(ScDocument*, double, const
String, const ScAddress, const ScRangeListRef)':
/home/grega/osebno/source/libo/clone/calc/sc/source/core/data/conditio.cxx:748:
error: 'class ScRangeList' has no member named 'Count'
/home/grega/osebno/source/libo/clone/calc/sc/source/core/data/conditio.cxx:751:
error: 'class ScRangeList' has no member named 'GetObject'
../../../inc/address.hxx: At global scope:
../../../inc/address.hxx:80: warning: 'SCCOLROW_MAX' defined but
not used
../../../inc/address.hxx:81: warning: 'SCSIZE_MAX' defined but
not used
../../../inc/address.hxx:105: warning: 'SC_TAB_APPEND' defined
but not used
../../../inc/address.hxx:106: warning: 'TABLEID_DOC' defined but
not used
../../../inc/address.hxx:108: warning: 'SCCOL_REPEAT_NONE'
defined but not used
../../../inc/address.hxx:109: warning: 'SCROW_REPEAT_NONE'
defined but not used
dmake:  Error code 1, while making
'../../../unxlngx6.pro/slo/conditio.obj'

-- 
Grega Fajdiga
Hacquetova 5
1113 Ljubljana
Slovenia
GSM: +386 40 923 635
E-pošta: gregor.fajd...@guest.arnes.si
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] Got an error while compiling latest libreoffice code

2011-01-06 Thread Caolán McNamara
On Thu, 2011-01-06 at 17:04 +0100, Grega Fajdiga wrote:
 Entering /home/grega/osebno/source/libo/sc/source/core/data
 
 Compiling: sc/source/core/data/conditio.cxx
 /home/grega/osebno/source/libo/clone/calc/sc/source/core/data/conditio.cxx:
 In function 'BOOL lcl_IsDuplicate(ScDocument*, double, const
 String, const ScAddress, const ScRangeListRef)':
 /home/grega/osebno/source/libo/clone/calc/sc/source/core/data/conditio.cxx:748:
 error: 'class ScRangeList' has no member named 'Count'

Yes, this was broken earlier today for a few hours. Should be fixed
again now if you ./g pull again.

C.

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] [PATCH] Removed dependencies on tools/solar.h in drawpage.hxx/.cxx

2011-01-06 Thread Kohei Yoshida
On Thu, 2011-01-06 at 11:27 +, Michael Meeks wrote:
 On Wed, 2011-01-05 at 19:56 +0100, Soeren Moeller wrote:
  It is not an attachment as I used git send-email 
 
   Heh - the kernel guys prefer in-line patches it seems; they find them
 easier to review and comment on that way; personally - I don't care, its
 easy enough to save a file as an mbox and throw it at either patch or
 git-am ;-)

Ah ok.  Fair enough. ;-)

  but - worth pleasing Kohei of course.

Nah, I didn't mean to set a rule that all patches must be attachments.
Rather, I was just curious because his last patch was an attachment and
this one wasn't.

It's fine by me as long as I can save the email itself and 'git am'
works fine with that.  Let's be flexible.  I'm sure it's easier to just
run 'git send-mail' than sending a patch as an attachment which is an
extra step. :-)

(though that means I need to avoid using the attachment symbols to look
for incoming patches, but I can adopt.)

Kohei

-- 
Kohei Yoshida, LibreOffice hacker, Calc
kyosh...@novell.com

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice] sign off for cherry-pick fix for crash on exit after using search in calc

2011-01-06 Thread Caolán McNamara
i.e. see bug: https://bugzilla.redhat.com/show_bug.cgi?id=666088 for
details. 

Attached is the proposed fix. Don't let the fancy-pants rtl::Static put
you off, its just making the maCache from a global to a static local.

Globals/static locals have their dtors called in reverse order of the
completion of their ctors. Leaving the complex global at its current
scope means it gets shutdown too late on final dlclose of libunotools,
at a point after which all the runtime structure that it needs has been
shutdown. Changing it to a local static ensures it gets dtored before
that.

C.
From b79c4f7daf165e4190b12a19382f4e5615b43163 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Caol=C3=A1n=20McNamara?= caol...@redhat.com
Date: Thu, 6 Jan 2011 17:05:14 +
Subject: [PATCH] Resolves: rhbz#666088 clean up search cache singleton in correct order

---
 unotools/inc/unotools/textsearch.hxx |9 
 unotools/source/i18n/textsearch.cxx  |   37 +
 2 files changed, 23 insertions(+), 23 deletions(-)

diff --git a/unotools/inc/unotools/textsearch.hxx b/unotools/inc/unotools/textsearch.hxx
index 0aaa123..16ae82f 100644
--- a/unotools/inc/unotools/textsearch.hxx
+++ b/unotools/inc/unotools/textsearch.hxx
@@ -128,15 +128,6 @@ public:
 
 class UNOTOOLS_DLLPUBLIC TextSearch
 {
-struct CachedTextSearch
-{
-::osl::Mutex mutex;
-::com::sun::star::util::SearchOptions Options;
-::com::sun::star::uno::Reference ::com::sun::star::util::XTextSearch  xTextSearch;
-};
-
-static CachedTextSearch maCache;
-
 static ::com::sun::star::uno::Reference ::com::sun::star::util::XTextSearch 
 getXTextSearch( const ::com::sun::star::util::SearchOptions rPara );
 
diff --git a/unotools/source/i18n/textsearch.cxx b/unotools/source/i18n/textsearch.cxx
index 8f8f780..8085045 100644
--- a/unotools/source/i18n/textsearch.cxx
+++ b/unotools/source/i18n/textsearch.cxx
@@ -36,6 +36,7 @@
 #include unotools/charclass.hxx
 #include comphelper/processfactory.hxx
 #include unotools/textsearch.hxx
+#include rtl/instance.hxx
 
 using namespace ::com::sun::star::util;
 using namespace ::com::sun::star::uno;
@@ -86,13 +87,6 @@ SearchParam::SearchParam( const SearchParam rParam )
 nTransliterationFlags = rParam.nTransliterationFlags;
 }
 
-//  Klasse zum Suchen eines Strings in einem Text. Es wird genau nach
-//  dem String gesucht.
-//  ( Die Unterscheidung der Gross/Klein-Schreibung kann mit einen Flag
-//  unterdrueckt werden )
-
-TextSearch::CachedTextSearch TextSearch::maCache;
-
 static bool lcl_Equals( const SearchOptions rSO1, const SearchOptions rSO2 )
 {
 return rSO1.algorithmType == rSO2.algorithmType 
@@ -108,27 +102,42 @@ static bool lcl_Equals( const SearchOptions rSO1, const SearchOptions rSO2 )
 rSO1.transliterateFlags == rSO2.transliterateFlags;
 }
 
+namespace
+{
+struct CachedTextSearch
+{
+::osl::Mutex mutex;
+::com::sun::star::util::SearchOptions Options;
+::com::sun::star::uno::Reference ::com::sun::star::util::XTextSearch  xTextSearch;
+};
+
+struct theCachedTextSearch
+: public rtl::Static CachedTextSearch, theCachedTextSearch  {};
+}
+
 ReferenceXTextSearch TextSearch::getXTextSearch( const SearchOptions rPara )
 {
-osl::MutexGuard aGuard(maCache.mutex);
+CachedTextSearch rCache = theCachedTextSearch::get();
+
+osl::MutexGuard aGuard(rCache.mutex);
 
-if ( lcl_Equals(maCache.Options, rPara) )
-return maCache.xTextSearch;
+if ( lcl_Equals(rCache.Options, rPara) )
+return rCache.xTextSearch;
 
 try
 {
 Reference XMultiServiceFactory  xMSF = ::comphelper::getProcessServiceFactory();
-maCache.xTextSearch.set( xMSF-createInstance(
+rCache.xTextSearch.set( xMSF-createInstance(
 ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM(
 com.sun.star.util.TextSearch ) ) ), UNO_QUERY_THROW );
-maCache.xTextSearch-setOptions( rPara );
-maCache.Options = rPara;
+rCache.xTextSearch-setOptions( rPara );
+rCache.Options = rPara;
 }
 catch ( Exception )
 {
 DBG_ERRORFILE( TextSearch ctor: Exception caught! );
 }
-return maCache.xTextSearch;
+return rCache.xTextSearch;
 }
 
 TextSearch::TextSearch(const SearchParam  rParam, LanguageType eLang )
-- 
1.7.3.3

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice] Strange sorting of top level menu

2011-01-06 Thread leif
When I see the Startcenter I see the Insert to the most right position
and not as expected between View and Tools. This occurs only when the
start center is visible. When I start or open a document the position is
correct.

Screendump:
http://picasaweb.google.com/lh/photo/LroFAuzV1Wmj04_1mY8cIQ?feat=directlink

Using Ubuntu Linux. I just changed from manual installation to
http://ppa.launchpad.net/libreoffice/ppa/ubuntu . I'm not sure that the
problem was before.

Tomorrow I will check with my computer at work if it occurs consequent.


Cheers,
Leif Lodahl
Danish Team

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] Strange sorting of top level menu

2011-01-06 Thread Andy Brown

On Thu Jan 06 2011 10:33:05 GMT-0800 (PST)  leif wrote:

When I see the Startcenter I see the Insert to the most right position
and not as expected between View and Tools. This occurs only when the
start center is visible. When I start or open a document the position is
correct.

Screendump:
http://picasaweb.google.com/lh/photo/LroFAuzV1Wmj04_1mY8cIQ?feat=directlink

Using Ubuntu Linux. I just changed from manual installation to
http://ppa.launchpad.net/libreoffice/ppa/ubuntu . I'm not sure that the
problem was before.

Tomorrow I will check with my computer at work if it occurs consequent.


Cheers,
Leif Lodahl
Danish Team



I has to be something that is being added by Ubuntu.  Here using the 
LibO download there is no Insert on the Startcenter, see attached.



inline: sc-ss.png___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice] [PATCH] Again cppcheck cleanliness

2011-01-06 Thread Guillaume Poussel
Hi everybody,

A new patch which correct cppcheck warnings in base/.

To be continued with other modules.

Regards,
Guillaume


0002-Clean-resourceleak-warning-in-cppcheck.patch
Description: Binary data


0001-cppcheck-cleanliness.patch
Description: Binary data
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] [PATCH] Updates for Lightproof

2011-01-06 Thread Petr Mladek
Kálmán „KAMI” Szalai píše v St 05. 01. 2011 v 20:52 +0100:
 Hi Michael,
 
 Sorry for bugging you but it seems I sent it two times, and David
 already pushed in:
 Re: [Libreoffice] [PATCH] [PUSHED] Updates for LightProof extension
 handling
 
 It is required If someone packs LightProof because of changed
 extension installation in 3.3.

Does it mean that LightProof is broken in libreoffice-3-3 branch?
If it is true, I would commit it even to the branch. It looks safe
enough.


Best Regards,
Petr

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] sign off for cherry-pick fix for crash on exit after using search in calc

2011-01-06 Thread Kohei Yoshida
On Thu, 2011-01-06 at 17:23 +, Caolán McNamara wrote:
 i.e. see bug: https://bugzilla.redhat.com/show_bug.cgi?id=666088 for
 details. 
 
 Attached is the proposed fix. Don't let the fancy-pants rtl::Static put
 you off, its just making the maCache from a global to a static local.
 
 Globals/static locals have their dtors called in reverse order of the
 completion of their ctors. Leaving the complex global at its current
 scope means it gets shutdown too late on final dlclose of libunotools,
 at a point after which all the runtime structure that it needs has been
 shutdown. Changing it to a local static ensures it gets dtored before
 that.

I can sign off on this.  I believe this code was originally my patch and
Eike changed it a bit.  I remember this one.

Nothing weird about this change, so looks good to me. :-)

Kohei

-- 
Kohei Yoshida, LibreOffice hacker, Calc
kyosh...@novell.com

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice] [PATCH] Fix configure check for BerkleyDB where db_create is macro

2011-01-06 Thread Petr Mladek
Hi,

this fixed the problem described in the attached mail.

Gökçen, Robert, could you please test it and sign it out for
libreoffice-3-3 branch?

It works here and it should be more safe than the current solution.


Analyze:


db_create function is defined by macro in some db version. I did a lot
of googling and have not found any ultimate solution.

Some people try to check all db_create variants, e.g. db_create_4002, 
db_create_4001, see
http://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/mail-filter/maildrop/files/maildrop-2.5.1-db.patch?view=diffr1=texttr1=1.1r2=texttr2=1.1diff_format=s

The problem is that there exists too many variants. Gökçen would need
db_create_4008, ...

I have found only one other variant for dbopen that is __db185_open.
This symbol has been introduced in db-3.x and seems to be provided by
any newer version for compatibility reasons. See also
http://lists.debian.org/debian-devel/2001/04/msg01770.html


Solution:
-

LibO requires db = 4.4. All these version should provide the symbol
__db185_open for compatibility reasons. Of course, it is better to
check for the original symbol dbopen, so I used __db185_open just as
fallback.

I am not 100% sure that it would work on all archaic systems but it
should be better than the current state.


Best Regards,
Petr
---BeginMessage---
Çarşamba 01 Aralık 2010 günü (saat 20:35:22) Petr Mladek şunları yazmıştı:
 Hi Robert,
 
 Robert Nagy píše v St 01. 12. 2010 v 11:27 +0100:
  Hi,
  
  So this is a modified versio nof patches/dev300/system-db-check.diff
  to find the proper berkeley db headers and libs.
  I really would like to get this into bootstrap so please test and
  comment.
 
 It looks reasonable and works here. Feel free to commit it.


With this patch (in RC2) I can no longer pass configure:

checking which db to use... external
checking for db-5.1/db.h... no
checking for db5.1/db.h... no
checking for db-5.0/db.h... no
checking for db5.0/db.h... no
checking for db-5/db.h... no
checking for db5/db.h... no
checking for db-4.8/db.h... no
checking for db4.8/db.h... no
checking for db-4.7/db.h... no
checking for db4.7/db.h... no
checking for db-4/db.h... no
checking for db4/db.h... no
checking for db/db.h... no
checking db.h usability... yes
checking db.h presence... yes
checking for db.h... yes
checking whether db is at least 4.1... yes
checking for db_create in -ldb... no
checking for db_create in -ldb-5.1... no
checking for db_create in -ldb5.1... no
checking for db_create in -ldb-5.0... no
checking for db_create in -ldb5.0... no
checking for db_create in -ldb-5... no
checking for db_create in -ldb5... no
checking for db_create in -ldb-4.8... no
checking for db_create in -ldb4.8... no
checking for db_create in -ldb-4.7... no
checking for db_create in -ldb4.7... no
checking for db_create in -ldb-4... no
checking for db_create in -ldb4... no
checking for db_create... no
configure: error: db not installed or functional

I'm using db4 version 4.8.30, and db4-devel package is installed. I can 
compile RC1 without any errors. 

When I try to compile this test file that configure generates:

/* confdefs.h */


#define PACKAGE_NAME  


#define PACKAGE_TARNAME   


#define PACKAGE_VERSION   


#define PACKAGE_STRING


#define PACKAGE_BUGREPORT 


#define PACKAGE_URL   


#define STDC_HEADERS 1  

 

Re: [Libreoffice] [PATCH] Updates for Lightproof

2011-01-06 Thread Kálmán „KAMI” Szalai
Hi Michael, Petr,

Please review the new rules of extension integration for 3.3. I did the
necessary changes for extensions except Lightproof and SUN templates
pack. I did it after RC stage started. I hope this was the last patch
for it.

David has already comited it:
http://cgit.freedesktop.org/libreoffice/bootstrap/commit/?h=libreoffice-3-3id=a1fc860c638a072924fd7733b4c09769acfa8ca3


All the best,
KAMI

2011-01-06 20:39 keltezéssel, Petr Mladek írta:
 Kálmán „KAMI” Szalai píše v St 05. 01. 2011 v 20:52 +0100:
 Hi Michael,

 Sorry for bugging you but it seems I sent it two times, and David
 already pushed in:
 Re: [Libreoffice] [PATCH] [PUSHED] Updates for LightProof extension
 handling

 It is required If someone packs LightProof because of changed
 extension installation in 3.3.
 Does it mean that LightProof is broken in libreoffice-3-3 branch?
 If it is true, I would commit it even to the branch. It looks safe
 enough.


 Best Regards,
 Petr





signature.asc
Description: OpenPGP digital signature
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] [PATCH] Fix configure check for BerkleyDB where db_create is macro

2011-01-06 Thread Hanno Meyer-Thurow
Hi,
I just checked for my db version. I, too, use

[IP-] [  ] sys-libs/db-4.8.30:4.8

which works just fine. So I wonder what differs.


Regards,
Hanno
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] segfault in libqa_unit.so test

2011-01-06 Thread Thomas Klausner
On Tue, Jan 04, 2011 at 03:01:53PM +, Caolán McNamara wrote:
 One horrific temporary hack to test a theory would be to call _exit to
 skip calling the massive dtor chain which OOo builds up. Though that's
 not a cure, just a temp hack, but maybe it'll help isolate this.

I'm confused why, but with this patch applied, I get failure much
earlier.

Module 'odk' delivered successfully. 3 files copied, 1 files unchanged

---
Oh dear - something failed during the build - sorry !
  For more help with debugging build errors, please see the section in:
http://wiki.documentfoundation.org/Development

  internal build errors:

ERROR: error 65280 occurred while making /disk/2/archive/libreoffice/tools/qa

 it seems that the error is inside 'tools', please re-run build
 inside this module to isolate the error and/or test your fix:
---

rm -Rf /disk/2/archive/libreoffice/tools/unxbsdx3.pro # optional module 'clean'
/bin/bash
cd /disk/2/archive/libreoffice
source ./NetBSDX86-64Env.Set.sh
cd tools
build

when the problem is isolated and fixed exit and re-run 'make' from the top-level


when I do that:


Making:test_tools.so
../unxbsdx3.pro/slo/test_reversemap.o: In function `(anonymous 
namespace)::Test::testEncoding(unsigned short)':
test_reversemap.cxx:(.text+0x3f0): undefined reference to 
`getBestMSEncodingByChar(unsigned short)'
dmake:  Error code 1, while making '../unxbsdx3.pro/lib/test_tools.so'
Forcing regeneration of dependency info

--
- start unit test #1 on library
--
:  
LD_LIBRARY_PATH=${LD_LIBRARY_PATH+${LD_LIBRARY_PATH}:}/disk/2/archive/libreoffice/clone/libs-gui/tools/unxbsdx3.pro/lib:/disk/2/archive/libreoffice/solver/330/unxbsdx3.pro/lib
  /disk/2/archive/libreoffice/solver/330/unxbsdx3.pro/bin/cppunittester 
Usage: cppunittester shared-library-path
dmake:  Error code 1, while making 'test1'
Retrying /disk/2/archive/libreoffice/tools/qa

Making:all_qa.dpslo
Making:test_tools.so
../unxbsdx3.pro/slo/test_reversemap.o: In function `(anonymous 
namespace)::Test::testEncoding(unsigned short)':
test_reversemap.cxx:(.text+0x3f0): undefined reference to 
`getBestMSEncodingByChar(unsigned short)'
dmake:  Error code 1, while making '../unxbsdx3.pro/lib/test_tools.so'

---
Oh dear - something failed during the build - sorry !
  For more help with debugging build errors, please see the section in:
http://wiki.documentfoundation.org/Development

 it seems that the error is inside 'tools', please re-run build
 inside this module to isolate the error and/or test your fix:
---

rm -Rf /disk/2/archive/libreoffice/tools/unxbsdx3.pro # optional module 'clean'
/bin/bash
cd /disk/2/archive/libreoffice
source ./NetBSDX86-64Env.Set.sh
cd tools
build


How can the diff cause that?
 Thomas
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] segfault in libqa_unit.so test

2011-01-06 Thread Caolán McNamara
On Thu, 2011-01-06 at 22:44 +0100, Thomas Klausner wrote:
 On Tue, Jan 04, 2011 at 03:01:53PM +, Caolán McNamara wrote:
  One horrific temporary hack to test a theory would be to call _exit to
  skip calling the massive dtor chain which OOo builds up. Though that's
  not a cure, just a temp hack, but maybe it'll help isolate this.
 
 I'm confused why, but with this patch applied, I get failure much
 earlier.

Unrelated I feel.
 
 Making:test_tools.so
 ../unxbsdx3.pro/slo/test_reversemap.o: In function `(anonymous 
 namespace)::Test::testEncoding(unsigned short)':
 test_reversemap.cxx:(.text+0x3f0): undefined reference to 
 `getBestMSEncodingByChar(unsigned short)'
 dmake:  Error code 1, while making '../unxbsdx3.pro/lib/test_tools.so'
 Forcing regeneration of dependency info

export VERBOSE=true here to get the command lines used.

fiddle with the equivalent of 

nm -D ../unxbsdx3.pro/lib/libtl*so | c++filt|grep getBest 
to see if the symbol in in libtl*.so like it should be to help find out
if we're linking against the right libtl locally e.g. as opposed to the
one inside the solver.

If its not then see if ../unxbsdx3.pro/bin/bestreversemap exists, and
see if ../unxbsdx3.pro/inc/reversemap.hxx exists with
getBestMSEncodingByChar inside it. That reversemap.hxx should be
generated by bestreversemap and included by source/string/reversemap.cxx
and built into the libtl.so

My *guess* is that for some reason ../unxbsdx3.pro/inc/reversemap.hxx
exists but is empty. Is that the case ?

C.

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] A file to show 8192 discontinuities crashes Calc

2011-01-06 Thread r_ouellette

Like I said earlier, I submitted also this bug to OpenOffice.org and it was
marked as RESOLVED yesterday. See 
http://www.openoffice.org/issues/show_bug.cgi?id=116164
http://www.openoffice.org/issues/show_bug.cgi?id=116164 

I don't know how are the contacts (if any) between LO developers and OOo
developers, but the bug was solved. Maybe someone there will give you a hint
;-)

Raymond

-- 
View this message in context: 
http://nabble.documentfoundation.org/A-file-to-show-8192-discontinuities-crashes-Calc-tp2135015p2208563.html
Sent from the Dev mailing list archive at Nabble.com.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] segfault in libqa_unit.so test

2011-01-06 Thread Caolán McNamara
On Thu, 2011-01-06 at 22:07 +, Caolán McNamara wrote:
 On Thu, 2011-01-06 at 22:44 +0100, Thomas Klausner wrote:
  On Tue, Jan 04, 2011 at 03:01:53PM +, Caolán McNamara wrote:
   One horrific temporary hack to test a theory would be to call _exit to
   skip calling the massive dtor chain which OOo builds up. Though that's
   not a cure, just a temp hack, but maybe it'll help isolate this.
  
  I'm confused why, but with this patch applied, I get failure much
  earlier.
 
 Unrelated I feel.

Bah, let me rethink that. I guess its very possible that _exit doesn't
flush standard I/O on NetBSD which might indeed cause an empty
generated .hxx. I've just pushed a fflush into that generator to help
test the theory. Try a pull and build again of tools.

reversemap.obj depends on reversemap.hxx which depends on
bestreversemap which I've added the extra fflush to so the rebuild
should re-generate it without any manual intervention. 

C.

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice] [PATCH] 1st try of translating German comments into English

2011-01-06 Thread Christoph Herzog

Tried hard and hope it's acceptable. Please check.
Criticism welcome.

From ab9c6699645c17d1743e94a8af77685415db688b Mon Sep 17 00:00:00 2001
From: Christoph Herzog rho...@googlemail.com
Date: Fri, 7 Jan 2011 00:22:24 +0100
Subject: [PATCH] First try of translating German comments into English.

---
 sw/inc/IDocumentContentOperations.hxx |   73 
+---

 1 files changed, 30 insertions(+), 43 deletions(-)

diff --git a/sw/inc/IDocumentContentOperations.hxx 
b/sw/inc/IDocumentContentOperations.hxx

index 040b09c..4a5c0ee 100644
--- a/sw/inc/IDocumentContentOperations.hxx
+++ b/sw/inc/IDocumentContentOperations.hxx
@@ -74,20 +74,20 @@
 };

  public:
-/** Kopieren eines Bereiches im oder in ein anderes Dokument !
-Die Position kann auch im Bereich liegen !!
-*/
+/** Copying of a range within or to another document.
+The position can also be within the range!
+ */
 virtual bool CopyRange(SwPaM, SwPosition, const bool bCopyAll ) 
const = 0;


-/** Loesche die Section, in der der Node steht.
+/** Delete section containing the node.
 */
 virtual void DeleteSection(SwNode* pNode) = 0;

-/** loeschen eines BereichesSwFlyFrmFmt
+/** Delete a range SwFlyFrmFmt.
 */
 virtual bool DeleteRange(SwPaM) = 0;

-/** loeschen gesamter Absaetze
+/** Delete full paragraphs.
 */
 virtual bool DelFullPara(SwPaM) = 0;

@@ -100,19 +100,15 @@
 virtual bool DeleteAndJoin( SwPaM,
 const bool bForceJoinNext = false ) = 0;

-/** verschieben eines Bereiches
-*/
 virtual bool MoveRange(SwPaM, SwPosition, SwMoveFlags) = 0;

-/** verschieben ganzer Nodes
-*/
 virtual bool MoveNodeRange(SwNodeRange, SwNodeIndex, 
SwMoveFlags) = 0;


-/** verschieben eines Bereiches
+/** Move a range.
 */
 virtual bool MoveAndJoin(SwPaM, SwPosition, SwMoveFlags) = 0;

-/** Ueberschreiben eines Strings in einem bestehenden Textnode.
+/** Overwrite string in an existing text node.
 */
 virtual bool Overwrite(const SwPaM rRg, const String rStr) = 0;

@@ -125,26 +121,24 @@
 */
 virtual void TransliterateText(const SwPaM rPaM, 
utl::TransliterationWrapper) = 0;


-/** Einfuegen einer Grafik, Formel. Die  werden kopiert.
+/** Insert graphic or formula. The  are copied.
 */
 virtual SwFlyFrmFmt* Insert(const SwPaM rRg, const String 
rGrfName, const String rFltName, const Graphic* pGraphic,
 const SfxItemSet* pFlyAttrSet, const 
SfxItemSet* pGrfAttrSet, SwFrmFmt*) = 0;


-/**
-*/
 virtual SwFlyFrmFmt* Insert(const SwPaM rRg, const 
GraphicObject rGrfObj,	const SfxItemSet* pFlyAttrSet,

 const SfxItemSet* pGrfAttrSet, SwFrmFmt*) = 0;

-/** austauschen einer Grafik (mit Undo)
+/** Transpose graphic (with undo)
 */
 virtual void ReRead(SwPaM, const String rGrfName, const String 
rFltName, const Graphic* pGraphic, const GraphicObject* pGrfObj) = 0;


-/** Einfuegen eines DrawObjectes. Das Object muss bereits im DrawModel
-angemeldet sein.
+/** Insert a DrawObject. The object must be already registered
+in DrawModel.
 */
 virtual SwDrawFrmFmt* Insert(const SwPaM rRg, SdrObject 
rDrawObj, const SfxItemSet* pFlyAttrSet, SwFrmFmt*) = 0;


-/** Einfuegen von OLE-Objecten.
+/** Insert OLE-objects.
 */
 virtual SwFlyFrmFmt* Insert(const SwPaM rRg, const 
svt::EmbeddedObjectRef xObj, const SfxItemSet* pFlyAttrSet,

 const SfxItemSet* pGrfAttrSet, SwFrmFmt*) = 0;
@@ -152,43 +146,36 @@
 virtual SwFlyFrmFmt* InsertOLE(const SwPaM rRg, const String 
rObjName, sal_Int64 nAspect, const SfxItemSet* pFlyAttrSet,

const SfxItemSet* pGrfAttrSet, SwFrmFmt*) = 0;

-/** Aufspalten eines Nodes an rPos (nur fuer den TxtNode implementiert)
+/** Split a node at rPos (implemented only for TxtNode).
 */
 virtual bool SplitNode(const SwPosition rPos, bool 
bChkTableStart) = 0;


-/**
-*/
 virtual bool AppendTxtNode(SwPosition rPos) = 0;

-/** Ersetz einen selektierten Bereich in einem TextNode mit dem
-String. Ist fuers SuchenErsetzen gedacht.
-bRegExpRplc - ersetze Tabs (\\t) und setze den gefundenen String
- ein ( nicht \ )
-z.B.: Fnd: zzz, Repl: xx\t\\t\
--- xx\tTab..zzz..
+/** Replace selected range in a TxtNode with string.
+Intended for search  replace.
+bRegExpRplc - replace tabs (\\t) and insert the found string
+( not \ ). E.g.: Find: zzz, Replace: xx\t\\t\
+-- xx\tTab..zzz..
 */
 virtual bool ReplaceRange(SwPaM rPam, const String rNewStr,
   const bool bRegExReplace) = 0;

-/** Einfuegen eines Attributs. Erstreckt sich rRg ueber
-mehrere Nodes, wird das Attribut aufgespaltet, 

Re: [Libreoffice] Oracle wordbook in LibO

2011-01-06 Thread Andrea Pescetti
On 06/01/2011 Petr Mladek wrote:
 Kálmán „KAMI” Szalai píše v Čt 06. 01. 2011 v 15:40 +0100:
  May we include vendor specific sfuffs from oracle.dic?
  http://cgit.freedesktop.org/libreoffice/extras/commit/?h=libreoffice-3-3id=7fbdfd0a0d2af30af92839ebf8bcf97c465519e8
 
 Looks good to me.

I might have lost track of changes due to the multiple patches here and
in https://bugs.freedesktop.org/show_bug.cgi?id=31798, but there are a
few things that make little sense to me:
- This would remove OpenOffice.org and everything that used to be in
  oracle.dic
- ...but it adds the word MySQL
- It adds the whole family tree of Ubuntu
- ...but it doesn't add GNU/Linux or just GNU and Linux, and it
  removes Linux instead
- It removes Oracle and it adds Novell, but it ignores all the
  companies listed in http://www.documentfoundation.org/supporters/
- It removes all the application names: Writer, Calc, Impress...

Are these choices deliberate?

Regards,
  Andrea.

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] [PATCH] Standard-color-palette-updates

2011-01-06 Thread Bernhard Dippold

Hi Kami, all,

Kálmán „KAMI” Szalai schrieb:

  Hi All,

Some colors of LibO are already part of palette, I just wanted to update
it to follow the latest colors in wiki. If not required to provide LibO
colors with the product (as a main or separated palette) then I we can
leave the current situation or roll back. I would like to provide these
colors but this is one opinion :o)


My personal opinion is near to Christoph's position:

We can't grant that the present LibO colors will stay the official 
branding colors when the community based branding will be developed 
(planned for LibO 3.5).


Changing colors while keeping their names is a no-go IMHO, because this 
would lead to modified documents when opening them with different 
versions of LibO.


As Christoph mentions, the LibO colors are relevant for all the people 
trying to create documents and artwork following the present LibO 
branding, but these people are a minority against all our standard users.


Therefore I would love to see these colors in an additional palette, but 
not in the main palette (perhaps except the main color 
LibreOffice_Green1).


Best regards

Bernhard

PS: If we provide the colors, we should ship the final version you got 
from the wiki - thanks for your work on this topic!

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] segfault in libqa_unit.so test

2011-01-06 Thread Thomas Klausner
On Thu, Jan 06, 2011 at 10:18:38PM +, Caolán McNamara wrote:
 Bah, let me rethink that. I guess its very possible that _exit doesn't
 flush standard I/O on NetBSD which might indeed cause an empty
 generated .hxx. I've just pushed a fflush into that generator to help
 test the theory. Try a pull and build again of tools.

With a new pull and your patch applied, I get to:
No EPM: do no packaging at this stage

Multiprocessing build is finished
Maximal number of processes run: 1

Is that the end of the build? (Where did the nice graphics go? ;) )

Thank you! How do we continue with this?
 Thomas
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice