[PUSHED] sax: com.sun.star.xml.sax.FastSerializer seems to be unused

2012-09-07 Thread Kohei Yoshida (via Code Review)
Hi,

Thank you for your patch! :-)  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/480

Approvals:
  Kohei Yoshida: Verified
  Michael Meeks: Looks good to me, approved


--
To view, visit https://gerrit.libreoffice.org/480
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I826da00c83589f468c499879d86546d60b2aa515
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Matúš Kukan 
Gerrit-Reviewer: Kohei Yoshida 
Gerrit-Reviewer: Matúš Kukan 
Gerrit-Reviewer: Michael Meeks 

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


[Libreoffice-commits] .: sax/inc sax/source

2012-09-07 Thread Libreoffice Gerrit user
 sax/inc/fastserializer.hxx  |  202 
 sax/source/fastparser/facreg.cxx|   17 --
 sax/source/fastparser/fastsax.component |3 
 sax/source/tools/fastserializer.cxx |2 
 sax/source/tools/fastserializer.hxx |  202 
 sax/source/tools/fshelper.cxx   |2 
 6 files changed, 204 insertions(+), 224 deletions(-)

New commits:
commit 2ecd5b0372e7953a57ece3c669c22e6545e19f2e
Author: Matúš Kukan 
Date:   Fri Aug 24 21:43:27 2012 +0200

sax: com.sun.star.xml.sax.FastSerializer seems to be unused

Change-Id: I826da00c83589f468c499879d86546d60b2aa515
Reviewed-on: https://gerrit.libreoffice.org/480
Reviewed-by: Michael Meeks 
Tested-by: Kohei Yoshida 

diff --git a/sax/inc/fastserializer.hxx b/sax/inc/fastserializer.hxx
deleted file mode 100644
index 765cafb..000
--- a/sax/inc/fastserializer.hxx
+++ /dev/null
@@ -1,202 +0,0 @@
-/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
-/*
- * This file is part of the LibreOffice project.
- *
- * This Source Code Form is subject to the terms of the Mozilla Public
- * License, v. 2.0. If a copy of the MPL was not distributed with this
- * file, You can obtain one at http://mozilla.org/MPL/2.0/.
- *
- * This file incorporates work covered by the following license notice:
- *
- *   Licensed to the Apache Software Foundation (ASF) under one or more
- *   contributor license agreements. See the NOTICE file distributed
- *   with this work for additional information regarding copyright
- *   ownership. The ASF licenses this file to you under the Apache
- *   License, Version 2.0 (the "License"); you may not use this file
- *   except in compliance with the License. You may obtain a copy of
- *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
- */
-
-#ifndef SAX_FASTSERIALIZER_HXX
-#define SAX_FASTSERIALIZER_HXX
-
-#include 
-#include 
-#include 
-#include 
-#include 
-#include 
-
-#include 
-#include 
-
-#include 
-
-#include "sax/fshelper.hxx"
-
-#define SERIALIZER_IMPLEMENTATION_NAME  
"com.sun.star.comp.extensions.xml.sax.FastSerializer"
-#define SERIALIZER_SERVICE_NAME "com.sun.star.xml.sax.FastSerializer"
-
-namespace sax_fastparser {
-
-class SAX_DLLPUBLIC FastSaxSerializer : public ::cppu::WeakImplHelper2< 
::com::sun::star::xml::sax::XFastSerializer, 
::com::sun::star::lang::XServiceInfo >
-{
-typedef ::com::sun::star::uno::Sequence< ::sal_Int8 > Int8Sequence;
-typedef ::com::sun::star::uno::Sequence< ::sal_Int32 > Int32Sequence;
-
-public:
-explicitFastSaxSerializer(  );
-virtual ~FastSaxSerializer();
-
-::com::sun::star::uno::Reference< ::com::sun::star::io::XOutputStream > 
getOutputStream() {return mxOutputStream;}
-
-// The implementation details
-static ::com::sun::star::uno::Sequence< ::rtl::OUString >   
getSupportedServiceNames_Static(void);
-
-// XFastSerializer
-virtual void SAL_CALL startDocument(  ) throw 
(::com::sun::star::xml::sax::SAXException, 
::com::sun::star::uno::RuntimeException);
-virtual void SAL_CALL endDocument(  ) throw 
(::com::sun::star::xml::sax::SAXException, 
::com::sun::star::uno::RuntimeException);
-virtual void SAL_CALL startFastElement( ::sal_Int32 Element, const 
::com::sun::star::uno::Reference< 
::com::sun::star::xml::sax::XFastAttributeList >& Attribs )
-throw (::com::sun::star::xml::sax::SAXException, 
::com::sun::star::uno::RuntimeException);
-virtual void SAL_CALL startUnknownElement( const ::rtl::OUString& 
Namespace, const ::rtl::OUString& Name, const ::com::sun::star::uno::Reference< 
::com::sun::star::xml::sax::XFastAttributeList >& Attribs )
-throw (::com::sun::star::xml::sax::SAXException, 
::com::sun::star::uno::RuntimeException);
-virtual void SAL_CALL endFastElement( ::sal_Int32 Element )
-throw (::com::sun::star::xml::sax::SAXException, 
::com::sun::star::uno::RuntimeException);
-virtual void SAL_CALL endUnknownElement( const ::rtl::OUString& Namespace, 
const ::rtl::OUString& Name )
-throw (::com::sun::star::xml::sax::SAXException, 
::com::sun::star::uno::RuntimeException);
-virtual void SAL_CALL singleFastElement( ::sal_Int32 Element, const 
::com::sun::star::uno::Reference< 
::com::sun::star::xml::sax::XFastAttributeList >& Attribs )
-throw (::com::sun::star::xml::sax::SAXException, 
::com::sun::star::uno::RuntimeException);
-virtual void SAL_CALL singleUnknownElement( const ::rtl::OUString& 
Namespace, const ::rtl::OUString& Name, const ::com::sun::star::uno::Reference< 
::com::sun::star::xml::sax::XFastAttributeList >& Attribs )
-throw (::com::sun::star::xml::sax::SAXException, 
::com::sun::star::uno::RuntimeException);
-virtual void SAL_CALL characters( const ::rtl::OUString& aChars )
-throw (::com::sun::star::xml::sax::SAXException, 
::com::sun::star::uno::RuntimeException);
-virtual voi

[Libreoffice-commits] Changes to 'feature/calc-xml-source'

2012-09-07 Thread Libreoffice Gerrit user
New branch 'feature/calc-xml-source' available with the following commits:
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED] OUString cleanup in accessibility

2012-09-07 Thread Olivier Hallot (via Code Review)
Hi,

Thank you for your patch! :-)  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/583

Approvals:
  Olivier Hallot: Verified; Looks good to me, approved


--
To view, visit https://gerrit.libreoffice.org/583
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ifceaa764231306c1f542183afc2e852cc418
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Ricardo Montania 
Gerrit-Reviewer: Olivier Hallot 

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


[Libreoffice-commits] .:

2012-09-07 Thread Libreoffice Gerrit user
 0 files changed

New commits:
commit ea744ff9e8d2d6bb463e6f8f44405f9b574b37d1
Author: Ricardo Montania 
Date:   Fri Sep 7 05:42:09 2012 -0300

OUString cleanup in accessibility

Change-Id: Ifceaa764231306c1f542183afc2e852cc418
Reviewed-on: https://gerrit.libreoffice.org/583
Reviewed-by: Olivier Hallot 
Tested-by: Olivier Hallot 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: valgrinding with python stuff active

2012-09-07 Thread julien2412
Sorry, I didn't get the point about G_SLICE or the Stephan's commit.
For a beginner like me, what must I change/add to this line:
valgrind --tool=memcheck --num-callers=50 --trace-children=yes ./soffice.bin
2>&1 | tee /tmp/valgrind.log

so I get no false positive with Python and/or Java?

I read this:
"
By default LibreOffice uses a custom memory allocator, so valgrind won't be
able to give full results unless this is disabled, disable it at runtime
with...

$ export G_SLICE=always-malloc
"
on this page http://wiki.documentfoundation.org/Development/How_to_debug
but what about if custom memory allocator is bugged? (BTW why do we use
this? Historical reason?) Wouldn't Valgrind be helpful to detect it?

I'm a bit lost here :-(

Julien



--
View this message in context: 
http://nabble.documentfoundation.org/valgrinding-with-python-stuff-active-tp4006046p4006167.html
Sent from the Dev mailing list archive at Nabble.com.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Design-related questions

2012-09-07 Thread Alexander Wilms
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Hi,

during the last IRC meeting of the design team a few questions were
raised and maybe some devs could answer them.

1) Would it be feasible to create a new UI element, a modal bar like
the one shown in Nautilus when e.g. an MP3 player is plugged. After
the user exits Impress' slideshow mode while using the Android remote,
a modal bar should appear, featuring a button to disconnect the phone
and one to stay connected for future presentations.

2) Do you need the option 'Size optimization for ODF format'. We're
not sure whether it's an advanced feature used by devs or if it's
unneccessary.

3) Is it possible to integrate warnings like 'You're saving to OOXML,
which means some formatting might get lost.' into the system file dialogs?

Thanks a lot

Alex
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://www.enigmail.net/

iQEcBAEBAgAGBQJQSnc8AAoJECO1/tAiWGrNSJMH/3ZVK8B2A1Geg63y84m3WnHo
LgjLUGQZiUjL1GUY8QOrKnNYaZ+RuRIeE6hWzNVQBtF9WOXXRz+lDBhxqS2dzBgF
RySiHOJWHoQqBdFA1GTaUndikYHIFfnZ+X70FTG0NPPdq9sUB3VpPjR4jl3LykqG
vPI8LxpvoZhvxEImGA8CtmVi46CFX6be4aeH+g5IxEWDqiJkuv67bwDoL6/VTui4
BOg1huMB57YwN/7Jt1c6qCuSfW+O02aRjfD5ly8eAq9dGkoOa1VnAcUFOThkPdAW
kO1M+oEn/6fICjkYkmu6VVy2NCjAHOMFS+03XTMY+FBolxaqWGvSFHRcdnnJEf8=
=Njy/
-END PGP SIGNATURE-

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


[PUSHED] OUString cleanup in animations

2012-09-07 Thread Olivier Hallot (via Code Review)
Hi,

Thank you for your patch! :-)  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/584

Approvals:
  Olivier Hallot: Verified; Looks good to me, approved


--
To view, visit https://gerrit.libreoffice.org/584
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib858f9fa709d72539f71a6473a4ecb519d68e98e
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Ricardo Montania 
Gerrit-Reviewer: Olivier Hallot 

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


[Libreoffice-commits] .: animations/source

2012-09-07 Thread Libreoffice Gerrit user
 animations/source/animcore/animcore.cxx|5 --
 animations/source/animcore/factreg.hxx |4 -
 animations/source/animcore/targetpropertiescreator.cxx |   36 -
 3 files changed, 22 insertions(+), 23 deletions(-)

New commits:
commit 1bbe1743c288392b9b2421f9d33efa8d05a01008
Author: Ricardo Montania 
Date:   Fri Sep 7 16:54:11 2012 -0300

OUString cleanup in animations

Change-Id: Ib858f9fa709d72539f71a6473a4ecb519d68e98e
Reviewed-on: https://gerrit.libreoffice.org/584
Reviewed-by: Olivier Hallot 
Tested-by: Olivier Hallot 

diff --git a/animations/source/animcore/animcore.cxx 
b/animations/source/animcore/animcore.cxx
index f372bbb..a194dcc 100644
--- a/animations/source/animcore/animcore.cxx
+++ b/animations/source/animcore/animcore.cxx
@@ -55,7 +55,6 @@
 
 using ::osl::Mutex;
 using ::osl::Guard;
-using ::rtl::OUString;
 using ::cppu::OInterfaceContainerHelper;
 using ::cppu::OInterfaceIteratorHelper;
 using ::com::sun::star::uno::Any;
@@ -515,12 +514,12 @@ Reference< XInterface > SAL_CALL createInstance_##N( 
const Reference< XComponent
 }\
 OUString getImplementationName_##N()\
 {\
-return OUString( RTL_CONSTASCII_USTRINGPARAM ( IN ) );\
+return OUString( IN );\
 }\
 Sequence getSupportedServiceNames_##N(void)\
 {\
 Sequence aRet(1);\
-aRet.getArray()[0] = OUString( RTL_CONSTASCII_USTRINGPARAM( SN ));\
+aRet.getArray()[0] = SN;\
 return aRet;\
 }
 
diff --git a/animations/source/animcore/factreg.hxx 
b/animations/source/animcore/factreg.hxx
index 09b7a36..a809c06 100644
--- a/animations/source/animcore/factreg.hxx
+++ b/animations/source/animcore/factreg.hxx
@@ -24,8 +24,8 @@ extern rtl_StandardModuleCount g_moduleCount;
 
 #define DECL_NODE_FACTORY(N)\
 extern ::com::sun::star::uno::Reference< ::com::sun::star::uno::XInterface > 
SAL_CALL createInstance_##N( const ::com::sun::star::uno::Reference< 
::com::sun::star::uno::XComponentContext > & rSMgr ) throw 
(::com::sun::star::uno::Exception);\
-extern ::rtl::OUString getImplementationName_##N();\
-extern ::com::sun::star::uno::Sequence< ::rtl::OUString> 
getSupportedServiceNames_##N(void)
+extern OUString getImplementationName_##N();\
+extern ::com::sun::star::uno::Sequence< OUString> 
getSupportedServiceNames_##N(void)
 
 DECL_NODE_FACTORY( PAR );
 DECL_NODE_FACTORY( SEQ );
diff --git a/animations/source/animcore/targetpropertiescreator.cxx 
b/animations/source/animcore/targetpropertiescreator.cxx
index 128b5f1..4ecdce4 100644
--- a/animations/source/animcore/targetpropertiescreator.cxx
+++ b/animations/source/animcore/targetpropertiescreator.cxx
@@ -70,12 +70,12 @@ namespace animcore
 virtual uno::Sequence< animations::TargetProperties > SAL_CALL 
createInitialTargetProperties( const uno::Reference< animations::XAnimationNode 
>& rootNode ) throw (uno::RuntimeException);
 
 // XServiceInfo
-virtual ::rtl::OUString SAL_CALL getImplementationName() throw( 
uno::RuntimeException );
-virtual sal_Bool SAL_CALL supportsService( const ::rtl::OUString& 
ServiceName ) throw( uno::RuntimeException );
-virtual uno::Sequence< ::rtl::OUString > SAL_CALL 
getSupportedServiceNames()  throw( uno::RuntimeException );
+virtual OUString SAL_CALL getImplementationName() throw( 
uno::RuntimeException );
+virtual sal_Bool SAL_CALL supportsService( const OUString& ServiceName 
) throw( uno::RuntimeException );
+virtual uno::Sequence< OUString > SAL_CALL getSupportedServiceNames()  
throw( uno::RuntimeException );
 
 // XServiceName
-virtual ::rtl::OUString SAL_CALL getServiceName(  ) throw 
(uno::RuntimeException);
+virtual OUString SAL_CALL getServiceName(  ) throw 
(uno::RuntimeException);
 
 protected:
 ~TargetPropertiesCreator(); // we're a ref-counted UNO class. _We_ 
destroy ourselves.
@@ -95,15 +95,15 @@ namespace animcore
 return TargetPropertiesCreator::createInstance( rSMgr );
 }
 
-::rtl::OUString getImplementationName_TargetPropertiesCreator()
+OUString getImplementationName_TargetPropertiesCreator()
 {
-return ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM ( 
IMPLEMENTATION_NAME ) );
+return OUString( IMPLEMENTATION_NAME );
 }
 
-uno::Sequence< ::rtl::OUString > 
getSupportedServiceNames_TargetPropertiesCreator(void)
+uno::Sequence< OUString > 
getSupportedServiceNames_TargetPropertiesCreator(void)
 {
-uno::Sequence< ::rtl::OUString > aRet(1);
-aRet.getArray()[0] = ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( 
SERVICE_NAME ) );
+uno::Sequence< OUString > aRet(1);
+aRet.getArray()[0] = SERVICE_NAME;
 return aRet;
 }
 
@@ -342,7 +342,7 @@ namespace animcore
 if( !(aAny >>= bVisible) )
 {
 // try to extract string
-::rtl::OUString aString;
+   

[Bug 37361] LibreOffice 3.5 most annoying bugs

2012-09-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=37361

--- Comment #373 from Joel Madero  2012-09-07 21:47:29 
UTC ---
Nominating fdo#50285
https://bugs.freedesktop.org/show_bug.cgi?id=50285

This bug has the potential of really creating an unprofessional document. This
could be a real problem for business people or students if they save as a
document and then submit the work as it completely changes the text.

-- 
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-qa] How Can I Provide More Info For Developers

