Re: [Libreoffice] [libreoffice-l10n] [ANN] LibreOffice 3.4.2 RC1 available

2011-07-19 Thread Cor Nouws

Hi all,

Nino Novak wrote (14-07-11 22:48)

Am Donnerstag, 14. Juli 2011, um 14:29:53 schrieb Florian Effenberger:


The list of fixed bugs in this release is here:

http://dev-builds.libreoffice.org/pre-releases/src/bugfixes-libreoffice-3-4-release-3.4.2.1.log


for a clickable version, look at
http://wiki.documentfoundation.org/User:Nnino/Drafts/bugfixes3.4.2


Thanks for the lists,
I see that bug #37488 is not listed, though it is fixed.
This not being listed is caused by the fact that it is (IMO 
unfortunately) marked as WORKSFORME. See my latest comment in the issue:

https://bugs.freedesktop.org/show_bug.cgi?id=37488#c67

Cheers

--
 - Cor
 - http://nl.libreoffice.org

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


Re: [Libreoffice] [PATCH] Replace List with std::vector ImplBmpObj*

2011-07-19 Thread Matúš Kukan
Hi Joe,

On 19 July 2011 06:40, Joseph Powers jpower...@cox.net wrote:
 I'd like someone doing a Unix build to review this for me. I compile Mac and 
 this is Unix only code so I don't just want to push and hope...

First I thought it would compile and want just to write something but
then I tried and it doesn't.
But my question is:
Would not it be better to replace List with std::list ? Or if vector I
don't like erase because it's not effective.
In this case I'd use maBmpList.pop_back(). On the first sight I
thought you have mistake in:

 void ImplSalBitmapCache::ImplRemove( X11SalBitmap* pBmp )
 {
-for( ImplBmpObj* pObj = (ImplBmpObj*) maBmpList.Last(); pObj;
pObj = (ImplBmpObj*) maBmpList.Prev() )
+for( size_t i = maBmpList.size(); i; )
 {
+ImplBmpObj* pObj = maBmpList[ --i ];
 if( pObj-mpBmp == pBmp )
 {
-maBmpList.Remove( pObj );
+maBmpList.erase( maBmpList.begin() + i );
 pObj-mpBmp-ImplRemovedFromCache();
 mnTotalSize -= pObj-mnMemSize;
 delete pObj;

But then I realized you are decreasing i in ImplBmpObj* pObj = maBmpList[ --i ];
So - maBmpList.erase( maBmpList.begin() + i ); is in fact pop_back and
it's effective but personally I'd use the latter to avoid mistakes.

Now here is what I got on 32bit Ubuntu:

vcl/unx/generic/gdi/salbmp.cxx: In member function ‘void
ImplSalBitmapCache::ImplAdd(X11SalBitmap*, sal_uLong, sal_uLong)’:
vcl/unx/generic/gdi/salbmp.cxx:1218: error: invalid use of incomplete
type ‘struct ImplSalBitmapCache::ImplBmpObj’
vcl/inc/unx/salbmp.h:253: error: forward declaration of ‘struct
ImplSalBitmapCache::ImplBmpObj’
vcl/unx/generic/gdi/salbmp.cxx:1226: error: invalid use of incomplete
type ‘struct ImplSalBitmapCache::ImplBmpObj’
vcl/inc/unx/salbmp.h:253: error: forward declaration of ‘struct
ImplSalBitmapCache::ImplBmpObj’
vcl/unx/generic/gdi/salbmp.cxx:1227: error: invalid use of incomplete
type ‘struct ImplSalBitmapCache::ImplBmpObj’
vcl/inc/unx/salbmp.h:253: error: forward declaration of ‘struct
ImplSalBitmapCache::ImplBmpObj’
vcl/unx/generic/gdi/salbmp.cxx:1227: error: invalid use of incomplete
type ‘struct ImplSalBitmapCache::ImplBmpObj’
vcl/inc/unx/salbmp.h:253: error: forward declaration of ‘struct
ImplSalBitmapCache::ImplBmpObj’
vcl/unx/generic/gdi/salbmp.cxx:1230: error: invalid use of incomplete
type ‘struct ImplSalBitmapCache::ImplBmpObj’
vcl/inc/unx/salbmp.h:253: error: forward declaration of ‘struct
ImplSalBitmapCache::ImplBmpObj’
vcl/unx/generic/gdi/salbmp.cxx: In member function ‘void
ImplSalBitmapCache::ImplRemove(X11SalBitmap*)’:
vcl/unx/generic/gdi/salbmp.cxx:1240: error: invalid use of incomplete
type ‘struct ImplSalBitmapCache::ImplBmpObj’
vcl/inc/unx/salbmp.h:253: error: forward declaration of ‘struct
ImplSalBitmapCache::ImplBmpObj’
vcl/unx/generic/gdi/salbmp.cxx:1243: error: invalid use of incomplete
type ‘struct ImplSalBitmapCache::ImplBmpObj’
vcl/inc/unx/salbmp.h:253: error: forward declaration of ‘struct
ImplSalBitmapCache::ImplBmpObj’
vcl/unx/generic/gdi/salbmp.cxx:1244: error: invalid use of incomplete
type ‘struct ImplSalBitmapCache::ImplBmpObj’
vcl/inc/unx/salbmp.h:253: error: forward declaration of ‘struct
ImplSalBitmapCache::ImplBmpObj’
vcl/unx/generic/gdi/salbmp.cxx:1245: warning: possible problem
detected in invocation of delete operator:
vcl/unx/generic/gdi/salbmp.cxx:1239: warning: ‘pObj’ has incomplete type
vcl/inc/unx/salbmp.h:253: warning: forward declaration of ‘struct
ImplSalBitmapCache::ImplBmpObj’
vcl/unx/generic/gdi/salbmp.cxx:1245: note: neither the destructor nor
the class-specific operator delete will be called, even if they are
declared when the class is defined.
vcl/unx/generic/gdi/salbmp.cxx: In member function ‘void
ImplSalBitmapCache::ImplClear()’:
vcl/unx/generic/gdi/salbmp.cxx:1257: error: invalid use of incomplete
type ‘struct ImplSalBitmapCache::ImplBmpObj’
vcl/inc/unx/salbmp.h:253: error: forward declaration of ‘struct
ImplSalBitmapCache::ImplBmpObj’
vcl/unx/generic/gdi/salbmp.cxx:1258: warning: possible problem
detected in invocation of delete operator:
vcl/unx/generic/gdi/salbmp.cxx:1258: warning: invalid use of
incomplete type ‘struct ImplSalBitmapCache::ImplBmpObj’
vcl/inc/unx/salbmp.h:253: warning: forward declaration of ‘struct
ImplSalBitmapCache::ImplBmpObj’
vcl/unx/generic/gdi/salbmp.cxx:1258: note: neither the destructor nor
the class-specific operator delete will be called, even if they are
declared when the class is defined.

I was not investigating where the problem is, I think you can handle it.

All the best,
Matus
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice] [Bug 35673] LibreOffice 3.4 most annoying bugs

2011-07-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=35673

Caolán McNamara caol...@redhat.com changed:

   What|Removed |Added

 Depends on||38095

-- 
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


[Libreoffice] [REVIEW] cherry-picking fix for fdo#38095, character classification for font selection

2011-07-19 Thread Caolán McNamara
On Tue, 2011-07-19 at 01:14 +0900, Takeshi Abe wrote:
 Hi all,
 
 According to a discussion on the Japanese local mailing list,
 the following issue may prevent some users from migrating
 to 3.4.x series:
 https://bugs.freedesktop.org/show_bug.cgi?id=38095

Looking for ack-s for 3-4 and 3-4-2.

The goal is to reset the classification of the entire full+half width
range back to ASIAN, i.e. consider the 0-9 and A-z full-width forms as
ASIAN despite their UAX24 classification as Latin, because that's the
way it used to be done and documents change otherwise, and because the
full-width Latin/Common Numerals exist in order to be the same cell
width as full-width CJK chars so they should be rendered with a CJK
font, which is all we want this particular classification to determine
for us.

I also reset back the entire number-forms range to WEAK as well, i.e.
the single-glyph roman-numerals i, IV, etc. seeing as they also
really only exist in order to be nicely rendered in CJK vertical text
mode.

http://cgit.freedesktop.org/libreoffice/libs-gui/commit/?id=e76c8d80009c8e29abf0447b7edc157eb42c9e56

looks big, but its basically just if old explicitly assigned range use
old assignment else do probably technically correct thing

C.

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


[Libreoffice] converting to new build system

2011-07-19 Thread Matúš Kukan
Hi Mike,

On 19 July 2011 01:07, Mike Eberdt libr...@comcast.net wrote:
 My 'dmake' fu is basically non-existent, so I haven't poked at this
 seriously yet.  And as I understand it, a transition is underway to GNU
 make, so I guess I've been a bit reluctant to put a lot of time into
 learning the old build system.  However, I didn't notice any info in the
 Developers section of the Wiki about using the new build system, or helping
 with the migration, etc.  Is this something that mere mortals can contribute
 to, or should I just wait for the grand unveiling? :)

I'm really glad you wrote this. I think there is always possibility to help.
For example adding visibility marks to exported objects should be
easy, there are still some .map files to be processed.

I'm not sure but may be it's already planned to write some info somewhere.
If there are people willing to help, that's great.
Moreover, there is always possibility to learn.

I have to work now on converting modules to gbuild, so I could move on
with my GSoC project.
Maybe you could help.

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


Re: [Libreoffice] [REVIEW] cherry-picking fix for fdo#38095, character classification for font selection -- 2 more reviews needed

2011-07-19 Thread Fridrich Strba
I cherry-picked to the 3-4 branch, 2 more reviews needed for the current
3.4.2 I guess.

F.

On Tue, 2011-07-19 at 09:50 +0100, Caolán McNamara wrote: 
 On Tue, 2011-07-19 at 01:14 +0900, Takeshi Abe wrote:
  Hi all,
  
  According to a discussion on the Japanese local mailing list,
  the following issue may prevent some users from migrating
  to 3.4.x series:
  https://bugs.freedesktop.org/show_bug.cgi?id=38095
 
 Looking for ack-s for 3-4 and 3-4-2.
 
 The goal is to reset the classification of the entire full+half width
 range back to ASIAN, i.e. consider the 0-9 and A-z full-width forms as
 ASIAN despite their UAX24 classification as Latin, because that's the
 way it used to be done and documents change otherwise, and because the
 full-width Latin/Common Numerals exist in order to be the same cell
 width as full-width CJK chars so they should be rendered with a CJK
 font, which is all we want this particular classification to determine
 for us.
 
 I also reset back the entire number-forms range to WEAK as well, i.e.
 the single-glyph roman-numerals i, IV, etc. seeing as they also
 really only exist in order to be nicely rendered in CJK vertical text
 mode.
 
 http://cgit.freedesktop.org/libreoffice/libs-gui/commit/?id=e76c8d80009c8e29abf0447b7edc157eb42c9e56
 
 looks big, but its basically just if old explicitly assigned range use
 old assignment else do probably technically correct thing
 
 C.
 
 ___
 LibreOffice mailing list
 LibreOffice@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/libreoffice


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


Re: [Libreoffice] [REVIEW] cherry-picking fix for fdo#38095, character classification for font selection -- 2 more reviews needed

2011-07-19 Thread Tor Lillqvist
 2 more reviews needed for the current 3.4.2

+1

--tml

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


Re: [Libreoffice] [PATCH] Get rid of SvULongs in calc

2011-07-19 Thread Maciej Rumianowski
Hi Kohei,
Thanks for review!

 First, for your svl patch:
 
 1) in SfxIntegerListItem::GetList(), you do
 
 +rList.insert( aIt, m_aList[n] );
 
 but you can simply call push_back here to append new data to the end of
 the vector.  Also, let's use pre-increment on iterators in the for
 loop there i.e. instead of doing aIt++, do ++aIt.  Post-increment on
 iterators is generally more expensive than pre-increment, so it's best
 to stick with pre-increments.

