Re: [Libreoffice] [libreoffice-l10n] [ANN] LibreOffice 3.4.2 RC1 available
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*
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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.
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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)
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
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
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
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*
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
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
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
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
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
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
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
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
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
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
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)
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
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
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*
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
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
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
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
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
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
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
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
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
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
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*
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
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)
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