2012-09-07 Thread julien2412
Just some thoughts here.
First backtrace are very useful only if they contain symbols.
It's more difficult to retrieve bt on Windows than on Linux (I don't know
how MacOs could be compared with them)
It seems no daily builds contain symbols or missed it
(http://dev-builds.libreoffice.org/daily/).
It's more complicated to compile on Windows (a little less for MacOs) than
for Linux. At least, it seems so according to what I read.
We can't expect any user build sources or have an idea about what's a
backtrace/strace/Valgrind trace is
As a simple QA contributor, I think preserving time of Devs is very
important, more important than QA people time, because:
- dev guys are few compared to the size of the project, especially in some
parts like Base
- it's easier to learn the basics of QA than C++. Of course, QA time mustn't
be spoiled because they're few too

But:
Main (all?) Linux distributions provide the LO symbols package so you don't
need to build LO to retrieve useful bt.
Doc efforts have been made to help retrieved bt in:
- Linux
(http://wiki.documentfoundation.org/BugReport#How_to_get_backtrace_.28on_Linux.29)
- Windows
(http://wiki.documentfoundation.org/BugReport#How_to_get_backtrace_.28on_WINDOWS.29)
A great video has been made by jesus corrius to explain how to install the
required tools on Windows and retrieve an useful bt and guys like bfo
retrieved a lot of bt (greatly appreciated for Windows only trackers).
- wiki pages (eg: Native build and bug reporting) and BSA bring and help a
lot new comers (computer student or simple user) who take some dev tasks or
do some QA. So let's keep on to improve these tools

The fact is we can't guess in general the computer skill of a reporter so we
can only ask politely these info by giving him/her the links quoted above.
It can't be an offense or a scarecrow to ask technical info to people ! :-)

Julien



--
View this message in context: 
http://nabble.documentfoundation.org/How-Can-I-Provide-More-Info-For-Developers-tp4006100p4006162.html
Sent from the Dev mailing list archive at Nabble.com.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: feature/vs2012: testing needed

2012-09-07 Thread Peter Foley
On 9/7/2012 2:47 AM, Noel Grandin wrote:
> picture attached.
> 
> $ ./oowintool --msvc-ver
> 10.0
> 
> This machine is a Windows7 64 bit box, with Visual Studio 2008 Express 
> Edition installed.
> 

Ok, try with the new commit I pushed.

Thanks,

Peter

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


[PUSHED] Better center status bar items between separators

2012-09-07 Thread Kohei Yoshida (via Code Review)
Hi,

Thank you for your patch! :-)  It has been merged to LibreOffice.

If you are interested in details, please visit

https://gerrit.libreoffice.org/298

Approvals:
  Kohei Yoshida: Verified; Looks good to me, approved


--
To view, visit https://gerrit.libreoffice.org/298
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I24553dda11b13c323b1a14813530175cdc563f8b
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Björn Michaelsen 
Gerrit-Reviewer: Jan Holesovsky 
Gerrit-Reviewer: Kohei Yoshida 

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


[Libreoffice-commits] .: vcl/source

2012-09-07 Thread Libreoffice Gerrit user
 vcl/source/window/status.cxx |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit e4b634fcd6420389fc77d0996d2351211804c865
Author: Ruslan Kabatsayev 
Date:   Sun Jul 8 21:20:07 2012 +0400

Better center status bar items between separators

Change-Id: I24553dda11b13c323b1a14813530175cdc563f8b
Reviewed-on: https://gerrit.libreoffice.org/298
Reviewed-by: Kohei Yoshida 
Tested-by: Kohei Yoshida 

diff --git a/vcl/source/window/status.cxx b/vcl/source/window/status.cxx
index 51c01c9..23c690a 100644
--- a/vcl/source/window/status.cxx
+++ b/vcl/source/window/status.cxx
@@ -479,10 +479,10 @@ void StatusBar::ImplDrawItem( sal_Bool bOffScreen, 
sal_uInt16 nPos, sal_Bool bDr
 {
 // draw separator
 Point aFrom( aRect.TopLeft() );
-aFrom.X()--;
+aFrom.X()-=4;
 aFrom.Y()++;
 Point aTo( aRect.BottomLeft() );
-aTo.X()--;
+aTo.X()-=4;
 aTo.Y()--;
 
 DecorationView aDecoView( this );
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: --headless broken with --enable-headless

2012-09-07 Thread Joop Kiefte
Run that command I would guess...

2012/9/7 John Smith :
> On Fri, Sep 7, 2012 at 8:33 PM, Riccardo Magliocchetti
>  wrote:
>> Does anyone can give it a try with a build without --enable-headless please?
> Urm. Do *what* exactly, without --enable-headless ?
>
> - John Smith.
> ___
> 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: --headless broken with --enable-headless

2012-09-07 Thread John Smith
On Fri, Sep 7, 2012 at 8:33 PM, Riccardo Magliocchetti
 wrote:
> Does anyone can give it a try with a build without --enable-headless please?
Urm. Do *what* exactly, without --enable-headless ?

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


Re: [Libreoffice-qa] How Can I Provide More Info For Developers

2012-09-07 Thread John Smith
On Fri, Sep 7, 2012 at 9:59 PM, Joel Madero  wrote:
> This isn't for end-users, it's for QA team..and it's our job to do as
> much as possible to help the developers. I know when I'm doing the
> development side I appreciate whatever previous info users and QA can
> provide. It's maximizing the efficiency of our abilitieslimited # of
> developers means we should be using their time wisely, not running
> backtraces that someone with 1/10th of their computer programming skills
> could manage just fine.
>
> Best Regards,
> Joel
>
Well, like I said: I guess it all depends on who your target audience
is. Just keep in mind who that is, and what their expected skill set
is, and adjust your procedures to that. IMHO.

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


[PATCH] OUString cleanup in accessibility

2012-09-07 Thread Ricardo Montania (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/583

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/83/583/1

OUString cleanup in accessibility

Change-Id: Ifceaa764231306c1f542183afc2e852cc418
---
M accessibility/source/extended/AccessibleBrowseBox.cxx
M accessibility/source/extended/AccessibleBrowseBoxBase.cxx
M accessibility/source/extended/AccessibleBrowseBoxCheckBoxCell.cxx
M accessibility/source/extended/AccessibleBrowseBoxHeaderBar.cxx
M accessibility/source/extended/AccessibleBrowseBoxHeaderCell.cxx
M accessibility/source/extended/AccessibleBrowseBoxTable.cxx
M accessibility/source/extended/AccessibleBrowseBoxTableBase.cxx
M accessibility/source/extended/AccessibleBrowseBoxTableCell.cxx
M accessibility/source/extended/AccessibleGridControl.cxx
M accessibility/source/extended/AccessibleGridControlBase.cxx
M accessibility/source/extended/AccessibleGridControlHeader.cxx
M accessibility/source/extended/AccessibleGridControlHeaderCell.cxx
M accessibility/source/extended/AccessibleGridControlTable.cxx
M accessibility/source/extended/AccessibleGridControlTableBase.cxx
M accessibility/source/extended/AccessibleGridControlTableCell.cxx
M accessibility/source/extended/AccessibleToolPanelDeck.cxx
M accessibility/source/extended/AccessibleToolPanelDeckTabBar.cxx
M accessibility/source/extended/AccessibleToolPanelDeckTabBarItem.cxx
M accessibility/source/extended/accessiblebrowseboxcell.cxx
M accessibility/source/extended/accessibleeditbrowseboxcell.cxx
M accessibility/source/extended/accessibleiconchoicectrl.cxx
M accessibility/source/extended/accessibleiconchoicectrlentry.cxx
M accessibility/source/extended/accessiblelistbox.cxx
M accessibility/source/extended/accessiblelistboxentry.cxx
M accessibility/source/extended/accessibletabbar.cxx
M accessibility/source/extended/accessibletabbarpage.cxx
M accessibility/source/extended/accessibletabbarpagelist.cxx
M accessibility/source/extended/accessibletablistboxtable.cxx
M accessibility/source/extended/textwindowaccessibility.cxx
M accessibility/source/helper/accresmgr.cxx
M accessibility/source/helper/characterattributeshelper.cxx
M accessibility/source/standard/accessiblemenubasecomponent.cxx
M accessibility/source/standard/accessiblemenucomponent.cxx
M accessibility/source/standard/accessiblemenuitemcomponent.cxx
M accessibility/source/standard/vclxaccessiblebox.cxx
M accessibility/source/standard/vclxaccessiblebutton.cxx
M accessibility/source/standard/vclxaccessiblecheckbox.cxx
M accessibility/source/standard/vclxaccessiblecombobox.cxx
M accessibility/source/standard/vclxaccessibledropdowncombobox.cxx
M accessibility/source/standard/vclxaccessibledropdownlistbox.cxx
M accessibility/source/standard/vclxaccessibleedit.cxx
M accessibility/source/standard/vclxaccessiblefixedhyperlink.cxx
M accessibility/source/standard/vclxaccessiblefixedtext.cxx
M accessibility/source/standard/vclxaccessiblelist.cxx
M accessibility/source/standard/vclxaccessiblelistbox.cxx
M accessibility/source/standard/vclxaccessiblelistitem.cxx
M accessibility/source/standard/vclxaccessiblemenu.cxx
M accessibility/source/standard/vclxaccessiblemenubar.cxx
M accessibility/source/standard/vclxaccessiblemenuitem.cxx
M accessibility/source/standard/vclxaccessiblemenuseparator.cxx
M accessibility/source/standard/vclxaccessiblepopupmenu.cxx
M accessibility/source/standard/vclxaccessibleradiobutton.cxx
M accessibility/source/standard/vclxaccessiblescrollbar.cxx
M accessibility/source/standard/vclxaccessiblestatusbar.cxx
M accessibility/source/standard/vclxaccessiblestatusbaritem.cxx
M accessibility/source/standard/vclxaccessibletabcontrol.cxx
M accessibility/source/standard/vclxaccessibletabpage.cxx
M accessibility/source/standard/vclxaccessibletextfield.cxx
M accessibility/source/standard/vclxaccessibletoolbox.cxx
M accessibility/source/standard/vclxaccessibletoolboxitem.cxx
60 files changed, 559 insertions(+), 605 deletions(-)


--
To view, visit https://gerrit.libreoffice.org/583
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifceaa764231306c1f542183afc2e852cc418
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Ricardo Montania 

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


[PATCH] OUString cleanup in animations

2012-09-07 Thread Ricardo Montania (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/584

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/84/584/1

OUString cleanup in animations

Change-Id: Ib858f9fa709d72539f71a6473a4ecb519d68e98e
---
M animations/source/animcore/animcore.cxx
M animations/source/animcore/factreg.hxx
M animations/source/animcore/targetpropertiescreator.cxx
3 files changed, 22 insertions(+), 23 deletions(-)


--
To view, visit https://gerrit.libreoffice.org/584
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib858f9fa709d72539f71a6473a4ecb519d68e98e
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Ricardo Montania 

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


Re: [Libreoffice-qa] How Can I Provide More Info For Developers

2012-09-07 Thread Joel Madero
This isn't for end-users, it's for QA team..and it's our job to do as
much as possible to help the developers. I know when I'm doing the
development side I appreciate whatever previous info users and QA can
provide. It's maximizing the efficiency of our abilitieslimited # of
developers means we should be using their time wisely, not running
backtraces that someone with 1/10th of their computer programming skills
could manage just fine.

Best Regards,
Joel

On Fri, Sep 7, 2012 at 12:56 PM, John Smith  wrote:

> On Fri, Sep 7, 2012 at 9:49 PM, Joel Madero  wrote:
> > I'll bring this up at our next conference call and see if there is a
> > possible solution.
> >
> >
> > Best Regards,
> > Joel
> >
> >
> May I humbly note that I personally feel that developers should be
> able to produce their own backtraces, given a solid reproducible
> test-case in the bug report ? Perhaps effort would be better spend on:
>
> 1.)
> teaching end-users how to provide a reproducible test case in a bug report
>
> 2.)
> teaching devs on how to produce backtraces
>
> instead of:
>
> 1.)
> teaching end-users how to install symbol binaries and backtrace them
> on their platform ?
>
>
> Just my 2$
>
>
> - John Smith.
>
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice-qa] How Can I Provide More Info For Developers

2012-09-07 Thread John Smith
On Fri, Sep 7, 2012 at 9:49 PM, Joel Madero  wrote:
> I'll bring this up at our next conference call and see if there is a
> possible solution.
>
>
> Best Regards,
> Joel
>
>
May I humbly note that I personally feel that developers should be
able to produce their own backtraces, given a solid reproducible
test-case in the bug report ? Perhaps effort would be better spend on:

1.)
teaching end-users how to provide a reproducible test case in a bug report

2.)
teaching devs on how to produce backtraces

instead of:

1.)
teaching end-users how to install symbol binaries and backtrace them
on their platform ?


Just my 2$


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


[Libreoffice-commits] .: desktop/source

2012-09-07 Thread Libreoffice Gerrit user
 desktop/source/app/cmdlineargs.cxx |3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit bf8f009cdb233b130363a808e194026f4dd84fc5
Author: Riccardo Magliocchetti 
Date:   Fri Sep 7 21:53:59 2012 +0200

desktop: enable --invisible by default when built with --enable-headless

Change-Id: I560759b91c5d37079a2b0d748cff5acf0a81ae4f

diff --git a/desktop/source/app/cmdlineargs.cxx 
b/desktop/source/app/cmdlineargs.cxx
index f836028..83b728f 100644
--- a/desktop/source/app/cmdlineargs.cxx
+++ b/desktop/source/app/cmdlineargs.cxx
@@ -591,11 +591,12 @@ bool CommandLineArgs::InterpretCommandLineParameter( 
const ::rtl::OUString& aArg
 void CommandLineArgs::InitParamValues()
 {
 m_minimized = false;
-m_invisible = false;
 m_norestore = false;
 #ifdef LIBO_HEADLESS
+m_invisible = true;
 m_headless = true;
 #else
+m_invisible = false;
 m_headless = false;
 #endif
 m_quickstart = false;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: [Libreoffice-qa] How Can I Provide More Info For Developers

2012-09-07 Thread Joel Madero
I'll bring this up at our next conference call and see if there is a
possible solution.


Best Regards,
Joel

On Fri, Sep 7, 2012 at 12:48 PM, John Smith  wrote:

> On Fri, Sep 7, 2012 at 9:42 PM, Joel Madero  wrote:
> >
> >
> > Sounds good, but I agree with John that better instructions are needed.
> I'm
> > working on a new triage page and hopefully in there we can get someone to
> > contribute incredibly precise, easy for noobs, instructions for every
> type
> > of backtrace/debug. Right now they are really written by experts for
> experts
> > (IMO). I would backtrace much more frequently if I could follow the
> > instructions on how to do it. Every time I feel like I have to go on IRC
> and
> > ask 100 questions.
> >
> >
> >
> >
> >
> >
> > I don't think we have enough people packaging to make pre-built binaries
> for
> > everyone. Plus, these are huge, my install is 22 gigs with all symbols
> on,
> > not very good for packaging purposes.
> >
> I guess it all depends on who your target audience is. If you really
> want end-users (libre office users) to provide backtraces, you really
> need to make the threshold for doing that as low as possible. I dont
> think it is reasonable to have end-users compile source code for
> themselves; especially on windows, where it requires msvc or cygwin to
> do so.
>
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice-qa] How Can I Provide More Info For Developers

2012-09-07 Thread John Smith
On Fri, Sep 7, 2012 at 9:42 PM, Joel Madero  wrote:
>
>
> Sounds good, but I agree with John that better instructions are needed. I'm
> working on a new triage page and hopefully in there we can get someone to
> contribute incredibly precise, easy for noobs, instructions for every type
> of backtrace/debug. Right now they are really written by experts for experts
> (IMO). I would backtrace much more frequently if I could follow the
> instructions on how to do it. Every time I feel like I have to go on IRC and
> ask 100 questions.
>
>
>
>
>
>
> I don't think we have enough people packaging to make pre-built binaries for
> everyone. Plus, these are huge, my install is 22 gigs with all symbols on,
> not very good for packaging purposes.
>
I guess it all depends on who your target audience is. If you really
want end-users (libre office users) to provide backtraces, you really
need to make the threshold for doing that as low as possible. I dont
think it is reasonable to have end-users compile source code for
themselves; especially on windows, where it requires msvc or cygwin to
do so.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice-qa] How Can I Provide More Info For Developers

2012-09-07 Thread khagaroth
The best solution would be to have a debug symbol server, then there
would be easily accessible debug symbols even for the release builds.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice-qa] How Can I Provide More Info For Developers

2012-09-07 Thread Joel Madero
On Fri, Sep 7, 2012 at 10:44 AM, John Smith  wrote:

> On Fri, Sep 7, 2012 at 7:24 PM, Michael Meeks 
> wrote:
> > Hi John,
> >
> > You can always make it easier for a developer to solve - and the
> easier
> > it is, the more likely it is to get solved quickly. Ways to do that are:
> >
> > * getting a stack-trace with full debugging symbols
> >
> > * running valgrind with full debugging symbols and
> >   attaching a trace.
> >
>

Sounds good, but I agree with John that better instructions are needed. I'm
working on a new triage page and hopefully in there we can get someone to
contribute incredibly precise, easy for noobs, instructions for every type
of backtrace/debug. Right now they are really written by experts for
experts (IMO). I would backtrace much more frequently if I could follow the
instructions on how to do it. Every time I feel like I have to go on IRC
and ask 100 questions.






> If this is the case, maybe pre-build binaries with full debugging
> symbols should be made available for download (more) easily: both for
> releases and daily master builds ? Also, a 'howto' on how to do this
> (on linux, with gdb, on windows, with windbg ?) would be nice to have
> ?
>

I don't think we have enough people packaging to make pre-built binaries
for everyone. Plus, these are huge, my install is 22 gigs with all symbols
on, not very good for packaging purposes.

Best Regards,
Joel



>
>
> - John Smith.
>
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] Changes to 'feature/base-preview'

2012-09-07 Thread Libreoffice Gerrit user
New branch 'feature/base-preview' available with the following commits:
commit 047dfc3c6e5f92bb24464e61dcdb7387890fa207
Author: Lionel Elie Mamane 
Date:   Tue Jul 24 17:59:11 2012 +0200

janitorial: C++-style cast instead of legacy C-style cast

Change-Id: I86cd5d87751c5a9dd31a392e2b3847e8f5741524

commit 2e55ab2b62834b5e291d5202e2f53ba2761a3ca4
Author: Lionel Elie Mamane 
Date:   Mon Jul 23 16:05:25 2012 +0200

TODO fdo#52179

Change-Id: I3144087cfe60b15a38853460f15a39ec730f0bf4

commit 34bede65812c70a1c63b7e5895b73ef50e9e0eac
Author: Lionel Elie Mamane 
Date:   Tue Jul 24 17:15:55 2012 +0200

forcing repaint has catastrophic performance implications; invalidate only

In Base:
1) Open a big table (hundreds of rows)
2) Only the first 40 to 100 rows are fetched into the cache and shown
   (depending on screen / window size).
3) Move to last row (click on icon |>|)
4) The last 40 to 100 rows are fetched into the cache; the data of the
   first 40/100 rows is pushed out of the cache.
5) The first 40 to 100 rows are fetched, just so that the call to
   rWindow.Paint() can do its job, because they are not in the cache
   anymore (!). The last 40/100 rows are pushed out of the cache.
6) The last 40 to 100 rows are fetched into the cache and displayed;
   the first 40/100 rows are pushed out of the cache *again*.

Steps 5 and 6 are obviously *very* stupid.

Change-Id: Ic11b893ea9440c2c5a142bd3c77c95d6730aa723

commit 66fa227f2b1bb9f1a8a963228c407d414dc463f3
Author: Terrence Enger 
Date:   Wed Jul 18 14:46:11 2012 -0400

stop some leaked statement handles

Change-Id: I06764e0569ea615e66de6cd5946614c7c538e60e
Signed-off-by: Lionel Elie Mamane 

commit 470d9922835a74b417287d3c302580722d219349
Author: Terrence Enger 
Date:   Wed Jul 18 14:46:11 2012 -0400

ODBMetaDataRS ctor: abort if statement handle allocation failed

Change-Id: Ieac069565bbc14c909eeecf3e67588191191992f
Signed-off-by: Lionel Elie Mamane 

commit ffb04635dbfd34cacd7874d1a9d1af79238e4ab1
Author: Terrence Enger 
Date:   Wed Jul 18 14:46:11 2012 -0400

avoid freeing the NULL handle

Change-Id: Id3f22bacfbf5e582656cc8ac38d60b781a25b4c3
Signed-off-by: Lionel Elie Mamane 

commit c237a94ab321d98919448402c8c2b9ca65399a6d
Author: Lionel Elie Mamane 
Date:   Mon Jul 16 23:58:18 2012 +0200

fdo#51239 refresh row lazily (when data is requested)

This avoids fetching data that will not be requested when the "cursor" is 
only moved and no data requested. That is typically what RowSetCache does when 
its own cursor is moved within its window.

This basically makes the whole {next,previous,absolute,...}_checked story 
obsolete, by basically always being as fast as the i_bFetchRow==false case, but 
in a safer way.

Change-Id: I89eaf277069736b3077bde8b45325929db290f2d

commit f33b2c1de33fe66413b03e6a28a70a482d2405e4
Author: Lionel Elie Mamane 
Date:   Thu Jul 12 23:28:10 2012 +0200

errors to stderr

Change-Id: I01756622dd7700d3918d156f118cd69c8a15879a

commit 69caae9dfaa37287fbab235fabeb2748c4dcefbc
Author: Lionel Elie Mamane 
Date:   Thu Jul 12 18:10:14 2012 +0200

debug unexpected exception

Change-Id: I404072caf6ddab0ed833586066507c7d332fcea4

commit 8dd4a862e659efb877ba3ca32679b14b4e5158b7
Author: Lionel Elie Mamane 
Date:   Thu Jul 12 13:07:03 2012 +0200

fix hsqldb java version detection

1) The values were not properly exclusive.
   E.g. when compiling under OpenJDK 1.6, iscjava12 was set (!)

2) On Java 1.6, should not enable the parts that are (only) for Java7!

3) When non-JAVA7, need to preprocess the files that have JAVA7-only code, 
so that it gets removed

Change-Id: I99b6bd9e781fee1007a04a4dc3d3f952c1b2eb00

commit 2540ca5f8ecbf0751e345db6604670ae06e87620
Author: Lionel Elie Mamane 
Date:   Tue Jul 3 20:17:01 2012 +0200

embedded HSQLDB: reclaim space occupied by deleted rows

Disadvantage is that saving/closing the file may take a significantly 
longer time

Change-Id: I1cc35c34b49e60d73e8d341549c5a3c5ef314b6d

commit 3c4428fdaa649c21d09f81bdfaa5688e0c7928b2
Author: Lionel Elie Mamane 
Date:   Mon Jul 2 19:33:38 2012 +0200

LEM notes

Change-Id: I16939e5d6e1f7c0a83f7f2b2dbc9a5b397b45c36

commit 1e4f693eb65e6eccd579d859543b63ee76722c6d
Author: Lionel Elie Mamane 
Date:   Mon Jul 2 14:48:03 2012 +0200

janitorial: remove unused xParamsAsNames variable

Change-Id: Ic4fe24faf75d38a8123a8f0e8304c054760bad85

commit c47c625d01feb357edb1ed937a2dfd5e7fab5cd1
Author: Lionel Elie Mamane 
Date:   Wed Jun 6 14:28:50 2012 +0200

legacy reports: unify treatment of query and table

In *both* cases, the value of hidden control "Sorting" (if non-empty)
decides the columns being sorted on.

Change-Id: I7f4b50c3af8c12e48e5dedd36b5877ad7a9e1b66

commit 74f934dcb65072c02

[Libreoffice-commits] .: Branch 'feature/base-preview' - 0 commits -

2012-09-07 Thread Libreoffice Gerrit user
Rebased ref, commits from common ancestor:
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


--headless broken with --enable-headless

2012-09-07 Thread Riccardo Magliocchetti

Hello,

i have libreoffice crashing early when running unoconv [1] test suite 
(cd tests && make). Attached gdbtrace.log


Libreoffice from yesterday master configured:

./autogen.sh --without-java --enable-headless

and started:
/usr/local/lib/libreoffice/program/soffice --backtrace 
"--accept=socket,host=localhost,port=2002;urp;"


Does anyone have a clue what can be broken? Does anyone can give it a 
try with a build without --enable-headless please?


Compiling latest master right now.

thanks,
riccardo

[1] https://github.com/dagwieers/unoconv
warning: Currently logging to gdbtrace.log.  Turn the logging off and on to make the new setting effective.
[Thread debugging using libthread_db enabled]
[New Thread 0x7fffec230700 (LWP 13053)]
[New Thread 0x7fffe99b2700 (LWP 13054)]
[New Thread 0x7fffe91b1700 (LWP 13055)]
[New Thread 0x7fffe8360700 (LWP 13056)]
[Thread 0x7fffe99b2700 (LWP 13054) exited]
[New Thread 0x7fffe99b2700 (LWP 13059)]
[New Thread 0x7fffdfb63700 (LWP 13060)]
[Thread 0x7fffdfb63700 (LWP 13060) exited]
[New Thread 0x7fffdfb63700 (LWP 13061)]
[New Thread 0x7fffddd32700 (LWP 13062)]
[Thread 0x7fffddd32700 (LWP 13062) exited]
[Thread 0x7fffe99b2700 (LWP 13059) exited]
[New Thread 0x7fffe99b2700 (LWP 13076)]
[New Thread 0x7fffddd32700 (LWP 13079)]
[New Thread 0x7fffdd2bd700 (LWP 13080)]
[New Thread 0x7fffdcabc700 (LWP 13081)]
[New Thread 0x7fffd64b2700 (LWP 13082)]
[Thread 0x7fffd64b2700 (LWP 13082) exited]
[New Thread 0x7fffd64b2700 (LWP 13083)]
[Thread 0x7fffd64b2700 (LWP 13083) exited]
[New Thread 0x7fffd64b2700 (LWP 13084)]

Program received signal SIGSEGV, Segmentation fault.
0x7616ec7e in com::sun::star::uno::WeakReferenceHelper::get() const () from /usr/local/lib/libreoffice/program/../ure-link/lib/libuno_cppuhelpergcc3.so.3
#0  0x7616ec7e in com::sun::star::uno::WeakReferenceHelper::get() const () from /usr/local/lib/libreoffice/program/../ure-link/lib/libuno_cppuhelpergcc3.so.3
#1  0x73727e0b in Window::~Window() () from /usr/local/lib/libreoffice/program/libvcllo.so
#2  0x7fffe76223f3 in framework::ToolBar::~ToolBar() () from /usr/local/lib/libreoffice/program/../program/libfwklo.so
#3  0x73730516 in vcl::LazyDeletor::~LazyDeletor() () from /usr/local/lib/libreoffice/program/libvcllo.so
#4  0x736932ff in vcl::LazyDelete::flush() () from /usr/local/lib/libreoffice/program/libvcllo.so
#5  0x73494525 in Application::Yield(bool) () from /usr/local/lib/libreoffice/program/libvcllo.so
#6  0x73494567 in Application::Execute() () from /usr/local/lib/libreoffice/program/libvcllo.so
#7  0x7791fd95 in desktop::Desktop::Main() () from /usr/local/lib/libreoffice/program/libsofficeapp.so
#8  0x7349deb9 in ImplSVMain() () from /usr/local/lib/libreoffice/program/libvcllo.so
#9  0x7349e7e5 in SVMain() () from /usr/local/lib/libreoffice/program/libvcllo.so
#10 0x779528c6 in soffice_main () from /usr/local/lib/libreoffice/program/libsofficeapp.so
#11 0x0040069b in main ()