Done, and with push_back we don't need iterator.

 2) Your new constructors take arrays of different integer type; one
 takes std::vectorsal_uLong while the other takes
 uno::Sequencesal_Int32.  Let's just stick with one type and use
 sal_Int32 in both cases since that's the integer type that this class
 uses internally.

Done and checked in other places to use right type.

 And for your sc patch:
 
 3) In ScTabViewShell::Execute() you do
 
 -SvULongs aIndexList( 4, 4 );
 +::std::vector  sal_uLong  aIndexList( 4, 4 );
 
 but this actually changes the behavior of the code.  The arguments to
 SvULong's constructor controls how the internal array is laid out during
 initialization but it doesn't populate the container with data.
 Vector's ctor arguments OTOH populate the container with data.  So, your
 new code initializes aIndexList with 4 elements of value 4, which is not
 what the code intends to do.  You can simply do without passing any
 arguments to aIndexList there.

Done, Now I understand the SvULong's constructor. Also changed type to
sal_uInt32.

 4) Last but not least, please state that you are submitting your patches
 under LGPLv3+/MPL 1.1.

Of course and future also.

Best Regards,
Maciej

From a4f2da5241afdd8062f9611ca1f580df1c2ffe54 Mon Sep 17 00:00:00 2001
From: Maciej Rumianowski maciej.rumianow...@gmail.com
Date: Tue, 19 Jul 2011 10:50:54 +0200
Subject: [PATCH] Get rid of SvULongs in calc

Instead of SvULongs use ::std::vector  sal_Int32 
---
 .../itemsetwrapper/DataPointItemConverter.cxx  |7 +--
 .../itemsetwrapper/SeriesOptionsItemConverter.cxx  |7 +--
 chart2/source/view/main/ChartItemPool.cxx  |8 +++-
 sc/source/ui/docshell/impex.cxx|   11 +--
 sc/source/ui/view/tabvwsh3.cxx |   11 +--
 5 files changed, 15 insertions(+), 29 deletions(-)

diff --git a/chart2/source/controller/itemsetwrapper/DataPointItemConverter.cxx b/chart2/source/controller/itemsetwrapper/DataPointItemConverter.cxx
index 3802ffc..76e7db0 100644
--- a/chart2/source/controller/itemsetwrapper/DataPointItemConverter.cxx
+++ b/chart2/source/controller/itemsetwrapper/DataPointItemConverter.cxx
@@ -54,8 +54,6 @@
 #include editeng/brshitem.hxx
 //SfxIntegerListItem
 #include svl/ilstitem.hxx
-#define _SVSTDARR_ULONGS
-#include svl/svstdarr.hxx
 #include vcl/graph.hxx
 #include com/sun/star/graphic/XGraphic.hpp
 
@@ -641,10 +639,7 @@ void DataPointItemConverter::FillSpecialItem(
 
 case SCHATTR_DATADESCR_AVAILABLE_PLACEMENTS:
 {
-SvULongs aList;
-for ( sal_Int32 nN=0; nNm_aAvailableLabelPlacements.getLength(); nN++ )
-aList.Insert( m_aAvailableLabelPlacements[nN], sal::static_int_cast sal_uInt16 (nN) );
-rOutItemSet.Put( SfxIntegerListItem( nWhichId, aList ) );
+rOutItemSet.Put( SfxIntegerListItem( nWhichId, m_aAvailableLabelPlacements ) );
 }
 break;
 
diff --git a/chart2/source/controller/itemsetwrapper/SeriesOptionsItemConverter.cxx b/chart2/source/controller/itemsetwrapper/SeriesOptionsItemConverter.cxx
index a199b66..7df2d61 100644
--- a/chart2/source/controller/itemsetwrapper/SeriesOptionsItemConverter.cxx
+++ b/chart2/source/controller/itemsetwrapper/SeriesOptionsItemConverter.cxx
@@ -51,8 +51,6 @@
 
 //SfxIntegerListItem
 #include svl/ilstitem.hxx
-#define _SVSTDARR_ULONGS
-#include svl/svstdarr.hxx
 
 #include rtl/math.hxx
 #include functional
@@ -433,10 +431,7 @@ void SeriesOptionsItemConverter::FillSpecialItem(
 }
 case SCHATTR_AVAILABLE_MISSING_VALUE_TREATMENTS:
 {
-SvULongs aList;
-for ( sal_Int32 nN=0; nNm_aSupportedMissingValueTreatments.getLength(); nN++ )
-aList.Insert( m_aSupportedMissingValueTreatments[nN], sal::static_int_cast sal_uInt16 (nN) );
-rOutItemSet.Put( SfxIntegerListItem( nWhichId, aList ) );
+rOutItemSet.Put( SfxIntegerListItem( nWhichId, m_aSupportedMissingValueTreatments ) );
 break;
 }
 case SCHATTR_INCLUDE_HIDDEN_CELLS:
diff --git a/chart2/source/view/main/ChartItemPool.cxx b/chart2/source/view/main/ChartItemPool.cxx
index 2cdac8f..aa81e06 100644
--- a/chart2/source/view/main/ChartItemPool.cxx
+++ b/chart2/source/view/main/ChartItemPool.cxx
@@ -39,10 +39,9 @@
 #include svl/stritem.hxx
 #include svl/rectitem.hxx
 #include svl/ilstitem.hxx
-#define _SVSTDARR_ULONGS
-#include svl/svstdarr.hxx
 #include 

Re: [Libreoffice] Make RPMs

2011-07-19 Thread Caolán McNamara
On Tue, 2011-07-19 at 04:20 +0700, Nguyen Vu Hung (KiEi) wrote:
 Hello all,
 
 How can I build *rpm installation set under CentOS 5.6?
 
 I know that
 
 ./autogen.sh  --with-distro=LibreOfficeLinux --with-lang=vi
 --with-max-jobs=20 --with-num-cpus=5  --with-package-format=rpm deb
 --without-junit --with-gcc-speedup=ccache --prefix=$HOME
 make
 make install
 make dev-install
 
 will create execution files but I don't know how to create rpms files.
 
 # It seems that --with-package-format=rpm deb  doesn't help.

If you use --enable-epm (you can drop the explicit
--with-package-format=rpm deb), then when you do a make there should
them be rpms created under
instsetoo_native/unxlng*/some/sub/dir/use/find/to/find/them

FWIW, distro-configs/LibreOfficeLinux.conf contains the flags used to
build the .rpms that appear on download.libreoffice.org (or at least
that's my understanding, it may be that some extra ones get added to
that, but it should be roughly right anyway)

C.

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


Re: [Libreoffice] converting to new build system

2011-07-19 Thread Bjoern Michaelsen
Hi all,

On Tue, 19 Jul 2011 10:54:32 +0200
Matúš Kukan matus.ku...@gmail.com wrote:

 Hi Mike,
 
 On 19 July 2011 01:07, Mike Eberdt
 libr...@comcast.net wrote:
  My 'dmake' fu is basically non-existent, so I haven't poked at this
  seriously yet.  And as I understand it, a transition is underway to
  GNU make, so I guess I've been a bit reluctant to put a lot of time
  into learning the old build system.  However, I didn't notice any
  info in the Developers section of the Wiki about using the new
  build system, or helping with the migration, etc.  Is this
  something that mere mortals can contribute to, or should I just
  wait for the grand unveiling? :)
 
 I'm really glad you wrote this. I think there is always possibility
 to help. For example adding visibility marks to exported objects
 should be easy, there are still some .map files to be processed.
 
 I'm not sure but may be it's already planned to write some info
 somewhere. If there are people willing to help, that's great.
 Moreover, there is always possibility to learn.

I just noted my original posts about that at:
http://blogs.oracle.com/GullFOSS/entry/gbuild_meet_the_new_boss
http://blogs.oracle.com/GullFOSS/entry/gbuild_eyecandy_for_developers
http://blogs.oracle.com/GullFOSS/entry/gbuild_how_to_migrate_a
http://blogs.oracle.com/GullFOSS/entry/gbuild_how_to_setup_a
http://blogs.oracle.com/GullFOSS/entry/build_scalability1
http://blogs.oracle.com/GullFOSS/entry/building_openoffice_org_with_gnu

are all gone. If somebody find an archived version of that and puts it
on the wiki, that would be great. At least the how to migrate a
module post is still there at:

http://wiki.services.openoffice.org/wiki/Build_Environment_Effort/Module_Migration
with some additional info at:
http://wiki.services.openoffice.org/wiki/Build_Environment_Effort/Gbuild_Bootstrapping
http://wiki.services.openoffice.org/wiki/Build_Environment_Effort/Zen_of_gbuild
http://wiki.services.openoffice.org/wiki/Build_Environment_Effort

If somebody collects all the relevant available info about gbuild and
puts it on the TDF wiki, that would be great -- I would help out with
that.

Best,

Bjoern

-- 
https://launchpad.net/~bjoern-michaelsen


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


Re: [Libreoffice] [REVIEW 3-4] [PUSHED 3-4] Update mdds to 0.5.3

2011-07-19 Thread Michael Meeks
Hi Kohei,

On Mon, 2011-07-18 at 16:57 -0400, Kohei Yoshida wrote:
 Just to tone down this statistics a bit.  The vast majority of changes
 are in fact code relocations with no changes e.g. the change in
 rectangle_set is purely moving code from one file to another.

:-) Any chance of a real-changes diff for review ?

 With that, the real change that directly affects LibreOffice code is
 much much smaller than a couple of thousands of LOC that you say.

Good - so it sounds much more do-able.

 I'm just mentioning this so that we can make a decision based on more
 meaning analysis than simply running stats on all changes when in fact
 not all changes are relevant to our code base.