Thread 16 (Thread 0x7fffd64b2700 (LWP 13084)):
#0  pthread_cond_timedwait@@GLIBC_2.3.2 () at ../nptl/sysdeps/unix/sysv/linux/x86_64/pthread_cond_timedwait.S:216
#1  0x77bb626c in osl_waitCondition () from /usr/local/lib/libreoffice/program/../ure-link/lib/libuno_sal.so.3
#2  0x7fffe9f07433 in configmgr::Components::WriteThread::execute() () from /usr/local/lib/libreoffice/program/../program/configmgr.uno.so
#3  0x75a7bcc6 in salhelper::Thread::run() () from /usr/local/lib/libreoffice/program/../ure-link/lib/libuno_salhelpergcc3.so.3
#4  0x75a7bf3a in threadFunc () from /usr/local/lib/libreoffice/program/../ure-link/lib/libuno_salhelpergcc3.so.3
#5  0x77b94a9f in osl_thread_start_Impl () from /usr/local/lib/libreoffice/program/../ure-link/lib/libuno_sal.so.3
#6  0x76ba9efc in start_thread (arg=0x7fffd64b2700) at pthread_create.c:304
#7  0x76ea089d in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:112
#8  0x in ?? ()

Thread 13 (Thread 0x7fffdcabc700 (LWP 13081)):
#0  pthread_cond_timedwait@@GLIBC_2.3.2 () at ../nptl/sysdeps/unix/sysv/linux/x86_64/pthread_cond_timedwait.S:216
#1  0x77bb626c in osl_waitCondition () from /usr/local/lib/libreoffice/program/../ure-link/lib/libuno_sal.so.3
#2  0x763a22ca in cppu_threadpool::ThreadPool::waitInPool(rtl::Reference const&) () from /usr/local/lib/libreoffice/program/../ure-link/lib/libuno_cppu.so.3
#3  0x763a118f in cppu_threadpool::ORequestThread::run() () from /usr/local/lib/libreoffice/program/../ure-link/lib/libuno_cppu.so.3
#4  0x763a165a in threadFunc () from /usr/local/lib/libreoffice/program/../ure-link/lib/libuno_cppu.so.3
#5  0x77b94a9f in osl_thread_start_Impl () from /usr/local/lib/libreoffice/program/../ure-link/lib/libuno_sal.so.3
#6  0x76ba9efc in start_thread (arg=0x7fffdcabc700) at pthread_create.c:304
#7  0x76ea089d in clone () at ../sysdeps/unix/sys

[Libreoffice-commits] .: 2 commits - desktop/CustomTarget_soffice.mk desktop/scripts scp2/source

2012-09-07 Thread Libreoffice Gerrit user
 desktop/CustomTarget_soffice.mk  |6 ++
 desktop/scripts/soffice.sh   |2 +-
 scp2/source/ooo/common_brand.scp |2 +-
 3 files changed, 8 insertions(+), 2 deletions(-)

New commits:
commit dae66b7b1df89d824017c7afb52f66ebfd2b8fe2
Author: Riccardo Magliocchetti 
Date:   Fri Sep 7 20:11:41 2012 +0200

desktop: use soffice.bin instead of oosplash if build with --enable-headless

Change-Id: I2e4742a6893267ea8d4ec2048dd0686fd309a7db

diff --git a/desktop/CustomTarget_soffice.mk b/desktop/CustomTarget_soffice.mk
index 74eb6c9..83b84ab 100644
--- a/desktop/CustomTarget_soffice.mk
+++ b/desktop/CustomTarget_soffice.mk
@@ -40,4 +40,10 @@ else
cp $< $@
 endif
 
+ifeq ($(strip $(ENABLE_HEADLESS)),TRUE)
+   sed -i "s/@SOFFICEBIN@/soffice.bin/" $@
+else
+   sed -i "s/@SOFFICEBIN@/oosplash/" $@
+endif
+
 # vim:set shiftwidth=4 tabstop=4 noexpandtab:
diff --git a/desktop/scripts/soffice.sh b/desktop/scripts/soffice.sh
index a259bf6..b0ba98d 100755
--- a/desktop/scripts/soffice.sh
+++ b/desktop/scripts/soffice.sh
@@ -175,4 +175,4 @@ for arg in $@ ; do
 done
 
 # oosplash does the rest: forcing pages in, javaldx etc. are
-exec $VALGRINDCHECK $STRACECHECK "$sd_prog/oosplash" "$@"
+exec $VALGRINDCHECK $STRACECHECK "$sd_prog/@SOFFICEBIN@" "$@"
commit 4d408cc2f09fcc645d0da736b74f2f5cae86edc4
Author: Riccardo Magliocchetti 
Date:   Fri Sep 7 18:21:06 2012 +0200

scp2: Don't deliver oosplash with --enable-headless

Change-Id: I6e12236d04e7ca19bea396e2aa79cba8280639a1

diff --git a/scp2/source/ooo/common_brand.scp b/scp2/source/ooo/common_brand.scp
index 968a550..5d89ae8 100644
--- a/scp2/source/ooo/common_brand.scp
+++ b/scp2/source/ooo/common_brand.scp
@@ -525,7 +525,7 @@ File gid_Brand_File_Bin_Soffice_Bin
 Name = "soffice.bin";
 End
 
-#if defined UNX && ! defined MACOSX
+#if defined UNX && ! defined MACOSX && ! defined LIBO_HEADLESS
 
 File gid_File_Bin_QStart_Bin
 BIN_FILE_BODY;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: [Libreoffice-qa] How Can I Provide More Info For Developers

2012-09-07 Thread John Smith
On Fri, Sep 7, 2012 at 7:24 PM, Michael Meeks  wrote:
> Hi John,
>
> You can always make it easier for a developer to solve - and the 
> easier
> it is, the more likely it is to get solved quickly. Ways to do that are:
>
> * getting a stack-trace with full debugging symbols
>
> * running valgrind with full debugging symbols and
>   attaching a trace.
>
If this is the case, maybe pre-build binaries with full debugging
symbols should be made available for download (more) easily: both for
releases and daily master builds ? Also, a 'howto' on how to do this
(on linux, with gdb, on windows, with windbg ?) would be nice to have
?


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


Re: [Libreoffice-qa] How Can I Provide More Info For Developers

2012-09-07 Thread Michael Meeks
Hi John,

On Fri, 2012-09-07 at 19:10 +0200, John Smith wrote:
> Maybe im just dumb, but: Once you have provided a reliable and
> reproducible test case (in this case, download the odt file attached
> to the report and save it as docx in libreoffice), is there still a
> need to provide further info at all ?

You can always make it easier for a developer to solve - and the easier
it is, the more likely it is to get solved quickly. Ways to do that are:

* getting a stack-trace with full debugging symbols

* running valgrind with full debugging symbols and
  attaching a trace.

For bonus points poking around at the code that's failing and thinking
hard about it - and (for some bugs) even a non-expert can see the
silliness by reading the relevant code in the backtrace, and looking at
the git history for that file ;-)

So anything to get more people drawn into development and bug fixing is
good :-) we have no-where near enough hackers / proctologists to match
the bugs (yet).

ATB,

Michael.

-- 
michael.me...@suse.com  <><, Pseudo Engineer, itinerant idiot

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


REMINDER: Release 3.6.2-rc1 from libreoffice-3-6 branch

2012-09-07 Thread Petr Mladek
Hi,

please note that the commit deadline for 3.6.2-rc1 is on Monday, September 10,
2012.

Andras, l10n teams, there as few more string to fix the nasty date-recognitions 
bugs[*].
Please, try to have them localized until Sep 23, 2012 when is the deadline for 
rc2.

The tag and branch libreoffice-3-6-1 is going to be created the day after.
Only commits with 3 reviewers will be allowed for that branch.

See also
http://wiki.documentfoundation.org/ReleasePlan#3.6_release
http://wiki.documentfoundation.org/Release_Criteria
http://wiki.documentfoundation.org/Development/Branches


Best Regards,
Petr

PS: Please, help with reviewing the pending commits.


[*] It is the commit
http://cgit.freedesktop.org/libreoffice/core/commit/?h=libreoffice-3-6&id=cfbfa26deb2776e5c07463e59517eaf68c1d5d6d
that solved bugs:
+ https://bugs.freedesktop.org/show_bug.cgi?id=52240
+ https://bugs.freedesktop.org/show_bug.cgi?id=52137
+ https://bugs.freedesktop.org/show_bug.cgi?id=52288
and many duplicates.


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


Re: How Can I Provide More Info For Developers

2012-09-07 Thread John Smith
On Fri, Sep 7, 2012 at 6:59 PM, Joel Madero  wrote:
> Wrong Bug, dammit, my bad. Here is the proper one, attachment is there and
> functioning, I had 5 open FDO tabs in FF and picked the wrong one.
>
> https://bugs.freedesktop.org/show_bug.cgi?id=48569
>
> My apologies
>
> Joel
>
Maybe im just dumb, but: Once you have provided a reliable and
reproducible test case (in this case, download the odt file attached
to the report and save it as docx in libreoffice), is there still a
need to provide further info at all ? The person tackling the bug can
get all the info required on his/her own system, then ?

Just my 2$.



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


Re: How Can I Provide More Info For Developers

2012-09-07 Thread Joel Madero
Wrong Bug, dammit, my bad. Here is the proper one, attachment is there and
functioning, I had 5 open FDO tabs in FF and picked the wrong one.

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

My apologies

Joel

On Fri, Sep 7, 2012 at 9:57 AM, John Smith  wrote:

> On Fri, Sep 7, 2012 at 6:38 PM, Joel Madero  wrote:
> > I don't see that helping much, as the developer who takes this one can
> > quickly do this themselves. A working doc only takes 2 seconds to make.
> >
> > Best Regards,
> > Joel
> >
> Another random thought then: Is there a way to reproduce the issue
> that doesnt require the installation of Lotus Domino ?
>
>
>
> - John Smith.
>
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: How Can I Provide More Info For Developers

2012-09-07 Thread John Smith
On Fri, Sep 7, 2012 at 6:38 PM, Joel Madero  wrote:
> I don't see that helping much, as the developer who takes this one can
> quickly do this themselves. A working doc only takes 2 seconds to make.
>
> Best Regards,
> Joel
>
Another random thought then: Is there a way to reproduce the issue
that doesnt require the installation of Lotus Domino ?



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


Re: Modern font features, hacky patch

2012-09-07 Thread Khaled Hosny
On Fri, Sep 07, 2012 at 05:18:46PM +0100, Caolán McNamara wrote:
> On Sat, 2012-07-21 at 18:20 +0200, Khaled Hosny wrote:
> > Here is a very crude, WIP patch. It adds --enable-harfbuzz configure
> > option, plus some (very broken) harfbuzz layout code, but I can't get
> > the ENABLE_HARFBUZZ to propagate in and thus can't do any testing
> > because the code is never compiled. I appreciate any insights on how to
> > get this to work.
> 
> You need to put harfbuzz in the gb_Library_use_externals section of
> Library_vcl.mk (or whichever is the right .mk file in vcl)
> 
> It would be nice to get this feature in as a compile time option for
> experimenting with.

I’ve some code that more or less works (as in displaying some text), but
it crashes like hell, I'm yet to figure out what is going on (I think
I've to override more members of ServerFontLayout to make things saner).

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


Re: How Can I Provide More Info For Developers

2012-09-07 Thread Joel Madero
I don't see that helping much, as the developer who takes this one can
quickly do this themselves. A working doc only takes 2 seconds to make.

Best Regards,
Joel

On Fri, Sep 7, 2012 at 9:35 AM, John Smith  wrote:

> On Fri, Sep 7, 2012 at 6:11 PM, Joel Madero  wrote:
> > Hi All,
> >
> > Came across this bug FDO#45570 and I've already nominated as a most
> annoying
> > bug but I think that devs might appreciate me getting some kind of a log
> or
> > something together for the crash. What type of log should I create and
> how
> > do I go about doing this (basic steps, my knowledge of terminology is
> still
> > in the first stages :) ). Thanks everyone
> >
> >
> >
> > Best Regards,
> > Joel
> >
>
> Just some random thoughts here: What happens when you create a
> 'working document', thats been opened and saved in Msoffice or
> OpenOffice, and then try to open that saved doc in LibreOffice ? Does
> the issue still appear ? Also, would it be helpful to attach such a
> document to the bug report for troubleshooting purposes ?
>
>
> - John Smith.
>
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: How Can I Provide More Info For Developers

2012-09-07 Thread John Smith
On Fri, Sep 7, 2012 at 6:11 PM, Joel Madero  wrote:
> Hi All,
>
> Came across this bug FDO#45570 and I've already nominated as a most annoying
> bug but I think that devs might appreciate me getting some kind of a log or
> something together for the crash. What type of log should I create and how
> do I go about doing this (basic steps, my knowledge of terminology is still
> in the first stages :) ). Thanks everyone
>
>
>
> Best Regards,
> Joel
>

Just some random thoughts here: What happens when you create a
'working document', thats been opened and saved in Msoffice or
OpenOffice, and then try to open that saved doc in LibreOffice ? Does
the issue still appear ? Also, would it be helpful to attach such a
document to the bug report for troubleshooting purposes ?


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


Re: How Can I Provide More Info For Developers

2012-09-07 Thread Stephan Bergmann

On 09/07/2012 06:11 PM, Joel Madero wrote:

Came across this bug FDO#45570 and I've already nominated as a most
annoying bug but I think that devs might appreciate me getting some kind
of a log or something together for the crash. What type of log should I
create and how do I go about doing this (basic steps, my knowledge of
terminology is still in the first stages :) ). Thanks everyone


One tip:  Please always provide a hyperlink and title when mentioning a 
bug in a mail, so readers can quickly tell whether it is relevant for 
them;  "Cannot embed 
Writer document in an IBM Lotus Notes rich text field."


Thanks,
Stephan

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


Re: Modern font features, hacky patch

2012-09-07 Thread Caolán McNamara
On Sat, 2012-07-21 at 18:20 +0200, Khaled Hosny wrote:
> Here is a very crude, WIP patch. It adds --enable-harfbuzz configure
> option, plus some (very broken) harfbuzz layout code, but I can't get
> the ENABLE_HARFBUZZ to propagate in and thus can't do any testing
> because the code is never compiled. I appreciate any insights on how to
> get this to work.

You need to put harfbuzz in the gb_Library_use_externals section of
Library_vcl.mk (or whichever is the right .mk file in vcl)

It would be nice to get this feature in as a compile time option for
experimenting with.

C.

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


Re: make_installer 'touching' files in solver ?

2012-09-07 Thread Stephan Bergmann

On 08/27/2012 08:53 AM, Stephan Bergmann wrote:

So looks like instsetoo_native still strips libraries, and does so
in-place in the solver.  (I at least lost track of the "when and what to
srip" discussion we had not too long ago.)


This was due to epm unconditionally being called without -g ("Don't 
strip executables in distributions"). 
 
"Prevent epm from stripping unless explicitly requested" fixes that.


However, one consequence of that fix is that "proper" installation sets 
(generated from --enable-epm builds in insetsetoo_native) are now always 
non-stripped (as the only case where make_installer.pl is called with 
-strip is from top-level "make install-strip" AFAIU), where before they 
were always stripped.  I have lost overview whether that matches 
expectations of people producing such installation sets (for 
 etc.).


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


How Can I Provide More Info For Developers

2012-09-07 Thread Joel Madero
Hi All,

Came across this bug FDO#45570 and I've already nominated as a most
annoying bug but I think that devs might appreciate me getting some kind of
a log or something together for the crash. What type of log should I create
and how do I go about doing this (basic steps, my knowledge of terminology
is still in the first stages :) ). Thanks everyone



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


Re: Access to http://dev-www.libreoffice.org/src

2012-09-07 Thread Kohei Yoshida

On 09/07/2012 11:31 AM, Kohei Yoshida wrote:

Hi,

Is there an instruction on how to upload a tarball on

http://dev-www.libreoffice.org/src


Nevermind, someone told me privately.

Kohei

--
Kohei Yoshida, LibreOffice hacker, Calc
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: Need help fixing fdo#38557

2012-09-07 Thread Niklas Johansson

Hi

No worries, it was my bad for failing to mention Impress in my original 
message. ;)


I've been stepping through the code a lot of times and well it seems to 
go over my head.


I found something interesting in the ViewShells [1] in the method 
UpdatePreview which is called from drviews3.cxx. The method only 
contains this comment.

// Do nothing.  After the actual preview has been removed,
// OutlineViewShell::UpdatePreview() is the place where something
// usefull is still done.

The method above would update the preview pages to the left I think so 
not exactly what I was aiming for. :(


[1] 
http://opengrok.libreoffice.org/xref/core/sd/source/ui/inc/ViewShell.hxx#223


Regards,
Niklas Johansson

Cedric Bosdonnat skrev 2012-09-07 11:26:

Hi Niklas,

Please excuse Michael as he redirected you to the wrong guy... this is
an Impress problem, better see with the Impress hackers ;) Thorsten,
Muthu, Radek could you help him?

--
Cedric

On Thu, 2012-09-06 at 17:22 +0100, Michael Meeks wrote:

Hi Niklas,

Sadly I'm no writer expert :-) did you get any further ? if not Cedric
is one person with a good overview there I suspect; Cedric ?

ATB,

Michael.

On Fri, 2012-08-31 at 17:40 +0200, Niklas Johansson wrote:

I'm currently looking at fdo#38557 [1] and could use some pointers.
Summary of the bug: After using the Header and footer dialog (View menu)
changes does not get displayed until you add a new page (or something
else that triggers an update of the page layout).

I tracked down the dialog to headerfooterdlg.cxx [2] the
HeaderFooterDialog:apply is called from ApplyToAll or Apply depending on
witch button you press.

In HeaderFooterDialog::apply the properties from the dialog is set but
it does not update the view on screen.

Where is such an update supposed to happen? In the execute function? Is
it even supposed to be handled in headerfooterdlg.cxx or do I have to go
deeper?


[1] https://bugs.freedesktop.org/show_bug.cgi?id=38557
[2]
http://opengrok.libreoffice.org/xref/core/sd/source/ui/dlg/headerfooterdlg.cxx#304





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


Re: Triage Project Update

2012-09-07 Thread Joel Madero
Hi Nino,

That's the beauty of our project, everyone's opinion is respected :) I'll
try to avoid adding any work to website team and see what method works best
to get these triaged and organized best. As of now, google doc + fdo seems
to be doing the trick :) Ultimately might just make a macro to auto sort
them once every other week to keep the google doc updated until the
"project" is done (ie. getting all bugs >30 days old triaged so we can try
to stick with a goal of triaging withing 30 days for new bugs). Thanks for
all the valuable input.


Best Regards,
Joel

On Tue, Sep 4, 2012 at 3:07 PM, Nino Novak  wrote:

> Am 04.09.2012 23:05 schrieb Joel Madero:
> > I agree that FDO has some benefits but the limitation is really that
> each user
> > is needed to query every time,  the possibility of overlap is great, and
> no one
> > is really responsible for an individual bug until the query is made and
> someone
> > takes the time to look into it. I'm not sure if others would agree but
> it seems
> > like having a "group" of 50 or so and being able to just do those at your
> > convenience makes people more likely to help and feel like their is an
> end in
> > sight for "their portion". This is vs. just seeing a never ending list
> from FDO
> > or even having to "teach" new users (or even not new users) exactly what
> to
> > search for every time with FDO.
>
> As for me (a rather unexperienced QA Newbie), I've chosen a somewhat
>  different
> approach: I've first created two custom searches,
>
> 1) all recent bugs (reported within the last two days) for curiosity (just
> to
> see what people report recently)
>
> 2) all UNCONFIRMED bugs from the last 14 days
>
> From query 2 I picked a couple of bugs every couple of days to
> reproduce/confirm/assign/close/whatever seemed appropriate.
>
> That's just to show a slightly different approach, which is rather simple
> and
> can be handled perfectly within bugzilla itself without any external tool.
>
> Ok, the only problem was, that when a person starts reproducing a bug, it
> can
> happen, that another triager just starts with the very same bug at the same
> time. So some kind of lock signal was the only missing thing to prevent
> duplication of work. However, this situation did not happen a single time
> during
> my self-chosen "BugReviewWeek" ;-)
>
> Another advantage: By the above process nobody (virtually) "blocks" 50
> bugs for
> a longer time period. Bugzilla queries are very adequate at every time, as
> all
> works with live data.
>
>
> > Similar to how developers assign themselves bugs and then can just go
> look at
> > their own bugs ("My Bugs") it would be nice to have this ability for QA
> triagers
> > but have it somewhat automated since it's just triaging, not
> programming. In the
> > long run (once we're through the back log of 650+ that are really old),
> it would
> > be amazing if we had a team of QA staff that signed up to have bugs "auto
> > assigned" to them for triaging.
>
> We have the libreoffice-bugs@fdo mailing list, which contains (nearly?)
> every
> new bug. Could we use it somehow for this purpose? E.g. by replying to a
> bug or
> forwarding it to the qa list or some such? (Just thoughts, nothing
> concrete)
>
>
>
>  What I imagine:
> >
> > QA triagers "sign up" for components they are willing to triage and
> their "max"
> > load
> > New bug is reported, if the bug has a component listed the bug gets "auto
> > assigned" for triaging purposes according to some rule(s)
>
> Personally, I prefer not to sign up for a special component but to pick a
> recent
> bug which kind of "attracts" me spontanously. But there might be other
> opinions/preferences/arguments/approaches.
>
>
> > For now the google docs works, FDO does not as it is now but I'll
> discuss this
> > further with Bjoern, Petr & Rainer to see if we can come up with
> something more
> > functional than the chaos that is FDO :) Or maybe I'm just not familiar
> enough
> > with FDO to really feel comfortable myself with it, this is more likely
> than not
> > true :)
>
> :-)
>
> I like your initiative. Please don't feel discouraged by my comments, I
> just
> wanted to add a slightly different view. If people like your approach,
> that's
> great! It does not contradict to mine (IMHO), as it's rather obvious if a
> bug
> has been triaged or not. So we can all work together towards our common
> goal.
>
> Regards,
> Nino
>
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: solenv/bin

2012-09-07 Thread Libreoffice Gerrit user
 solenv/bin/modules/installer/epmfile.pm |2 ++
 1 file changed, 2 insertions(+)

New commits:
commit 2ad7f82e4af8a894b42157ccaabfeb90870d080f
Author: Stephan Bergmann 
Date:   Fri Sep 7 18:01:54 2012 +0200

Prevent epm from stripping unless explicitly requested

Change-Id: I9f89d78d5772a1a60c89ab4aa4b6ef9bd6179754

diff --git a/solenv/bin/modules/installer/epmfile.pm 
b/solenv/bin/modules/installer/epmfile.pm
index 7425a78..8045536 100644
--- a/solenv/bin/modules/installer/epmfile.pm
+++ b/solenv/bin/modules/installer/epmfile.pm
@@ -829,6 +829,8 @@ sub call_epm
 my $extraflags = "";
 if ($ENV{'EPM_FLAGS'}) { $extraflags = $ENV{'EPM_FLAGS'}; }
 
+$extraflags .= ' -g' unless $installer::globals::strip;
+
 my $systemcall = $ldpreloadstring . $epmname . " -f " . $packageformat . " 
" . $extraflags . " " . $localpackagename . " " . $epmlistfilename . 
$outdirstring . " -v " . " 2\>\&1 |";
 
 installer::logger::print_message( "... $systemcall ...\n" );
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: sw/source

2012-09-07 Thread Libreoffice Gerrit user
 sw/source/core/doc/doc.cxx   |4 
 sw/source/filter/xml/xmlmeta.cxx |2 --
 2 files changed, 6 deletions(-)

New commits:
commit 2195e0c699be6093f200ba6225a80ec4b9a545b8
Author: Caolán McNamara 
Date:   Fri Sep 7 16:47:32 2012 +0100

Resolves: fdo#53590 you can trust no one to tell you the truth

Change-Id: I6850d2060c1c7490c15c8eff7e0338433e8dab32

diff --git a/sw/source/core/doc/doc.cxx b/sw/source/core/doc/doc.cxx
index f7b5bfd..8b439b3 100644
--- a/sw/source/core/doc/doc.cxx
+++ b/sw/source/core/doc/doc.cxx
@@ -1941,10 +1941,6 @@ void SwDoc::ResetModified()
 //  Bit 1:  -> new state
 long nCall = mbModified ? 1 : 0;
 mbModified = sal_False;
-// If there is already a document statistic, we assume that
-// it is correct. In this case we reset the modified flag.
-if ( 0 != pDocStat->nCharExcludingSpaces )
-pDocStat->bModified = sal_False;
 GetIDocumentUndoRedo().SetUndoNoModifiedPosition();
 if( nCall && aOle2Link.IsSet() )
 {
diff --git a/sw/source/filter/xml/xmlmeta.cxx b/sw/source/filter/xml/xmlmeta.cxx
index 765ac10..0b80ece 100644
--- a/sw/source/filter/xml/xmlmeta.cxx
+++ b/sw/source/filter/xml/xmlmeta.cxx
@@ -159,8 +159,6 @@ void SwXMLImport::SetStatistics(
 }
 }
 
-if( 255 == nTokens )
-aDocStat.bModified = sal_False;
 if( nTokens )
 pDoc->SetDocStat( aDocStat );
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: 2 commits - sal/qa vcl/generic vcl/source

2012-09-07 Thread Libreoffice Gerrit user
 sal/qa/rtl/strings/test_oustring_startswith.cxx |   37 +++-
 vcl/generic/fontmanager/fontmanager.cxx |3 -
 vcl/source/app/svmain.cxx   |5 +++
 3 files changed, 11 insertions(+), 34 deletions(-)

New commits:
commit 63a4eac4269910d577b9fae7f48d3bd946d08a1f
Author: Luboš Luňák 
Date:   Fri Sep 7 17:34:28 2012 +0200

move temporary font cleaning to a place that is run on all platforms

Change-Id: Idc647684772b53d36a364afd91d9695210545e19

diff --git a/vcl/generic/fontmanager/fontmanager.cxx 
b/vcl/generic/fontmanager/fontmanager.cxx
index 209970d..0fd8d28 100644
--- a/vcl/generic/fontmanager/fontmanager.cxx
+++ b/vcl/generic/fontmanager/fontmanager.cxx
@@ -1035,7 +1035,6 @@ PrintFontManager::~PrintFontManager()
 delete m_pAtoms;
 if( m_pFontCache )
 delete m_pFontCache;
-TemporaryFonts::clear();
 }
 
 // -
@@ -1662,8 +1661,6 @@ void PrintFontManager::initialize()
 CALLGRIND_ZERO_STATS();
 #endif
 
-TemporaryFonts::clear();
-
 long aDirEntBuffer[ (sizeof(struct dirent)+_PC_NAME_MAX)+1 ];
 
 if( ! m_pFontCache )
diff --git a/vcl/source/app/svmain.cxx b/vcl/source/app/svmain.cxx
index 0bcdb64..fe895a5 100644
--- a/vcl/source/app/svmain.cxx
+++ b/vcl/source/app/svmain.cxx
@@ -49,6 +49,7 @@
 #include "vcl/unowrap.hxx"
 #include "vcl/configsettings.hxx"
 #include "vcl/lazydelete.hxx"
+#include "vcl/temporaryfonts.hxx"
 
 #ifdef WNT
 #include 
@@ -257,6 +258,8 @@ sal_Bool InitVCL( const ::com::sun::star::uno::Reference< 
::com::sun::star::lang
 if( pExceptionHandler != NULL )
 return sal_False;
 
+TemporaryFonts::clear();
+
 if( ! ImplGetSVData() )
 ImplInitSVData();
 
@@ -594,6 +597,8 @@ void DeInitVCL()
 delete pOwnSvApp;
 pOwnSvApp = NULL;
 }
+
+TemporaryFonts::clear();
 }
 
 // only one call is allowed
commit ff97ebe1cac06fe6887854b1daa6dbf23b968cd2
Author: Luboš Luňák 
Date:   Fri Sep 7 16:29:44 2012 +0200

use new file header, this is actually a new file

Change-Id: If12e05b0d46fbbd815b9af5dc9551e2f07d859be

diff --git a/sal/qa/rtl/strings/test_oustring_startswith.cxx 
b/sal/qa/rtl/strings/test_oustring_startswith.cxx
index a16a244..fc4a414 100644
--- a/sal/qa/rtl/strings/test_oustring_startswith.cxx
+++ b/sal/qa/rtl/strings/test_oustring_startswith.cxx
@@ -1,41 +1,16 @@
 /* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
-/*
+/*
+ * This file is part of the LibreOffice project.
  *
- * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
- *
- * Copyright 2000, 2010 Oracle and/or its affiliates.
- *
- * OpenOffice.org - a multi-platform office productivity suite
- *
- * This file is part of OpenOffice.org.
- *
- * OpenOffice.org is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License version 3
- * only, as published by the Free Software Foundation.
- *
- * OpenOffice.org is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License version 3 for more details
- * (a copy is included in the LICENSE file that accompanied this code).
- *
- * You should have received a copy of the GNU Lesser General Public License
- * version 3 along with OpenOffice.org.  If not, see
- * 
- * for a copy of the LGPLv3 License.
- *
- /
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/.
+ */
 
 #include 
 #include 
 #include 
-#include "rtl/strbuf.hxx"
-#include "rtl/string.h"
-#include "rtl/string.hxx"
-#include "rtl/textenc.h"
 #include "rtl/ustring.hxx"
-#include "sal/types.h"
-#include 
 
 namespace test { namespace oustring {
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Access to http://dev-www.libreoffice.org/src

2012-09-07 Thread Kohei Yoshida

Hi,

Is there an instruction on how to upload a tarball on

http://dev-www.libreoffice.org/src

so that I can upload tarballs required for the build instead of always 
bugging other people to do it?


Thanks,

Kohei

--
Kohei Yoshida, LibreOffice hacker, Calc
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: liborcus/makefile.mk

2012-09-07 Thread Libreoffice Gerrit user
 liborcus/makefile.mk |8 +---
 1 file changed, 1 insertion(+), 7 deletions(-)

New commits:
commit a6610e9e48327d2f4cc2a0c1cc0dea3e2f199917
Author: Kohei Yoshida 
Date:   Fri Sep 7 11:18:00 2012 -0400

Revert "Specify boost header directory for internal boost."

This reverts commit f721ae074deef795a32ad28289286b5d2667af0d.

diff --git a/liborcus/makefile.mk b/liborcus/makefile.mk
index 0f6b72c..fd04605 100644
--- a/liborcus/makefile.mk
+++ b/liborcus/makefile.mk
@@ -53,13 +53,7 @@ MY_CXXFLAGS = CXXFLAGS=-I$(OUTDIR)/inc/external
 .END
 
 CONFIGURE_DIR=
-CONFIGURE_ACTION=
-
-.IF "$(SYSTEM_BOOST)" != "YES"
-CONFIGURE_ACTION+=export CPPFLAGS="$(CPPFLAGS) -I$(OUTDIR)/inc/external" &&
-.ENDIF
-
-CONFIGURE_ACTION+=./configure \
+CONFIGURE_ACTION=./configure \
--with-pic \
--enable-static \
--disable-shared \
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: liborcus/makefile.mk

2012-09-07 Thread Libreoffice Gerrit user
 liborcus/makefile.mk |8 +++-
 1 file changed, 7 insertions(+), 1 deletion(-)

New commits:
commit f721ae074deef795a32ad28289286b5d2667af0d
Author: Kohei Yoshida 
Date:   Fri Sep 7 11:15:54 2012 -0400

Specify boost header directory for internal boost.

Change-Id: I60e8533552161933f48a4dfbe17e5108f77fa2eb

diff --git a/liborcus/makefile.mk b/liborcus/makefile.mk
index fd04605..0f6b72c 100644
--- a/liborcus/makefile.mk
+++ b/liborcus/makefile.mk
@@ -53,7 +53,13 @@ MY_CXXFLAGS = CXXFLAGS=-I$(OUTDIR)/inc/external
 .END
 
 CONFIGURE_DIR=
-CONFIGURE_ACTION=./configure \
+CONFIGURE_ACTION=
+
+.IF "$(SYSTEM_BOOST)" != "YES"
+CONFIGURE_ACTION+=export CPPFLAGS="$(CPPFLAGS) -I$(OUTDIR)/inc/external" &&
+.ENDIF
+
+CONFIGURE_ACTION+=./configure \
--with-pic \
--enable-static \
--disable-shared \
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: 2 commits - liborcus/liborcus_0.1.0-warnings.patch liborcus/makefile.mk

2012-09-07 Thread Libreoffice Gerrit user
 liborcus/liborcus_0.1.0-warnings.patch |   62 +
 liborcus/makefile.mk   |9 +++-
 2 files changed, 69 insertions(+), 2 deletions(-)

New commits:
commit 127d842cbb41db987ac5d03ca78520a32b6fb206
Author: Stephan Bergmann 
Date:   Fri Sep 7 17:04:51 2012 +0200

Fix liborcus warnings

Change-Id: I6811e7d451c96d5e52e152e693b1c4739be308dc

diff --git a/liborcus/liborcus_0.1.0-warnings.patch 
b/liborcus/liborcus_0.1.0-warnings.patch
new file mode 100644
index 000..a7e796a
--- /dev/null
+++ b/liborcus/liborcus_0.1.0-warnings.patch
@@ -0,0 +1,62 @@
+--- misc/liborcus_0.1.0/src/liborcus/pstring.cpp   2012-09-07 
06:00:35.0 +0200
 misc/build/liborcus_0.1.0/src/liborcus/pstring.cpp 2012-09-07 
16:56:14.0 +0200
+@@ -57,25 +57,25 @@
+ 
+ pstring pstring::intern(const char* str, size_t n)
+ {
+-::boost::mutex::scoped_lock(interned_strings.mtx);
++::boost::mutex::scoped_lock lock(interned_strings.mtx);
+ return interned_strings.store.intern(str, n);
+ }
+ 
+ void pstring::intern::dispose()
+ {
+-::boost::mutex::scoped_lock(interned_strings.mtx);
++::boost::mutex::scoped_lock lock(interned_strings.mtx);
+ interned_strings.store.clear();
+ }
+ 
+ size_t pstring::intern::size()
+ {
+-::boost::mutex::scoped_lock(interned_strings.mtx);
++::boost::mutex::scoped_lock lock(interned_strings.mtx);
+ return interned_strings.store.size();
+ }
+ 
+ void pstring::intern::dump()
+ {
+-::boost::mutex::scoped_lock(interned_strings.mtx);
++::boost::mutex::scoped_lock lock(interned_strings.mtx);
+ interned_strings.store.dump();
+ }
+ 
+--- misc/liborcus_0.1.0/src/liborcus/xml_map_tree.cpp  2012-09-07 
06:00:35.0 +0200
 misc/build/liborcus_0.1.0/src/liborcus/xml_map_tree.cpp2012-09-07 
16:59:23.0 +0200
+@@ -55,15 +55,14 @@
+ {
+ const char* mp_char;
+ const char* mp_end;
+-size_t m_size;
+ public:
+-xpath_parser(const char* p, size_t n) : mp_char(p), mp_end(p+n), m_size(n)
++xpath_parser(const char* p, size_t n) : mp_char(p), mp_end(p+n)
+ {
+ if (!n)
+-xml_map_tree::xpath_error("empty path");
++throw xml_map_tree::xpath_error("empty path");
+ 
+ if (*p != '/')
+-xml_map_tree::xpath_error("first character must be '/'.");
++throw xml_map_tree::xpath_error("first character must be '/'.");
+ 
+ ++mp_char;
+ }
+@@ -414,7 +413,7 @@
+ {
+ // Make sure the root element's names are the same.
+ if (mp_root->name != name)
+-xpath_error("path begins with inconsistent root level name.");
++throw xpath_error("path begins with inconsistent root level 
name.");
+ }
+ else
+ {
diff --git a/liborcus/makefile.mk b/liborcus/makefile.mk
index 8d92109..fd04605 100644
--- a/liborcus/makefile.mk
+++ b/liborcus/makefile.mk
@@ -35,7 +35,8 @@ TARGET=orcus
 TARFILE_NAME=liborcus_0.1.0
 TARFILE_MD5=46d9f4cf8b145c21ce1056e116d2ce71
 
-PATCH_FILES=
+PATCH_FILES=liborcus_0.1.0-warnings.patch
+# -Werror,-Wunused-variable -Werror,-Wunused-private-field
 
 .IF "$(GUI)$(COM)"=="WNTMSC"
 
commit f006e75f1876bf7e6249352092d0ed7e7a5be20f
Author: Stephan Bergmann 
Date:   Fri Sep 7 17:01:54 2012 +0200

Make liborcus work --without-system-boost

Change-Id: I4e199c167a67ca73c1967a7cf45bb8cc5b0017e7

diff --git a/liborcus/makefile.mk b/liborcus/makefile.mk
index b108fce..8d92109 100644
--- a/liborcus/makefile.mk
+++ b/liborcus/makefile.mk
@@ -47,6 +47,10 @@ BUILD_ACTION= \
 
 .ELSE
 
+.IF "$(SYSTEM_BOOST)" == "NO"
+MY_CXXFLAGS = CXXFLAGS=-I$(OUTDIR)/inc/external
+.END
+
 CONFIGURE_DIR=
 CONFIGURE_ACTION=./configure \
--with-pic \
@@ -54,7 +58,7 @@ CONFIGURE_ACTION=./configure \
--disable-shared \
--without-libzip \
--disable-debug \
-   --disable-spreadsheet-model
+   --disable-spreadsheet-model $(MY_CXXFLAGS)
 
 BUILD_ACTION=make
 BUILD_DIR=
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: [PROPOSED] Changes to General Options FDO#40656

2012-09-07 Thread Caolán McNamara
On Fri, 2012-09-07 at 07:54 -0700, Joel Madero wrote:

> As for your other comment about changing name of help, how do I go
> about doing this? I've never messed with the help stuff before so I'll
> need a bit of guidance.

Its kind of gruesome, I generally just get the help pages up and cut and
paste some likely unique text from them and grep for that in
helpcontent2/source. e.g. the help page for options has a "Java" entry
in it and "Use this dialog to create general settings as the top", so
grepped for the second string and get
source/text/shared/optionen/0101.xhp and so on. Its an xml format so
I just tend to copy and paste bits of it around. the "id"s you'll see in
the source apparently have to be unique. If I need a new one I pick a
random number and grep to make sure its unique.

C.

> 
> Also now that Java page has changed to Advanced, does it make sense to
> have files named "optjavacxx,src, etc..."? Should I consider
> changing the names to something more sane since it's not just java?
> Also the page name is now not really complete (svxJavaOptionsPage). If
> you give me some guidance I'll move forward on that, until then I'll
> get at least the first part of the measurements moved. I'm not sure if
> I'm going to redo the measurements completely (ie. combine each
> components measurement into one) or just have the option in general
> control all of them as is (ie. separately). I'll figure that out as I
> go along.
> 
> Thanks for pushing that one out, it'll be nice when this change is
> done :)
> 
> 
> Best Regards,
> Joel
> 
> 
> ___
> 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: [PROPOSED] Changes to General Options FDO#40656

2012-09-07 Thread Joel Madero
 
SvxJavaOptionsPage
::SvxJavaOptionsPage
(


On Fri, Sep 7, 2012 at 7:28 AM, Caolán McNamara  wrote:

> On Thu, 2012-08-09 at 14:07 -0700, Joel Madero wrote:
> > Hi All,
> >
> > This is in response to
> > FDO#40656: https://bugs.freedesktop.org/show_bug.cgi?id=40656
> >
> >
> > In order to accomplish this I need to move some things around. I did
> > the visual changes - didn't do any of the functional changes yet, nor
> > did I ensure that spacing and what not is perfect. Looking for general
> > input before I put the time that is needed into this. Thanks all.
>
> Seems sane to me (seeing as I pushed the first patch in this process, I
> thought I'd better go back and have a look at the final picture). Though
> maybe (more work) "Measurements" would be better at the top, the rest of
> the general options are sort of obscure.
>
> C.
>
>
>
Shouldn't be a problem at all.



As for your other comment about changing name of help, how do I go about
doing this? I've never messed with the help stuff before so I'll need a bit
of guidance.

Also now that Java page has changed to Advanced, does it make sense to have
files named "optjavacxx,src, etc..."? Should I consider changing the
names to something more sane since it's not just java? Also the page name
is now not really complete (svxJavaOptionsPage). If you give me some
guidance I'll move forward on that, until then I'll get at least the first
part of the measurements moved. I'm not sure if I'm going to redo the
measurements completely (ie. combine each components measurement into one)
or just have the option in general control all of them as is (ie.
separately). I'll figure that out as I go along.

Thanks for pushing that one out, it'll be nice when this change is done :)


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


Re: [PATCH] fdo#44925: list box contents should be same direction/alignment as sheet, not the interface

2012-09-07 Thread Caolán McNamara
On Thu, 2012-08-30 at 20:52 +, Manal M. Alhassoun wrote:
> 
> Hello,
> 
> Could you please review the attached patch.

Looks reasonable to me. Or maybe
if (Application::GetSettings().GetLayoutRTL() != bLayoutRTL)
pFilterBox->EnableMirroring();
is a little shorter with the same effect ?

Either way, could you add yourself to
http://wiki.documentfoundation.org/Development/Developers
and send a mail like
http://lists.freedesktop.org/archives/libreoffice/2012-July/035301.html
to this list to state that your contributions are under our preferred
MPL+/LGPLv3+ license combination, and then we can merge it in.

C.


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


[Libreoffice-commits] .: sc/inc

2012-09-07 Thread Libreoffice Gerrit user
 sc/inc/orcusfilters.hxx |2 ++
 1 file changed, 2 insertions(+)

New commits:
commit a7c157e801928d2d113ec654c1d4fd61e7b7d6b5
Author: Stephan Bergmann 
Date:   Fri Sep 7 16:36:33 2012 +0200

-Werror=non-virtual-dtor

Change-Id: Iac56ada41977e58e6e36d6efc31125f1d1ae1dec

diff --git a/sc/inc/orcusfilters.hxx b/sc/inc/orcusfilters.hxx
index 2464ede..b7e3198 100644
--- a/sc/inc/orcusfilters.hxx
+++ b/sc/inc/orcusfilters.hxx
@@ -20,6 +20,8 @@ class ScDocument;
 class ScOrcusFilters
 {
 public:
+virtual ~ScOrcusFilters() {}
+
 virtual bool importCSV(ScDocument& rDoc, const rtl::OUString& rPath) const 
= 0;
 };
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: [PROPOSED] Changes to General Options FDO#40656

2012-09-07 Thread Caolán McNamara
On Thu, 2012-08-09 at 14:07 -0700, Joel Madero wrote:
> Hi All,
> 
> This is in response to
> FDO#40656: https://bugs.freedesktop.org/show_bug.cgi?id=40656
> 
> 
> In order to accomplish this I need to move some things around. I did
> the visual changes - didn't do any of the functional changes yet, nor
> did I ensure that spacing and what not is perfect. Looking for general
> input before I put the time that is needed into this. Thanks all.

Seems sane to me (seeing as I pushed the first patch in this process, I
thought I'd better go back and have a look at the final picture). Though
maybe (more work) "Measurements" would be better at the top, the rest of
the general options are sort of obscure.

C.


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


[Libreoffice-commits] .: vbahelper/source

2012-09-07 Thread Libreoffice Gerrit user
 vbahelper/source/vbahelper/vbaglobalbase.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 006b76a785a9829bfb19f068eb49414611fe0611
Author: Stephan Bergmann 
Date:   Fri Sep 7 16:23:36 2012 +0200

Requires explicit ctor

Change-Id: I9c3a8b7366cda75d25341afaf35da9e1ea7e4f9f

diff --git a/vbahelper/source/vbahelper/vbaglobalbase.cxx 
b/vbahelper/source/vbahelper/vbaglobalbase.cxx
index 62091b6..b5327c1 100644
--- a/vbahelper/source/vbahelper/vbaglobalbase.cxx
+++ b/vbahelper/source/vbahelper/vbaglobalbase.cxx
@@ -152,7 +152,7 @@ VbaGlobalsBase::getAvailableServiceNames(  ) throw 
(uno::RuntimeException)
 {
 static const OUString names[] = {
 // common
-"ooo.vba.msforms.UserForm" ,
+OUString("ooo.vba.msforms.UserForm"),
   };
 static uno::Sequence< OUString > serviceNames( names, sizeof( names )/ 
sizeof( names[0] ) );
 return serviceNames;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: valgrinding with python stuff active

2012-09-07 Thread Michael Meeks

On Fri, 2012-09-07 at 15:44 +0200, Stephan Bergmann wrote:
> On 09/07/2012 03:29 PM, Michael Meeks wrote:
> > Now if only we could do something similar for Java :-)
> 
> FYI, works reasonably well for me with 
> 
>  
> "Force JVM into interpreted mode when running under Valgrind, to avoid 
> false error reports" and a valgrind --suppressions= file containing just

Ooh ! I missed that :-) also awesome work - thanks Stephan !

All the best,

Michael.

-- 
michael.me...@suse.com  <><, Pseudo Engineer, itinerant idiot

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


[Libreoffice-commits] .: liborcus/prj

2012-09-07 Thread Libreoffice Gerrit user
 liborcus/prj/build.lst |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit c9d99268b434f6bf154de673ad08203cfe7bf15e
Author: Kohei Yoshida 
Date:   Fri Sep 7 09:56:34 2012 -0400

Ensure to build liborcus after solenv.

Change-Id: Ie78b7857ac60d1337814ffb1507aadf48e05478c

diff --git a/liborcus/prj/build.lst b/liborcus/prj/build.lst
index 705..cc7e934 100644
--- a/liborcus/prj/build.lst
+++ b/liborcus/prj/build.lst
@@ -1,3 +1,3 @@
-orcus  liborcus: BOOST:boost MDDS:mdds NULL
+orcus  liborcus: solenv BOOST:boost MDDS:mdds NULL
 orcus  liborcususr1-   all orcus_mkout NULL
 orcus  liborcusnmake   -   all orcus_liborcus NULL
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED] Fix docx 'absolute' frame position import

2012-09-07 Thread Gerrit
>From Luboš Luňák :

Luboš Luňák has submitted this change and it was merged.

Change subject: Fix docx 'absolute' frame position import
..


Fix docx 'absolute' frame position import

Frames with absolute position style must be vertically placed relative
to 'Margin', otherwise parent paragraph style may modify their Y coord.

Change-Id: Ifae8f73ad9c6aa98b67283663cfc37dd847ff095
---
M oox/source/vml/vmlshape.cxx
1 file changed, 4 insertions(+), 0 deletions(-)

Approvals:
  Luboš Luňák: Verified; Looks good to me, approved


--
To view, visit https://gerrit.libreoffice.org/561
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ifae8f73ad9c6aa98b67283663cfc37dd847ff095
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Pierre-Eric Pelloux-Prayer 
Gerrit-Reviewer: Luboš Luňák 

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


[Libreoffice-commits] .: oox/source

2012-09-07 Thread Libreoffice Gerrit user
 oox/source/vml/vmlshape.cxx |4 
 1 file changed, 4 insertions(+)

New commits:
commit 711958d8afcf7704b19e8f656aeac3cd7569e0f5
Author: Pierre-Eric Pelloux-Prayer 
Date:   Wed Sep 5 10:17:27 2012 +0200

Fix docx 'absolute' frame position import

Frames with absolute position style must be vertically placed relative
to 'Margin', otherwise parent paragraph style may modify their Y coord.

Change-Id: Ifae8f73ad9c6aa98b67283663cfc37dd847ff095
Reviewed-on: https://gerrit.libreoffice.org/561
Reviewed-by: Luboš Luňák 
Tested-by: Luboš Luňák 

diff --git a/oox/source/vml/vmlshape.cxx b/oox/source/vml/vmlshape.cxx
index a32030b..759947a 100644
--- a/oox/source/vml/vmlshape.cxx
+++ b/oox/source/vml/vmlshape.cxx
@@ -384,8 +384,12 @@ void lcl_SetAnchorType(PropertySet& rPropSet, const 
ShapeTypeModel& rTypeModel)
 rPropSet.setProperty(PROP_AnchorType, 
text::TextContentAnchorType_AT_PAGE);
 }
 else
+{
 // Map to as-character by default, that fixes vertical position of 
some textframes.
 rPropSet.setProperty(PROP_AnchorType, 
text::TextContentAnchorType_AT_CHARACTER);
+}
+// Vertical placement relative to margin, because parent style must 
not modify vertical position
+rPropSet.setProperty(PROP_VertOrientRelation, 
text::RelOrientation::FRAME);
 }
 else if( rTypeModel.maPosition == "relative" )
 {   // I'm not very sure this is correct either.
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: valgrinding with python stuff active

2012-09-07 Thread Stephan Bergmann

On 09/07/2012 03:29 PM, Michael Meeks wrote:

Now if only we could do something similar for Java :-)


FYI, works reasonably well for me with 
 
"Force JVM into interpreted mode when running under Valgrind, to avoid 
false error reports" and a valgrind --suppressions= file containing just



{
 java-1
 Memcheck:Addr4
 ...
 
obj:/usr/lib/jvm/java-1.7.0-openjdk-1.7.0.6.x86_64/jre/lib/amd64/server/libjvm.so
}
{
 java-2
 Memcheck:Cond
 
obj:/usr/lib/jvm/java-1.7.0-openjdk-1.7.0.6.x86_64/jre/lib/amd64/server/libjvm.so
}


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


valgrinding with java stuff active

2012-09-07 Thread Noel Grandin


On 2012-09-07 15:29, Michael Meeks wrote:

On Fri, 2012-09-07 at 13:30 +0100, Caolán McNamara wrote:

FWIW, I patched the *internal* to be valgrind friendly when G_SLICE is
set. So if you have to debug something where python is alive you could
try building against the internal python and it should "just work" with
your usual export VALGRIND=memcheck foo

Oooh ! that's insanely cool :-)

Was just digging through a load of false-positives there myself.

Now if only we could do something similar for Java :-)




Does valgrind's --smc-check=all not help?

Otherwise you could turn off the VM's JIT, there is a way of doing it 
from JNI. Search this page:
http://docs.oracle.com/javase/1.4.2/docs/guide/jni/jni-12.html 


for "JIT".

Regards, Noel Grandin


Disclaimer: http://www.peralex.com/disclaimer.html


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


[PUSHED] More OUString cleanup in vbahelper

2012-09-07 Thread Gerrit
>From Luboš Luňák :

Luboš Luňák has submitted this change and it was merged.

Change subject: More OUString cleanup in vbahelper
..


More OUString cleanup in vbahelper

Change-Id: Ie7e350698900f1c1cacb6ebc3fa6716ef03ab782
---
M vbahelper/source/vbahelper/vbaglobalbase.cxx
M vbahelper/source/vbahelper/vbapagesetupbase.cxx
2 files changed, 39 insertions(+), 39 deletions(-)

Approvals:
  Luboš Luňák: Verified; Looks good to me, approved


--
To view, visit https://gerrit.libreoffice.org/575
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie7e350698900f1c1cacb6ebc3fa6716ef03ab782
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Olivier Hallot 
Gerrit-Reviewer: Luboš Luňák 

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


[PUSHED: 3-6] fdo#54450 install evolution datasource driver

2012-09-07 Thread Michael Meeks
On Fri, 2012-09-07 at 13:26 +0200, David Tardon wrote:
> This does not affect the official build, because it is configured with
> --enable-gnomevfs, but I think it does not hurt to have it in 3.6, if
> there is anyone who wants to review the fix :-)