Of course, of course :-) It is good to dig into exactly what changed,
and of course it'd be nice to ship this. If you can generate the real
diff, (after all we've had 2 reviews already), and it is small enough,
I'm happy.

On Mon, 2011-07-18 at 22:28 +0200, Markus Mohrhard wrote:
 I understand your concerns but in my opinion the current situation is
 even worse. I think we will call 3.4.2 ready for enterprise use but
 calc is limited to simple documents because complex documents need
 some minutes to load and most people won't expect that.

Well, this is one of the joys of a time-based release. If the bug was
-that- startling, unexpected, urgent etc. surely we wouldn't have either
introduced it, or discovered it only three months into the 3.4.x series
- right ? [ and then just between RC1 and final ] ;-)

Having said that - I'm not a pure PITA if there is an accurate and
complete patch of what actually changed here to review - then we can
review it - but I'm not a fan of approving big, confused, un-quantified
changes at the very last minute, and then shipping them for a month to
all our users, without widespread testing first.

I look forward to reading the actual change; and I'd like to know more
of the history of this issue really, how it came to light so late etc.

ATB,

Michale.

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

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


Re: [Libreoffice] [MHST][Easy Hack] Get current page number for printing

2011-07-19 Thread Ta Duc Tung

On 07/18/2011 11:07 AM, Andrew Douglas Pitonyak wrote:

On 07/16/2011 05:02 AM, Ta Duc Tung wrote:

Hi,

1. How to get current page number for printing?
We've found a variable mnCurPage from printdlg.cxx but it 
seems not to be the right one. I think it is current page number for 
preview in print dialog.


If I were writing a macro and I wanted to know, I would probably get 
this from the view cursor (but that is just a guess).



Hi,

Last mail, I said that I can get the current page number from 
SwDocShell. However, that number is the page where the cursor is, not 
always the page which user see. Could you give me some hint where to get 
the page number where user is seeing? Is that view cursor? Where can I 
find it?


Thanks and Best Regards,
Ta Tung
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] converting to new build system

2011-07-19 Thread Matúš Kukan
It seems like Bjoern unintentionally wrote this just to me:

Hi all,

On Tue, 19 Jul 2011 10:54:32 +0200
Matúš Kukan matus.kukan-re5jqeeqqe8avxtiumw...@public.gmane.org wrote:

 Hi Mike,

 On 19 July 2011 01:07, Mike Eberdt
 libreme-wuw85uim5zdr7s880jo...@public.gmane.org wrote:
  My 'dmake' fu is basically non-existent, so I haven't poked at this
  seriously yet.  And as I understand it, a transition is underway to
  GNU make, so I guess I've been a bit reluctant to put a lot of time
  into learning the old build system.  However, I didn't notice any
  info in the Developers section of the Wiki about using the new
  build system, or helping with the migration, etc.  Is this
  something that mere mortals can contribute to, or should I just
  wait for the grand unveiling? :)
 
 I'm really glad you wrote this. I think there is always possibility
 to help. For example adding visibility marks to exported objects
 should be easy, there are still some .map files to be processed.

 I'm not sure but may be it's already planned to write some info
 somewhere. If there are people willing to help, that's great.
 Moreover, there is always possibility to learn.

I just noted my original posts about that at:
http://blogs.oracle.com/GullFOSS/entry/gbuild_meet_the_new_boss
http://blogs.oracle.com/GullFOSS/entry/gbuild_eyecandy_for_developers
http://blogs.oracle.com/GullFOSS/entry/gbuild_how_to_migrate_a
http://blogs.oracle.com/GullFOSS/entry/gbuild_how_to_setup_a
http://blogs.oracle.com/GullFOSS/entry/build_scalability1
http://blogs.oracle.com/GullFOSS/entry/building_openoffice_org_with_gnu

are all gone. If somebody find an archived version of that and puts it
on the wiki, that would be great. At least the how to migrate a
module post is still there at:

http://wiki.services.openoffice.org/wiki/Build_Environment_Effort/Module_Migration
with some additional info at:
http://wiki.services.openoffice.org/wiki/Build_Environment_Effort/Gbuild_Bootstrapping
http://wiki.services.openoffice.org/wiki/Build_Environment_Effort/Zen_of_gbuild
http://wiki.services.openoffice.org/wiki/Build_Environment_Effort

If somebody collects all the relevant available info about gbuild and
puts it on the TDF wiki, that would be great -- I would help out with
that.

Best,

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


Re: [Libreoffice] Fwd: https://bugs.freedesktop.org/show_bug.cgi?id=37996

2011-07-19 Thread Michael Meeks
Hi Imacat,

This really belongs on the design list; please reply only to there,
unless you have a patch or are seriously considering doing the work;
as/when you are committed there - then:

libreoffice-ux-adv...@lists.freedesktop.org

is a better forum. My personal take is that the longer name is more
descriptive and helpful - if it fits into the normal Unix app selection
guis, but we should have a holistic solution for all apps, and the UX
guys would want to have some input here. Please take it off the dev list
though.

Thanks,

Michael.

On Tue, 2011-07-19 at 00:23 +0800, imacat wrote:
 Dear all,
 
 Hi.  This is imacat from Taiwan.  I received a mail from some friend
 in Canonical, Taiwan.  What do you think about this, to change the title
 from LibreOffice Calc to LibreOffice Calc Spreadsheet?
 
 I know that this is technically very easy.  The problem is:  Is this
 appropriate?
 
 --- Original Mail ---
 Subject: https://bugs.freedesktop.org/show_bug.cgi?id=37996
 Date: Fri, 15 Jul 2011 19:47:07 +0800
 From: Kevin Huang kevin.hu...@canonical.com
 To: ima...@mail.imacat.idv.tw
 
 Hi Imacat,
 
 Sorry for the cold email for LibreOffice feature request.  I am
 wondering you might be able to help it.
 
 I submitted a feature request below.
 
 https://bugs.freedesktop.org/show_bug.cgi?id=37996
 
 The application name showing on menu is LibreOffice, LibreOffice Calc...
 It
 
 would definitely help LibreOffice adoption from new users and from users in
 none-English speaking countries, ex: CJK countries, if the application name
 showing LibreOffice Calc Spreadsheet instead of LibreOffice Calc only.
 
 
 This feature might not be important to English speaking people, but
 certainly help to improve Libreoffice adoption in none-English speaking
 countries.  For example, LibreOffice Calc Spreadsheet will be shown
 LibreOffice Calc 試算表 in Traditional Chinese.  I believe most
 Traditional Chinese speaking people will feel friendly on it and are
 willing to try it.
 
 ___
 LibreOffice mailing list
 LibreOffice@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/libreoffice

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


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


[Libreoffice] [PUSHED] Re: [PATCH] Re: svtools/source/contnr/imivctl1.cxx not matching imivctl1.hxx

2011-07-19 Thread Jan Holesovsky
Hi Maciej,

On 2011-07-16 at 17:22 +0200, Maciej Rumianowski wrote:

  I have changed it and now building again if it works, I will send a
  Patch. It will be my first :)

 Patch in attachment, ofcourse on LGPL3+/GPL3+/MPL.

Thank you very much for the patch! :-)  Joseph has pushed it the other
way around in the end (changing the .hxx so that it uses size_t too).

Looking forward to seeing more patches from you!

Regards,
Kendy

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


Re: [Libreoffice] X-GNOME-FullName: https://bugs.freedesktop.org/show_bug.cgi?id=37996

2011-07-19 Thread Caolán McNamara
On Tue, 2011-07-19 at 00:23 +0800, imacat wrote:
 Dear all,
 
 Hi.  This is imacat from Taiwan.  I received a mail from some friend
 in Canonical, Taiwan.  What do you think about this, to change the title
 from LibreOffice Calc to LibreOffice Calc Spreadsheet?
 
 I know that this is technically very easy.  The problem is:  Is this
 appropriate?

Well, FWIW the .desktop spec is at
http://standards.freedesktop.org/desktop-entry-spec/desktop-entry-spec-latest.html
There is a Name field and a Genericname field, we do set the Name
field as suggested there, e.g. Specific name of the application, for
example Mozilla., while we do set the Genericname to Spreadsheet
for calc, along the Generic name of the application, for example Web
Browser. recommendation.

I do see that on RHEL-6 we tweaked e.g. the firefox .desktop to use
Name=Firefox Web Browser while on Fedora 16 we use Name=Firefox so
in practice the fashion changes every now and then, but not
consistently, so it ends up a bit of a mess, never one way or the
other :-(

I see from the bug-title you list that some thinking is floats towards
an *additional* tag in the .desktop files for the long name, e.g.
http://live.gnome.org/GnomeGoals/CorrectDesktopFiles

I'm a little dubious that X-GNOME-FullName is still a desirable tag,
e.g. my GNOME-3 desktop doesn't seem to use it, but I can't see any real
objection to add X-GNOME-FullName entries to our .desktop files if it
would be used/useful if you want to do that.

Alternatively, I doubt there would be much of an objection to change the
translations of the Name field to include a generic term for at least
the languages which don't use a Latin script.

C.

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


Re: [Libreoffice] problem of master build cannot launch.

2011-07-19 Thread Caolán McNamara
On Sun, 2011-07-17 at 15:06 -0700, julien2412 wrote:
 Yifan J wrote:
  
  Firstly, I found smoketest fails on the HEAD (with error
  officeconnection.cxx:140), then I tried to launch soffice.bin in
  solver/350
  manually, but soffice.bin crashes immedialy even before splash screen
  appears.
  
 I came back yesterday and made from scratch a new local repository and I've
 still got this same pb.

it crashed doesn't give anyone enough to hazard a guess on. If
launching soffice crashes for e.g, then gdb ./soffice.bin and get a bt.

C.

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


Re: [Libreoffice] [GSoC][performance] report - week #8

2011-07-19 Thread Caolán McNamara
On Sun, 2011-07-17 at 22:53 +0200, Matúš Kukan wrote:
 And also when converting to new build system I still don't understand
 some things. For example with libraries and RepositoryFixes.
 And I found commit:
 http://cgit.freedesktop.org/libreoffice/bootstrap/commit/?id=f0d67eeeac0c615a928004082a0a7aadc5c03106
 Using new build system we want to have libs declared with STD_LIB_FILE ?

A lot of these things are far more obvious when doing it than reading it
afterwards, it just so happened that the hwpfilter module created a
lib[name].so instead of the standard lib[name]lo.so library. The
gbuild stuff creates output .sos in standard naming format. So easiest
fix is was to just to allow it to output a standard libhwplo.so name and
change the scp2 packaging stuff to use that instead. More than likely
this won't be necessary in the general case.

C.

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


Re: [Libreoffice] [PATCH] Work around a bash-ism in configure.in

2011-07-19 Thread Michael Meeks
Hi Mike,

On Mon, 2011-07-18 at 16:07 -0700, Mike Eberdt wrote:
 seriously yet.  And as I understand it, a transition is underway to GNU 
 make, so I guess I've been a bit reluctant to put a lot of time into 
 learning the old build system.

Yes - there is some horrible sterilisation of build related things due
to the change there - unfortunately, the change is taking a while :-)
not at all helped by the stop until gnumake4 is merged putting a big
mutex on any gnumake work for the last month+. Bjoern promises to
release that mutex this week.

  However, I didn't notice any info in the 
 Developers section of the Wiki about using the new build system, or 
 helping with the migration, etc.  Is this something that mere mortals can 
 contribute to, or should I just wait for the grand unveiling? :)

Yes - something for mere mortals, and it'd be fantastic to have you
working on it. By far the best thing to do is to check out the
feature/gnumake4 branch - and build on top of that work - pick a module
to convert and work on it I guess. I don't think it's -that- difficult,
instead of reading any docs, I'd read the existing makefiles that are in
already converted projects - it shouldn't be -so- hard to untangle what
to do from that. Best to let others know what module you're working on
though.

HTH,

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] Preserve RTF \'00 sequences (NUL) in strings

2011-07-19 Thread Caolán McNamara
On Mon, 2011-07-18 at 00:32 +1000, Troy Rollo wrote:
 +/* \'00 is a valid internal character in  a
 +* string in RTF, however ByteString::Append
 +* does nothing if '\0' is passed in. It is
 +* otherwise capable of handling strings with
 +* embedded NULs, so add a '\1' and then
 +* change it, as ByteString::SetChar does not
 +* care if the character is '\0'.
 +*/

Well, this'll work I guess, and I'm minded to push it, but why ;-) Is
there a concrete bug this fixes ?

C.

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


Re: [Libreoffice] Error compiling from master inside vcl

2011-07-19 Thread Caolán McNamara
On Sun, 2011-07-17 at 00:17 +0200, Andreas Mantke wrote:
 Hi all,
 
 I tried to build LibO on Linux-x64 with KDE 4 (openSUSE 11.4-x64) from a 
 fresh cloned 
 master and the build failed inside the module vcl. I then did source 
 ./LinuxX86-64Env.Set.sh and walked to vcl and run a make -r. That gave me the 
 following output in the shell:

Freshly built, is it on a machine which previously successfully built
Libreoffice ?, or is it on a fresh install ?

 /local/libreoffice/gitrepro/libreoffice1/libo/solver/350/unxlngx6/workdir/CxxObject/vcl/unx/generic/app/randrwrapper.o:
  
 In function `(anonymous 
 namespace)::RandRWrapper::XRRQueryExtension(_XDisplay*, int*, 
 int*)':
 /local/libreoffice/gitrepro/libreoffice1/libo/vcl/unx/generic/app/randrwrapper.cxx:214:
  
 undefined reference to `XRRQueryExtension'

This suggests that the code in vcl/unx/generic/app/randrwrapper.cxx was
compiled with USE_RANDR and without XRANDR_DLOPEN. I see -lXrandr *is*
in the link line, but at the beginning again as is the current style. I
suspect it might be the positioning of -lXrandr on the link line, which
will reportedly get fixed with the gnumake4 work.

As a workaround, you could try configuring with an extra
--disable-randr-link (do a make -sr clean in vcl first)

C.

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


[Libreoffice] Review Request: fdo#39023 for 3.4.* branch

2011-07-19 Thread Muthu Subramanian K
Hi,

It would be great if somebody could review+push the attached patch to
3.4 branch(s). This is _not_ a problem in master.
Without this patch shared formula(s) would not be imported correctly in
builds from 3.4 branches.

Thanks to Markus (moggi) for debugging this to quite some level (and
also for fixing one part of the problem (defined name ranges) - that fix
is already reviewed+pushed).

Thanks  Regards,
Muthu Subramanian
diff --git a/oox/source/xls/sheetdatacontext.cxx b/oox/source/xls/sheetdatacontext.cxx
index 2f0be89..d203b88 100644
--- a/oox/source/xls/sheetdatacontext.cxx
+++ b/oox/source/xls/sheetdatacontext.cxx
@@ -227,9 +227,6 @@ void SheetDataContext::onCharacters( const OUString rChars )
 {
 if( rChars.getLength()  0 )
 getSharedFormulas().importSharedFmla( rChars, maCurrCell.maFormulaRef, maCurrCell.mnSharedId, maCurrCell.maAddress );
-Reference XFormulaTokens  xTokens( maCurrCell.mxCell, UNO_QUERY_THROW );
-ExtCellFormulaContext aContext( *this, xTokens, maCurrCell.maAddress );
-getSharedFormulas().setSharedFormulaCell( aContext, maCurrCell.mnSharedId );
 }
 break;
 
@@ -279,6 +276,12 @@ void SheetDataContext::onEndElement()
 maCurrCell.mnCellType = XML_TOKEN_INVALID;
 }
 }
+if( maCurrCell.mnSharedId = 0 )
+{
+Reference XFormulaTokens  xTokens( maCurrCell.mxCell, UNO_QUERY_THROW );
+ExtCellFormulaContext aContext( *this, xTokens, maCurrCell.maAddress );
+getSharedFormulas().setSharedFormulaCell( aContext, maCurrCell.mnSharedId );
+}
 
 // store the cell formatting data
 setCellFormat( maCurrCell );
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] sylk unit tests

2011-07-19 Thread Michael Meeks
Hi there,

On Sun, 2011-07-17 at 18:50 +0200, Maciej Rumianowski wrote:
  Hmm, well I've now added a sylk import unit test to sc, i.e.
  http://cgit.freedesktop.org/libreoffice/calc/commit/?id=8ac49ef8ab4f065d7b729dcb15505c602c77
   
 
 Thanks, but as Nigel pointed out it's hard to unit test every peace of
 LO and I don't think my small change is worth it.

Hey ! - so unit tests are -never- useless, particularly if they are
fast, small and run during the build. Indeed - I would -love- to have
much more unit test coverage of calc.

 Later on I will mail my Patch getting rid of SvULongs in calc.

It would be simply excellent if we could find a way of doing this. One
thing I would strongly recommend is not trying to create your own,
stand-alone unit test harness, but instead building on top of an
existing test by adding a new test method in sc/qa/unit/ucalc.cxx :-)
[ indeed, reading that it is encouraging to see the growth in
unit-testage there particularly for data-pilots etc. ;-].

Anyhow - thanks for your work !

Regards,

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] [REVIEW] cherry-picking fix for fdo#38095, character classification for font selection

2011-07-19 Thread Petr Mladek
Caolán McNamara píše v Út 19. 07. 2011 v 09:50 +0100:
 On Tue, 2011-07-19 at 01:14 +0900, Takeshi Abe wrote:
  Hi all,
  
  According to a discussion on the Japanese local mailing list,
  the following issue may prevent some users from migrating
  to 3.4.x series:
  https://bugs.freedesktop.org/show_bug.cgi?id=38095
 
 Looking for ack-s for 3-4 and 3-4-2.

 http://cgit.freedesktop.org/libreoffice/libs-gui/commit/?id=e76c8d80009c8e29abf0447b7edc157eb42c9e56

Looks fine except for a typo in scriptListCount definition. I have fixed
it by
http://cgit.freedesktop.org/libreoffice/libs-gui/commit/?id=bfa455e31ab2356b4ee0cf499bb71138e837e6c7

Could you please approve this typo fix?


Best Regards,
Petr

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


Re: [Libreoffice] [REVIEWED] cherry-picking fix for fdo#38095, character classification for font selection -- 2 more reviews needed

2011-07-19 Thread Michael Meeks

On Tue, 2011-07-19 at 03:06 -0600, Tor Lillqvist wrote:
  2 more reviews needed for the current 3.4.2
 +1

And pushed to libreoffice-3-4-2; thanks - and lovely to see the unit
test there to prevent regression :-)

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] Review Request: fdo#39023 for 3.4.* branch

2011-07-19 Thread Markus Mohrhard
Looks good to me. Will push it to 3.4.

2011/7/19 Muthu Subramanian K sumu...@novell.com

 Hi,

 It would be great if somebody could review+push the attached patch to
 3.4 branch(s). This is _not_ a problem in master.
 Without this patch shared formula(s) would not be imported correctly in
 builds from 3.4 branches.

 Thanks to Markus (moggi) for debugging this to quite some level (and
 also for fixing one part of the problem (defined name ranges) - that fix
 is already reviewed+pushed).

 Thanks  Regards,
 Muthu Subramanian

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


Re: [Libreoffice] [REVIEW] cherry-picking fix for fdo#38095, character classification for font selection

2011-07-19 Thread Caolán McNamara
On Tue, 2011-07-19 at 12:49 +0200, Petr Mladek wrote:
 Looks fine except for a typo in scriptListCount definition. I have fixed
 it by
 http://cgit.freedesktop.org/libreoffice/libs-gui/commit/?id=bfa455e31ab2356b4ee0cf499bb71138e837e6c7
 
 Could you please approve this typo fix?

Oops, yes. That looks right.

C.

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


[Libreoffice] [REVIEWED 3-4-2] [PUSHED 3-4-2] Update mdds to 0.5.3

2011-07-19 Thread Michael Meeks

On Mon, 2011-07-18 at 16:57 -0400, Kohei Yoshida wrote:
 Just to tone down this statistics a bit.  The vast majority of changes
 are in fact code relocations with no changes

So - I've pushed it to -3-4-2 - after doing my best at reviewing the
changes; but I'm really unhappy with this.

I would like to understand:

* why mdds is maintained in a separate place - can we not
  whack it into the calc/ git module ?

* why it is not branched and frozen with the same cadence - so
  we have small fixes back-ported, rather than having to review
  large code movements / renaming / etc.
+ I'm well up for API cleaning and big beautification
  patches etc. but prefer them on the unstable branch.

Also - some UI pointers for QA to do some more hard-core testing of its
various uses cases here would be much appreciated.

Thanks,

Michael.

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


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


[Libreoffice] [REVIEWED] [PUSHED 3-4-2] patch for fdo#39023: shared formulas are not imported correctly from xlsx, first part

2011-07-19 Thread Michael Meeks

On Tue, 2011-07-12 at 11:18 +0200, Petr Mladek wrote:
 Yes, it will be in 3.4.2. I am going to make sure that all patches are
 in, do test build, tag, and branch.

So this got into 3.4.2 - for the record.

Thanks,

Michael.

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


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


[Libreoffice] [REVIEW] fix for fdo#39333: database ranges don't work in formulas

2011-07-19 Thread Markus Mohrhard
Hello,

I fear I'm a bit late for this patch to be included into 3.4.2 but at least
we should push it to 3.4

It fixes that we can't use database ranges in formulas because ScNameToken
always used ocName but it could also be ocDBArea for database ranges. It's
nothing special, just added an optional parameter to the constructor of
ScNameToken that will be passed to ScToken.

Regards,
Markus
From 9e098c065220ff56497c25edc0a92df5462fafc5 Mon Sep 17 00:00:00 2001
From: Markus Mohrhard markus.mohrh...@googlemail.com
Date: Tue, 19 Jul 2011 13:09:20 +0200
Subject: [PATCH] fix for fdo#39333: database ranges don't work in formulas

---
 sc/inc/token.hxx  |2 +-
 sc/source/core/tool/token.cxx |6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/sc/inc/token.hxx b/sc/inc/token.hxx