Looks great to me, pushed.

Thanks,

Michael.

-- 
michael.me...@suse.com  <><, Pseudo Engineer, itinerant idiot

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


[Libreoffice-commits] .: Branch 'libreoffice-3-6' - scp2/InstallModule_gnome.mk scp2/InstallScript_setup_osl.mk scp2/Module_scp2.mk scp2/source

2012-09-07 Thread Libreoffice Gerrit user
 scp2/InstallModule_gnome.mk|2 ++
 scp2/InstallScript_setup_osl.mk|2 +-
 scp2/Module_scp2.mk|2 +-
 scp2/source/gnome/module_gnome.scp |   12 
 4 files changed, 12 insertions(+), 6 deletions(-)

New commits:
commit bb1511c00acb25c607e8c885f33f1f313b5ea098
Author: David Tardon 
Date:   Fri Sep 7 12:40:05 2012 +0200

fdo#54450 install evolution DB driver correctly

Change-Id: I0b6eb67014a61e22981f1f6be488fa762d639554
Signed-off-by: Michael Meeks 

diff --git a/scp2/InstallModule_gnome.mk b/scp2/InstallModule_gnome.mk
index ea20b37..15e8116 100644
--- a/scp2/InstallModule_gnome.mk
+++ b/scp2/InstallModule_gnome.mk
@@ -33,9 +33,11 @@ $(eval $(call gb_InstallModule_use_packages,scp2/gnome,\
 ))
 
 $(eval $(call gb_InstallModule_define_if_set,scp2/gnome,\
+   ENABLE_EVOAB2 \
ENABLE_GCONF \
ENABLE_GIO \
ENABLE_GNOMEVFS \
+   ENABLE_GTK \
ENABLE_GTK3 \
ENABLE_LOCKDOWN \
ENABLE_SYSTRAY_GTK \
diff --git a/scp2/InstallScript_setup_osl.mk b/scp2/InstallScript_setup_osl.mk
index 5eca444..cbe59b4 100644
--- a/scp2/InstallScript_setup_osl.mk
+++ b/scp2/InstallScript_setup_osl.mk
@@ -58,7 +58,7 @@ $(eval $(call gb_InstallScript_use_modules,setup_osl,\
$(if $(filter YES,$(WITH_EXTENSION_INTEGRATION)),\
scp2/extensions \
) \
-   $(if $(filter TRUE,$(ENABLE_GCONF) $(ENABLE_GNOMEVFS) $(ENABLE_GIO)),\
+   $(if $(filter TRUE,$(ENABLE_EVOAB2) $(ENABLE_GCONF) $(ENABLE_GNOMEVFS) 
$(ENABLE_GIO) $(ENABLE_GTK) $(ENABLE_GTK3)),\
scp2/gnome \
) \
$(if $(filter TRUE,$(SOLAR_JAVA)),\
diff --git a/scp2/Module_scp2.mk b/scp2/Module_scp2.mk
index 48f4686..c621cc5 100644
--- a/scp2/Module_scp2.mk
+++ b/scp2/Module_scp2.mk
@@ -66,7 +66,7 @@ $(eval $(call gb_Module_add_targets,scp2,\
$(if $(filter TRUE,$(ENABLE_CRASHDUMP)),\
InstallModule_crashrep \
) \
-   $(if $(filter TRUE,$(ENABLE_GCONF) $(ENABLE_GNOMEVFS) $(ENABLE_GIO)),\
+   $(if $(filter TRUE,$(ENABLE_EVOAB2) $(ENABLE_GCONF) $(ENABLE_GNOMEVFS) 
$(ENABLE_GIO) $(ENABLE_GTK) $(ENABLE_GTK3)),\
InstallModule_gnome \
) \
$(if $(filter TRUE,$(SOLAR_JAVA)),\
diff --git a/scp2/source/gnome/module_gnome.scp 
b/scp2/source/gnome/module_gnome.scp
index 88a2926..9194825 100644
--- a/scp2/source/gnome/module_gnome.scp
+++ b/scp2/source/gnome/module_gnome.scp
@@ -41,14 +41,16 @@ Module gid_Module_Optional_Gnome
 XpdCheckSolaris = "SUNWgnome-vfs";
 #endif
 Files = (
+#ifdef ENABLE_EVOAB2
   gid_File_Lib_Evoab,
+#endif
+#ifdef ENABLE_GTK
   gid_File_Lib_Fps_Gnome,
   gid_File_Lib_Vclplug_Gtk,
+#endif
 #ifdef ENABLE_GTK3
   gid_File_Lib_Vclplug_Gtk3,
 #endif
-  gid_File_Bin_Gnome_Open_Url,
-  gid_File_Bin_Gnome_Open_Url_Bin,
 #ifdef ENABLE_SYSTRAY_GTK
   gid_File_Lib_QStart_Gtk,
 #endif
@@ -59,11 +61,13 @@ Module gid_Module_Optional_Gnome
   gid_File_Lib_GIO,
 #endif
 #ifdef ENABLE_GCONF
-  gid_File_Lib_Gconfbe
+  gid_File_Lib_Gconfbe,
 #endif
 #if (defined ENABLE_GNOMEVFS && defined ENABLE_LOCKDOWN) || defined ENABLE_GIO
-  gid_File_Share_Registry_Gnome_Xcd
+  gid_File_Share_Registry_Gnome_Xcd,
 #endif
+  gid_File_Bin_Gnome_Open_Url,
+  gid_File_Bin_Gnome_Open_Url_Bin
 );
 End
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: vbahelper/source

2012-09-07 Thread Libreoffice Gerrit user
 vbahelper/source/vbahelper/vbaglobalbase.cxx|   22 -
 vbahelper/source/vbahelper/vbapagesetupbase.cxx |   56 
 2 files changed, 39 insertions(+), 39 deletions(-)

New commits:
commit 419d074df87d24d4612a9e5ef5f70cb9dcefef93
Author: Olivier Hallot 
Date:   Thu Sep 6 18:58:56 2012 -0300

More OUString cleanup in vbahelper

Change-Id: Ie7e350698900f1c1cacb6ebc3fa6716ef03ab782
Reviewed-on: https://gerrit.libreoffice.org/575
Reviewed-by: Luboš Luňák 
Tested-by: Luboš Luňák 

diff --git a/vbahelper/source/vbahelper/vbaglobalbase.cxx 
b/vbahelper/source/vbahelper/vbaglobalbase.cxx
index 9b38f52..62091b6 100644
--- a/vbahelper/source/vbahelper/vbaglobalbase.cxx
+++ b/vbahelper/source/vbahelper/vbaglobalbase.cxx
@@ -33,7 +33,7 @@ const char sAppService[] = "ooo.vba.Application";
 
 VbaGlobalsBase::VbaGlobalsBase(
 const uno::Reference< ov::XHelperInterface >& xParent,
-const uno::Reference< uno::XComponentContext >& xContext, const rtl::OUString& 
sDocCtxName )
+const uno::Reference< uno::XComponentContext >& xContext, const OUString& 
sDocCtxName )
 : Globals_BASE( xParent, xContext )
 , msDocCtxName( sDocCtxName )
 , msApplication( RTL_CONSTASCII_USTRINGPARAM("Application") )
@@ -43,14 +43,14 @@ const uno::Reference< uno::XComponentContext >& xContext, 
const rtl::OUString& s
 uno::Reference< uno::XInterface > aSrvMgr;
 if ( xContext.is() && xContext->getServiceManager().is() )
 {
-aSrvMgr = xContext->getServiceManager()->createInstanceWithContext( 
rtl::OUString( 
RTL_CONSTASCII_USTRINGPARAM("com.sun.star.comp.stoc.OServiceManagerWrapper") ), 
xContext );
+aSrvMgr = xContext->getServiceManager()->createInstanceWithContext( 
"com.sun.star.comp.stoc.OServiceManagerWrapper" , xContext );
 }
 
 ::cppu::ContextEntry_Init aHandlerContextInfo[] =
 {
 ::cppu::ContextEntry_Init( msApplication, uno::Any() ),
 ::cppu::ContextEntry_Init( sDocCtxName, uno::Any() ),
-::cppu::ContextEntry_Init( rtl::OUString( 
RTL_CONSTASCII_USTRINGPARAM("/singletons/com.sun.star.lang.theServiceManager" ) 
), uno::makeAny( aSrvMgr ) )
+::cppu::ContextEntry_Init( 
"/singletons/com.sun.star.lang.theServiceManager" , uno::makeAny( aSrvMgr ) )
 };
 // don't pass a delegate, this seems to introduce yet another cyclic 
dependency ( and
 // some strange behavior
@@ -117,7 +117,7 @@ VbaGlobalsBase::init(  const uno::Sequence< 
beans::PropertyValue >& aInitArgs )
 }
 
 uno::Reference< uno::XInterface > SAL_CALL
-VbaGlobalsBase::createInstance( const ::rtl::OUString& aServiceSpecifier ) 
throw (uno::Exception, uno::RuntimeException)
+VbaGlobalsBase::createInstance( const OUString& aServiceSpecifier ) throw 
(uno::Exception, uno::RuntimeException)
 {
 uno::Reference< uno::XInterface > xReturn;
 if ( aServiceSpecifier == sAppService )
@@ -132,7 +132,7 @@ VbaGlobalsBase::createInstance( const ::rtl::OUString& 
aServiceSpecifier ) throw
 }
 
 uno::Reference< uno::XInterface > SAL_CALL
-VbaGlobalsBase::createInstanceWithArguments( const ::rtl::OUString& 
aServiceSpecifier, const uno::Sequence< uno::Any >& Arguments ) throw 
(uno::Exception, uno::RuntimeException)
+VbaGlobalsBase::createInstanceWithArguments( const OUString& 
aServiceSpecifier, const uno::Sequence< uno::Any >& Arguments ) throw 
(uno::Exception, uno::RuntimeException)
 {
 
 uno::Reference< uno::XInterface > xReturn;
@@ -147,21 +147,21 @@ VbaGlobalsBase::createInstanceWithArguments( const 
::rtl::OUString& aServiceSpec
 return xReturn;
 }
 
-uno::Sequence< ::rtl::OUString > SAL_CALL
+uno::Sequence< OUString > SAL_CALL
 VbaGlobalsBase::getAvailableServiceNames(  ) throw (uno::RuntimeException)
 {
-static const rtl::OUString names[] = {
+static const OUString names[] = {
 // common
-::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM ( 
"ooo.vba.msforms.UserForm" ) ),
+"ooo.vba.msforms.UserForm" ,
   };
-static uno::Sequence< rtl::OUString > serviceNames( names, sizeof( names 
)/ sizeof( names[0] ) );
+static uno::Sequence< OUString > serviceNames( names, sizeof( names )/ 
sizeof( names[0] ) );
 return serviceNames;
 }
 
 bool
-VbaGlobalsBase::hasServiceName( const rtl::OUString& serviceName )
+VbaGlobalsBase::hasServiceName( const OUString& serviceName )
 {
-uno::Sequence< rtl::OUString > sServiceNames( getAvailableServiceNames() );
+uno::Sequence< OUString > sServiceNames( getAvailableServiceNames() );
 sal_Int32 nLen = sServiceNames.getLength();
 for ( sal_Int32 index = 0; index < nLen; ++index )
 {
diff --git a/vbahelper/source/vbahelper/vbapagesetupbase.cxx 
b/vbahelper/source/vbahelper/vbapagesetupbase.cxx
index 885abed..a47f591 100644
--- a/vbahelper/source/vbahelper/vbapagesetupbase.cxx
+++ b/vbahelper/source/vbahelper/vbapagesetupbase.cxx
@@ -34,15 +34,15 @@ double SAL_CALL VbaPageSetupBase::getTopMargin() throw 
(css::uno::RuntimeExcepti
 
 try
 

Re: valgrinding with python stuff active

2012-09-07 Thread Michael Meeks

On Fri, 2012-09-07 at 13:30 +0100, Caolán McNamara wrote:
> FWIW, I patched the *internal* to be valgrind friendly when G_SLICE is
> set. So if you have to debug something where python is alive you could
> try building against the internal python and it should "just work" with
> your usual export VALGRIND=memcheck foo

Oooh ! that's insanely cool :-)

Was just digging through a load of false-positives there myself.

Now if only we could do something similar for Java :-)

Nice work,

Michael.

-- 
michael.me...@suse.com  <><, Pseudo Engineer, itinerant idiot

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


[PATCH] next attempt to fix bug 35079

2012-09-07 Thread Gerrit
>From Lennard Wasserthal :

Lennard Wasserthal has uploaded a new change for review.

Change subject: next attempt to fix bug 35079
..

next attempt to fix bug 35079

Removes the hidden frame. That frame really doesn't seem to be needed, not even 
for exporting, though.

Change-Id: I37285a9dfe46fe9263ba1f767fe69a59226e342a
Signed-off-by: Lennard Wasserthal 
---
M svx/source/sdr/contact/viewcontactofsdrobjcustomshape.cxx
1 file changed, 0 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/82/582/1
--
To view, visit https://gerrit.libreoffice.org/582
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I37285a9dfe46fe9263ba1f767fe69a59226e342a
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Lennard Wasserthal 

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


Re: [PATCH][PUSHED] Higher‐resolution OpenDocument icons for Mac OS X

2012-09-07 Thread Caolán McNamara
On Fri, 2012-09-07 at 14:57 +0200, Stefan Knorr (Astron) wrote:
> Hi Caolan,
> 
> > So, re the contents of the icons.tar file, they look fine to me. I'm
> > minded to add them in unless someone has some argument against that ?
> 
> +1 from me, they look good.

Righteo, pushed to master now. Thanks Nicholas.

C.

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


[Libreoffice-commits] .: sysui/desktop

2012-09-07 Thread Libreoffice Gerrit user
 sysui/desktop/icons/hicolor/1024x1024/apps/main.png
 |binary
 sysui/desktop/icons/hicolor/1024x1024/mimetypes/oasis-database.png 
 |binary
 sysui/desktop/icons/hicolor/1024x1024/mimetypes/oasis-drawing-template.png 
 |binary
 sysui/desktop/icons/hicolor/1024x1024/mimetypes/oasis-drawing.png  
 |binary
 sysui/desktop/icons/hicolor/1024x1024/mimetypes/oasis-formula.png  
 |binary
 sysui/desktop/icons/hicolor/1024x1024/mimetypes/oasis-master-document.png  
 |binary
 
sysui/desktop/icons/hicolor/1024x1024/mimetypes/oasis-presentation-template.png 
|binary
 sysui/desktop/icons/hicolor/1024x1024/mimetypes/oasis-presentation.png 
 |binary
 sysui/desktop/icons/hicolor/1024x1024/mimetypes/oasis-spreadsheet-template.png 
 |binary
 sysui/desktop/icons/hicolor/1024x1024/mimetypes/oasis-spreadsheet.png  
 |binary
 sysui/desktop/icons/hicolor/1024x1024/mimetypes/oasis-text-template.png
 |binary
 sysui/desktop/icons/hicolor/1024x1024/mimetypes/oasis-text.png 
 |binary
 sysui/desktop/icons/hicolor/1024x1024/mimetypes/oasis-web-template.png 
 |binary
 sysui/desktop/icons/hicolor/512x512/apps/main.png  
 |binary
 sysui/desktop/icons/hicolor/512x512/mimetypes/oasis-database.png   
 |binary
 sysui/desktop/icons/hicolor/512x512/mimetypes/oasis-drawing-template.png   
 |binary
 sysui/desktop/icons/hicolor/512x512/mimetypes/oasis-drawing.png
 |binary
 sysui/desktop/icons/hicolor/512x512/mimetypes/oasis-formula.png
 |binary
 sysui/desktop/icons/hicolor/512x512/mimetypes/oasis-master-document.png
 |binary
 sysui/desktop/icons/hicolor/512x512/mimetypes/oasis-presentation-template.png  
 |binary
 sysui/desktop/icons/hicolor/512x512/mimetypes/oasis-presentation.png   
 |binary
 sysui/desktop/icons/hicolor/512x512/mimetypes/oasis-spreadsheet-template.png   
 |binary
 sysui/desktop/icons/hicolor/512x512/mimetypes/oasis-spreadsheet.png
 |binary
 sysui/desktop/icons/hicolor/512x512/mimetypes/oasis-text-template.png  
 |binary
 sysui/desktop/icons/hicolor/512x512/mimetypes/oasis-text.png   
 |binary
 sysui/desktop/icons/hicolor/512x512/mimetypes/oasis-web-template.png   
 |binary
 sysui/desktop/icons/main.icns  
 |binary
 sysui/desktop/icons/oasis-database.icns
 |binary
 sysui/desktop/icons/oasis-drawing-template.icns
 |binary
 sysui/desktop/icons/oasis-drawing.icns 
 |binary
 sysui/desktop/icons/oasis-formula.icns 
 |binary
 sysui/desktop/icons/oasis-master-document.icns 
 |binary
 sysui/desktop/icons/oasis-presentation-template.icns   
 |binary
 sysui/desktop/icons/oasis-presentation.icns
 |binary
 sysui/desktop/icons/oasis-spreadsheet-template.icns
 |binary
 sysui/desktop/icons/oasis-spreadsheet.icns 
 |binary
 sysui/desktop/icons/oasis-text-template.icns   
 |binary
 sysui/desktop/icons/oasis-text.icns
 |binary
 38 files changed

New commits:
commit 0cb1b9685877f6aeaddf1088ae5008e38bade6f9
Author: Nicholas Shanks 
Date:   Fri Sep 7 14:14:29 2012 +0100

higher-resolution OpenDocument icons for MacOSX

Change-Id: I5bd4e5a8f80520f9cdf3f5a223dfe5cebadc4c6a

diff --git a/sysui/desktop/icons/hicolor/1024x1024/apps/main.png 
b/sysui/desktop/icons/hicolor/1024x1024/apps/main.png
new file mode 100644
index 000..a19d7ed
Binary files /dev/null and 
b/sysui/desktop/icons/hicolor/1024x1024/apps/main.png differ
diff --git a/sysui/desktop/icons/hicolor/1024x1024/mimetypes/oasis-database.png 
b/sysui/desktop/icons/hicolor/1024x1024/mimetypes/oasis-database.png
new file mode 100644
index 000..a3b1878
Binary files /dev/null and 
b/sysui/desktop/icons/hicolor/1024x1024/mimetypes/oasis-database.png differ
diff --git 
a/sysui/desktop/icons/hicolor/1024x1024/mimetypes/oasis-drawing-template.png 
b/sysui/desktop/icons/hicolor/1024x1024/mimetypes/oasis-drawing-template.png
new file mode 100644
index 000..2b09705
Binary files /dev/null and 
b/sysui/desktop/icons/hicolor/1024x1024/mimetypes/oasis-drawing-template.png 
differ
diff --git a/sysui/desktop/icons/hicolor/1024x1024/mimetypes/oasis-drawing.png 
b/sysui/desktop/icons/hicolor/1024x1024/mimetypes/oasis-drawing.png
new file mode 100644
index 000..99c209a
Binary files /dev/null and 
b/sysui/desktop/icons/hicolor/1024x1024/mimetypes/oasis-drawing.png differ
diff --git a/sysui/desktop/icons/hicolor/1024x1024/mimetypes/oasis-formula.png 
b/sysui/desktop/icons/hicolor/1024x1024/mimetypes/oasis-formula.png
n

feature/orcus-integration about to be merged

2012-09-07 Thread Kohei Yoshida
This is just to inform you that I'm about to merge the 
feature/orcus-integration branch into master.  This will add another 
external library to the build, with new configure switch 
--with-system-orcus.  I've spent the past week ensuring its buildability 
on Linux and Windows (well, more on Windows than Linux), but do please 
watch out for any potential disruptions.


Best,

Kohei

--
Kohei Yoshida, LibreOffice hacker, Calc
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [PATCH][PUSHED] Changes to Tools ->Options Tree

2012-09-07 Thread Caolán McNamara
On Tue, 2012-09-04 at 13:03 -0700, Joel Madero wrote:
> Hi All,
> 
> This is the first of multiple patches to address fdo#40656 (moving 
> measurements to the LibreOffice General tab). In this patch:
> 
> 1. Changed "Java" to "Advanced"
> 2. Shrunk Java stuff to about half of tab
> 3. Moved experimental options to Advanced tab below Java section

Looks reasonable to me. Pushed to master now. You should adjust the help
as well to change the title of the help page from "Java" to "Advanced"
there too.

C.

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


[Libreoffice-commits] .: 2 commits - cui/source

2012-09-07 Thread Libreoffice Gerrit user
 cui/source/options/optgdlg.cxx |   29 -
 cui/source/options/optgdlg.hrc |3 ---
 cui/source/options/optgdlg.hxx |4 
 cui/source/options/optgdlg.src |   18 --
 cui/source/options/optjava.cxx |   32 +++-
 cui/source/options/optjava.hrc |   10 ++
 cui/source/options/optjava.hxx |4 
 cui/source/options/optjava.src |   22 --
 cui/source/options/treeopt.src |2 +-
 9 files changed, 70 insertions(+), 54 deletions(-)

New commits:
commit 948350d60c031904c7daffc7be95ef9c146e700c
Author: Caolán McNamara 
Date:   Fri Sep 7 14:01:19 2012 +0100

WaE: member init order

Change-Id: Idf7def16e5a9bb4a44624dbc1039e7df05d07c40

diff --git a/cui/source/options/optjava.cxx b/cui/source/options/optjava.cxx
index 65bde82..c1d6bfb 100644
--- a/cui/source/options/optjava.cxx
+++ b/cui/source/options/optjava.cxx
@@ -117,12 +117,11 @@ SvxJavaOptionsPage::SvxJavaOptionsPage( Window* pParent, 
const SfxItemSet& rSet
 m_sAccessibilityText(   CUI_RES( STR_ACCESSIBILITY ) ),
 m_sAddDialogText(   CUI_RES( STR_ADDDLGTEXT ) ),
 
-xDialogListener ( new ::svt::DialogClosedListener() ),
-
 m_aExperimental ( this, CUI_RES( FL_EXPERIMENTAL ) ),
 m_aExperimentalCB   ( this, CUI_RES( CB_EXPERIMENTAL ) ),
-m_aMacroCB  ( this, CUI_RES( CB_MACRO ) )
+m_aMacroCB  ( this, CUI_RES( CB_MACRO ) ),
 
+xDialogListener ( new ::svt::DialogClosedListener() )
 {
 m_aJavaEnableCB.SetClickHdl( LINK( this, SvxJavaOptionsPage, 
EnableHdl_Impl ) );
 m_aJavaList.SetCheckButtonHdl( LINK( this, SvxJavaOptionsPage, 
CheckHdl_Impl ) );
commit 818d269971e3119047a3bcc78995f4067d9d9ddd
Author: Joel Madero 
Date:   Tue Sep 4 07:03:03 2012 -0700

Moving experimental section to java, changing section name

First of a few patches to move measurements to the general section in the
LibreOffice main tree. This patch moves experimental section to "Java"
and then changes the name "Java" to "Advanced". The space which was cleared
by moving experimental section will be used for measurements in the future.

modified:   cui/source/options/optgdlg.cxx
modified:   cui/source/options/optgdlg.hrc
modified:   cui/source/options/optgdlg.hxx
modified:   cui/source/options/optgdlg.src
modified:   cui/source/options/optjava.cxx
modified:   cui/source/options/optjava.hrc
modified:   cui/source/options/optjava.hxx
modified:   cui/source/options/optjava.src
modified:   cui/source/options/treeopt.src

Change-Id: I20073296ce81aace4b041ef5cb27e4313637a58d

diff --git a/cui/source/options/optgdlg.cxx b/cui/source/options/optgdlg.cxx
index 5f7015b..3ff3b3c 100644
--- a/cui/source/options/optgdlg.cxx
+++ b/cui/source/options/optgdlg.cxx
@@ -212,10 +212,7 @@ OfaMiscTabPage::OfaMiscTabPage(Window* pParent, const 
SfxItemSet& rSet ) :
 aTwoFigureFL( this, CUI_RES( FL_TWOFIGURE ) ),
 aInterpretFT( this, CUI_RES( FT_INTERPRET ) ),
 aYearValueField ( this, CUI_RES( NF_YEARVALUE ) ),
-aToYearFT   ( this, CUI_RES( FT_TOYEAR ) ),
-aExperimentalFL ( this, CUI_RES( FL_EXPERIMENTAL ) ),
-aExperimentalCB ( this, CUI_RES( CB_EXPERIMENTAL ) ),
-aMacroRecorderCB( this, CUI_RES( CB_MACRORECORDER ) )
+aToYearFT   ( this, CUI_RES( FT_TOYEAR ) )
 {
 FreeResource();
 
@@ -245,7 +242,7 @@ OfaMiscTabPage::OfaMiscTabPage(Window* pParent, const 
SfxItemSet& rSet ) :
 Window* pWins[] =
 {
 &aPrintDlgFL, &aPrintDlgCB, &aDocStatusFL, &aDocStatusCB, 
&aSaveAlwaysCB,
-&aTwoFigureFL, &aInterpretFT, &aYearValueField, &aToYearFT, 
&aExperimentalFL, &aExperimentalCB, &aMacroRecorderCB
+&aTwoFigureFL, &aInterpretFT, &aYearValueField, &aToYearFT
 };
 Window** pCurrent = pWins;
 const sal_Int32 nCount = SAL_N_ELEMENTS( pWins );
@@ -266,7 +263,7 @@ OfaMiscTabPage::OfaMiscTabPage(Window* pParent, const 
SfxItemSet& rSet ) :
 Window* pWins[] =
 {
 &aPrintDlgFL, &aPrintDlgCB, &aDocStatusFL, &aDocStatusCB, 
&aSaveAlwaysCB,
-&aTwoFigureFL, &aInterpretFT, &aYearValueField, &aToYearFT, 
&aExperimentalFL, &aExperimentalCB, &aMacroRecorderCB
+&aTwoFigureFL, &aInterpretFT, &aYearValueField, &aToYearFT
 };
 Window** pCurrent = pWins;
 const sal_Int32 nCount = SAL_N_ELEMENTS( pWins );
@@ -292,7 +289,7 @@ OfaMiscTabPage::OfaMiscTabPage(Window* pParent, const 
SfxItemSet& rSet ) :
 Window* pWins[] =
 {
 &aDocStatusFL, &aDocStatusCB, &aSaveAlwaysCB, &aTwoFigureFL,
-&aInterpretFT, &aYearValueField, &aToYearFT, &aExperimentalFL, 
&aExperimentalCB, &aMacroRecorderCB
+&aInterpretFT, &aYearValueField, &aToYearFT
 };
 Window** pCurrent = pWins;
 

Re: [PATCH] Higher‐resolution OpenDocument icons for Mac OS X

2012-09-07 Thread Stefan Knorr (Astron)
Hi Caolan,

> So, re the contents of the icons.tar file, they look fine to me. I'm
> minded to add them in unless someone has some argument against that ?

+1 from me, they look good.
(You could always add more detail, but that's not necessary right now.)

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


Re: [PUSHED: 3-6] fdo#54620 crash when merging table cells in writer

2012-09-07 Thread Caolán McNamara
On Fri, 2012-09-07 at 13:30 +0200, David Tardon wrote:
> Hi all,
> 
> commit 06ae26b5676c29c4d1cc9c65f5bbaede9483a21d fixes crash than happens
> in specific situation when merging table cells in writer. It is a
> regression caused by STL conversion. The cause is use of vector iterator
> after calling insert on the vector.

Looks good to me, pushed to 3-6.

C.

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


[Libreoffice-commits] .: Branch 'libreoffice-3-6' - sw/source

2012-09-07 Thread Libreoffice Gerrit user
 sw/source/core/doc/gctable.cxx |   13 +++--
 1 file changed, 7 insertions(+), 6 deletions(-)

New commits:
commit 032b29619484a2a4ade0600cd978df86d5d522a5
Author: David Tardon 
Date:   Fri Sep 7 10:00:44 2012 +0200

fdo#54620 do not use vector iterator after insert

Change-Id: I5e809f38a500ab818e9acef80b419dfece7a29fa
(cherry picked from commit 06ae26b5676c29c4d1cc9c65f5bbaede9483a21d)

Signed-off-by: Caolán McNamara 

diff --git a/sw/source/core/doc/gctable.cxx b/sw/source/core/doc/gctable.cxx
index 5507c54..f71cb2a 100644
--- a/sw/source/core/doc/gctable.cxx
+++ b/sw/source/core/doc/gctable.cxx
@@ -357,19 +357,20 @@ static bool lcl_MergeGCBox(SwTableBox* pTblBox, 
void*const pPara)
 
 if( 1 == pTblBox->GetTabLines().Count() )
 {
-// Box with a Line, then move all the Line's Boxes after this Box
-// into the parent Line and delete this Box
+// we have a box with a single line, so we just replace it by the 
line's boxes
 SwTableLine* pInsLine = pTblBox->GetUpper();
 SwTableLine* pCpyLine = pTblBox->GetTabLines()[0];
 SwTableBoxes::iterator it = std::find( 
pInsLine->GetTabBoxes().begin(), pInsLine->GetTabBoxes().end(), pTblBox );
 for( n = 0; n < pCpyLine->GetTabBoxes().size(); ++n )
 pCpyLine->GetTabBoxes()[n]->SetUpper( pInsLine );
 
-pInsLine->GetTabBoxes().insert( it + 1, 
pCpyLine->GetTabBoxes().begin(), pCpyLine->GetTabBoxes().end());
+// remove the old box from its parent line
+it = pInsLine->GetTabBoxes().erase( it );
+// insert the nested line's boxes in its place
+pInsLine->GetTabBoxes().insert( it, 
pCpyLine->GetTabBoxes().begin(), pCpyLine->GetTabBoxes().end());
 pCpyLine->GetTabBoxes().clear();
-// Delete the old Box with the Line
-delete *it;
-pInsLine->GetTabBoxes().erase( it );
+// destroy the removed box
+delete pTblBox;
 
 return false; // set up anew
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: 2 commits - sw/source vcl/generic vcl/inc vcl/Library_vcl.mk vcl/Package_inc.mk vcl/source writerfilter/source

2012-09-07 Thread Libreoffice Gerrit user
 sw/source/filter/ww8/docxattributeoutput.cxx |4 +-
 vcl/Library_vcl.mk   |1 
 vcl/Package_inc.mk   |1 
 vcl/generic/fontmanager/fontmanager.cxx  |   45 +-
 vcl/inc/vcl/fontmanager.hxx  |   25 
 vcl/inc/vcl/temporaryfonts.hxx   |   54 +++
 vcl/source/gdi/temporaryfonts.cxx|   54 +++
 writerfilter/source/dmapper/FontTable.cxx|6 +--
 8 files changed, 119 insertions(+), 71 deletions(-)

New commits:
commit 063966ee0e8b98231cb622311eb2131d19531926
Author: Luboš Luňák 
Date:   Fri Sep 7 14:35:10 2012 +0200

Revert "vcl: export PrintFontManager"

This reverts commit 23cd24bd4d4df9247f8f0496f9934a2ce6d110dd.
Not needed after the temporary font code has been moved elsewhere.

diff --git a/vcl/inc/vcl/fontmanager.hxx b/vcl/inc/vcl/fontmanager.hxx
index 3049be0..f0f 100644
--- a/vcl/inc/vcl/fontmanager.hxx
+++ b/vcl/inc/vcl/fontmanager.hxx
@@ -153,7 +153,7 @@ class FontCache;
 
 class FontCache;
 
-class VCL_DLLPUBLIC PrintFontManager
+class VCL_PLUGIN_PUBLIC PrintFontManager
 {
 struct PrintFont;
 struct TrueTypeFontFile;
commit fc169270eaeb8156d40740cd088cd8ed958ea99c
Author: Luboš Luňák 
Date:   Fri Sep 7 14:33:54 2012 +0200

move the code for adding temporary fonts to a separate class

I'm a bit confused on how this stuff is separated between platforms,
so better do it this way.

Change-Id: I2dbd9baef587c81ee37b509bde272ef970f5b118

diff --git a/sw/source/filter/ww8/docxattributeoutput.cxx 
b/sw/source/filter/ww8/docxattributeoutput.cxx
index f9de95a..e4cd092 100644
--- a/sw/source/filter/ww8/docxattributeoutput.cxx
+++ b/sw/source/filter/ww8/docxattributeoutput.cxx
@@ -122,7 +122,7 @@
 #include 
 #include 
 #include 
-#include 
+#include 
 
 #include 
 
@@ -2829,7 +2829,7 @@ static inline char toHexChar( int value )
 
 void DocxAttributeOutput::EmbedFontStyle( const OUString& name, int tag, const 
char* style )
 {
-OUString fontUrl = psp::PrintFontManager::get().fileUrlForTemporaryFont( 
name, style );
+OUString fontUrl = TemporaryFonts::fileUrlForFont( name, style );
 // If a temporary font file exists for this font, assume it was embedded
 // and embed it again.
 // TODO IDocumentSettingAccess::EMBED_SYSTEM_FONTS
diff --git a/vcl/Library_vcl.mk b/vcl/Library_vcl.mk
index 0a7c807..f8afea0 100644
--- a/vcl/Library_vcl.mk
+++ b/vcl/Library_vcl.mk
@@ -234,6 +234,7 @@ $(eval $(call gb_Library_add_exception_objects,vcl,\
 vcl/source/gdi/salmisc \
 vcl/source/gdi/salnativewidgets-none \
 vcl/source/gdi/svgread \
+vcl/source/gdi/temporaryfonts \
 vcl/source/gdi/textlayout \
 vcl/source/gdi/virdev \
 vcl/source/gdi/wall \
diff --git a/vcl/Package_inc.mk b/vcl/Package_inc.mk
index 3f95e97..e791793 100644
--- a/vcl/Package_inc.mk
+++ b/vcl/Package_inc.mk
@@ -150,6 +150,7 @@ $(eval $(call 
gb_Package_add_file,vcl_inc,inc/vcl/tabctrl.hxx,vcl/tabctrl.hxx))
 $(eval $(call gb_Package_add_file,vcl_inc,inc/vcl/tabdlg.hxx,vcl/tabdlg.hxx))
 $(eval $(call gb_Package_add_file,vcl_inc,inc/vcl/tabpage.hxx,vcl/tabpage.hxx))
 $(eval $(call 
gb_Package_add_file,vcl_inc,inc/vcl/taskpanelist.hxx,vcl/taskpanelist.hxx))
+$(eval $(call 
gb_Package_add_file,vcl_inc,inc/vcl/temporaryfonts.hxx,vcl/temporaryfonts.hxx))
 $(eval $(call 
gb_Package_add_file,vcl_inc,inc/vcl/textdata.hxx,vcl/textdata.hxx))
 $(eval $(call gb_Package_add_file,vcl_inc,inc/vcl/texteng.hxx,vcl/texteng.hxx))
 $(eval $(call 
gb_Package_add_file,vcl_inc,inc/vcl/textview.hxx,vcl/textview.hxx))
diff --git a/vcl/generic/fontmanager/fontmanager.cxx 
b/vcl/generic/fontmanager/fontmanager.cxx
index 7b9294a..209970d 100644
--- a/vcl/generic/fontmanager/fontmanager.cxx
+++ b/vcl/generic/fontmanager/fontmanager.cxx
@@ -43,8 +43,7 @@
 #include "vcl/fontmanager.hxx"
 #include "vcl/strhelper.hxx"
 #include "vcl/ppdparser.hxx"
-#include 
-#include 
+#include 
 
 #include "tools/urlobj.hxx"
 #include "tools/stream.hxx"
@@ -53,7 +52,6 @@
 #include "osl/file.hxx"
 #include "osl/process.h"
 
-#include 
 #include "rtl/tencinfo.h"
 #include "rtl/ustrbuf.hxx"
 #include "rtl/strbuf.hxx"
@@ -1037,7 +1035,7 @@ PrintFontManager::~PrintFontManager()
 delete m_pAtoms;
 if( m_pFontCache )
 delete m_pFontCache;
-cleanTemporaryFonts();
+TemporaryFonts::clear();
 }
 
 // -
@@ -1664,7 +1662,7 @@ void PrintFontManager::initialize()
 CALLGRIND_ZERO_STATS();
 #endif
 
-cleanTemporaryFonts();
+TemporaryFonts::clear();
 
 long aDirEntBuffer[ (sizeof(struct dirent)+_PC_NAME_MAX)+1 ];
 
@@ -3063,41 +3061,4 @@ bool PrintFontManager::readOverrideMetrics()
 return true;
 }
 
-void PrintFontManager::cleanTemporaryFonts()
-{
-OUString path = "${$BRAND_BASE_DIR/program/" SAL_CONFIGFILE( "bootstrap") 
":

valgrinding with python stuff active

2012-09-07 Thread Caolán McNamara
FWIW, I patched the *internal* to be valgrind friendly when G_SLICE is
set. So if you have to debug something where python is alive you could
try building against the internal python and it should "just work" with
your usual export VALGRIND=memcheck foo

C.

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


[PUSHED] OUString cleanup in accessibility

2012-09-07 Thread Gerrit
>From Olivier Hallot :

Olivier Hallot has submitted this change and it was merged.

Change subject: OUString cleanup in accessibility
..


OUString cleanup in accessibility

Change-Id: Ie4e51632bf81053f0d32428bb7ef5b01ec301ccc
---
M accessibility/source/extended/AccessibleBrowseBox.cxx
M accessibility/source/extended/AccessibleBrowseBoxBase.cxx
M accessibility/source/extended/AccessibleBrowseBoxCheckBoxCell.cxx
M accessibility/source/extended/AccessibleBrowseBoxHeaderBar.cxx
M accessibility/source/extended/AccessibleBrowseBoxHeaderCell.cxx
M accessibility/source/extended/AccessibleBrowseBoxTable.cxx
M accessibility/source/extended/AccessibleBrowseBoxTableBase.cxx
M accessibility/source/extended/AccessibleBrowseBoxTableCell.cxx
M accessibility/source/extended/AccessibleGridControl.cxx
M accessibility/source/extended/AccessibleGridControlBase.cxx
M accessibility/source/extended/AccessibleGridControlHeader.cxx
M accessibility/source/extended/AccessibleGridControlHeaderCell.cxx
M accessibility/source/extended/AccessibleGridControlTable.cxx
M accessibility/source/extended/AccessibleGridControlTableBase.cxx
M accessibility/source/extended/AccessibleGridControlTableCell.cxx
M accessibility/source/extended/AccessibleToolPanelDeck.cxx
M accessibility/source/extended/AccessibleToolPanelDeckTabBar.cxx
M accessibility/source/extended/AccessibleToolPanelDeckTabBarItem.cxx
M accessibility/source/extended/accessiblebrowseboxcell.cxx
M accessibility/source/extended/accessibleeditbrowseboxcell.cxx
M accessibility/source/extended/accessibleiconchoicectrl.cxx
M accessibility/source/extended/accessibleiconchoicectrlentry.cxx
M accessibility/source/extended/accessiblelistbox.cxx
M accessibility/source/extended/accessiblelistboxentry.cxx
M accessibility/source/extended/accessibletabbar.cxx
M accessibility/source/extended/accessibletabbarpage.cxx
M accessibility/source/extended/accessibletabbarpagelist.cxx
M accessibility/source/extended/accessibletablistboxtable.cxx
M accessibility/source/extended/textwindowaccessibility.cxx
M accessibility/source/helper/accresmgr.cxx
M accessibility/source/helper/characterattributeshelper.cxx
M accessibility/source/standard/accessiblemenubasecomponent.cxx
M accessibility/source/standard/accessiblemenucomponent.cxx
M accessibility/source/standard/accessiblemenuitemcomponent.cxx
M accessibility/source/standard/vclxaccessiblebox.cxx
M accessibility/source/standard/vclxaccessiblebutton.cxx
M accessibility/source/standard/vclxaccessiblecheckbox.cxx
M accessibility/source/standard/vclxaccessiblecombobox.cxx
M accessibility/source/standard/vclxaccessibledropdowncombobox.cxx
M accessibility/source/standard/vclxaccessibledropdownlistbox.cxx
M accessibility/source/standard/vclxaccessibleedit.cxx
M accessibility/source/standard/vclxaccessiblefixedhyperlink.cxx
M accessibility/source/standard/vclxaccessiblefixedtext.cxx
M accessibility/source/standard/vclxaccessiblelist.cxx
M accessibility/source/standard/vclxaccessiblelistbox.cxx
M accessibility/source/standard/vclxaccessiblelistitem.cxx
M accessibility/source/standard/vclxaccessiblemenu.cxx
M accessibility/source/standard/vclxaccessiblemenubar.cxx
M accessibility/source/standard/vclxaccessiblemenuitem.cxx
M accessibility/source/standard/vclxaccessiblemenuseparator.cxx
M accessibility/source/standard/vclxaccessiblepopupmenu.cxx
M accessibility/source/standard/vclxaccessibleradiobutton.cxx
M accessibility/source/standard/vclxaccessiblescrollbar.cxx
M accessibility/source/standard/vclxaccessiblestatusbar.cxx
M accessibility/source/standard/vclxaccessiblestatusbaritem.cxx
M accessibility/source/standard/vclxaccessibletabcontrol.cxx
M accessibility/source/standard/vclxaccessibletabpage.cxx
M accessibility/source/standard/vclxaccessibletextfield.cxx
M accessibility/source/standard/vclxaccessibletoolbox.cxx
M accessibility/source/standard/vclxaccessibletoolboxitem.cxx
60 files changed, 559 insertions(+), 605 deletions(-)

Approvals:
  Olivier Hallot: Verified; Looks good to me, approved


--
To view, visit https://gerrit.libreoffice.org/579
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie4e51632bf81053f0d32428bb7ef5b01ec301ccc
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Ricardo Montania 
Gerrit-Reviewer: Olivier Hallot 

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


[Libreoffice-commits] .: Branch 'feature/unitymenus' - vcl/inc vcl/unx

2012-09-07 Thread Libreoffice Gerrit user
 vcl/inc/unx/gtk/gtksalmenu.hxx|2 
 vcl/unx/gtk/window/gtksalmenu.cxx |  124 +-
 2 files changed, 33 insertions(+), 93 deletions(-)

New commits:
commit 0363a1a01b579bfba5e608420b211e26b923d85e
Author: Antonio Fernandez 
Date:   Fri Sep 7 13:19:08 2012 +0100

Radio buttons are now displayed correctly.

Change-Id: I8a07fc7ca31b3f39e4fd2fd2c926d9ab461b4af3

diff --git a/vcl/inc/unx/gtk/gtksalmenu.hxx b/vcl/inc/unx/gtk/gtksalmenu.hxx
index dc30260..38ac784 100644
--- a/vcl/inc/unx/gtk/gtksalmenu.hxx
+++ b/vcl/inc/unx/gtk/gtksalmenu.hxx
@@ -89,7 +89,7 @@ public:
 voidNativeSetItemText( unsigned nSection, unsigned 
nItemPos, const rtl::OUString& rText );
 voidNativeSetItemCommand( unsigned nSection, 
unsigned nItemPos, GtkSalMenuItem* pItem, const gchar* aCommandStr );
 voidNativeSetEnableItem( gchar* aCommand, gboolean 
bEnable );
-voidNativeCheckItem( unsigned nSection, unsigned 
nItemPos, GtkSalMenuItem* pItem, gboolean bCheck );
+voidNativeCheckItem( unsigned nSection, unsigned 
nItemPos, MenuItemBits bits, gboolean bCheck );
 voidNativeSetAccelerator( unsigned nSection, 
unsigned nItemPos, const KeyCode& rKeyCode, const rtl::OUString& rKeyName );
 };
 
diff --git a/vcl/unx/gtk/window/gtksalmenu.cxx 
b/vcl/unx/gtk/window/gtksalmenu.cxx
index af4df22..d6c96a4 100644
--- a/vcl/unx/gtk/window/gtksalmenu.cxx
+++ b/vcl/unx/gtk/window/gtksalmenu.cxx
@@ -79,6 +79,7 @@ static void UpdateNativeMenu( GtkSalMenu* pMenu )
 sal_Bool itemEnabled = pVCLMenu->IsItemEnabled( nId );
 KeyCode nAccelKey = pVCLMenu->GetAccelKey( nId );
 sal_Bool itemChecked = pVCLMenu->IsItemChecked( nId );
+MenuItemBits itemBits = pVCLMenu->GetItemBits( nId );
 
 // Convert internal values to native values.
 gboolean bChecked = ( itemChecked == sal_True ) ? TRUE : FALSE;
@@ -93,7 +94,9 @@ static void UpdateNativeMenu( GtkSalMenu* pMenu )
 {
 pMenu->NativeSetItemCommand( nSection, nItemPos, pSalMenuItem, 
aNativeCommand );
 pMenu->NativeSetEnableItem( aNativeCommand, bEnabled );
-pMenu->NativeCheckItem( nSection, nItemPos, pSalMenuItem, bChecked 
);
+
+if ( ( itemBits & MIB_CHECKABLE ) || ( itemBits & MIB_RADIOCHECK ) 
)
+pMenu->NativeCheckItem( nSection, nItemPos, itemBits, bChecked 
);
 }
 
 g_free( aNativeCommand );
@@ -193,49 +196,6 @@ rtl::OUString GetGtkKeyName( rtl::OUString keyName )
 return aGtkKeyName;
 }
 
-//GVariant* GetRadioButtonHints( GtkSalMenuItem *pSalMenuItem )
-//{
-//GVariantBuilder *pBuilder;
-//GVariant *pHints;
-
-//pBuilder = g_variant_builder_new( G_VARIANT_TYPE_STRING );
-
-//Menu* pMenu = pSalMenuItem->mpVCLMenu;
-
-//gboolean bItemIncluded = FALSE;
-
-//for ( sal_uInt16 i = 0; i < pMenu->GetItemCount(); i++ )
-//{
-//sal_uInt16 nId = pMenu->GetItemId( i );
-//MenuItemBits itemBits = pMenu->GetItemBits( nId );
-//MenuItemType itemType = pMenu->GetItemType( nId );
-
-//if ( itemBits & MIB_RADIOCHECK )
-//{
-//rtl::OString aValue = rtl::OUStringToOString( 
pMenu->GetItemText( nId ), RTL_TEXTENCODING_UTF8 );
-//g_variant_builder_add( pBuilder, "s", aValue.getStr() );
-
-//if ( nId == pSalMenuItem->mnId )
-//bItemIncluded = TRUE;
-//}
-//else if ( itemType == MENUITEM_SEPARATOR )
-//{
-//if ( bItemIncluded == FALSE )
-//{
-//g_variant_builder_clear( pBuilder );
-//}
-//else
-//break;
-//}
-//}
-
-//// Build an array of G_VARIANT_TYPE_STRING.
-//pHints = g_variant_new ("as", pBuilder);
-//g_variant_builder_unref ( pBuilder );
-
-//return pHints;
-//}
-
 /*
  * GtkSalMenu
  */
@@ -269,7 +229,7 @@ GtkSalMenu::~GtkSalMenu()
 
 sal_Bool GtkSalMenu::VisibleMenuBar()
 {
-return sal_True;
+return sal_False;
 }
 
 void GtkSalMenu::InsertItem( SalMenuItem* pSalMenuItem, unsigned nPos )
@@ -310,17 +270,15 @@ void GtkSalMenu::SetFrame( const SalFrame* pFrame )
 GdkWindow *gdkWindow = gtk_widget_get_window( widget );
 
 if (gdkWindow) {
-GLOMenu* pMainMenu = G_LO_MENU( g_object_get_data( G_OBJECT( gdkWindow 
), "g-lo-menubar" ) );
-GLOActionGroup* pActionGroup = G_LO_ACTION_GROUP( g_object_get_data( 
G_OBJECT( gdkWindow ), "g-lo-action-group" ) );
+mpMenuModel = G_MENU_MODEL( g_object_get_data( G_OBJECT( gdkWindow ), 
"g-lo-menubar" ) );
+mpActionGroup = G_ACTION_GROUP( g_object_get_data( G_OBJECT( gdkWindow 
), "g-lo-action-group" ) );
 
-if ( pMainMenu && pActionGroup ) {
-g_lo_menu_remove( pMainMenu, 0 );
-} else {
-pMainMenu = g_lo_menu_n

[PUSHED 3-6] fdo#53337 crash in search in help

2012-09-07 Thread Michael Meeks

On Wed, 2012-08-29 at 08:51 +0100, Caolán McNamara wrote:
> This is reported on the vanilla MacOSX and Linux build binaries, but
> doesn't happen locally for more recent tool chains. My guess is that its
> visibility related and missing typeinfo so the exception doesn't get
> caught where it should be.

Looks sensible, pushed :-)

Thanks,

Michael.

-- 
michael.me...@suse.com  <><, Pseudo Engineer, itinerant idiot

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


[Libreoffice-commits] .: Branch 'libreoffice-3-6' - l10ntools/source xmlhelp/source

2012-09-07 Thread Libreoffice Gerrit user
 l10ntools/source/help/LuceneHelper.hxx|8 
 xmlhelp/source/cxxhelp/provider/resultsetforquery.cxx |6 ++
 2 files changed, 14 insertions(+)

New commits:
commit a251a353300e27625741338b6290a109c5588952
Author: Caolán McNamara 
Date:   Fri Aug 24 11:01:51 2012 +0100

Related: fdo#53337 Maybe this is a typeinfo visibility problem

Change-Id: I024de5b400d0f7a443d4cce7bfe4e88841e6049b
Signed-off-by: Michael Meeks 

diff --git a/l10ntools/source/help/LuceneHelper.hxx 
b/l10ntools/source/help/LuceneHelper.hxx
index c990647..a0248f8 100644
--- a/l10ntools/source/help/LuceneHelper.hxx
+++ b/l10ntools/source/help/LuceneHelper.hxx
@@ -35,9 +35,17 @@
 #pragma warning(disable : 4068 4263 4264 4266)
 #endif
 
+#if defined(__GNUC__) && defined(HAVE_GCC_VISIBILITY_FEATURE)
+#  pragma GCC visibility push (default)
+#endif
+
 #include 
 #include 
 
+#if defined(__GNUC__) && defined(HAVE_GCC_VISIBILITY_FEATURE)
+#  pragma GCC visibility pop
+#endif
+
 #if defined _MSC_VER
 #pragma warning(pop)
 #endif
diff --git a/xmlhelp/source/cxxhelp/provider/resultsetforquery.cxx 
b/xmlhelp/source/cxxhelp/provider/resultsetforquery.cxx
index 240edff..4f221f6 100644
--- a/xmlhelp/source/cxxhelp/provider/resultsetforquery.cxx
+++ b/xmlhelp/source/cxxhelp/provider/resultsetforquery.cxx
@@ -40,7 +40,13 @@
 #pragma warning(disable : 4068 4263 4264 4266)
 #endif
 
+#if defined(__GNUC__) && defined(HAVE_GCC_VISIBILITY_FEATURE)
+#  pragma GCC visibility push (default)
+#endif
 #include 
+#if defined(__GNUC__) && defined(HAVE_GCC_VISIBILITY_FEATURE)
+#  pragma GCC visibility pop
+#endif
 
 #if defined _MSC_VER
 #pragma warning(pop)
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: need help fixing some files with weird line feeds in the LO tree

2012-09-07 Thread khagaroth
git config --global core.autocrlf true

Also on Windows I think it's better to use msysgit than the one in cygwin.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


impress / trivial crash after undo ...

2012-09-07 Thread Michael Meeks

IIRC this was a HardHack, and what a rats nest it seems to be:

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

I have an horrible hack around it ;-) IMHO the design and attempt to
track those ordinals looks fragile, over-optimised and generally
problematic. My guess would be that a better data structure might make
things simpler & faster; if we could work out what that looked like:
presumably with a more sensible lifecycle mechanism, and perhaps leaving
undo objects in the page, but making them invisible might work better
(?) - with a plain list of refs for ultimate removal when pruning the
undo queue size (?).

Anyhow - thoughts / better fixes much appreciated.

ATB,

Michael.

-- 
michael.me...@suse.com  <><, Pseudo Engineer, itinerant idiot

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


[Libreoffice-commits] .: i18npool/qa i18npool/source

2012-09-07 Thread Libreoffice Gerrit user
 i18npool/qa/cppunit/test_breakiterator.cxx |   16 
 i18npool/source/breakiterator/data/README  |8 +---
 2 files changed, 17 insertions(+), 7 deletions(-)

New commits:
commit 346cf4ee5d2f82b59900de1f71160c0d90ffab41
Author: Caolán McNamara 
Date:   Fri Sep 7 12:39:02 2012 +0100

Related: #i50172# add regression test for Tamil cursoring

Change-Id: I8f6c3814aa3630f5f640f611fb51ce72641715c6

diff --git a/i18npool/qa/cppunit/test_breakiterator.cxx 
b/i18npool/qa/cppunit/test_breakiterator.cxx
index c00b086..aa4e943 100644
--- a/i18npool/qa/cppunit/test_breakiterator.cxx
+++ b/i18npool/qa/cppunit/test_breakiterator.cxx
@@ -573,6 +573,7 @@ void TestBreakIterator::testWordBoundaries()
 //See https://bugs.freedesktop.org/show_bug.cgi?id=40292
 //See https://issues.apache.org/ooo/show_bug.cgi?id=80412
 //See https://issues.apache.org/ooo/show_bug.cgi?id=52
+//See https://issues.apache.org/ooo/show_bug.cgi?id=50172
 void TestBreakIterator::testGraphemeIteration()
 {
 lang::Locale aLocale;
@@ -640,6 +641,21 @@ void TestBreakIterator::testGraphemeIteration()
 }
 
 {
+const sal_Unicode KA_VOWELSIGNU[] = { 0x0B95, 0x0BC1 };
+rtl::OUString aTest(KA_VOWELSIGNU, SAL_N_ELEMENTS(KA_VOWELSIGNU));
+
+sal_Int32 nDone=0;
+sal_Int32 nPos = 0;
+
+nPos = m_xBreak->nextCharacters(aTest, 0, aLocale,
+i18n::CharacterIteratorMode::SKIPCELL, 1, nDone);
+CPPUNIT_ASSERT_MESSAGE("Should skip full grapheme", nPos == 
SAL_N_ELEMENTS(KA_VOWELSIGNU));
+nPos = m_xBreak->previousCharacters(aTest, 
SAL_N_ELEMENTS(KA_VOWELSIGNU), aLocale,
+i18n::CharacterIteratorMode::SKIPCELL, 1, nDone);
+CPPUNIT_ASSERT_MESSAGE("Should skip full grapheme", nPos == 0);
+}
+
+{
 const sal_Unicode 
CA_VOWELSIGNI_TA_VIRAMA_TA_VOWELSIGNI_RA_VOWELSIGNAI[] =
 { 0x0B9A, 0x0BBF, 0x0BA4, 0x0BCD, 0x0BA4, 0x0BBF, 0x0BB0, 0x0BC8 };
 rtl::OUString 
aTest(CA_VOWELSIGNI_TA_VIRAMA_TA_VOWELSIGNI_RA_VOWELSIGNAI,
diff --git a/i18npool/source/breakiterator/data/README 
b/i18npool/source/breakiterator/data/README
index a4dc9e8..2b4e7cd 100644
--- a/i18npool/source/breakiterator/data/README
+++ b/i18npool/source/breakiterator/data/README
@@ -378,13 +378,6 @@ Date:   Mon Oct 17 14:43:17 2005 +
 INTEGRATION: CWS i18n20 (1.1.2); FILE ADDED
 2005/09/22 23:47:49 khong 1.1.2.1: #i51661# add quotation mark as middle 
letter for Hebrew in word breakiterator rule.
 
-commit 268567aa88a991e055c41d9115bccce951f7e308
-Author: Rüdiger Timm 
-Date:   Mon Oct 17 14:42:47 2005 +
-
-INTEGRATION: CWS i18n20 (1.1.2); FILE ADDED
-2005/08/26 23:33:38 khong 1.1.2.1: #i50172# add cell breakiterator rule 
for Tamil
-
 commit a428a8927006a10ccfe7182e6fe5a8b677281eca
 Author: Rüdiger Timm 
 Date:   Mon Oct 17 14:42:30 2005 +
@@ -536,6 +529,7 @@ Date:   Mon Mar 8 16:17:05 2004 +
 
 done, regression tests added:
 
+#i50172# add cell breakiterator rule for Tamil
 #i80412# indic cursoring
 #i107843# em-dash/en-dash breakiterator fix for spell checking
 #i103552# Japanese word for 'shutdown' added to ja.dic
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


need help fixing some files with weird line feeds in the LO tree

2012-09-07 Thread Noel Grandin

Hi

In my java cleanup work, the Java files in
   nlpsolver/ThirdParty/EvolutionarySolver/src/net/adaptivebox/
appear to have weird line endings that make it hard for me to create 
patches.


When I do a normal
  git commit
  git format-patch
from my Windows Cygwin shell, the resulting patches don't apply cleanly 
on my linux box.


Anybody willing to fix this up?
Don't know enough about GIT's line encoding strategy to sort this out 
myself.


Thanks, Noel Grandin

Disclaimer: http://www.peralex.com/disclaimer.html


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


[REVIEW: 3-6] fdo#54620 crash when merging table cells in writer

2012-09-07 Thread David Tardon
Hi all,

commit 06ae26b5676c29c4d1cc9c65f5bbaede9483a21d fixes crash than happens
in specific situation when merging table cells in writer. It is a
regression caused by STL conversion. The cause is use of vector iterator
after calling insert on the vector.

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


[REVIEW: 3-6] fdo#54450 install evolution datasource driver

2012-09-07 Thread David Tardon
Hi all,

commit 5e19849479ee25e1ed1c7965588ce5497831614b fixes inclusion of some
gnome/gtk related files in the installation package(s), which would be
missing if the configuration did not contain at least one of
--enable-gio, --enable-gnomevfs or --enable-gtk3 options: Notably:
* if configuring with --enable-gtk (which is the default),
  libvclplug_gtklo.so would not be installed
* if configuring with --enable-evolution2, libevoablo.so would not be
  installed

This does not affect the official build, because it is configured with
--enable-gnomevfs, but I think it does not hurt to have it in 3.6, if
there is anyone who wants to review the fix :-)

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


[Libreoffice-commits] .: i18npool/qa i18npool/source

2012-09-07 Thread Libreoffice Gerrit user
 i18npool/qa/cppunit/test_breakiterator.cxx |   21 -
 i18npool/source/breakiterator/data/README  |   14 +-
 2 files changed, 21 insertions(+), 14 deletions(-)

New commits:
commit 9b69085c5da49aa64c531b27d078a3f4a353103c
Author: Caolán McNamara 
Date:   Fri Sep 7 12:20:24 2012 +0100

Related: #i80412# add regression test for Indic language cursoring

Change-Id: Ia1cc6ade8d2122abf5469ec521b2883961121a04

diff --git a/i18npool/qa/cppunit/test_breakiterator.cxx 
b/i18npool/qa/cppunit/test_breakiterator.cxx
index f8f4d71..c00b086 100644
--- a/i18npool/qa/cppunit/test_breakiterator.cxx
+++ b/i18npool/qa/cppunit/test_breakiterator.cxx
@@ -570,8 +570,9 @@ void TestBreakIterator::testWordBoundaries()
 }
 }
 
-//See http://qa.openoffice.org/issues/show_bug.cgi?id=52
 //See https://bugs.freedesktop.org/show_bug.cgi?id=40292
+//See https://issues.apache.org/ooo/show_bug.cgi?id=80412
+//See https://issues.apache.org/ooo/show_bug.cgi?id=52
 void TestBreakIterator::testGraphemeIteration()
 {
 lang::Locale aLocale;
@@ -681,6 +682,24 @@ void TestBreakIterator::testGraphemeIteration()
 
 CPPUNIT_ASSERT_MESSAGE("Should be considered 1 grapheme", 
nGraphemeCount == 1);
 }
+
+aLocale.Language = rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("hi"));
+aLocale.Country = rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("IN"));
+
+{
+const sal_Unicode SHA_VOWELSIGNII[] = { 0x936, 0x940 };
+rtl::OUString aTest(SHA_VOWELSIGNII, SAL_N_ELEMENTS(SHA_VOWELSIGNII));
+
+sal_Int32 nDone=0;
+sal_Int32 nPos = 0;
+
+nPos = m_xBreak->nextCharacters(aTest, 0, aLocale,
+i18n::CharacterIteratorMode::SKIPCELL, 1, nDone);
+CPPUNIT_ASSERT_MESSAGE("Should skip full grapheme", nPos == 
SAL_N_ELEMENTS(SHA_VOWELSIGNII));
+nPos = m_xBreak->previousCharacters(aTest, 
SAL_N_ELEMENTS(SHA_VOWELSIGNII), aLocale,
+i18n::CharacterIteratorMode::SKIPCELL, 1, nDone);
+CPPUNIT_ASSERT_MESSAGE("Should skip full grapheme", nPos == 0);
+}
 }
 
 //A test to ensure that certain ranges and codepoints that are categorized as
diff --git a/i18npool/source/breakiterator/data/README 
b/i18npool/source/breakiterator/data/README
index 22136e7..a4dc9e8 100644
--- a/i18npool/source/breakiterator/data/README
+++ b/i18npool/source/breakiterator/data/README
@@ -23,19 +23,6 @@ Date:   Thu Aug 6 18:13:57 2009 +
 CWS-TOOLING: integrate CWS tl73
 2009-07-31 15:29:33 +0200 tl  r274535 : #i64400# dash/hyphen should not 
break words
 
-commit 4e1fd2fa161708049f1c3b6039659d0bf4f4539e
-Author: Jens-Heiner Rechtien 
-Date:   Tue Jan 6 12:54:16 2009 +
-
-2008-12-03 16:14:17 +0100 erack  r264797 : #i93694# update script+language 
-> unicode digit mapping; patch from 
-2008-11-25 02:01:18 +0100 erack  r264270 : #i83349# apply remaining parts 
of the patch, now that we use ICU 4.0; contributed by 
-
-commit 16b796fcce6430448535e3692f624cb469380442
-Author: Release Engineers 
-Date:   Thu Oct 2 13:51:29 2008 +
-
-#i80412#
-
 commit 9964a76ef58786bba47d409970512d7ded6c8889
 Author: Rüdiger Timm 
 Date:   Wed Jul 2 07:53:05 2008 +
@@ -549,6 +536,7 @@ Date:   Mon Mar 8 16:17:05 2004 +
 
 done, regression tests added:
 
+#i80412# indic cursoring
 #i107843# em-dash/en-dash breakiterator fix for spell checking
 #i103552# Japanese word for 'shutdown' added to ja.dic
 #i113785# ligatures for spell checking will no longer break words
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: [PATCH] Higher‐resolution OpenDocument icons for Mac OS X

2012-09-07 Thread Caolán McNamara
On Tue, 2012-08-28 at 16:19 +0200, Stefan Knorr (Astron) wrote:
> Hi Nicholas,
> 
> I am taking the liberty to add the design list into CC here...
> 
> Astron.
> 
> On 27 August 2012 16:58, Nicholas Shanks  wrote:
> > ... The Mac icons were
> > only provided up to 128px in size.
> > So I have created revised versions of the
> > sysui/desktop/icons/oasis-*.icns files, derived directly from the PNG
> > files.
> >
> > The Mac icon format also supports 512x512 and 1024x1024 sized icons,
> > so I have generated PNGs for these sizes and included them in the icns
> > ...
> > If you extract the attached tape archive into the sysui/desktop/
> > directory everything should be placed in the correct location.

So, re the contents of the icons.tar file, they look fine to me. I'm
minded to add them in unless someone has some argument against that ?

C.

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


[Libreoffice-commits] .: scp2/InstallModule_gnome.mk scp2/InstallScript_setup_osl.mk scp2/Module_scp2.mk scp2/source

2012-09-07 Thread Libreoffice Gerrit user
 scp2/InstallModule_gnome.mk|2 ++
 scp2/InstallScript_setup_osl.mk|2 +-
 scp2/Module_scp2.mk|2 +-
 scp2/source/gnome/module_gnome.scp |   12 
 4 files changed, 12 insertions(+), 6 deletions(-)

New commits:
commit 5e19849479ee25e1ed1c7965588ce5497831614b
Author: David Tardon 
Date:   Fri Sep 7 12:40:05 2012 +0200

fdo#54450 install evolution DB driver correctly

Change-Id: I0b6eb67014a61e22981f1f6be488fa762d639554

diff --git a/scp2/InstallModule_gnome.mk b/scp2/InstallModule_gnome.mk
index ea20b37..15e8116 100644
--- a/scp2/InstallModule_gnome.mk
+++ b/scp2/InstallModule_gnome.mk
@@ -33,9 +33,11 @@ $(eval $(call gb_InstallModule_use_packages,scp2/gnome,\
 ))
 
 $(eval $(call gb_InstallModule_define_if_set,scp2/gnome,\
+   ENABLE_EVOAB2 \
ENABLE_GCONF \
ENABLE_GIO \
ENABLE_GNOMEVFS \
+   ENABLE_GTK \
ENABLE_GTK3 \
ENABLE_LOCKDOWN \
ENABLE_SYSTRAY_GTK \
diff --git a/scp2/InstallScript_setup_osl.mk b/scp2/InstallScript_setup_osl.mk
index 5eca444..cbe59b4 100644
--- a/scp2/InstallScript_setup_osl.mk
+++ b/scp2/InstallScript_setup_osl.mk
@@ -58,7 +58,7 @@ $(eval $(call gb_InstallScript_use_modules,setup_osl,\
$(if $(filter YES,$(WITH_EXTENSION_INTEGRATION)),\
scp2/extensions \
) \
-   $(if $(filter TRUE,$(ENABLE_GCONF) $(ENABLE_GNOMEVFS) $(ENABLE_GIO)),\
+   $(if $(filter TRUE,$(ENABLE_EVOAB2) $(ENABLE_GCONF) $(ENABLE_GNOMEVFS) 
$(ENABLE_GIO) $(ENABLE_GTK) $(ENABLE_GTK3)),\
scp2/gnome \
) \
$(if $(filter TRUE,$(SOLAR_JAVA)),\
diff --git a/scp2/Module_scp2.mk b/scp2/Module_scp2.mk
index 48f4686..c621cc5 100644
--- a/scp2/Module_scp2.mk
+++ b/scp2/Module_scp2.mk
@@ -66,7 +66,7 @@ $(eval $(call gb_Module_add_targets,scp2,\
$(if $(filter TRUE,$(ENABLE_CRASHDUMP)),\
InstallModule_crashrep \
) \
-   $(if $(filter TRUE,$(ENABLE_GCONF) $(ENABLE_GNOMEVFS) $(ENABLE_GIO)),\
+   $(if $(filter TRUE,$(ENABLE_EVOAB2) $(ENABLE_GCONF) $(ENABLE_GNOMEVFS) 
$(ENABLE_GIO) $(ENABLE_GTK) $(ENABLE_GTK3)),\
InstallModule_gnome \
) \
$(if $(filter TRUE,$(SOLAR_JAVA)),\
diff --git a/scp2/source/gnome/module_gnome.scp 
b/scp2/source/gnome/module_gnome.scp
index 88a2926..9194825 100644
--- a/scp2/source/gnome/module_gnome.scp
+++ b/scp2/source/gnome/module_gnome.scp
@@ -41,14 +41,16 @@ Module gid_Module_Optional_Gnome
 XpdCheckSolaris = "SUNWgnome-vfs";
 #endif
 Files = (
+#ifdef ENABLE_EVOAB2
   gid_File_Lib_Evoab,
+#endif
+#ifdef ENABLE_GTK
   gid_File_Lib_Fps_Gnome,
   gid_File_Lib_Vclplug_Gtk,
+#endif
 #ifdef ENABLE_GTK3
   gid_File_Lib_Vclplug_Gtk3,
 #endif
-  gid_File_Bin_Gnome_Open_Url,
-  gid_File_Bin_Gnome_Open_Url_Bin,
 #ifdef ENABLE_SYSTRAY_GTK
   gid_File_Lib_QStart_Gtk,
 #endif
@@ -59,11 +61,13 @@ Module gid_Module_Optional_Gnome
   gid_File_Lib_GIO,
 #endif
 #ifdef ENABLE_GCONF
-  gid_File_Lib_Gconfbe
+  gid_File_Lib_Gconfbe,
 #endif
 #if (defined ENABLE_GNOMEVFS && defined ENABLE_LOCKDOWN) || defined ENABLE_GIO
-  gid_File_Share_Registry_Gnome_Xcd
+  gid_File_Share_Registry_Gnome_Xcd,
 #endif
+  gid_File_Bin_Gnome_Open_Url,
+  gid_File_Bin_Gnome_Open_Url_Bin
 );
 End
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: Branch 'libreoffice-3-6' - vcl/source

2012-09-07 Thread Libreoffice Gerrit user
 vcl/source/app/help.cxx |6 --
 1 file changed, 4 insertions(+), 2 deletions(-)

New commits:
commit f3563cb13d679953abb5975dcd246ad60ed8045d
Author: László Németh 
Date:   Fri Sep 7 08:52:52 2012 +0200

multiline tooltip for very long (eg. footnote) texts #i42424#

Change-Id: Ib6383220c3cdd6fef55dee82fcfdc3472da45163
Signed-off-by: Andras Timar 

diff --git a/vcl/source/app/help.cxx b/vcl/source/app/help.cxx
index b968ca1..2dc76fb 100644
--- a/vcl/source/app/help.cxx
+++ b/vcl/source/app/help.cxx
@@ -52,6 +52,8 @@
 #define HELPDELAY_SHORT 2
 #define HELPDELAY_NONE  3
 
+#define HELPTEXTMAXLEN150
+
 // ===
 
 Help::Help()
@@ -359,7 +361,7 @@ HelpTextWindow::~HelpTextWindow()
 void HelpTextWindow::SetHelpText( const String& rHelpText )
 {
 maHelpText = rHelpText;
-if ( mnHelpWinStyle == HELPWINSTYLE_QUICK )
+if ( mnHelpWinStyle == HELPWINSTYLE_QUICK && maHelpText.Len() < 
HELPTEXTMAXLEN)
 {
 Size aSize;
 aSize.Height() = GetTextHeight();
@@ -421,7 +423,7 @@ void HelpTextWindow::Paint( const Rectangle& )
 }
 
 // paint text
-if ( mnHelpWinStyle == HELPWINSTYLE_QUICK )
+if ( mnHelpWinStyle == HELPWINSTYLE_QUICK && maHelpText.Len() < 
HELPTEXTMAXLEN)
 {
 if ( mnStyle & QUICKHELP_CTRLTEXT )
 DrawCtrlText( maTextRect.TopLeft(), maHelpText );
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: Branch 'libreoffice-3-5' - vcl/source

2012-09-07 Thread Libreoffice Gerrit user
 vcl/source/app/help.cxx |6 --
 1 file changed, 4 insertions(+), 2 deletions(-)

New commits:
commit a5d8d122dbef5c33f65d9e667989936a940b66de
Author: László Németh 
Date:   Fri Sep 7 08:52:52 2012 +0200

multiline tooltip for very long (eg. footnote) texts #i42424#

Change-Id: Ib6383220c3cdd6fef55dee82fcfdc3472da45163
Signed-off-by: Andras Timar 

diff --git a/vcl/source/app/help.cxx b/vcl/source/app/help.cxx
index b968ca1..2dc76fb 100644
--- a/vcl/source/app/help.cxx
+++ b/vcl/source/app/help.cxx
@@ -52,6 +52,8 @@
 #define HELPDELAY_SHORT 2
 #define HELPDELAY_NONE  3
 
+#define HELPTEXTMAXLEN150
+
 // ===
 
 Help::Help()
@@ -359,7 +361,7 @@ HelpTextWindow::~HelpTextWindow()
 void HelpTextWindow::SetHelpText( const String& rHelpText )
 {
 maHelpText = rHelpText;
-if ( mnHelpWinStyle == HELPWINSTYLE_QUICK )
+if ( mnHelpWinStyle == HELPWINSTYLE_QUICK && maHelpText.Len() < 
HELPTEXTMAXLEN)
 {
 Size aSize;
 aSize.Height() = GetTextHeight();
@@ -421,7 +423,7 @@ void HelpTextWindow::Paint( const Rectangle& )
 }
 
 // paint text
-if ( mnHelpWinStyle == HELPWINSTYLE_QUICK )
+if ( mnHelpWinStyle == HELPWINSTYLE_QUICK && maHelpText.Len() < 
HELPTEXTMAXLEN)
 {
 if ( mnStyle & QUICKHELP_CTRLTEXT )
 DrawCtrlText( maTextRect.TopLeft(), maHelpText );
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PATCH] bnc#759197: adjust chart layout distance

2012-09-07 Thread Gerrit
>From Felix Zhang :

Felix Zhang has uploaded a new change for review.

Change subject: bnc#759197: adjust chart layout distance
..

bnc#759197: adjust chart layout distance

Change-Id: Ifae4faeb234630dcce79851a0ad8bb0c6960f9f7
---
M chart2/source/view/main/ChartView.cxx
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/81/581/1
--
To view, visit https://gerrit.libreoffice.org/581
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifae4faeb234630dcce79851a0ad8bb0c6960f9f7
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Felix Zhang 

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


[PUSHED] Change in core[libreoffice-3-6]: resolved fdo#52240 fdo#52137 fdo#52288 user editable date pa...

2012-09-07 Thread Gerrit
>From Caolán McNamara :

Caolán McNamara has submitted this change and it was merged.

Change subject: resolved fdo#52240 fdo#52137 fdo#52288 user editable date 
patterns
..


resolved fdo#52240 fdo#52137 fdo#52288 user editable date patterns

Implemented user editable date acceptance patterns.

The introduction of strict date parsing using locale dependent date
acceptance patterns in 3.6.0 wasn't always welcomed. Besides that not
every locale had patterns for incomplete (only day and month) date
input, users also complained about not being able to key in dates on
numeric keypads if the locale's date separator wasn't '/' or '-'

This commit implements a "Date acceptance patterns" edit field under
Tools->Options->LanguageSettings->Languages that follows the selected
locale and enables the user to add patterns.

Example de-DE locale:
* default patterns: D.M.Y;D.M.
* to enable additional input on numeric keypad: D.M.Y;D.M.;D-M-Y;D-M
  * if 3-4 shall not result in a date, D-M- could be used instead of D-M
  * note that to enter an ISO 8601 Y-M-D date with a D-M-Y pattern
active one needs to enter a year >31 or with at least 3 digits, e.g.
011

(cherry picked from commit bf10f4d62a5fe308ea47f9a0aac4f6f7e264ae3e)

Conflicts:

cui/source/options/optgdlg.hrc
svl/source/numbers/zforfind.cxx
unotools/source/config/syslocaleoptions.cxx

Change-Id: Ic1ce91b1f9d29f1837d56b45ba0ae16f6d9cb17c
---
M cui/source/options/optgdlg.cxx
M cui/source/options/optgdlg.hrc
M cui/source/options/optgdlg.hxx
M cui/source/options/optgdlg.src
M officecfg/registry/schema/org/openoffice/Setup.xcs
M svl/inc/svl/zforlist.hxx
M svl/source/numbers/zforfind.cxx
M svl/source/numbers/zforfind.hxx
M svl/source/numbers/zforlist.cxx
M unotools/inc/unotools/localedatawrapper.hxx
M unotools/inc/unotools/syslocaleoptions.hxx
M unotools/source/config/syslocaleoptions.cxx
M unotools/source/i18n/localedatawrapper.cxx
M unotools/source/misc/syslocale.cxx
14 files changed, 429 insertions(+), 46 deletions(-)

Approvals:
  Caolán McNamara: Looks good to me, approved
  Markus Mohrhard: Looks good to me, but someone else must approve
  Petr Mladek: Verified; Looks good to me, but someone else must approve


--
To view, visit https://gerrit.libreoffice.org/511
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic1ce91b1f9d29f1837d56b45ba0ae16f6d9cb17c
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-3-6
Gerrit-Owner: Eike Rathke 
Gerrit-Reviewer: Caolán McNamara 
Gerrit-Reviewer: Markus Mohrhard 
Gerrit-Reviewer: Petr Mladek 

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


[Libreoffice-commits] .: Branch 'libreoffice-3-6' - cui/source officecfg/registry svl/inc svl/source unotools/inc unotools/source

2012-09-07 Thread Libreoffice Gerrit user
 cui/source/options/optgdlg.cxx |  128 -
 cui/source/options/optgdlg.hrc |3 
 cui/source/options/optgdlg.hxx |4 
 cui/source/options/optgdlg.src |   52 +---
 officecfg/registry/schema/org/openoffice/Setup.xcs |8 +
 svl/inc/svl/zforlist.hxx   |4 
 svl/source/numbers/zforfind.cxx|   55 +++--
 svl/source/numbers/zforfind.hxx|2 
 svl/source/numbers/zforlist.cxx|   37 +++---
 unotools/inc/unotools/localedatawrapper.hxx|6 
 unotools/inc/unotools/syslocaleoptions.hxx |   18 ++
 unotools/source/config/syslocaleoptions.cxx|   68 ++-
 unotools/source/i18n/localedatawrapper.cxx |   59 +
 unotools/source/misc/syslocale.cxx |   31 +
 14 files changed, 429 insertions(+), 46 deletions(-)

New commits:
commit cfbfa26deb2776e5c07463e59517eaf68c1d5d6d
Author: Eike Rathke 
Date:   Wed Aug 29 20:49:01 2012 +0200

resolved fdo#52240 fdo#52137 fdo#52288 user editable date patterns

Implemented user editable date acceptance patterns.

The introduction of strict date parsing using locale dependent date
acceptance patterns in 3.6.0 wasn't always welcomed. Besides that not
every locale had patterns for incomplete (only day and month) date
input, users also complained about not being able to key in dates on
numeric keypads if the locale's date separator wasn't '/' or '-'

This commit implements a "Date acceptance patterns" edit field under
Tools->Options->LanguageSettings->Languages that follows the selected
locale and enables the user to add patterns.

Example de-DE locale:
* default patterns: D.M.Y;D.M.
* to enable additional input on numeric keypad: D.M.Y;D.M.;D-M-Y;D-M
  * if 3-4 shall not result in a date, D-M- could be used instead of D-M
  * note that to enter an ISO 8601 Y-M-D date with a D-M-Y pattern
active one needs to enter a year >31 or with at least 3 digits, e.g.
011

(cherry picked from commit bf10f4d62a5fe308ea47f9a0aac4f6f7e264ae3e)

Conflicts:

cui/source/options/optgdlg.hrc
svl/source/numbers/zforfind.cxx
unotools/source/config/syslocaleoptions.cxx

Change-Id: Ic1ce91b1f9d29f1837d56b45ba0ae16f6d9cb17c
Reviewed-on: https://gerrit.libreoffice.org/511
Reviewed-by: Markus Mohrhard 
Reviewed-by: Petr Mladek 
Tested-by: Petr Mladek 
Reviewed-by: Caolán McNamara 

diff --git a/cui/source/options/optgdlg.cxx b/cui/source/options/optgdlg.cxx
index 72e66d2..8826a09 100644
--- a/cui/source/options/optgdlg.cxx
+++ b/cui/source/options/optgdlg.cxx
@@ -1248,6 +1248,22 @@ static OUString 
sUserLocalePath(RTL_CONSTASCII_USTRINGPARAM("org.openoffice.Offi
 static const OUString sUserLocaleKey(RTL_CONSTASCII_USTRINGPARAM("UILocale"));
 static Sequence< OUString > seqInstalledLanguages;
 
+static OUString lcl_getDatePatternsConfigString( const LocaleDataWrapper& 
rLocaleWrapper )
+{
+Sequence< OUString > aDateAcceptancePatterns = 
rLocaleWrapper.getDateAcceptancePatterns();
+sal_Int32 nPatterns = aDateAcceptancePatterns.getLength();
+rtl::OUStringBuffer aBuf( nPatterns * 6 );   // 6 := length of Y-M-D;
+SAL_WARN_IF( !nPatterns, "cui.options", "No date acceptance pattern");
+if (nPatterns)
+{
+const OUString* pPatterns = aDateAcceptancePatterns.getConstArray();
+aBuf.append( pPatterns[0]);
+for (sal_Int32 i=1; i < nPatterns; ++i)
+aBuf.append(';').append( pPatterns[i]);
+}
+return aBuf.makeStringAndClear();
+}
+
 OfaLanguagesTabPage::OfaLanguagesTabPage( Window* pParent, const SfxItemSet& 
rSet ) :
 SfxTabPage( pParent, CUI_RES( OFA_TP_LANGUAGES ), rSet ),
 aUILanguageGB(this, CUI_RES(FL_UI_LANG  )),
@@ -1261,6 +1277,9 @@ OfaLanguagesTabPage::OfaLanguagesTabPage( Window* 
pParent, const SfxItemSet& rSe
 aDecimalSeparatorCB(this,   CUI_RES(CB_DECIMALSEPARATOR)),
 aCurrencyFT( this,  CUI_RES(FT_CURRENCY   )),
 aCurrencyLB( this,  CUI_RES(LB_CURRENCY   )),
+aDatePatternsFI( this,  CUI_RES(FI_DATEPATTERNS   )),
+aDatePatternsFT( this,  CUI_RES(FT_DATEPATTERNS   )),
+aDatePatternsED( this,  CUI_RES(ED_DATEPATTERNS   )),
 aLinguLanguageGB(this,  CUI_RES(FL_LINGU_LANG   )),
 aWesternLanguageFI(this,CUI_RES(FI_WEST_LANG  )),
 aWesternLanguageFT(this,CUI_RES(FT_WEST_LANG  )),
@@ -1383,6 +1402,8 @@ OfaLanguagesTabPage::OfaLanguagesTabPage( Window* 
pParent, const SfxItemSet& rSe
 delete pLanguageTable;
 
 aLocaleSettingLB.SetSelectHdl( LINK( this, OfaLanguagesTabPage, 
LocaleSettingHdl ) );
+aDatePatternsED.SetModifyHdl( LINK( this, OfaLanguagesTabPage, 
DatePatternsHdl ) );
+
 Link aLink( LINK( this, OfaLa

Re: .: vcl/inc

2012-09-07 Thread Caolán McNamara
On Thu, 2012-09-06 at 21:40 +0200, Michael Stahl wrote:
> this didn't fix the mac tinderbox like i hoped; on further investigation
> it looks like the psp::PrintFontManager that is used in the newfangled
> embedded font support is actually UNX-specific and not built on Win/Mac
> platforms.
> 
> i don't know what to do about that...

The additions to fontmanager are just a method to return a file path to
use as a temp file based on a given name and some dtor stuff to delete
all those on exit and an activateTempFont which just sets the font on
the Application::GetDefaultDevice so at the moment the three methods are
very self-contained, and the three of them could be moved into e.g.
Application or OutputDevice or something new and all would probably work
out fine.

C.

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


  1   2   >