index 78364e9..ce16c17 100644
--- a/sc/inc/token.hxx
+++ b/sc/inc/token.hxx
@@ -244,7 +244,7 @@ private:
 private:
 ScNameToken(); // disabled
 public:
-ScNameToken(sal_uInt16 nIndex, bool bGlobal);
+ScNameToken(sal_uInt16 nIndex, bool bGlobal, OpCode eOpCode = ocName);
 ScNameToken(const ScNameToken r);
 virtual ~ScNameToken();
 virtual sal_uInt8GetByte() const;
diff --git a/sc/source/core/tool/token.cxx b/sc/source/core/tool/token.cxx
index 0153ca4..1cf448e 100644
--- a/sc/source/core/tool/token.cxx
+++ b/sc/source/core/tool/token.cxx
@@ -380,7 +380,7 @@ FormulaToken* ScRawToken::CreateToken() const
 IF_NOT_OPCODE_ERROR( ocPush, ScMatrixToken);
 return new ScMatrixToken( pMat );
 case svIndex :
-return new ScNameToken(name.nIndex, name.bGlobal);
+return new ScNameToken(name.nIndex, name.bGlobal, eOp);
 case svExternalSingleRef:
 {
 String aTabName(extref.cTabName);
@@ -949,8 +949,8 @@ bool ScExternalDoubleRefToken::operator ==( const FormulaToken r ) const
 
 // 
 
-ScNameToken::ScNameToken(sal_uInt16 nIndex, bool bGlobal) :
-ScToken(svIndex, ocName), mnIndex(nIndex), mbGlobal(bGlobal) {}
+ScNameToken::ScNameToken(sal_uInt16 nIndex, bool bGlobal, OpCode eOpCode) :
+ScToken(svIndex, eOpCode), mnIndex(nIndex), mbGlobal(bGlobal) {}
 
 ScNameToken::ScNameToken(const ScNameToken r) :
 ScToken(r), mnIndex(r.mnIndex), mbGlobal(r.mbGlobal) {}
-- 
1.7.3.4

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


Re: [Libreoffice] Review Request: fdo#39023 for 3.4.* branch

2011-07-19 Thread Katarina Machalkova
 Looks good to me. Will push it to 3.4.
 
 2011/7/19 Muthu Subramanian K sumu...@novell.com
 
  Hi,
  
  It would be great if somebody could review+push the attached patch to
  3.4 branch(s). This is _not_ a problem in master.
  Without this patch shared formula(s) would not be imported correctly in
  builds from 3.4 branches.

 +1
Need 1 more review

B.
-- 
  \  Katarina Machalkova
  \\\__o  LibO developer
__\\\'/_   hedgehog painter


signature.asc
Description: This is a digitally signed message part.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice] [Bug 35673] LibreOffice 3.4 most annoying bugs

2011-07-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=35673

Bug 35673 depends on bug 38095, which changed state.

Bug 38095 Summary: Character classification for Western or Asian text font  
differ since conventional version.
https://bugs.freedesktop.org/show_bug.cgi?id=38095

   What|Old Value   |New Value

 Resolution||FIXED
 Status|ASSIGNED|RESOLVED

-- 
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


[Libreoffice] [REVIEW] Fix wrapping around objects in headers (fdo#39155)

2011-07-19 Thread Lubos Lunak

 Hello,

 could I get reviews for the attached patch for 3-4 and 3-4-2 too, if 
possible? It is actually just removing checking for header in the 
bWrapAllowed expression. This is fdo#39155 and I do not see a reason for the 
check there anyway, the change applied only makes text in a header wrap 
around objects in the header (and not in the main text), so I see no 
regression in the behavior.

-- 
 Lubos Lunak
 l.lu...@suse.cz
diff --git a/sw/source/core/text/txtfly.cxx b/sw/source/core/text/txtfly.cxx
index 513c2d5..a900867 100644
--- a/sw/source/core/text/txtfly.cxx
+++ b/sw/source/core/text/txtfly.cxx
@@ -1380,14 +1380,7 @@ SwAnchoredObjList* SwTxtFly::InitAnchoredObjList()
 
 const SwSortedObjs *pSorted = pPage-GetSortedObjs();
 const sal_uInt32 nCount = pSorted ? pSorted-Count() : 0;
-// -- #108724# Page header/footer content doesn't have to wrap around
-//  floating screen objects
-const bool bFooterHeader = 0 != pCurrFrm-FindFooterOrHeader();
-const IDocumentSettingAccess* pIDSA = pCurrFrm-GetTxtNode()-getIDocumentSettingAccess();
-// #i40155# - check, if frame is marked not to wrap
-const sal_Bool bWrapAllowed = ( pIDSA-get(IDocumentSettingAccess::USE_FORMER_TEXT_WRAPPING) ||
-( !pCurrFrm-IsInFtn()  !bFooterHeader ) ) 
-  !SwLayouter::FrmNotToWrap( *pCurrFrm-GetTxtNode()-getIDocumentLayoutAccess(), *pCurrFrm );
+const sal_Bool bWrapAllowed = !SwLayouter::FrmNotToWrap( *pCurrFrm-GetTxtNode()-getIDocumentLayoutAccess(), *pCurrFrm );
 
 bOn = sal_False;
 
@@ -1399,6 +1392,7 @@ SwAnchoredObjList* SwTxtFly::InitAnchoredObjList()
 // #i28701# - consider complete frame area for new
 // text wrapping
 SwRect aRect;
+const IDocumentSettingAccess* pIDSA = pCurrFrm-GetTxtNode()-getIDocumentSettingAccess();
 if ( pIDSA-get(IDocumentSettingAccess::USE_FORMER_TEXT_WRAPPING) )
 {
 aRect = pCurrFrm-Prt();
@@ -1414,6 +1408,7 @@ SwAnchoredObjList* SwTxtFly::InitAnchoredObjList()
 const long nRight = (aRect.*fnRect-fnGetRight)() - 1;
 const long nLeft = (aRect.*fnRect-fnGetLeft)() + 1;
 const sal_Bool bR2L = pCurrFrm-IsRightToLeft();
+const bool bFooterHeader = ( pCurrFrm-FindFooterOrHeader() != NULL );
 
 const IDocumentDrawModelAccess* pIDDMA = pCurrFrm-GetTxtNode()-getIDocumentDrawModelAccess();
 
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice] [REVIEWED, 1 more for 3-4-2] Re: [REVIEW] cherry-picking fix for fdo#38095, character classification for font selection

2011-07-19 Thread Jan Holesovsky
Hi,

On 2011-07-19 at 12:05 +0100, Caolán McNamara wrote:

  Looks fine except for a typo in scriptListCount definition. I have fixed
  it by
  http://cgit.freedesktop.org/libreoffice/libs-gui/commit/?id=bfa455e31ab2356b4ee0cf499bb71138e837e6c7
  
  Could you please approve this typo fix?
 
 Oops, yes. That looks right.

One more review from me; I have pushed it to libreoffice-3-4.

One last review + push to -3-4-2 needed.

Regards,
Kendy

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


Re: [Libreoffice] [PATCH] Preserve RTF \'00 sequences (NUL) in strings

2011-07-19 Thread Troy Rollo
On Tuesday 19 July 2011, Caolán McNamara wrote:
 Well, this'll work I guess, and I'm minded to push it, but why ;-) Is
 there a concrete bug this fixes ?

Yes, but it is a little obscure. If you take the attached ODT file, save it as 
an RTF, and then load the RTF, the prefix to the paragraph number will be 
wrong (\x05( instead of ([). With the patch, it works. The problem is that 
in this file, the leveltext field in the number format is (correctly) exported 
to RTF as follows:

  \'05([\'00])

Without the patch the embedded NUL gets dropped, so the resulting string on 
reading is:

\x05([])

With that string, the following test in rtfnum.cxx fails:

   if( DelCharAtEnd( sLvlText, ';' ).Len() 
sLvlText.Len()  sLvlText.Len() ==
(sal_uInt16)(sLvlText.GetChar( 0 )) + 1 )
sLvlText.Erase( 0, 1 );

If the correct string (\x05([\0])) were read, the test would succeed, and 
the first character would be stripped, giving ([\0])

Later, when the string is decoded in the number format we get \x05( as the 
prefix instead of ([. The suffix comes out OK because the missing NUL is 
made up for by the additional \x05 - so the suffix is in the right position in 
the string.  However the prefix was shifted one position to the right.

This particular manifestation of the problem only shows up clearly when the 
first list level has a visible prefix before the number.


Number RTF test.odt
Description: application/vnd.oasis.opendocument.text
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] [PATCH] Fix 33898 - import/export xrefs to numbered paras

2011-07-19 Thread Troy Rollo
On Tuesday 19 July 2011, Korrawit Pruegsanusak wrote:
 Hello Troy,
 
 Digging mailing-list archive and just glancing at your patch,
 isn't this patch against fdo#33960 ?
 (https://bugs.freedesktop.org/show_bug.cgi?id=33960)
 IMHO it isn't likely to be 33898.

It should have said 33893 (although I also posted a fix to 33960 separately).
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] [PATCH] Replace List with std::vector ImplBmpObj*

2011-07-19 Thread Joseph Powers

On Jul 19, 2011, at 12:51 AM, Matúš Kukan wrote:

 Hi Joe,
 
 On 19 July 2011 06:40, Joseph Powers jpower...@cox.net wrote:
 I'd like someone doing a Unix build to review this for me. I compile Mac and 
 this is Unix only code so I don't just want to push and hope...
 
 First I thought it would compile and want just to write something but
 then I tried and it doesn't.
 But my question is:
 Would not it be better to replace List with std::list ? Or if vector I
 don't like erase because it's not effective.
 In this case I'd use maBmpList.pop_back(). On the first sight I
 thought you have mistake in:

A List would  be better; however, it's a list of pointers so the size isn't 
that big.

 void ImplSalBitmapCache::ImplRemove( X11SalBitmap* pBmp )
 {
 -for( ImplBmpObj* pObj = (ImplBmpObj*) maBmpList.Last(); pObj;
 pObj = (ImplBmpObj*) maBmpList.Prev() )
 +for( size_t i = maBmpList.size(); i; )
 {
 +ImplBmpObj* pObj = maBmpList[ --i ];
 if( pObj-mpBmp == pBmp )
 {
 -maBmpList.Remove( pObj );
 +maBmpList.erase( maBmpList.begin() + i );
 pObj-mpBmp-ImplRemovedFromCache();
 mnTotalSize -= pObj-mnMemSize;
 delete pObj;
 
 But then I realized you are decreasing i in ImplBmpObj* pObj = maBmpList[ --i 
 ];
 So - maBmpList.erase( maBmpList.begin() + i ); is in fact pop_back and
 it's effective but personally I'd use the latter to avoid mistakes.

It's a loop and it's not just removing the last entry. It's only removing the 
entry that matches the one passed. (I don't know why we're starting at the end 
since the same pointer shouldn't be in the list twice; however, if the same 
pointer gets in the list twice, then we'll always remove the last one instead 
of the first one and I didn't wont to change this behavior).

 Now here is what I got on 32bit Ubuntu:
 
 vcl/unx/generic/gdi/salbmp.cxx: In member function ‘void
 ImplSalBitmapCache::ImplAdd(X11SalBitmap*, sal_uLong, sal_uLong)’:
 vcl/unx/generic/gdi/salbmp.cxx:1218: error: invalid use of incomplete
 type ‘struct ImplSalBitmapCache::ImplBmpObj’
 vcl/inc/unx/salbmp.h:253: error: forward declaration of ‘struct
 ImplSalBitmapCache::ImplBmpObj’

Ok, just guessing but:

+struct ImplBmpObj;
+
 class ImplSalBitmapCache
 {
 private:
+typedef ::std::vector ImplBmpObj*  BmpList_impl;
 
-ListmaBmpList;
+BmpList_implmaBmpList;
 sal_uIntPtr mnTotalSize;

Would most likely work better. I was defining struct ImplBmpObj inside the 
ImplSalBitmapCache class and in the .cxx the actual struct was defined outside 
the class; thus, we never defined the expected structure.

 I was not investigating where the problem is, I think you can handle it.

Can you try the new version of the patch?

 All the best,
 Matus

Thanks for helping,
Joe P.


0001-Replace-List-with-std-vector-ImplBmpObj.patch
Description: Binary data
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] sylk unit tests

2011-07-19 Thread Maciej Rumianowski
Hi Michael,

   Hey ! - so unit tests are -never- useless, particularly if they are
 fast, small and run during the build. Indeed - I would -love- to have
 much more unit test coverage of calc.

I also like the Idea of unit tests and an Easy Hack with calc unit tests
is on my radar :)

   It would be simply excellent if we could find a way of doing this. One
 thing I would strongly recommend is not trying to create your own,
 stand-alone unit test harness, but instead building on top of an
 existing test by adding a new test method in sc/qa/unit/ucalc.cxx :-)
 [ indeed, reading that it is encouraging to see the growth in
 unit-testage there particularly for data-pilots etc. ;-].

My first attempt was to write test there.

   Anyhow - thanks for your work !
I'm glad to work on LibreOffice! I'm following Document Foundation and
LO from the beginning ;)

Best Regards,
Maciej


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


Re: [Libreoffice] Review Request: fdo#39023 for 3.4.* branch

2011-07-19 Thread Markus Mohrhard
Hello David,

I think the original version should be safe. Our formula api can't deal with
broken documents during import especially the oox import can't deal with
errors of any kinds in formulas. But a broken document won't create a crash
only a #Name entry in the cell that is not correct.

I know that this situation is not ideal but we can't change the formula api
at the moment.

Regards, Markus

2011/7/19 David Tardon dtar...@redhat.com

 On Tue, Jul 19, 2011 at 04:29:32PM +0530, Muthu Subramanian K wrote:
  Hi,
 
  +if( maCurrCell.mnSharedId = 0 )
  +{
  +Reference XFormulaTokens  xTokens( maCurrCell.mxCell,
 UNO_QUERY_THROW );
  +ExtCellFormulaContext aContext( *this, xTokens,
 maCurrCell.maAddress );
  +getSharedFormulas().setSharedFormulaCell( aContext,
 maCurrCell.mnSharedId );
  +}

 This may cause problems with (broken) documents with cells that have
 type different from shared but contain si attribute. I am giving my +1
 on the condition that

 +if( maCurrCell.mnSharedId = 0 )

 is changed to

 +if( maCurrCell.mnFormulaType == XML_shared 
 maCurrCell.mnSharedId = 0 )

 (or that someone convinces me the scenario I described cannot happen :)

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

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


Re: [Libreoffice] [REVIEW] more distro-pack-install fixes

2011-07-19 Thread Petr Mladek
Andreas Radke píše v Po 18. 07. 2011 v 18:18 +0200:
 Am Mon, 18 Jul 2011 16:58:03 +0200
 schrieb Petr Mladek pmla...@suse.cz:
 
  Hi,
  
  the below patches should fix two more problems reported by Andy. I
  would like to get them into libreoffice-3-4-2 branch.
  
  Best Regards,
  Petr
  
  
 
 The first patch works as expected. 
 
 The second patch doesn't fix the file list creation for me.

As discussed on irc. Andy forgot to run autogen.sh. All the fixes worked
for him in the end:

--- cut ---
[23:12:38] andyrtr_laptop pmladek, thanks. all your patches so far
work well now. now applying the permission fix.
--- cut ---


Best Regards,
Petr

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


[Libreoffice] [REVIEW for 3-4-2] more distro-pack-install fixes

2011-07-19 Thread Michael Meeks
Hi guys,

On Mon, 2011-07-18 at 16:58 +0200, Petr Mladek wrote:
 the below patches should fix two more problems reported by Andy. I would
 like to get them into libreoffice-3-4-2 branch.

I've pushed them to libreoffice-3-4; and I attach them here, two more
ack's needed, I understand Andreas' problem was solved too.

Thanks,

Michael.

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

From 2046eaeaaabbc58326066ee37f53cc97ebf679bb Mon Sep 17 00:00:00 2001
From: Petr Mladek pmla...@suse.cz
Date: Mon, 18 Jul 2011 19:47:39 +0200
Subject: [PATCH 4/4] distro-install-desktop-integration: put $PREFIXDIR/bin/soffice into file list

---
 bin/distro-install-desktop-integration |1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/bin/distro-install-desktop-integration b/bin/distro-install-desktop-integration
index 9e3b499..77ccf3a 100755
--- a/bin/distro-install-desktop-integration
+++ b/bin/distro-install-desktop-integration
@@ -161,6 +161,7 @@ add_wrapper unopkg unopkg  unopkg  
 #   For example, the Novell package mark this symlink as %ghost
 #	and update it in %post and %postun
 ln -sf $INSTALLDIR/program/soffice $DESTDIR$PREFIXDIR/bin/soffice
+test -f $DESTDIR/gid_Module_Root_Brand  echo $PREFIXDIR/bin/soffice $DESTDIR/gid_Module_Root_Brand
 
 # create bash completion
 mkdir -p $DESTDIR/etc/bash_completion.d
-- 
1.7.3.4

From 72d5c4c3f7d2a5d329eb74fa1d1f69c1fb8c15b9 Mon Sep 17 00:00:00 2001
From: Petr Mladek pmla...@suse.cz
Date: Mon, 18 Jul 2011 16:50:18 +0200
Subject: [PATCH 3/3] distro-install: fix en-US-only installation

generate en-US file list when --with-distro=

sigh, we should set WITH_LANG=en-US when no language is selected; the empty
string is pretty ugly; unforrunately, many makefile tests check for
this empty variable
---
 configure.in |3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/configure.in b/configure.in
index 18f93d2..65813cb 100755
--- a/configure.in
+++ b/configure.in
@@ -7906,6 +7906,9 @@ for lang in $WITH_LANG ; do
 done
 # list with substituted ALL
 WITH_LANG_LIST=`echo $WITH_LANG | sed s/ALL/$ALL_LANGS/`
+# this variable is used only by bin/distro-install-* helper scripts
+# they need a real list of languages
+test -z $WITH_LANG_LIST  WITH_LANG_LIST=en-US
 AC_SUBST(ALL_LANGS)
 AC_SUBST(WITH_LANG)
 AC_SUBST(WITH_LANG_LIST)
-- 
1.7.3.4

From 7b202653f352a6c1bd18066bc71c93b1a5b33c13 Mon Sep 17 00:00:00 2001
From: Petr Mladek pmla...@suse.cz
Date: Mon, 18 Jul 2011 16:48:49 +0200
Subject: [PATCH 2/3] distro-install-desktop-integration: do not pack libreoffice-base.desktop twice

---
 bin/distro-install-desktop-integration |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/bin/distro-install-desktop-integration b/bin/distro-install-desktop-integration
index 600c54e..9e3b499 100755
--- a/bin/distro-install-desktop-integration
+++ b/bin/distro-install-desktop-integration
@@ -148,7 +148,7 @@ add_wrapper lomath soffice --math   libreoffice libreoffice-mat
 add_wrapper loimpress  soffice --impresslibreoffice libreoffice-impress.desktop gid_Module_Brand_Prg_Impress
 add_wrapper loweb  soffice --weblibreoffice gid_Module_Brand_Prg_Wrt
 add_wrapper lowriter   soffice --writer libreoffice libreoffice-writer.desktop  gid_Module_Brand_Prg_Wrt
-add_wrapper lofromtemplate soffice .uno:NewDoc libreoffice libreoffice-base.desktopgid_Module_Root_Brand
+add_wrapper lofromtemplate soffice .uno:NewDoc  libreoffice gid_Module_Root_Brand
 add_wrapper libreofficesoffice libreoffice libreoffice-startcenter.desktop gid_Module_Root_Brand
 add_wrapper lofficesoffice libreoffice gid_Module_Root_Brand
 add_wrapper unopkg unopkg  unopkg  gid_Module_Root_Brand
-- 
1.7.3.4

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


Re: [Libreoffice] Review Request: fdo#39023 for 3.4.* branch

2011-07-19 Thread David Tardon
On Tue, Jul 19, 2011 at 02:35:42PM +0200, Markus Mohrhard wrote:
 Hello David,
 
 I think the original version should be safe. Our formula api can't deal with
 broken documents during import especially the oox import can't deal with
 errors of any kinds in formulas. But a broken document won't create a crash
 only a #Name entry in the cell that is not correct.
 
 I know that this situation is not ideal but we can't change the formula api
 at the moment.

Okay.

I have pushed the fix to 3.4.1 as is (but the addendum is still in 3.4,
should we want it).

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


Re: [Libreoffice] [REVIEW for 3-4-2] more distro-pack-install fixes

2011-07-19 Thread Fridrich Strba
Looks perfectly OK.

+1 from me

F.

On Tue, 2011-07-19 at 13:55 +0100, Michael Meeks wrote: 
 Hi guys,
 
 On Mon, 2011-07-18 at 16:58 +0200, Petr Mladek wrote:
  the below patches should fix two more problems reported by Andy. I would
  like to get them into libreoffice-3-4-2 branch.
 
   I've pushed them to libreoffice-3-4; and I attach them here, two more
 ack's needed, I understand Andreas' problem was solved too.
 
   Thanks,
 
   Michael.
 
 ___
 LibreOffice mailing list
 LibreOffice@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/libreoffice


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


Re: [Libreoffice] [REVIEW for 3-4-2][PUSHED:3-4-2] more distro-pack-install fixes

2011-07-19 Thread Petr Mladek
Fridrich Strba píše v Út 19. 07. 2011 v 15:08 +0200:
 Looks perfectly OK.
 
 +1 from me

Andy told me on irc that it worked for hit. I took it as 3rd approval
and pushed into 3-4-2 branch.


Best Regards,
Petr

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


Re: [Libreoffice] [PUSHED:3-4; 3-4-2] Review Request: fdo#39023 for 3.4.* branch

2011-07-19 Thread Petr Mladek
David Tardon píše v Út 19. 07. 2011 v 15:02 +0200:
 On Tue, Jul 19, 2011 at 02:35:42PM +0200, Markus Mohrhard wrote:
 Okay.
 
 I have pushed the fix to 3.4.1 as is (but the addendum is still in 3.4,
 should we want it).

I see it in 3-4-2 branch = updated subject.

Thanks for help.

Best Regards,
Petr


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


[Libreoffice] [Bug 35673] LibreOffice 3.4 most annoying bugs

2011-07-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=35673

Jan Holesovsky ke...@suse.cz changed:

   What|Removed |Added

 Depends on|39155   |

--- Comment #171 from Jan Holesovsky ke...@suse.cz 2011-07-19 06:28:17 PDT ---
Removing bug 39155, the reason described in the bug 39155 comment 6.

-- 
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] [tdf-discuss] Re: LibO 3.4.2 RC1 under Windows

2011-07-19 Thread Jesús Corrius
On Tue, Jul 19, 2011 at 3:25 PM, Italo Vignoli italo.vign...@gmail.com wrote:
 On 7/19/11 2:53 PM, plino wrote:

 I understand that it is the version you are supposed to distribute. BUT it
 still needs a CHECK because there are NEWER security fixes.

 Although I don't use Windows, I've seen a recent update of the C++
 Redistributable Package under Windows 7 on a friend's PC. Online, I
 could just find this link, which I don't think is the right one, but
 mentions a Microsoft Visual C++ 2010 Redistributable Package:
 http://www.microsoft.com/download/en/details.aspx?id=. Maybe, the
 new package is what gets installed on updated PCs, and might be included
 in LibreOffice (but I'm not a developer, and I don't know if there are
 differences between C++ 2008 and C++ 2010).

Hi Italo,

Yes, there are diferences.
We should distribute the C++ 2008 as we are using the Visual C++ 2008
compiler to build it.

-- 
Jesús Corrius je...@softcatala.org
Document Foundation founding member
Mobile: +34 661 11 38 26
Skype: jcorrius | Twitter: @jcorrius
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] [REVIEW] Fix wrapping around objects in headers (fdo#39155)

2011-07-19 Thread Jan Holesovsky
Hi Lubos, all,

On 2011-07-19 at 13:49 +0200, Lubos Lunak wrote:

  could I get reviews for the attached patch for 3-4 and 3-4-2 too, if 
 possible? It is actually just removing checking for header in the 
 bWrapAllowed expression. This is fdo#39155 and I do not see a reason for the 
 check there anyway, the change applied only makes text in a header wrap 
 around objects in the header (and not in the main text), so I see no 
 regression in the behavior.

I have reviewed it, and it turns out that the behavior we have in
LibreOffice 3.4 is what one can see in the MS Office too.  I mean, in
LibreOffice, we have an additional feature that we can wrap around
pictures in headers, but it got disabled sometime between 3.2 and 3.3.

To fix this, we need to find a good balance between the feature, and
interoperability - which needs a bit more thought than a quick fix for
3.4.2.  Additionally, the 3.4 behavior has already been in 3.3, and as
such, it is not a regression against the last stable release.

Based on the above, I removed this bug from the 3.4 Most Annoying bugs,
and we should fix it in master for 3.5.0 so that it gets more testing,
and more feedback.

Regards,
Kendy

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


[Libreoffice] [PUSHED] Preserve RTF \'00 sequences (NUL) in strings

2011-07-19 Thread Caolán McNamara
On Tue, 2011-07-19 at 22:18 +1000, Troy Rollo wrote:
 On Tuesday 19 July 2011, Caolán McNamara wrote:
  Well, this'll work I guess, and I'm minded to push it, but why ;-) Is
  there a concrete bug this fixes ?
 
 Yes, but it is a little obscure. If you take the attached ODT file, save it 
 as 
 an RTF, and then load the RTF, the prefix to the paragraph number will be 
 wrong (\x05( instead of ([). With the patch, it works. The problem is 
 that 
 in this file, the leveltext field in the number format is (correctly) 
 exported 
 to RTF as follows:
 
   \'05([\'00])

Ahhh, gotcha. Indeed, nice catch. Looks good to me, and gives the
expected results. Pushed. Can you confirm the patch is under our
preferred LGPLv3+/MPLv1.1 combo ?

C.

As an aside we should check if the rtl::O[U]StringBuffer append's with a
0 character actually do append a 0 to the end of a already null
terminated string so the ugliness can go away if/when the conversion
train hits that code.

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


Re: [Libreoffice] [PATCH][REVIEW][PUSHED 3-4] SetNamedRanges speedup fdo#35357

2011-07-19 Thread Kohei Yoshida
On Tue, 2011-07-19 at 07:24 +0200, David Tardon wrote:

 (I haven't pushed Laurent's patch.)

Thanks, David.  I've just pushed Laurent's patch with my sign-off.

Kohei

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

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


Re: [Libreoffice] [PATCH] Replace List with std::vector ImplBmpObj*

2011-07-19 Thread Matúš Kukan
On 19 July 2011 14:22, Joseph Powers jpower...@cox.net wrote:

 A List would  be better; however, it's a list of pointers so the size isn't 
 that big.

So why not use it ?
I did not mean the actual size in memory but the number of elements.
I've seen there around 150 elements when I tried to print the size.
That's not really much but I think when we can use something better we should.
I don't really know how many elements there can be and how often we
are removing not from the end and what's the real difference in
effectiveness between list and vector but..

May be someone has opinion about this?

 void ImplSalBitmapCache::ImplRemove( X11SalBitmap* pBmp )
 {
 -    for( ImplBmpObj* pObj = (ImplBmpObj*) maBmpList.Last(); pObj;
 pObj = (ImplBmpObj*) maBmpList.Prev() )
 +    for( size_t i = maBmpList.size(); i; )
     {
 +        ImplBmpObj* pObj = maBmpList[ --i ];
         if( pObj-mpBmp == pBmp )
         {
 -            maBmpList.Remove( pObj );
 +            maBmpList.erase( maBmpList.begin() + i );
             pObj-mpBmp-ImplRemovedFromCache();
             mnTotalSize -= pObj-mnMemSize;
             delete pObj;

 But then I realized you are decreasing i in ImplBmpObj* pObj = maBmpList[ 
 --i ];
 So - maBmpList.erase( maBmpList.begin() + i ); is in fact pop_back and
 it's effective but personally I'd use the latter to avoid mistakes.

 It's a loop and it's not just removing the last entry. It's only removing the 
 entry that matches the one passed. (I don't know why we're starting at the 
 end since the same pointer shouldn't be in the list twice; however, if the 
 same pointer gets in the list twice, then we'll always remove the last one 
 instead of the first one and I didn't wont to change this behavior).

Ah, right, my fault. But now it's better to use list if you do not
need random access to elements. I mean maBmpList[ i ];

 Can you try the new version of the patch?

One more error:
maBmpList[ i ]-ImplRemovedFromCache(); should be
maBmpList[ i ]-mpBmp-ImplRemovedFromCache();
Now, just warning:
unx/generic/gdi/salbmp.cxx:1212: warning: ‘pObj’ may be used
uninitialized in this function
but that's not really true

I wonder if that was possible to compile for you but I have no
experience with other systems, so there may be big differences I'm not
used to.

Anyway, good job, I like removing old things or replacing them with new.

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


Re: [Libreoffice] [REVIEWED] [PUSHED 3-4] fix for fdo#39333: database ranges don't work in formulas

2011-07-19 Thread Kohei Yoshida
Hi Markus,

On Tue, 2011-07-19 at 13:19 +0200, Markus Mohrhard wrote:

 It fixes that we can't use database ranges in formulas because
 ScNameToken always used ocName but it could also be ocDBArea for
 database ranges. It's nothing special, just added an optional
 parameter to the constructor of ScNameToken that will be passed to
 ScToken.

Looks good  simple enough.  Pushed to the -3-4 branch.  Feel free to
push it to master if you haven't already.

Kohei

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

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


Re: [Libreoffice] [libreoffice-l10n] [ANN] LibreOffice 3.4.2 RC1 available

2011-07-19 Thread Petr Mladek
Cor Nouws píše v Út 19. 07. 2011 v 09:12 +0200:
 Hi all,
 
 Nino Novak wrote (14-07-11 22:48)
  Am Donnerstag, 14. Juli 2011, um 14:29:53 schrieb Florian Effenberger:
 
  The list of fixed bugs in this release is here:
 
  http://dev-builds.libreoffice.org/pre-releases/src/bugfixes-libreoffice-3-4-release-3.4.2.1.log
 
  for a clickable version, look at
  http://wiki.documentfoundation.org/User:Nnino/Drafts/bugfixes3.4.2
 
 Thanks for the lists,
 I see that bug #37488 is not listed, though it is fixed.

The list of changes is generated from the git commit logs. The bug
#37488 started to work accidentally. There was no commit associated with
it.

I am afraid that the list newer will be 100% because people sometimes
forget to mention bug numbers in commit messages. I guess that we do not
want to force bug number in every trivial commit. If it is not
mentioned, it is rather a detective work to find it ;-)


Best Regards,
Petr

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


Re: [Libreoffice] permission 444/555 of installation files

2011-07-19 Thread Petr Mladek
Andreas Radke píše v Út 19. 07. 2011 v 00:09 +0200:
 Nice, you've almost fixed all of them expect some *rc files:
 
 -r--r--r-- 1 root root 126 Jul 18 22:01 
 /build/pkg/libreoffice-common/usr/lib/libreoffice/ure/bin/versionrc
 -r--r--r-- 1 root root 1.1K Jul 18 22:01 
 /build/pkg/libreoffice-common/usr/lib/libreoffice/basis3.4/program/unorc
 -r--r--r-- 1 root root 44 Jul 18 22:01 
 /build/pkg/libreoffice-common/usr/lib/libreoffice/basis3.4/program/pythonloader.unorc
 -r--r--r-- 1 root root 1.5K Jul 18 22:01 
 /build/pkg/libreoffice-common/usr/lib/libreoffice/basis3.4/program/fundamentalbasisrc
 -r--r--r-- 1 root root 150 Jul 18 22:01 
 /build/pkg/libreoffice-common/usr/lib/libreoffice/basis3.4/program/versionrc
 -r--r--r-- 1 root root 50 Jul 18 22:01 
 /build/pkg/libreoffice-common/usr/lib/libreoffice/program/redirectrc
 -r--r--r-- 1 root root 56 Jul 18 22:01 
 /build/pkg/libreoffice-common/usr/lib/libreoffice/program/setuprc
 -r--r--r-- 1 root root 1.5K Jul 18 22:01 
 /build/pkg/libreoffice-common/usr/lib/libreoffice/program/fundamentalrc
 -r--r--r-- 1 root root 421 Jul 18 22:01 
 /build/pkg/libreoffice-common/usr/lib/libreoffice/program/versionrc

These might get fixed by the attached patch. They are created by the
perl installer from scratch.


Best Regards,
Petr
--- solenv/bin/modules/installer/profiles.pm.old	2011-05-18 17:51:47.0 +0200
+++ solenv/bin/modules/installer/profiles.pm	2011-07-19 18:45:47.0 +0200
@@ -107,7 +107,7 @@ sub add_profile_into_filelist
 $profile{'ismultilingual'} = 0;
 $profile{'sourcepath'} = $completeprofilename;
 $profile{'Name'} = $oneprofile-{'Name'};
-$profile{'UnixRights'} = 444;
+$profile{'UnixRights'} = 644;
 $profile{'gid'} = $oneprofile-{'gid'};
 $profile{'Dir'} = $oneprofile-{'Dir'};
 $profile{'destination'} = $oneprofile-{'destination'};
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice] hackfest agenda

2011-07-19 Thread Florian Effenberger

Hello,

sorry to bother you, but I'm getting continous requests from our 
co-sponsor to give them some details on the hackfest agenda. :-)


The proposed items list at 
http://wiki.documentfoundation.org/Hackfest2011#Ideas is rather empty 
for now. Anyone would like to add their ideas and plans to it? ;)


Thanks,
Florian

--
Florian Effenberger flo...@documentfoundation.org
Steering Committee and Founding Member of The Document Foundation
Tel: +49 8341 99660880 | Mobile: +49 151 14424108
Skype: floeff | Twitter/Identi.ca: @floeff
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice] [REVIEW 3-4] [PUSHED 3-4] Update mdds to 0.5.3

2011-07-19 Thread Kohei Yoshida
Hi guys,

On Tue, 2011-07-19 at 10:13 +0100, Michael Meeks wrote:

   Having said that - I'm not a pure PITA if there is an accurate and
 complete patch of what actually changed here to review - then we can
 review it - but I'm not a fan of approving big, confused, un-quantified
 changes at the very last minute, and then shipping them for a month to
 all our users, without widespread testing first.

Let's not go with the upgrade then.  I'm happy enough that this will go
into 3.4.3.  There is no point rushing this into 3.4.2 when one of us
feels uncomfortable.  That's the whole point of having a 3-person
review.

   I look forward to reading the actual change; 

I'm afraid the actual change would still be around 2 to 3 hundred lines
of code, and unfortunately I don't have time to tinker with the history
to generate such patch right now.  Let's just not worry about this.  I
was hoping that the amount of unit test code we have in the mdds
repository would be enough to give us comfort (despite the amount of
code change), but apparently that's not enough. :-P

So, no hard feelings.  Let's move on.

Kohei

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

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


Re: [Libreoffice] hackfest agenda

2011-07-19 Thread Regina Henschel

Florian Effenberger schrieb:

Hello,

sorry to bother you, but I'm getting continous requests from our
co-sponsor to give them some details on the hackfest agenda. :-)

The proposed items list at
http://wiki.documentfoundation.org/Hackfest2011#Ideas is rather empty
for now. Anyone would like to add their ideas and plans to it? ;)


I have added some subjects.

Kind regards
Regina

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


[Libreoffice] [Bug 35673] LibreOffice 3.4 most annoying bugs

2011-07-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=35673

Bug 35673 depends on bug 37763, which changed state.

Bug 37763 Summary: Define a range name prevents recognition of a label
https://bugs.freedesktop.org/show_bug.cgi?id=37763

   What|Old Value   |New Value

 Resolution||FIXED
 Status|NEW |RESOLVED

-- 
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


[Libreoffice] [REVIEW] Fix for fdo#39236

2011-07-19 Thread Kohei Yoshida
Hi there,

I'd like to have the following commit

http://cgit.freedesktop.org/libreoffice/calc/commit/?id=8da78944e7c4fcae9ae40bea7162898d20db6329

cherry-picked to -3-4 and -3-4-2 in order to fix

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

which is a nasty crasher which took me almost a full day to figure
out. :-/

In the end, it was because of my mis-use of the remove-erase idiom to
remove elements from ptr_vector, which caused double-delete.

Kohei

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

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


[Libreoffice] [Bug 35673] LibreOffice 3.4 most annoying bugs

2011-07-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=35673

mariosv mari...@miguelangel.mobi changed:

   What|Removed |Added

 Depends on||36555

-- 
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] Make RPMs

2011-07-19 Thread Nguyen Vu Hung (KiEi)

Hello,

(2011/07/19 16:12), Caolán McNamara wrote:


If you use --enable-epm (you can drop the explicit
--with-package-format=rpm deb), then when you do a make there should
them be rpms created under
instsetoo_native/unxlng*/some/sub/dir/use/find/to/find/them

FWIW, distro-configs/LibreOfficeLinux.conf contains the flags used to
build the .rpms that appear on download.libreoffice.org (or at least
that's my understanding, it may be that some extra ones get added to
that, but it should be roughly right anyway)


Executive summary: By adding --enable-epm, I got an error:
rpm -bb: option not found.

Which version of rpm should I use?

My environment:

CentOS 5.6
rpm-4.4.2.3-22.el5

Build options:
./autogen.sh  --with-distro=LibreOfficeLinux --with-lang=vi 
--with-max-jobs=20 --with-num-cpus=5   --without-junit 
--with-gcc-speedup=ccache --prefix=$HOME --enable-epm


Build error:

=
Building module setup_native
=
Entering /libo/vuhung/000.git/bootstrap/setup_native/source/ulfconv

Entering /libo/vuhung/000.git/bootstrap/setup_native/scripts/source

Entering /libo/vuhung/000.git/bootstrap/setup_native/scripts

mkdir -p ../unxlngi6.pro/misc/install/fake-db-root
rpm --define _builddir 
/libo/vuhung/000.git/bootstrap/setup_native/unxlngi6.pro/misc/install/fake-db-root 
--define _rpmdir 
/libo/vuhung/000.git/bootstrap/setup_native/unxlngi6.pro/bin -bb 
fake-db.spec

-bb: unknown option
dmake:  Error code 1, while making 
'../unxlngi6.pro/bin/noarch/fake-db-1.0-0.noarch.rpm'





--
Nguyen Vu Hung (Nguyyễn Vũ Hưng)
LibreOffice, Vietnamese Translation Team.

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


Re: [Libreoffice] [PATCH] Replace List with std::vector ImplBmpObj*

2011-07-19 Thread Joseph Powers

On Jul 19, 2011, at 7:34 AM, Matúš Kukan wrote:

 On 19 July 2011 14:22, Joseph Powers jpower...@cox.net wrote:
 
 A List would  be better; however, it's a list of pointers so the size 
 isn't that big.
 
 So why not use it ?
 I did not mean the actual size in memory but the number of elements.
 I've seen there around 150 elements when I tried to print the size.
 That's not really much but I think when we can use something better we should.
 I don't really know how many elements there can be and how often we
 are removing not from the end and what's the real difference in
 effectiveness between list and vector but..
 
 May be someone has opinion about this?
 
 void ImplSalBitmapCache::ImplRemove( X11SalBitmap* pBmp )
 {
 -for( ImplBmpObj* pObj = (ImplBmpObj*) maBmpList.Last(); pObj;
 pObj = (ImplBmpObj*) maBmpList.Prev() )
 +for( size_t i = maBmpList.size(); i; )
 {
 +ImplBmpObj* pObj = maBmpList[ --i ];
 if( pObj-mpBmp == pBmp )
 {
 -maBmpList.Remove( pObj );
 +maBmpList.erase( maBmpList.begin() + i );
 pObj-mpBmp-ImplRemovedFromCache();
 mnTotalSize -= pObj-mnMemSize;
 delete pObj;
 
 But then I realized you are decreasing i in ImplBmpObj* pObj = maBmpList[ 
 --i ];
 So - maBmpList.erase( maBmpList.begin() + i ); is in fact pop_back and
 it's effective but personally I'd use the latter to avoid mistakes.
 
 It's a loop and it's not just removing the last entry. It's only removing 
 the entry that matches the one passed. (I don't know why we're starting at 
 the end since the same pointer shouldn't be in the list twice; however, if 
 the same pointer gets in the list twice, then we'll always remove the last 
 one instead of the first one and I didn't wont to change this behavior).
 
 Ah, right, my fault. But now it's better to use list if you do not
 need random access to elements. I mean maBmpList[ i ];

Ok, I changed from a stl::vector to a stl::list; I also rewrote the loops 
to use an iterator instead of [] addressing since [] addressing can be 
expensive with lists. I also rewrote the loop in question to find the 1st match 
(there should only be one match) because it makes the logic cleaner and easier 
to read.

 Can you try the new version of the patch?
 
 One more error:
 maBmpList[ i ]-ImplRemovedFromCache(); should be
 maBmpList[ i ]-mpBmp-ImplRemovedFromCache();
 Now, just warning:
 unx/generic/gdi/salbmp.cxx:1212: warning: ‘pObj’ may be used
 uninitialized in this function
 but that's not really true

Ok, I missed that one.  I also added code to initialize pObj to NULL; some of 
the developers like to compile with error on warnings and they'ed get really 
mad if we left a warning.

 I wonder if that was possible to compile for you but I have no
 experience with other systems, so there may be big differences I'm not
 used to.

I only compile for Mac OS; thus, I only compile /vcl/aqua  /vcl/source. If you 
watch your compile, you should only compile /vcl/unx  /vcl/source. We also 
have /vcl/win for those brave people who compile on Windows.

 Anyway, good job, I like removing old things or replacing them with new.
 
 Matus

Hopefully this is the last time...
Joe P.


0001-Replace-List-with-std-list-ImplBmpObj.patch
Description: Binary data

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


Re: [Libreoffice] [REVIEW][PUSHED 3-4] Fix for fdo#39236

2011-07-19 Thread David Tardon
On Tue, Jul 19, 2011 at 02:56:04PM -0400, Kohei Yoshida wrote:
 Hi there,
 
 I'd like to have the following commit
 
 http://cgit.freedesktop.org/libreoffice/calc/commit/?id=8da78944e7c4fcae9ae40bea7162898d20db6329
 
 cherry-picked to -3-4 and -3-4-2 in order to fix
 
 https://bugs.freedesktop.org/show_bug.cgi?id=39236
 
 which is a nasty crasher which took me almost a full day to figure
 out. :-/
 
 In the end, it was because of my mis-use of the remove-erase idiom to
 remove elements from ptr_vector, which caused double-delete.
 

Looks okay, so pushed to 3.4. It's too late for 3.4.2, unfortunately.

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


[Libreoffice] [PATCH] Splash screen fix for multi-head on Linux (bug 33214)

2011-07-19 Thread Josh Heidenreich
Uses xinerama to get the size of the primary screen


0001-Splash-screen-fix-for-multi-head-on-Linux-bug-33214.patch
Description: Binary data
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice