[Libreoffice-commits] .: solenv/inc

2012-09-20 Thread Libreoffice Gerrit user
 solenv/inc/premac.h |6 ++
 1 file changed, 6 insertions(+)

New commits:
commit 6a8694d91e051412108601806cc5325b0641b5f2
Author: Tor Lillqvist t...@iki.fi
Date:   Thu Sep 20 08:51:33 2012 +0300

Add typedefs not in the 10.4 SDK

Change-Id: I67f6c5392feda674b3d7d53b5dd20ec86e4f9023

diff --git a/solenv/inc/premac.h b/solenv/inc/premac.h
index b5fff62..5dc085c 100644
--- a/solenv/inc/premac.h
+++ b/solenv/inc/premac.h
@@ -56,4 +56,10 @@ typedef int NSColorRenderingIntent;
 #endif
 #endif
 
+#if MACOSX_SDK_VERSION  1050
+typedef float CGFloat;
+typedef int NSInteger;
+typedef unsigned NSUInteger;
+#endif
+
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2012-09-20 Thread Libreoffice Gerrit user
 liborcus/UnpackedTarball_orcus.mk  |2 +-
 liborcus/liborcus_0.1.0-warnings.patch |   14 ++
 2 files changed, 15 insertions(+), 1 deletion(-)

New commits:
commit bed5557e8bdfe5c698a45a4cc82619a7c22de001
Author: Tor Lillqvist t...@iki.fi
Date:   Thu Sep 20 09:05:57 2012 +0300

WaE: declaration of 'size' shadows a member of 'this'

Change-Id: Ie620d906184efcba0eb92cd8757a90a97d61ff87

diff --git a/liborcus/UnpackedTarball_orcus.mk 
b/liborcus/UnpackedTarball_orcus.mk
index 030eebd..f6a054b 100644
--- a/liborcus/UnpackedTarball_orcus.mk
+++ b/liborcus/UnpackedTarball_orcus.mk
@@ -14,7 +14,7 @@ $(eval $(call 
gb_UnpackedTarball_set_tarball,orcus,$(ORCUS_TARBALL)))
 $(eval $(call gb_UnpackedTarball_set_patchlevel,orcus,0))
 
 orcus_patches :=
-# -Werror,-Wunused-variable -Werror,-Wunused-private-field
+# -Werror=unused-variable -Werror=unused-private-field -Werror=shadow
 orcus_patches += liborcus_0.1.0-warnings.patch
 # make config.sub recognize arm-linux-androideabi
 orcus_patches += liborcus_0.1.0-configure.patch
diff --git a/liborcus/liborcus_0.1.0-warnings.patch 
b/liborcus/liborcus_0.1.0-warnings.patch
index 8ce6533..8eb98bb 100644
--- a/liborcus/liborcus_0.1.0-warnings.patch
+++ b/liborcus/liborcus_0.1.0-warnings.patch
@@ -60,3 +60,17 @@
  }
  else
  {
+--- include/orcus/pstring.hpp
 include/orcus/pstring.hpp
+@@ -82,8 +82,8 @@
+ };
+ 
+ pstring() : m_pos(NULL), m_size(0) {}
+-pstring(const char* pos) : m_pos(pos) { m_size = ::std::strlen(pos); }
+-pstring(const char* pos, size_t size) : m_pos(pos), m_size(size) {}
++pstring(const char* pos_) : m_pos(pos_) { m_size = ::std::strlen(pos_); }
++pstring(const char* pos_, size_t size_) : m_pos(pos_), m_size(size_) {}
+ 
+ ::std::string str() const { return ::std::string(m_pos, m_size); }
+ 
+
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2012-09-20 Thread Libreoffice Gerrit user
 sc/source/filter/inc/lotattr.hxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit acc7209c97469c6e4c7633f2a206f345139818e2
Author: Stephan Bergmann sberg...@redhat.com
Date:   Mon Sep 10 10:08:30 2012 +0200

-Werror,-Wmismatched-tags

Change-Id: I69219a86b6530a7bac5f1726e980655bf5f0
(cherry picked from commit d3bc83f04314108cda530fb6663107a45a4a1cfc)

diff --git a/sc/source/filter/inc/lotattr.hxx b/sc/source/filter/inc/lotattr.hxx
index 844da07..cc8971a 100644
--- a/sc/source/filter/inc/lotattr.hxx
+++ b/sc/source/filter/inc/lotattr.hxx
@@ -42,7 +42,7 @@ class ScPatternAttr;
 class SvxColorItem;
 class Color;
 class LotAttrTable;
-class LOTUS_ROOT;
+struct LOTUS_ROOT;
 
 namespace editeng { class SvxBorderLine; }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2012-09-20 Thread Libreoffice Gerrit user
 liborcus/liborcus_0.1.0-warnings.patch |   16 
 1 file changed, 16 insertions(+)

New commits:
commit 21b5706f83edecad8cd3d0ab3ce5110b82e75a46
Author: Tor Lillqvist t...@iki.fi
Date:   Thu Sep 20 09:29:21 2012 +0300

WaE: declaration of 'str' shadows a member of 'this'

Change-Id: Ibbbe331421434e15c4659b5c566dadecf441dff8

diff --git a/liborcus/liborcus_0.1.0-warnings.patch 
b/liborcus/liborcus_0.1.0-warnings.patch
index 8eb98bb..8ddcdad 100644
--- a/liborcus/liborcus_0.1.0-warnings.patch
+++ b/liborcus/liborcus_0.1.0-warnings.patch
@@ -73,4 +73,20 @@
  
  ::std::string str() const { return ::std::string(m_pos, m_size); }
  
+@@ -110,11 +110,11 @@
+ 
+ bool operator (const pstring r) const;
+ 
+-bool operator== (const char* str) const;
++bool operator== (const char* str_) const;
+ 
+-bool operator!= (const char* str) const
++bool operator!= (const char* str_) const
+ {
+-return !operator==(str);
++return !operator==(str_);
+ }
+ 
+ pstring trim() const;
+
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2012-09-20 Thread Libreoffice Gerrit user
 sc/source/filter/oox/extlstcontext.cxx |2 --
 1 file changed, 2 deletions(-)

New commits:
commit 973574cbd4d9ec39efff3961b8121e27890a56da
Author: Michael Stahl mst...@redhat.com
Date:   Thu Jul 12 23:45:35 2012 +0200

ExtGlobalContext::onCreateContext: warning C4702: unreachable code

Change-Id: Iccbf66d8838fcfa2f2c620dd768aba75f9e071f4
(cherry picked from commit d0e4215b653b0fab6718b06a5dc0e566170f538d)

Signed-off-by: Stephan Bergmann sberg...@redhat.com

diff --git a/sc/source/filter/oox/extlstcontext.cxx 
b/sc/source/filter/oox/extlstcontext.cxx
index 6435128..740a841 100644
--- a/sc/source/filter/oox/extlstcontext.cxx
+++ b/sc/source/filter/oox/extlstcontext.cxx
@@ -180,8 +180,6 @@ ContextHandlerRef ExtGlobalContext::onCreateContext( 
sal_Int32 nElement, const A
 else
 return this;
 }
-
-return 0;
 }
 
 void ExtGlobalContext::onStartElement( const AttributeList rAttribs )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Options included in CC/CXX vs. in gb_CFLAGS/gb_CXXFLAGS

2012-09-20 Thread Tor Lillqvist
I think there is some fuzziness about what compiler options should be
part of the CC/CXX variables and what should be in
gb_CFLAGS/gb_CXXFLAGS.

In some cases, like my Mac build tree that uses the 10.4 SDK, from
Xcode 3 installed in /Xcode3, gb_CXXFLAGS contains at least one flag
that apparently *must* be used for also external projects (well,
liborcus in particular) to compile successfully.  This is the
-isysroot from solenv/gbuild/platform/macosx.mk.

In different Mac build environments, this apparently is not the case,
there the gcc from Xcode 3 (or 2 even) works fine for liborcus even if
no -isysroot is used. These are perhaps older versions of the OS,
and/or with Xcode 3 in a different location?

In other cases (normal Linuxes?) gb_CXXFLAGS contains just optional
flags that will in fact hurt an external project that isn't prepared
for them, like -Werror.

Should the -isysroot flag actually be part ot CC/CXX, not gb_CXXFLAGS?
Should configure.in and macosx.mk be modified thusly?

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


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

2012-09-20 Thread Libreoffice Gerrit user
 liborcus/liborcus_0.1.0-warnings.patch |   15 +++
 1 file changed, 15 insertions(+)

New commits:
commit 7e871bbaf7b208dd884528259211385b8074da39
Author: Tor Lillqvist t...@iki.fi
Date:   Thu Sep 20 09:52:13 2012 +0300

WaE: declaration of 'content' shadows a member of 'this'

Change-Id: I1cc06ac0608ef159eb3027bea614b2dade8b857c

diff --git a/liborcus/liborcus_0.1.0-warnings.patch 
b/liborcus/liborcus_0.1.0-warnings.patch
index 8ddcdad..9ba566c 100644
--- a/liborcus/liborcus_0.1.0-warnings.patch
+++ b/liborcus/liborcus_0.1.0-warnings.patch
@@ -90,3 +90,18 @@
  pstring trim() const;
 
 
+--- include/orcus/sax_parser.hpp
 include/orcus/sax_parser.hpp
+@@ -159,9 +159,9 @@
+ 
+ templatetypename _Handler
+ sax_parser_Handler::sax_parser(
+-const char* content, const size_t size, handler_type handler) :
+-m_content(content),
+-m_char(content),
++const char* content_, const size_t size, handler_type handler) :
++m_content(content_),
++m_char(content_),
+ m_size(size),
+ m_pos(0),
+ m_nest_level(0),
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: Options included in CC/CXX vs. in gb_CFLAGS/gb_CXXFLAGS

2012-09-20 Thread Stephan Bergmann

On 09/20/2012 08:48 AM, Tor Lillqvist wrote:

I think there is some fuzziness about what compiler options should be
part of the CC/CXX variables and what should be in
gb_CFLAGS/gb_CXXFLAGS.

In some cases, like my Mac build tree that uses the 10.4 SDK, from
Xcode 3 installed in /Xcode3, gb_CXXFLAGS contains at least one flag
that apparently *must* be used for also external projects (well,
liborcus in particular) to compile successfully.  This is the
-isysroot from solenv/gbuild/platform/macosx.mk.

[...]


In other cases (normal Linuxes?) gb_CXXFLAGS contains just optional
flags that will in fact hurt an external project that isn't prepared
for them, like -Werror.

Should the -isysroot flag actually be part ot CC/CXX, not gb_CXXFLAGS?
Should configure.in and macosx.mk be modified thusly?


I think there has traditionally not been much consistency in this area 
in OOo.  Originally, I think this was exclusively handled via flags 
variables (and re-crafting similar such variables for passing to 
external projects, with values more or less similar to the primary OOo 
variables).  Much cruft here also stems from the fact that Sun's 
internal OOo build environment required lots of overrides (as builds on 
arbitrary machines used a centralized toolchain instead of a local one), 
and this was typically achieved with ad-hoc fixes instead of a truly 
principled approach.


That said, your rule of when to put something into CC/CXX vs. 
gb_CXXFLAGS etc. sounds good to me.


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


[Libreoffice-commits] .: 2 commits - configure.in liborcus/ExternalProject_liborcus.mk liborcus/liborcus_0.1.0-warnings.patch liborcus/UnpackedTarball_orcus.mk solenv/gbuild solenv/inc

2012-09-20 Thread Libreoffice Gerrit user
 configure.in   |   12 
 liborcus/ExternalProject_liborcus.mk   |2 -
 liborcus/UnpackedTarball_orcus.mk  |2 -
 liborcus/liborcus_0.1.0-warnings.patch |   45 -
 solenv/gbuild/platform/macosx.mk   |2 -
 solenv/inc/unxmacx.mk  |2 -
 6 files changed, 9 insertions(+), 56 deletions(-)

New commits:
commit 288d4bf31399f474aadcd16b9a800aade0741aa2
Author: Tor Lillqvist t...@iki.fi
Date:   Thu Sep 20 10:37:10 2012 +0300

Move -isysroot from gb_CFLAGS/gb_CXXFLAGS to CC/CXX

That is a much better place for it. The gb_C(XX)FLAGS don't get passed
on to external modules, so they should not contain flags that are
essential for the compilation to work at all. I think.

At least for me, in my MacOSX SDK 10.4 -based build tree, using Xcode
3 installed in /Xcode3, on OS X 10.8, the -isysroot is essential for
liborcus to compile.

Change-Id: I1f11c690585e4ba512eb87d2fddf11872b91dd57

diff --git a/configure.in b/configure.in
index c719859..1010672 100644
--- a/configure.in
+++ b/configure.in
@@ -2740,19 +2740,19 @@ if test $_os = Darwin; then
 AC_MSG_ERROR([Cannot guess gcc location for this SDK])
 ;;
 esac
-CC=${gccprefix}gcc-4.0 
-mmacosx-version-min=$with_macosx_version_min_required
-CXX=${gccprefix}g++-4.0 
-mmacosx-version-min=$with_macosx_version_min_required
+CC=${gccprefix}gcc-4.0 
-mmacosx-version-min=$with_macosx_version_min_required -isysroot 
$MACOSX_SDK_PATH
+CXX=${gccprefix}g++-4.0 
-mmacosx-version-min=$with_macosx_version_min_required -isysroot 
$MACOSX_SDK_PATH
 ;;
 10.6)
 # Is similar logic as above needed? Is it likely somebody
 # has both an older Xcode with the 10.6 SDK and a current
 # Xcode?
-CC=gcc-4.2 $bitness 
-mmacosx-version-min=$with_macosx_version_min_required
-CXX=g++-4.2 $bitness 
-mmacosx-version-min=$with_macosx_version_min_required
+CC=gcc-4.2 $bitness 
-mmacosx-version-min=$with_macosx_version_min_required -isysroot 
$MACOSX_SDK_PATH
+CXX=g++-4.2 $bitness 
-mmacosx-version-min=$with_macosx_version_min_required -isysroot 
$MACOSX_SDK_PATH
 ;;
 10.7|10.8)
-CC=`xcrun -find clang` $bitness 
-mmacosx-version-min=$with_macosx_version_min_required
-CXX=`xcrun -find clang++` $bitness 
-mmacosx-version-min=$with_macosx_version_min_required
+CC=`xcrun -find clang` $bitness 
-mmacosx-version-min=$with_macosx_version_min_required -isysroot 
$MACOSX_SDK_PATH
+CXX=`xcrun -find clang++` $bitness 
-mmacosx-version-min=$with_macosx_version_min_required -isysroot 
$MACOSX_SDK_PATH
 ;;
 esac
 AC_MSG_RESULT([$CC and $CXX])
diff --git a/solenv/gbuild/platform/macosx.mk b/solenv/gbuild/platform/macosx.mk
index 3d6810b..e81f0f5 100644
--- a/solenv/gbuild/platform/macosx.mk
+++ b/solenv/gbuild/platform/macosx.mk
@@ -49,7 +49,6 @@ gb_OSDEFS := \
 
 
 gb_CFLAGS := \
-   -isysroot $(gb_SDKDIR) \
$(gb_CFLAGS_COMMON) \
-fPIC \
-fno-strict-aliasing \
@@ -59,7 +58,6 @@ gb_CFLAGS := \
 # Re: [dev] warnings01: -Wnon-virtual-dtor message to d...@openoffice.org 
from
 # Feb 1, 2006:
 gb_CXXFLAGS := \
-   -isysroot $(gb_SDKDIR) \
$(gb_CXXFLAGS_COMMON) \
-fPIC \
-Wno-ctor-dtor-privacy \
diff --git a/solenv/inc/unxmacx.mk b/solenv/inc/unxmacx.mk
index a5ea287..0a58f4c 100644
--- a/solenv/inc/unxmacx.mk
+++ b/solenv/inc/unxmacx.mk
@@ -49,7 +49,7 @@ CDEFS+=-DGLIBC=2 -D_PTHREADS -D_REENTRANT 
-DNO_PTHREAD_PRIORITY $(PROCESSOR_DEFI
 #.EXPORT: MACOSX_DEPLOYMENT_TARGET
 CDEFS+:=-DQUARTZ
 
-EXTRA_CDEFS+:=-isysroot $(MACOSX_SDK_PATH)  
-DMAC_OS_X_VERSION_MIN_REQUIRED=$(MAC_OS_X_VERSION_MIN_REQUIRED) 
-DMAC_OS_X_VERSION_MAX_ALLOWED=$(MAC_OS_X_VERSION_MAX_ALLOWED) 
-DMACOSX_SDK_VERSION=$(MACOSX_SDK_VERSION)
+EXTRA_CDEFS+:=-DMAC_OS_X_VERSION_MIN_REQUIRED=$(MAC_OS_X_VERSION_MIN_REQUIRED) 
-DMAC_OS_X_VERSION_MAX_ALLOWED=$(MAC_OS_X_VERSION_MAX_ALLOWED) 
-DMACOSX_SDK_VERSION=$(MACOSX_SDK_VERSION)
 
 # Name of library where static data members are initialized
 # STATICLIBNAME=static$(DLLPOSTFIX)
commit 90fc5593a44f41489719bc882db21813416f80da
Author: Tor Lillqvist t...@iki.fi
Date:   Thu Sep 20 10:25:01 2012 +0300

Revert Pass also gb_CXXFLAGS to the liborcus configury

Causes way too much problems with the tinderboxes, thanks to -Werror
getting passed to the liborcus configury, and liborcus apparently is
quite -Werror-unclean.

Thus also revert the initial fixes for those WaEs. (Many more would be
needed, it seems. I rather leave it to Kohei to make upstream liborcus
warning-free.

This reverts commit 083fa89dd8f894f9cd0ef217a900fab777db9f2a.
This reverts commit 7e871bbaf7b208dd884528259211385b8074da39.

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

2012-09-20 Thread Libreoffice Gerrit user
 pyuno/source/module/pyuno_runtime.cxx |3 ---
 1 file changed, 3 deletions(-)

New commits:
commit ed39debbc379628ce74b096f7b7b179ab826551b
Author: Stephan Bergmann sberg...@redhat.com
Date:   Fri Jul 20 10:32:47 2012 +0200

warning C4702: unreachable code

(cherry picked from commit 618376bcb55906913bc76365d7db9b71eb422daa)

Conflicts:
pyuno/source/module/pyuno_runtime.cxx

Change-Id: Ie2cc83dde7e0522e0bf669a1dd3d7a8ca680f7e2

diff --git a/pyuno/source/module/pyuno_runtime.cxx 
b/pyuno/source/module/pyuno_runtime.cxx
index 56ee7fa..0cf2998 100644
--- a/pyuno/source/module/pyuno_runtime.cxx
+++ b/pyuno/source/module/pyuno_runtime.cxx
@@ -598,9 +598,6 @@ PyRef Runtime::any2PyObject (const Any a ) const
 throw RuntimeException(buf.makeStringAndClear( ), Reference 
XInterface  () );
 }
 }
-//We shouldn't be here...
-Py_INCREF( Py_None );
-return Py_None;
 }
 
 static Sequence Type  invokeGetTypes( const Runtime  r , PyObject * o )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: [PUSHED 3-6] acclerate large OLE2 file reading ...

2012-09-20 Thread Miklos Vajna
Marking as pushed.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


preventing gerrit changes from being merged

2012-09-20 Thread d . ostrovsky

Hi,

recently some gerrit changes were accidentally merged into master  
through gerrit UI.
While there are ways to prevent it, like voting -2 for a change or  
prefixing the
commit message with work in progress, please don't merge or  
something like that,

I want to share with you the gerrit way to achieve that: draft queue feature.

Default queue in gerrit is publish queue. All changes are submitted to  
it per default.
Recently draft queue was introduced in gerrit. If you submit a change  
to draft queue it

cannot be merged. That change must be first promoted to publish queue.

To use draft queue, all you need is to pass -D switch to git-review:
git-review -D
If you don't use git-review, you can specify the draft queue on command line:
git push logerrit master:refs/draft/master
(or whatever branch you need).

When you are ready you can promote your change to publish queue with  
UI or with command line:

ssh logerrit gerrit review --publish COMMIT | CHANGEID

See https://gerrit.libreoffice.org/Documentation/cmd-review.html for  
more details.


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


[Libreoffice-commits] .: Branch 'libreoffice-3-6' - solenv/inc

2012-09-20 Thread Libreoffice Gerrit user
 solenv/inc/wntmsc.mk |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 6779927cfcfbd386c8d9129ac49385cd54af6739
Author: Michael Stahl mst...@redhat.com
Date:   Fri Jul 13 16:10:02 2012 +0200

wntmsc.mk: disable warnings-as-errors for binfilter

Change-Id: If5ac0aa620d514ca4affe2b20d647a9e9129032e
(cherry picked from commit 158a7c73bb0a701b12fc3e2e96eff965c73435cc)

Signed-off-by: Stephan Bergmann sberg...@redhat.com

diff --git a/solenv/inc/wntmsc.mk b/solenv/inc/wntmsc.mk
index afafe38..ec84e98 100644
--- a/solenv/inc/wntmsc.mk
+++ b/solenv/inc/wntmsc.mk
@@ -195,7 +195,7 @@ CFLAGSWERRCC=-WX -DLIBO_WERROR
 # Once all modules on this platform compile without warnings, set
 # COMPILER_WARN_ERRORS=TRUE here instead of setting MODULES_WITH_WARNINGS (see
 # settings.mk):
-MODULES_WITH_WARNINGS :=
+MODULES_WITH_WARNINGS := binfilter
 
 CDEFS+=-DWIN32 -D_MT -DWINVER=0x0500 -D_WIN32_WINNT=0x0500 -D_WIN32_IE=0x0500
 .IF $(COMEX) == 11
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: Dialogs - Why so Inconsistent

2012-09-20 Thread Miklos Vajna
On Wed, Sep 19, 2012 at 10:10:04AM -0700, Joel Madero jmadero@gmail.com 
wrote:
 Even simple things like position are handled different, some define columns
 as COL and then use those in the src file while others directly put in
 positions (X,Y).
 
 My questions are:
 
 1. Is this okay?
 2. Is this efficient?
 3. Out of the dialogs, which has the best methodology and then maybe I'l
 slowly start cleaning the others to be similar to the best one

The best is what Caolán does in the feature/cmclayouttrans branch to get
rid of the src format for dialog definitions. If you want to help, I'm
sure you can join the effort there. :-)
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [PATCH] i18npool Add locale data for ebo_CG and tyx_CG

2012-09-20 Thread Miklos Vajna
Hi Jeremy,

On Wed, Sep 19, 2012 at 10:30:52PM +0100, Jeremy Brown 
jeremy.r.br...@gmail.com wrote:
 This is my first patch submission to LibreOffice or anywhere else so
 let me know if there's something I haven't done right or should do
 differently.  Thanks.

All you need is to submit a license mail what we can link from
http://wiki.documentfoundation.org/Development/Developers, stating:

All my contributions, past and future, to LibreOffice are licensed
under the terms of the MPL / LGPLv3+.

 + * 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 .

Is this really based on work from the ASF or you only copied the wrong
license header? See TEMPLATE.SOURCECODE.HEADER in the root directory on
what header to use when the new file is based only on your own work.

 diff --git a/solenv/inc/langlist.mk b/solenv/inc/langlist.mk
 index 5ba2dc8..b35dd20 100644
 --- a/solenv/inc/langlist.mk
 +++ b/solenv/inc/langlist.mk
 @@ -47,6 +47,7 @@ da \
  de \
  dgo \
  dz \
 +ebo \
  el \
  en-GB \
  en-US \
 @@ -127,6 +128,7 @@ tn \
  tr \
  ts \
  tt \
 +tyx \
  ug \
  uk \
  uz \

Hmm, all languages in langlist.mk have a relevant English description in
scp2/source/ooo/module_langpack.ulf. Is it intentional that you left out
the descriptions for these two languages?

Thanks,

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


[Libreoffice-commits] .: avmedia/source

2012-09-20 Thread Libreoffice Gerrit user
 avmedia/source/quicktime/manager.hxx |8 
 avmedia/source/quicktime/manager.mm  |2 +-
 avmedia/source/quicktime/player.hxx  |   10 +-
 avmedia/source/quicktime/player.mm   |2 +-
 avmedia/source/quicktime/quicktimeuno.mm |2 +-
 5 files changed, 12 insertions(+), 12 deletions(-)

New commits:
commit 37d21245b5f03fd261c681951f1d819a0bc4423a
Author: Ricardo Montania rica...@linuxafundo.com.br
Date:   Tue Sep 18 13:10:32 2012 -0300

Some OUString cleanup and ::rtl::OUString to OUString in avmedia

Change-Id: If79fffec7388e73fd85f5279e773af363c5f97e3
Reviewed-on: https://gerrit.libreoffice.org/645
Reviewed-by: Miklos Vajna vmik...@suse.cz
Tested-by: Miklos Vajna vmik...@suse.cz

diff --git a/avmedia/source/quicktime/manager.hxx 
b/avmedia/source/quicktime/manager.hxx
index adab01f..ecd344c 100644
--- a/avmedia/source/quicktime/manager.hxx
+++ b/avmedia/source/quicktime/manager.hxx
@@ -48,12 +48,12 @@ public:
 ~Manager();
 
 // XManager
-virtual ::com::sun::star::uno::Reference ::com::sun::star::media::XPlayer 
 SAL_CALL createPlayer( const ::rtl::OUString aURL ) throw 
(::com::sun::star::uno::RuntimeException);
+virtual ::com::sun::star::uno::Reference ::com::sun::star::media::XPlayer 
 SAL_CALL createPlayer( const OUString aURL ) throw 
(::com::sun::star::uno::RuntimeException);
 
 // XServiceInfo
-virtual ::rtl::OUString SAL_CALL getImplementationName(  ) throw 
(::com::sun::star::uno::RuntimeException);
-virtual sal_Bool SAL_CALL supportsService( const ::rtl::OUString 
ServiceName ) throw (::com::sun::star::uno::RuntimeException);
-virtual ::com::sun::star::uno::Sequence ::rtl::OUString  SAL_CALL 
getSupportedServiceNames(  ) throw (::com::sun::star::uno::RuntimeException);
+virtual OUString SAL_CALL getImplementationName(  ) throw 
(::com::sun::star::uno::RuntimeException);
+virtual sal_Bool SAL_CALL supportsService( const OUString ServiceName ) 
throw (::com::sun::star::uno::RuntimeException);
+virtual ::com::sun::star::uno::Sequence OUString  SAL_CALL 
getSupportedServiceNames(  ) throw (::com::sun::star::uno::RuntimeException);
 private:
 
 ::com::sun::star::uno::Reference 
::com::sun::star::lang::XMultiServiceFactory  mxMgr;
diff --git a/avmedia/source/quicktime/manager.mm 
b/avmedia/source/quicktime/manager.mm
index ca8da8e..ec36906 100644
--- a/avmedia/source/quicktime/manager.mm
+++ b/avmedia/source/quicktime/manager.mm
@@ -88,7 +88,7 @@ uno::Sequence ::rtl::OUString  SAL_CALL 
Manager::getSupportedServiceNames(  )
 throw (uno::RuntimeException)
 {
 uno::Sequence ::rtl::OUString  aRet(1);
-aRet[0] = ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM ( 
AVMEDIA_QUICKTIME_MANAGER_SERVICENAME ) );
+aRet[0] = ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( 
AVMEDIA_QUICKTIME_MANAGER_SERVICENAME ) );
 
 return aRet;
 }
diff --git a/avmedia/source/quicktime/player.hxx 
b/avmedia/source/quicktime/player.hxx
index 9bffa99..8bd8a77 100644
--- a/avmedia/source/quicktime/player.hxx
+++ b/avmedia/source/quicktime/player.hxx
@@ -50,7 +50,7 @@ public:
 Player( const ::com::sun::star::uno::Reference 
::com::sun::star::lang::XMultiServiceFactory  rxMgr );
 ~Player();
 
-bool create( const ::rtl::OUString rURL );
+bool create( const OUString rURL );
 
 // XPlayer
 virtual void SAL_CALL start(  ) throw 
(::com::sun::star::uno::RuntimeException);
@@ -70,16 +70,16 @@ public:
 virtual ::com::sun::star::uno::Reference 
::com::sun::star::media::XPlayerWindow  SAL_CALL createPlayerWindow( const 
::com::sun::star::uno::Sequence ::com::sun::star::uno::Any  aArguments ) 
throw (::com::sun::star::uno::RuntimeException);
 virtual ::com::sun::star::uno::Reference 
::com::sun::star::media::XFrameGrabber  SAL_CALL createFrameGrabber(  ) throw 
(::com::sun::star::uno::RuntimeException);
 // XServiceInfo
-virtual ::rtl::OUString SAL_CALL getImplementationName(  ) throw 
(::com::sun::star::uno::RuntimeException);
-virtual sal_Bool SAL_CALL supportsService( const ::rtl::OUString 
ServiceName ) throw (::com::sun::star::uno::RuntimeException);
-virtual ::com::sun::star::uno::Sequence ::rtl::OUString  SAL_CALL 
getSupportedServiceNames(  ) throw (::com::sun::star::uno::RuntimeException);
+virtual OUString SAL_CALL getImplementationName(  ) throw 
(::com::sun::star::uno::RuntimeException);
+virtual sal_Bool SAL_CALL supportsService( const OUString ServiceName ) 
throw (::com::sun::star::uno::RuntimeException);
+virtual ::com::sun::star::uno::Sequence OUString  SAL_CALL 
getSupportedServiceNames(  ) throw (::com::sun::star::uno::RuntimeException);
 
 QTMovie* getMovie();
 
 private:
 ::com::sun::star::uno::Reference 
::com::sun::star::lang::XMultiServiceFactory  mxMgr;
 
-::rtl::OUString maURL;
+OUStringmaURL;
 
 QTMovie *mpMovie;  // the Movie object
 
diff --git 

Re: [PATCH] i18npool Add locale data for ebo_CG and tyx_CG

2012-09-20 Thread Andras Timar
2012/9/20 Miklos Vajna vmik...@suse.cz:
 Hi Jeremy,

 On Wed, Sep 19, 2012 at 10:30:52PM +0100, Jeremy Brown  wrote:
 This is my first patch submission to LibreOffice or anywhere else so
 let me know if there's something I haven't done right or should do
 differently.  Thanks.

 All you need is to submit a license mail what we can link from
 http://wiki.documentfoundation.org/Development/Developers, stating:

 All my contributions, past and future, to LibreOffice are licensed
 under the terms of the MPL / LGPLv3+.

 + * 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 .

 Is this really based on work from the ASF or you only copied the wrong
 license header? See TEMPLATE.SOURCECODE.HEADER in the root directory on
 what header to use when the new file is based only on your own work.

 diff --git a/solenv/inc/langlist.mk b/solenv/inc/langlist.mk
 index 5ba2dc8..b35dd20 100644
 --- a/solenv/inc/langlist.mk
 +++ b/solenv/inc/langlist.mk
 @@ -47,6 +47,7 @@ da \
  de \
  dgo \
  dz \
 +ebo \
  el \
  en-GB \
  en-US \
 @@ -127,6 +128,7 @@ tn \
  tr \
  ts \
  tt \
 +tyx \
  ug \
  uk \
  uz \

 Hmm, all languages in langlist.mk have a relevant English description in
 scp2/source/ooo/module_langpack.ulf. Is it intentional that you left out
 the descriptions for these two languages?

Please don't patch langlist.mk unless you want to provide UI (and
help) localization in these languages.

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


[PUSHED] Some OUString cleanup and ::rtl::OUString to OUString in avm...

2012-09-20 Thread Miklos Vajna (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/645

Approvals:
  Miklos Vajna: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: If79fffec7388e73fd85f5279e773af363c5f97e3
Gerrit-PatchSet: 3
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Ricardo Montania rica...@linuxafundo.com.br
Gerrit-Reviewer: Miklos Vajna vmik...@suse.cz
Gerrit-Reviewer: Ricardo Montania rica...@linuxafundo.com.br

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


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

2012-09-20 Thread Libreoffice Gerrit user
 sd/source/ui/remotecontrol/BluetoothServer.cxx |   63 ++---
 1 file changed, 47 insertions(+), 16 deletions(-)

New commits:
commit cf480d6c37e8c58972ee8a3e3ce04747f3ae0b5e
Author: Andrzej J.R. Hunt andr...@ahunt.org
Date:   Thu Sep 20 09:54:29 2012 +0200

Move the DBusGObjectPath typedef and formatting changes to Bluetooth 
includes.

Change-Id: I7c1338ef2dae7831cddda8378a031a58011d7b17

diff --git a/sd/source/ui/remotecontrol/BluetoothServer.cxx 
b/sd/source/ui/remotecontrol/BluetoothServer.cxx
index 69841c8..56fd471 100644
--- a/sd/source/ui/remotecontrol/BluetoothServer.cxx
+++ b/sd/source/ui/remotecontrol/BluetoothServer.cxx
@@ -12,17 +12,19 @@
 #include sal/log.hxx
 
 #if (defined(LINUX)  !defined(__FreeBSD_kernel__))  defined(ENABLE_DBUS)
-#include glib.h
-#include dbus/dbus-glib.h
-#include sys/unistd.h
-#include sys/socket.h
-#include bluetooth/bluetooth.h
-#include bluetooth/rfcomm.h
-#define DBUS_TYPE_G_STRING_ANY_HASHTABLE (dbus_g_type_get_map( GHashTable, 
G_TYPE_STRING, G_TYPE_VALUE ))
-
-#ifndef G_VALUE_INIT
-  #define G_VALUE_INIT {0,{{0}}} // G_VALUE_INIT only present in glib = 2.30
-#endif
+  #include glib.h
+  #include dbus/dbus-glib.h
+  #include sys/unistd.h
+  #include sys/socket.h
+  #include bluetooth/bluetooth.h
+  #include bluetooth/rfcomm.h
+  #define DBUS_TYPE_G_STRING_ANY_HASHTABLE (dbus_g_type_get_map( GHashTable, 
G_TYPE_STRING, G_TYPE_VALUE ))
+  #ifndef G_VALUE_INIT
+#define G_VALUE_INIT {0,{{0}}} // G_VALUE_INIT only present in glib = 2.30
+  #endif
+  #ifndef DBusGObjectPath
+#define DBusGObjectPath char // DBusGObjectPath is only present in newer 
version of dbus-glib
+  #endif
 #endif
 
 #ifdef WIN32
@@ -59,9 +61,7 @@ DBusGProxy* bluezGetDefaultAdapter( DBusGConnection* 
aConnection,
 }
 
 gboolean aResult;
-// The following should be a DBusGObjectPath, however the necessary
-// typedef is missing in older version of dbus-glib.
-char *aAdapterPath = NULL;
+DBusGObjectPath* aAdapterPath = NULL;
 aResult = dbus_g_proxy_call( aManager, DefaultAdapter, aError,
  G_TYPE_INVALID,
  DBUS_TYPE_G_OBJECT_PATH, aAdapterPath,
commit 85267897633bffe791d581140c0397aeddcad674
Author: Andrzej J.R. Hunt andr...@ahunt.org
Date:   Thu Sep 20 09:49:29 2012 +0200

Add SAL_INFO and SAL_WARN to linux bluetooth implementation.

Change-Id: Id2b7d3ca0881bf5b446692e1351517f1e8971fd7

diff --git a/sd/source/ui/remotecontrol/BluetoothServer.cxx 
b/sd/source/ui/remotecontrol/BluetoothServer.cxx
index 352275d..69841c8 100644
--- a/sd/source/ui/remotecontrol/BluetoothServer.cxx
+++ b/sd/source/ui/remotecontrol/BluetoothServer.cxx
@@ -9,6 +9,8 @@
 #include BluetoothServer.hxx
 #include stdio.h
 
+#include sal/log.hxx
+
 #if (defined(LINUX)  !defined(__FreeBSD_kernel__))  defined(ENABLE_DBUS)
 #include glib.h
 #include dbus/dbus-glib.h
@@ -51,6 +53,7 @@ DBusGProxy* bluezGetDefaultAdapter( DBusGConnection* 
aConnection,
 
 if ( aManager == NULL )
 {
+SAL_WARN( sdremote.bluetooth, getting org.bluez.Manager failed );
 dbus_g_connection_unref( aConnection );
 return NULL;
 }
@@ -67,6 +70,7 @@ DBusGProxy* bluezGetDefaultAdapter( DBusGConnection* 
aConnection,
 g_object_unref( G_OBJECT( aManager ));
 if ( !aResult || aError )
 {
+SAL_WARN( sdremote.bluetooth, getting DefaultAdapter path failed );
 if ( aError )
 g_error_free( aError );
 return NULL;
@@ -77,6 +81,7 @@ DBusGProxy* bluezGetDefaultAdapter( DBusGConnection* 
aConnection,
   aAdapterPath, aInterfaceType );
 g_free( aAdapterPath );
 
+SAL_INFO( sdremote.bluetooth, DefaultAdapter retrieved );
 return aAdapter;
 }
 #endif // defined(LINUX)  defined(ENABLE_DBUS)
@@ -94,6 +99,7 @@ BluetoothServer::~BluetoothServer()
 bool BluetoothServer::isDiscoverable()
 {
 #if (defined(LINUX)  !defined(__FreeBSD_kernel__))  defined(ENABLE_DBUS)
+SAL_INFO( sdremote.bluetooth, BluetoothServer::isDiscoverable called );
 g_type_init();
 gboolean aResult;
 
@@ -141,6 +147,7 @@ bool BluetoothServer::isDiscoverable()
 void BluetoothServer::setDiscoverable( bool aDiscoverable )
 {
 #if (defined(LINUX)  !defined(__FreeBSD_kernel__))  defined(ENABLE_DBUS)
+SAL_INFO( sdremote.bluetooth, BluetoothServer::isDiscoverable called );
 g_type_init();
 gboolean aResult;
 
@@ -170,8 +177,12 @@ void BluetoothServer::setDiscoverable( bool aDiscoverable )
 
 if ( !aResult || aError )
 {
+SAL_WARN( sdremote.bluetooth, GetProperties failed );
 if ( aError )
+{
 g_error_free( aError );
+SAL_WARN( sdremote.bluetooth, with error   aError-message );
+}
 return;
 }
 
@@ -181,6 +192,7 @@ void BluetoothServer::setDiscoverable( bool aDiscoverable )
 g_free( aProperties );
 if ( !aPowered )
 {
+SAL_INFO( 

Re: Options included in CC/CXX vs. in gb_CFLAGS/gb_CXXFLAGS

2012-09-20 Thread Tor Lillqvist
 Should the -isysroot flag actually be part ot CC/CXX, not gb_CXXFLAGS?
 Should configure.in and macosx.mk be modified thusly?

That's what I did, and reverted my other attempts from this morning to
get liborcus to build, and it seems that it was a good move, the
tinderboxes are getting greener now.

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


[Libreoffice-commits] .: offapi/com offapi/UnoApi_offapi.mk sax/source unusedcode.easy

2012-09-20 Thread Libreoffice Gerrit user
 offapi/UnoApi_offapi.mk |1 
 offapi/com/sun/star/xml/sax/XFastSerializer.idl |  165 
 sax/source/tools/fastserializer.cxx |   97 --
 sax/source/tools/fastserializer.hxx |   93 -
 unusedcode.easy |1 
 5 files changed, 61 insertions(+), 296 deletions(-)

New commits:
commit b7c41f1d1fcabc8f4abc670d5a18546a117b540a
Author: Matúš Kukan matus.ku...@gmail.com
Date:   Tue Sep 18 18:06:36 2012 +0200

unusedcode: FastSaxSerializer is not used through UNO

Change-Id: I07d7b78be200f5f4a0674361237ef45b2bd1e3e1
Reviewed-on: https://gerrit.libreoffice.org/651
Reviewed-by: Miklos Vajna vmik...@suse.cz
Tested-by: Miklos Vajna vmik...@suse.cz

diff --git a/offapi/UnoApi_offapi.mk b/offapi/UnoApi_offapi.mk
index e5606da..b053b9b 100644
--- a/offapi/UnoApi_offapi.mk
+++ b/offapi/UnoApi_offapi.mk
@@ -4241,7 +4241,6 @@ $(eval $(call 
gb_UnoApi_add_idlfiles,offapi,offapi/com/sun/star/xml/sax,\
XFastDocumentHandler \
XFastParser \
XFastSAXSerializable \
-   XFastSerializer \
XFastShapeContextHandler \
XFastTokenHandler \
XLocator \
diff --git a/offapi/com/sun/star/xml/sax/XFastSerializer.idl 
b/offapi/com/sun/star/xml/sax/XFastSerializer.idl
deleted file mode 100644
index 6601a00..000
--- a/offapi/com/sun/star/xml/sax/XFastSerializer.idl
+++ /dev/null
@@ -1,165 +0,0 @@
-/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
-/*
- *
- * 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
- * http://www.openoffice.org/license.html
- * for a copy of the LGPLv3 License.
- *
- /
-#ifndef __com_sun_star_xml_sax_XFastSerializer_idl__
-#define __com_sun_star_xml_sax_XFastSerializer_idl__
-
-#include com/sun/star/uno/XInterface.idl
-#include com/sun/star/xml/sax/SAXException.idl
-#include com/sun/star/xml/sax/XFastDocumentHandler.idl
-#include com/sun/star/io/XOutputStream.idl
-#include com/sun/star/xml/sax/XFastTokenHandler.idl
-
-
-module com {  module sun {  module star {  module xml {  module sax {
-
-
-/** receives notification of sax document events to write into an 
XOutputStream.
- */
-interface XFastSerializer: com::sun::star::uno::XInterface
-{
-
-/** called by the parser when parsing of an XML stream is started.
- */
-void startDocument()
-raises( com::sun::star::xml::sax::SAXException );
-
-
-/** called by the parser after the last XML element of a stream is 
processed.
- */
-void endDocument()
-raises( com::sun::star::xml::sax::SAXException );
-
-
-/** receives notification of the beginning of an element .
-
-@param Element
-contains the integer token from the typeXFastTokenHandler/type
-registered at the typeXFastParser/type.br
-
-If the element has a namespace that was registered with the
-typeXFastParser/type, paramElement/param contains the 
integer
-token of the elements local name from the 
typeXFastTokenHandler/type
-and the integer token of the namespace combined with an arithmetic
-bor/b operation.
-
-@param Attribs
-Contains a typeXFastAttrbitueList/type to access the attributes
-from the element.
-
-*/
-void startFastElement( [in] long Element, [in] XFastAttributeList Attribs )
-raises( com::sun::star::xml::sax::SAXException );
-
-
-/** receives notification of the beginning of an unknown element .
-
-@param Namespace
-contains the namespace url (not the prefix!) of this element.
-@param Name
-contains the elements local name.
-@param Attribs
-Contains a typeXFastAttrbitueList/type to access the attributes
-from the element.
- */
-void startUnknownElement( [in] string Namespace, [in] string Name, [in] 

[PUSHED] unusedcode: FastSaxSerializer is not used through UNO

2012-09-20 Thread Miklos Vajna (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/651

Approvals:
  Miklos Vajna: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I07d7b78be200f5f4a0674361237ef45b2bd1e3e1
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Matúš Kukan matus.ku...@gmail.com
Gerrit-Reviewer: Miklos Vajna vmik...@suse.cz

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


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

2012-09-20 Thread Libreoffice Gerrit user
 sw/source/core/crsr/crsrsh.cxx |   15 ---
 1 file changed, 8 insertions(+), 7 deletions(-)

New commits:
commit 6bcbf68b29b609cf06dbea0e192533e11eb523c3
Author: Michael Stahl mst...@redhat.com
Date:   Wed Sep 19 20:47:48 2012 +0200

rhbz#689053: fix crash following delete at last table cell:

The IsSelOvr() in SwCrsrShell::Combine() detects that the PaM with mark
in the last table cell and point ouside the table is invalid, but
unfortunatley restores the wrong position, which does not actually
correct the PaM; the deletion on the invalid PaM then removes the
last SwTxtNode in the document, which leads to crash later.

(cherry picked from commit af4b6c94c68b5f67b931cde8d0acda6ec8b288bb)

Conflicts:
sw/source/core/crsr/crsrsh.cxx

Change-Id: Ib2ae0b54e6c15dbe5b6811d8696531bf2022e1a4
Reviewed-on: https://gerrit.libreoffice.org/652
Reviewed-by: Miklos Vajna vmik...@suse.cz
Tested-by: Miklos Vajna vmik...@suse.cz

diff --git a/sw/source/core/crsr/crsrsh.cxx b/sw/source/core/crsr/crsrsh.cxx
index ca62c55..1493d14 100644
--- a/sw/source/core/crsr/crsrsh.cxx
+++ b/sw/source/core/crsr/crsrsh.cxx
@@ -2012,26 +2012,27 @@ void SwCrsrShell::Combine()
 return;
 
 SwCallLink aLk( *this );// Crsr-Moves ueberwachen, evt. Link callen
-SwCrsrSaveState aSaveState( *pCurCrsr );
+// rhbz#689053: IsSelOvr must restore the saved stack position, not the
+// current one, because current point + stack mark may be invalid PaM
+SwCrsrSaveState aSaveState(*pCrsrStk);
 if( pCrsrStk-HasMark() )   // nur wenn GetMark gesetzt wurde
 {
 bool const bResult =
 CheckNodesRange( pCrsrStk-GetMark()-nNode, 
pCurCrsr-GetPoint()-nNode, sal_True );
 OSL_ENSURE(bResult, StackCrsr  act. Crsr not in same Section.);
 (void) bResult; // non-debug: unused
-// kopiere das GetMark
-if( !pCurCrsr-HasMark() )
-pCurCrsr-SetMark();
-*pCurCrsr-GetMark() = *pCrsrStk-GetMark();
-pCurCrsr-GetMkPos() = pCrsrStk-GetMkPos();
 }
+*pCrsrStk-GetPoint() = *pCurCrsr-GetPoint();
+pCrsrStk-GetPtPos() = pCurCrsr-GetPtPos();
 
 SwShellCrsr * pTmp = 0;
 if( pCrsrStk-GetNext() != pCrsrStk )
 {
 pTmp = dynamic_castSwShellCrsr*(pCrsrStk-GetNext());
 }
-delete pCrsrStk;
+delete pCurCrsr;
+pCurCrsr = pCrsrStk;
+pCrsrStk-MoveTo(0); // remove from ring
 pCrsrStk = pTmp;
 if( !pCurCrsr-IsInProtectTable( sal_True ) 
 !pCurCrsr-IsSelOvr( nsSwCursorSelOverFlags::SELOVER_TOGGLE |
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED] Change in core[libreoffice-3-6]: rhbz#689053: fix crash following delete at last table cell:

2012-09-20 Thread Miklos Vajna (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/652

Approvals:
  Miklos Vajna: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib2ae0b54e6c15dbe5b6811d8696531bf2022e1a4
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-3-6
Gerrit-Owner: Michael Stahl mst...@redhat.com
Gerrit-Reviewer: Miklos Vajna vmik...@suse.cz

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


Re: sal/config.g sal/types.h and sal/osl/rtl includes in general

2012-09-20 Thread Lubos Lunak
On Wednesday 19 of September 2012, Norbert Thiebaud wrote:
 So there is no automatic way to verify that an include set is
 'minimal', but there isn't even a way to do that with eyeballing...

 Actually, there possibly is an automatic way to do that: 
http://code.google.com/p/include-what-you-use/ . I haven't tried it though.

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


[ANN] LibreOffice 3.5.7 RC1 available

2012-09-20 Thread Thorsten Behrens
Dear Community,

The Document Foundation is happy to announce the first release
candidate of LibreOffice 3.5.7. The upcoming 3.5.7 will be the seventh
in a series of frequent bugfix releases for our 3.5 code line. Please
be aware that LibreOffice 3.5.7 RC1 is not yet ready for production
use, you should continue to use LibreOffice 3.5.6 for that.

The release is available for Windows, Linux and Mac OS X from our QA
builds download page at

  http://www.libreoffice.org/download/pre-releases/

Should you find bugs, please report them to the FreeDesktop Bugzilla:

  https://bugs.freedesktop.org

A good way to assess the beta quality is to run some specific manual
tests on it, our TCM wiki page has more details:

 
http://wiki.documentfoundation.org/QA/Testing/Regression_Tests#Full_Regression_Test

 (and the announcement mail: 
http://lists.freedesktop.org/archives/libreoffice/2011-December/022464.html)
 
For other ways to get involved with this exciting project - you can
e.g. contribute code:

  http://www.libreoffice.org/get-involved/developers/

translate LibreOffice to your language:

  http://wiki.documentfoundation.org/Translation_for_3.5

or help with funding our operations:

  http://challenge.documentfoundation.org/

A list of known issues and fixed bugs with 3.5.7 RC1 is available
from our wiki:

  http://wiki.documentfoundation.org/Releases/3.5.7/RC1

Let us close again with a BIG Thank You! to all of you having
contributed to the LibreOffice project - this release would not have
been possible without your help.

Yours,

The Document Foundation Board of Directors

-- 
The Document Foundation, Zimmerstr. 69, 10117 Berlin, Germany
Rechtsfähige Stiftung des bürgerlichen Rechts
Legal details: http://www.documentfoundation.org/imprint


pgp50QicSJl2f.pgp
Description: PGP signature
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: Branch 'libreoffice-3-6' - sal/inc

2012-09-20 Thread Libreoffice Gerrit user
 sal/inc/rtl/string.hxx  |7 ++-
 sal/inc/rtl/ustring.hxx |7 ++-
 2 files changed, 4 insertions(+), 10 deletions(-)

New commits:
commit ab76bfc0366d72d80698e4145130a072aed64d31
Author: Stephan Bergmann sberg...@redhat.com
Date:   Mon Jul 23 12:09:12 2012 +0200

Make rtl::O[U]String::isEmpty return true bool

...instead of sal_Bool, to work around MSVC warning C4805: unsafe mix of 
type
'sal_Bool' and type 'bool'.

Change-Id: Ia3477539ccc23aa9daec49d633f023d414f2b4bf
(cherry picked from commit 8ee7c47404cdf408bcd3b6ea0267fbe54193c93f)
Reviewed-on: https://gerrit.libreoffice.org/650
Reviewed-by: Luboš Luňák l.lu...@suse.cz
Reviewed-by: Miklos Vajna vmik...@suse.cz
Tested-by: Miklos Vajna vmik...@suse.cz

diff --git a/sal/inc/rtl/string.hxx b/sal/inc/rtl/string.hxx
index 4e76b74..e70f3ac 100644
--- a/sal/inc/rtl/string.hxx
+++ b/sal/inc/rtl/string.hxx
@@ -320,12 +320,9 @@ public:
 
   @since LibreOffice 3.4
 */
-sal_Bool isEmpty() const SAL_THROW(())
+bool isEmpty() const SAL_THROW(())
 {
-if ( pData-length )
-return sal_False;
-else
-return sal_True;
+return pData-length == 0;
 }
 
 /**
diff --git a/sal/inc/rtl/ustring.hxx b/sal/inc/rtl/ustring.hxx
index f3d224e..5962752 100644
--- a/sal/inc/rtl/ustring.hxx
+++ b/sal/inc/rtl/ustring.hxx
@@ -405,12 +405,9 @@ public:
 
   @since LibreOffice 3.4
 */
-sal_Bool isEmpty() const SAL_THROW(())
+bool isEmpty() const SAL_THROW(())
 {
-if ( pData-length )
-return sal_False;
-else
-return sal_True;
+return pData-length == 0;
 }
 
 /**
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED] Change in core[libreoffice-3-6]: Make rtl::O[U]String::isEmpty return true bool

2012-09-20 Thread Miklos Vajna (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/650

Approvals:
  Miklos Vajna: Verified; Looks good to me, approved
  Luboš Luňák: Looks good to me, but someone else must approve


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia3477539ccc23aa9daec49d633f023d414f2b4bf
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-3-6
Gerrit-Owner: Stephan Bergmann sberg...@redhat.com
Gerrit-Reviewer: Luboš Luňák l.lu...@suse.cz
Gerrit-Reviewer: Michael Stahl mst...@redhat.com
Gerrit-Reviewer: Miklos Vajna vmik...@suse.cz
Gerrit-Reviewer: Norbert Thiebaud nthieb...@gmail.com
Gerrit-Reviewer: Stephan Bergmann sberg...@redhat.com

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


[Libreoffice-commits] .: config_host.mk.in configure.in filter/Module_filter.mk sc/CppunitTest_sc_filters_test.mk sc/qa sd/CppunitTest_sd_filters_test.mk sd/qa svtools/Module_svtools.mk sw/Module_sw.m

2012-09-20 Thread Libreoffice Gerrit user
 config_host.mk.in |1 +
 configure.in  |   17 +
 filter/Module_filter.mk   |2 ++
 sc/CppunitTest_sc_filters_test.mk |6 ++
 sc/qa/unit/filters-test.cxx   |2 ++
 sd/CppunitTest_sd_filters_test.mk |6 ++
 sd/qa/unit/filters-test.cxx   |2 ++
 svtools/Module_svtools.mk |2 ++
 sw/Module_sw.mk   |7 ++-
 9 files changed, 44 insertions(+), 1 deletion(-)

New commits:
commit e0f90c3060f542885d834314a9097353d0cb2ba2
Author: David Ostrovsky da...@ostrovsky.org
Date:   Wed Sep 19 22:13:36 2012 +0200

introduce new option --disable-cve-tests

Change-Id: I288638c4a70396ccb7354b20e901aa0c46ffe442
Reviewed-on: https://gerrit.libreoffice.org/653
Reviewed-by: Michael Meeks michael.me...@suse.com
Tested-by: Michael Meeks michael.me...@suse.com

diff --git a/config_host.mk.in b/config_host.mk.in
index b6d1ce9..753c901 100644
--- a/config_host.mk.in
+++ b/config_host.mk.in
@@ -103,6 +103,7 @@ export DISABLE_EXTENSIONS=@DISABLE_EXTENSIONS@
 export DISABLE_SCRIPTING=@DISABLE_SCRIPTING@
 export DISABLE_LINKOO=@DISABLE_LINKOO@
 export DISABLE_NEON=@DISABLE_NEON@
+export DISABLE_CVE_TESTS=@DISABLE_CVE_TESTS@
 export DISABLE_PYTHON=@DISABLE_PYTHON@
 export DLLTOOL=@DLLTOOL@
 export DMAKEROOT=@SRC_ROOT@/solenv/inc/startup
diff --git a/configure.in b/configure.in
index 1010672..e29a0e8 100644
--- a/configure.in
+++ b/configure.in
@@ -988,6 +988,11 @@ AC_ARG_ENABLE(neon,
 [Disable neon and the compilation of webdav binding.]),
 ,)
 
+AC_ARG_ENABLE(cve-tests,
+AS_HELP_STRING([--disable-cve-tests],
+[Prevent CVE tests to be executed]),
+,)
+
 AC_ARG_ENABLE(build-unowinreg,
 AS_HELP_STRING([--enable-build-unowinreg],
 [Do not use the prebuilt unowinreg.dll. Build it instead. The MinGW C++
@@ -8623,6 +8628,18 @@ AC_SUBST([MINGW_TASN1_DLL])
 fi
 
 dnl ===
+dnl Check for disabling cve_tests
+dnl ===
+AC_MSG_CHECKING([whether to execute CVE tests])
+if test $enable_cve_tests = no; then
+AC_MSG_RESULT([no])
+DISABLE_CVE_TESTS=TRUE
+AC_SUBST(DISABLE_CVE_TESTS)
+else
+AC_MSG_RESULT([yes])
+fi
+
+dnl ===
 dnl Check for system openssl
 dnl ===
 if test $_os = Darwin -o $_os = NetBSD -o $_os = OpenBSD -o \
diff --git a/filter/Module_filter.mk b/filter/Module_filter.mk
index 2a3ef79..b3ec2a3 100644
--- a/filter/Module_filter.mk
+++ b/filter/Module_filter.mk
@@ -80,11 +80,13 @@ $(eval $(call gb_Module_add_targets,filter,\
 ))
 endif
 
+ifneq ($(DISABLE_CVE_TESTS),TRUE)
 $(eval $(call gb_Module_add_check_targets,filter,\
 CppunitTest_filter_pict_test \
 CppunitTest_filter_tga_test \
 CppunitTest_filter_tiff_test \
 ))
+endif
 
 # TODO
 #$(eval $(call gb_Module_add_subsequentcheck_targets,filter,\
diff --git a/sc/CppunitTest_sc_filters_test.mk 
b/sc/CppunitTest_sc_filters_test.mk
index 37a4b14..ef47365 100644
--- a/sc/CppunitTest_sc_filters_test.mk
+++ b/sc/CppunitTest_sc_filters_test.mk
@@ -31,6 +31,12 @@ $(eval $(call 
gb_CppunitTest_add_exception_objects,sc_filters_test, \
 sc/qa/unit/filters-test \
 ))
 
+ifeq ($(DISABLE_CVE_TESTS),TRUE)
+$(eval $(call gb_CppunitTest_add_defs,sc_filters_test,\
+-DDISABLE_CVE_TESTS \
+))
+endif
+
 $(eval $(call gb_CppunitTest_use_externals,sc_filters_test, \
mdds_headers \
orcus \
diff --git a/sc/qa/unit/filters-test.cxx b/sc/qa/unit/filters-test.cxx
index dc6df6d..a7b050d 100644
--- a/sc/qa/unit/filters-test.cxx
+++ b/sc/qa/unit/filters-test.cxx
@@ -196,6 +196,7 @@ void ScFiltersTest::createCSVPath(const rtl::OUString 
aFileBase, rtl::OUString
 
 void ScFiltersTest::testCVEs()
 {
+#ifndef DISABLE_CVE_TESTS
 testDir(rtl::OUString(RTL_CONSTASCII_USTRINGPARAM(Quattro Pro 6.0)),
 getURLFromSrc(/sc/qa/unit/data/qpro/), rtl::OUString());
 
@@ -207,6 +208,7 @@ void ScFiltersTest::testCVEs()
 
 testDir(rtl::OUString(RTL_CONSTASCII_USTRINGPARAM(MS Excel 97)),
 getURLFromSrc(/sc/qa/unit/data/xls/), rtl::OUString());
+#endif
 }
 
 #if TEST_BUG_FILES
diff --git a/sd/CppunitTest_sd_filters_test.mk 
b/sd/CppunitTest_sd_filters_test.mk
index 6afe52d..4394230 100644
--- a/sd/CppunitTest_sd_filters_test.mk
+++ b/sd/CppunitTest_sd_filters_test.mk
@@ -33,6 +33,12 @@ $(eval $(call 
gb_CppunitTest_add_exception_objects,sd_filters_test, \
 sd/qa/unit/filters-test \
 ))
 
+ifeq ($(DISABLE_CVE_TESTS),TRUE)
+$(eval $(call gb_CppunitTest_add_defs,sd_filters_test,\
+-DDISABLE_CVE_TESTS \
+))
+endif
+
 $(eval $(call gb_CppunitTest_use_libraries,sd_filters_test, \
 avmedia \
 basegfx \
diff --git a/sd/qa/unit/filters-test.cxx b/sd/qa/unit/filters-test.cxx
index b5ae803..76fd560 100644
--- 

[PUSHED] introduce new option --disable-cve-tests

2012-09-20 Thread Michael Meeks (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/653

Approvals:
  Michael Meeks: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I288638c4a70396ccb7354b20e901aa0c46ffe442
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: David Ostrovsky david.ostrov...@gmx.de
Gerrit-Reviewer: Michael Meeks michael.me...@suse.com

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


[Libreoffice-commits] .: offapi/com offapi/UnoApi_offapi.mk officecfg/Configuration_officecfg.mk officecfg/registry

2012-09-20 Thread Libreoffice Gerrit user
 offapi/UnoApi_offapi.mk|1 
 offapi/com/sun/star/setup/ProductRegistration.idl  |   82 -
 officecfg/Configuration_officecfg.mk   |1 
 officecfg/registry/data/org/openoffice/Office/Jobs.xcu |   39 
 4 files changed, 123 deletions(-)

New commits:
commit 1f77f1ac177521698ee7c978f218127cb5840046
Author: Matúš Kukan matus.ku...@gmail.com
Date:   Thu Sep 20 11:46:48 2012 +0200

remove remnants of ProductRegistration

Implementation is gone since b2d7ede98716806cc4b3ea8547247175d5f04d18

Change-Id: I3986ca99748f0010d80d2c788723a42a1725e83c

diff --git a/offapi/UnoApi_offapi.mk b/offapi/UnoApi_offapi.mk
index b053b9b..047a2ec 100644
--- a/offapi/UnoApi_offapi.mk
+++ b/offapi/UnoApi_offapi.mk
@@ -1051,7 +1051,6 @@ $(eval $(call 
gb_UnoApi_add_idlfiles_noheader,offapi,offapi/com/sun/star/sdbcx,\
ViewDescriptor \
 ))
 $(eval $(call 
gb_UnoApi_add_idlfiles_noheader,offapi,offapi/com/sun/star/setup,\
-   ProductRegistration \
Setup \
 ))
 $(eval $(call 
gb_UnoApi_add_idlfiles_noheader,offapi,offapi/com/sun/star/sheet,\
diff --git a/offapi/com/sun/star/setup/ProductRegistration.idl 
b/offapi/com/sun/star/setup/ProductRegistration.idl
deleted file mode 100644
index 3dbdf57..000
--- a/offapi/com/sun/star/setup/ProductRegistration.idl
+++ /dev/null
@@ -1,82 +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 __com_sun_star_setup_ProductRegistration_idl__
-#define __com_sun_star_setup_ProductRegistration_idl__
-
-#include com/sun/star/task/JobExecutor.idl
-#include com/sun/star/task/XJob.idl
-
-module com {  module sun {  module star {  module setup {
-
-/** provides a mechanism for registering the product.
-
-pTwo main functionalities are covered by this service:
-ulilemTriggering Online Registration/embr/
-If, by any user interface of your choice, the user tells she wants 
to register
-their product copy, ProductRegistration is the service of your 
choice.
-/ilbr/
-ilemExecuting Registration Requests/embr/
-ProductRegistration allows to remind your users to register, using 
a dialog with various
-choices.
-/il
-/ul/p
-
-@since OOo 1.1.2
-*/
-published service ProductRegistration
-{
-/** allows to trigger certain events.
-
-pThe string argument of method 
scope=com::sun::star::taskXJobExecutor::trigger/method is
-used to determine the type of the event. Currently supported 
events are:br/
-ullibRegistrationRequired/bbr/
-Determines that the registration is required to be 
executed.br/
-This is usually triggered by some user interaction, for 
instance upon choosing a menu item
-called emRegistration/em./p
-/li
-/ul
-/p
-*/
-service com::sun::star::task::JobExecutor;
-
-/** allows the registration request to be scheduled automatically.
-
-pThere are processes which allow to schedule events and jobs, for 
instance there
-can be configuration settings scheduling which operations are to be 
executed upon an program
-startup. Such processes usually require the jobs to be executed to 
support the
-type scope=com::sun::star::taskXJob/type interface./p
-
-pThat's why this service supports the type 
scope=com::sun::star::taskXJob/type, too.br/
-Calling the method 
scope=com::sun::star::taskXJob::execute/method method results in 
displaying a
-dialog which asks the user to register. This is done once per session, 
means you need to restart the
-process which called this./p
-
-pAt the moment, there is no specification about how to use the 
arguments given in the
-method scope=com::sun::star::taskXJob::execute/method method, 
but this may change in future versions of the service
-descriptions, so to keep your implementation compatible, you should 
not use these arguments./p
-*/
-interface com::sun::star::task::XJob;
-};
-

[Libreoffice-commits] .: bridges/source cppu/inc cppu/source

2012-09-20 Thread Libreoffice Gerrit user
 bridges/source/jni_uno/jni_bridge.cxx |9 +
 cppu/inc/uno/environment.h|7 ++-
 cppu/source/uno/lbenv.cxx |4 
 3 files changed, 19 insertions(+), 1 deletion(-)

New commits:
commit 69765868e30494eb39a2b4b7566f161ef1f3db04
Author: Tor Lillqvist t...@iki.fi
Date:   Thu Sep 20 12:42:48 2012 +0300

DISABLE_DYNLOADING magic also for the jni_uno environment

Change-Id: I5e966a5734308381ad305b891ecfc830dc1419f6

diff --git a/bridges/source/jni_uno/jni_bridge.cxx 
b/bridges/source/jni_uno/jni_bridge.cxx
index 87d210d..f8fdbce 100644
--- a/bridges/source/jni_uno/jni_bridge.cxx
+++ b/bridges/source/jni_uno/jni_bridge.cxx
@@ -456,6 +456,10 @@ void SAL_CALL java_env_disposing( uno_Environment * 
java_env )
 }
 }
 
+#ifdef DISABLE_DYNLOADING
+#define uno_initEnvironment java_uno_initEnvironment
+#endif
+
 
//--
 void SAL_CALL uno_initEnvironment( uno_Environment * java_env )
 SAL_THROW_EXTERN_C()
@@ -554,12 +558,17 @@ void SAL_CALL uno_ext_getMapping(
 }
 }
 
+#ifndef DISABLE_DYNLOADING
+
 
//--
 SAL_DLLPUBLIC_EXPORT sal_Bool SAL_CALL component_canUnload( TimeValue * pTime )
 SAL_THROW_EXTERN_C()
 {
 return (*g_moduleCount.canUnload)( g_moduleCount, pTime );
 }
+
+#endif
+
 }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/cppu/inc/uno/environment.h b/cppu/inc/uno/environment.h
index bbba4c9..e76ed2f 100644
--- a/cppu/inc/uno/environment.h
+++ b/cppu/inc/uno/environment.h
@@ -258,9 +258,14 @@ typedef void (SAL_CALL * uno_initEnvironmentFunc)( 
uno_Environment * pEnv );
 #define UNO_INIT_ENVIRONMENT uno_initEnvironment
 
 #ifdef DISABLE_DYNLOADING
-/* We link statically and have just one kind of environment */
+/* We link statically and have just the C++ environment */
 void SAL_CALL CPPU_ENV_uno_initEnvironment( uno_Environment * Env )
  SAL_THROW_EXTERN_C();
+#ifdef SOLAR_JAVA
+/* We also have the Java environment */
+void SAL_CALL java_uno_initEnvironment( uno_Environment * Env )
+ SAL_THROW_EXTERN_C();
+#endif
 #endif
 
 /** Gets a specific environment. If the specified environment does not exist, 
then a default one
diff --git a/cppu/source/uno/lbenv.cxx b/cppu/source/uno/lbenv.cxx
index ff6de9a..439d49b 100644
--- a/cppu/source/uno/lbenv.cxx
+++ b/cppu/source/uno/lbenv.cxx
@@ -1058,6 +1058,10 @@ static bool loadEnv(OUString const   cLibStem,
 
 if ( cLibStem == CPPU_CURRENT_LANGUAGE_BINDING_NAME _uno )
 fpInit = CPPU_ENV_uno_initEnvironment;
+#ifdef SOLAR_JAVA
+else if ( cLibStem == java_uno )
+fpInit = java_uno_initEnvironment;
+#endif
 else
 {
 #if OSL_DEBUG_LEVEL  1
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: postprocess/packregistry

2012-09-20 Thread Libreoffice Gerrit user
 postprocess/packregistry/makefile.mk |1 -
 1 file changed, 1 deletion(-)

New commits:
commit b590d87b380389371eca94ffb0e6f47f86d1c917
Author: Matúš Kukan matus.ku...@gmail.com
Date:   Thu Sep 20 11:57:41 2012 +0200

forgot to remove this one

Change-Id: Idd33388f61461a55021b93498e2c992d7ae560d0

diff --git a/postprocess/packregistry/makefile.mk 
b/postprocess/packregistry/makefile.mk
index 9a8b330..aab9e01 100644
--- a/postprocess/packregistry/makefile.mk
+++ b/postprocess/packregistry/makefile.mk
@@ -227,7 +227,6 @@ MY_FILES_main = \
 $(MY_XCU)/Office/FormWizard.xcu \
 $(MY_XCU)/Office/Histories.xcu \
 $(MY_XCU)/Office/Impress.xcu \
-$(MY_XCU)/Office/Jobs.xcu \
 $(MY_XCU)/Office/Labels.xcu \
 $(MY_XCU)/Office/Logging.xcu \
 $(MY_XCU)/Office/Math.xcu \
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: sd/source

2012-09-20 Thread Libreoffice Gerrit user
 sd/source/ui/remotecontrol/Server.cxx |   17 ++---
 1 file changed, 14 insertions(+), 3 deletions(-)

New commits:
commit e2e701b5747982d037f76021ed64ab15a49cb573
Author: Andrzej J.R. Hunt andr...@ahunt.org
Date:   Thu Sep 20 12:02:05 2012 +0200

Add SAL_WARN/INFO to network server + remove static reference on exit.

Change-Id: I76e737f59aa36bf7982255fa9bb41ef9d6abfe8d

diff --git a/sd/source/ui/remotecontrol/Server.cxx 
b/sd/source/ui/remotecontrol/Server.cxx
index 388ef9d..8ec942d 100644
--- a/sd/source/ui/remotecontrol/Server.cxx
+++ b/sd/source/ui/remotecontrol/Server.cxx
@@ -19,6 +19,7 @@
 
 #include comphelper/processfactory.hxx
 #include comphelper/configuration.hxx
+#include sal/log.hxx
 
 #include sddll.hxx
 
@@ -75,23 +76,26 @@ RemoteServer::~RemoteServer()
 
 void RemoteServer::execute()
 {
+SAL_INFO( sdremote, RemoteServer::execute called );
 osl::SocketAddr aAddr( 0, PORT );
 if ( !mSocket.bind( aAddr ) )
 {
-// Error binding
+SAL_WARN( sdremote, bind failed  mSocket.getErrorAsString() );
 return;
 }
 
 if ( !mSocket.listen(3) )
 {
-// Error listening
+SAL_WARN( sdremote, listen failed  mSocket.getErrorAsString() );
 return;
 }
 while ( true )
 {
 StreamSocket aSocket;
+SAL_INFO( sdremote, waiting on accept );
 if ( mSocket.acceptConnection( aSocket ) == osl_Socket_Error )
 {
+SAL_WARN( sdremote, accept failed  
mSocket.getErrorAsString() );
 return; // Closed, or other issue.
 }
 BufferedStreamSocket *pSocket = new BufferedStreamSocket( aSocket);
@@ -138,6 +142,7 @@ void RemoteServer::execute()
 axPin = sPin;
 
 if ( sPin.equals( pClient-mPin ) ) {
+SAL_INFO( sdremote, client found on validated list 
-- connecting );
 connectClient( pClient, sPin );
 aFound = true;
 break;
@@ -147,13 +152,17 @@ void RemoteServer::execute()
 }
 // Pin not found so inform the client.
 if ( !aFound )
+{
+SAL_INFO( sdremote, client not found on validated list );
 pSocket-write( LO_SERVER_VALIDATING_PIN\n\n,
 strlen( LO_SERVER_VALIDATING_PIN\n\n ) );
+}
 } else {
+SAL_INFO( sdremote, client failed to send 
LO_SERVER_CLIENT_PAIR, ignoring );
 delete pSocket;
 }
 }
-
+spServer = NULL; // Object is destroyed when Thread::execute() ends.
 }
 
 RemoteServer *sd::RemoteServer::spServer = NULL;
@@ -224,6 +233,7 @@ std::vectorClientInfo* RemoteServer::getClients()
 
 sal_Bool RemoteServer::connectClient( ClientInfo* pClient, rtl::OUString aPin )
 {
+SAL_INFO( sdremote, RemoteServer::connectClient called );
 if ( !spServer )
 return false;
 
@@ -285,6 +295,7 @@ sal_Bool RemoteServer::connectClient( ClientInfo* pClient, 
rtl::OUString aPin )
 void SdDLL::RegisterRemotes()
 {
 // Disable unless in experimental mode for now
+SAL_INFO( sdremote, SdDLL::RegisterRemotes called );
 uno::Reference uno::XComponentContext  xContext = 
comphelper::getProcessComponentContext();
 if (!xContext.is() || 
!officecfg::Office::Common::Misc::ExperimentalMode::get(xContext))
 return;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED:3-6-2] [PATCH] fdo#37529 clear DataColumns read-only information when we cl...

2012-09-20 Thread Michael Meeks

On Thu, 2012-09-13 at 17:28 +0200, Miklos Vajna wrote:
 On Thu, Sep 13, 2012 at 04:38:34PM +0200, Lionel Elie Mamane 
 lio...@mamane.lu wrote:
  It is (yet another) Base crash. 't would be nice to have it in
  libreoffice-3-6-2, too.
 
 Looks OK to me, one more needed.

Pushed to -3-6-2 :-)

Thanks Lionel !

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-2' - dbaccess/source

2012-09-20 Thread Libreoffice Gerrit user
 dbaccess/source/core/api/RowSet.cxx |3 +++
 1 file changed, 3 insertions(+)

New commits:
commit 2fca05d5e4dee32feb6f9c9ba5f2c3792b72d5d1
Author: Lionel Elie Mamane lio...@mamane.lu
Date:   Thu Sep 13 16:24:38 2012 +0200

fdo#37529 clear DataColumns read-only information when we clear DataColumns

Change-Id: I6f3b4f1646df1244d97ae98f56ba054bc26e3314
Reviewed-on: https://gerrit.libreoffice.org/608
Reviewed-by: Michael Stahl mst...@redhat.com
Tested-by: Michael Stahl mst...@redhat.com
Signed-off-by: Miklos Vajna vmik...@suse.cz
Signed-off-by: Michael Meeks michael.me...@suse.com

diff --git a/dbaccess/source/core/api/RowSet.cxx 
b/dbaccess/source/core/api/RowSet.cxx
index afad13c..06a84a8 100644
--- a/dbaccess/source/core/api/RowSet.cxx
+++ b/dbaccess/source/core/api/RowSet.cxx
@@ -569,6 +569,8 @@ void ORowSet::freeResources( bool _bComplete )
 // the columns must be disposed before the querycomposer is disposed 
because
 // their owner can be the composer
 TDataColumns().swap(m_aDataColumns);// clear and resize capacity
+::std::vectorbool().swap(m_aReadOnlyDataColumns);
+
 m_xColumns  = NULL;
 if ( m_pColumns )
 m_pColumns-disposing();
@@ -1250,6 +1252,7 @@ void ORowSet::impl_setDataColumnsWriteable_throw()
 
 void ORowSet::impl_restoreDataColumnsWriteable_throw()
 {
+assert(m_aDataColumns.size() == m_aReadOnlyDataColumns.size() || 
m_aReadOnlyDataColumns.size() == 0 );
 TDataColumns::iterator aIter = m_aDataColumns.begin();
 ::std::vectorbool, std::allocatorbool ::iterator aReadIter = 
m_aReadOnlyDataColumns.begin();
 for(;aReadIter != m_aReadOnlyDataColumns.end();++aIter,++aReadIter)
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: sal/config.g sal/types.h and sal/osl/rtl includes in general

2012-09-20 Thread Norbert Thiebaud
On Thu, Sep 20, 2012 at 4:00 AM, Lubos Lunak l.lu...@suse.cz wrote:
 On Wednesday 19 of September 2012, Norbert Thiebaud wrote:
 So there is no automatic way to verify that an include set is
 'minimal', but there isn't even a way to do that with eyeballing...

  Actually, there possibly is an automatic way to do that:
 http://code.google.com/p/include-what-you-use/ . I haven't tried it though.

thanks... looks like exactly what we need... /me will have to build a
clang and build that plugin to try it :-)

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


Re: LO build can not find external dependencies

2012-09-20 Thread Michael Meeks
Hi there,

On Wed, 2012-09-19 at 10:35 +0200, Henrik /KaarPoSoft wrote:
 I am trying to build version 3.6.2.1 from source on a Linux box.

Sounds great.

 I want to use headers already on system and system installed versions of 
 libraries as much as possible, so I pass --with-system-headers 
 --with-system-libs to ./configure.

Seems reasonable.

 generates this error:
 --
 /home/kaarpux/KX_OO/kaarpux/linux/build/opt/libreoffice-core-3.6.2.1/solver/unxlngi6.pro/bin/cppunit/cppunittester:
  
 error while loading shared libraries: libcppunit-1.12.so.1: cannot open 
 shared object file: No such file or directory

Ok.

 --
 Even though the shared library IS present under /opt/lib, and the 
 symlinks are not broken.
 Workaround: --without-system-cppunit

So - is /opt/lib in your system linker path ? ie. in /etc/ld.so.conf -
it's an unusual place to have in there ;-) and have you run
sudo /sbin/ldconfig to re-build whatever caches there are there ?

Out of interest, what Linux are you using ? :-)

ATB,

Michael.

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

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


Re: LO build can not find external dependencies

2012-09-20 Thread Michael Stahl
On 19/09/12 10:35, Henrik /KaarPoSoft wrote:
 Dear all,
 
 I am trying to build version 3.6.2.1 from source on a Linux box.
 
 I want to use headers already on system and system installed versions of 
 libraries as much as possible, so I pass --with-system-headers 
 --with-system-libs to ./configure.
 
 The external dependencies are installed in /opt, so I
 export CPPFLAGS=-I/opt/include
 export LDFLAGS=-L/opt/lib
 before running ./autogen.sh

LibreOffice currently has 2 build systems; the new one should respect
these variables, but the old one probably does not, which effectively
makes them useless today.  you will probably have a better time by
setting PATH to /opt/bin and whatever variable is used by pkg-config, as
long as all the externals you have installed there have a foo-config
script or are pkg-config enabled the LO configure script should set
things up the right way.

another alternative would be to install a complete baseline (i.e. incl.
libc and everything needed) into some prefix and make use of the
existing GCC --sysroot support, but i've actually never done that in a
configure environment (i just remember that it was used to build OOo at
Sun), and you probably have to fix up some bitrot to get it to actually
work (reading configure it seems you'd need to set SYSBASE var).

 I also have
 export LD_LIBRARY_PATH=/opt/lib

apparently the new build system overwrites this always, guess nobody
ever needed to set this from the outside; you could try to edit various
occurrences in solenv/gbuild to prepend the values if it is already defined.

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


Re: sal/config.g sal/types.h and sal/osl/rtl includes in general

2012-09-20 Thread Michael Meeks

On Thu, 2012-09-20 at 11:00 +0200, Lubos Lunak wrote:
 On Wednesday 19 of September 2012, Norbert Thiebaud wrote:
  So there is no automatic way to verify that an include set is
  'minimal', but there isn't even a way to do that with eyeballing...
 
  Actually, there possibly is an automatic way to do that: 
 http://code.google.com/p/include-what-you-use/ . I haven't tried it though.

I -believe- this was tried - someone in bugzilla had an easy hack to
look at it, and it produced pretty awful / unusable results IIRC: mostly
adding dozens of new includes.

HTH,

Michael.

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

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


[PATCH] Change in core[libreoffice-3-6]: Don't access broken service mgr during bootstrap failure

2012-09-20 Thread Stephan Bergmann (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/654

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/54/654/1

Don't access broken service mgr during bootstrap failure

...so that displaying a (non-translated) error box upon BE_UNO_SERVICEMANAGER
works after all.  Augment the error text with an exception message where
appropriate.  This allows to revert fdfb7a3c4b3a89b73ab5546b9620348bc4984d8f
Related fdo#51252: Report uncaught exceptions with MessageBox on Windows as
that was to catch and display failures from instantiating the service mgr.

(cherry picked from commit 6bcfa095121c91e8bbc396f5bcf7e95424b9)

Change-Id: I049a38e95342634796eb0e940e2ee8e55193c9d3
---
M desktop/inc/app.hxx
M desktop/source/app/app.cxx
M desktop/source/app/desktop.hrc
M desktop/source/app/desktop.src
M desktop/source/app/sofficemain.cxx
M vcl/aqua/source/app/salsys.cxx
M vcl/generic/app/gensys.cxx
M vcl/inc/aqua/salsys.h
M vcl/inc/generic/gensys.h
M vcl/inc/ios/salsys.h
M vcl/inc/salsys.hxx
M vcl/inc/win/salsys.h
M vcl/ios/source/app/salsys.cxx
M vcl/source/app/svapp.cxx
M vcl/source/app/svdata.cxx
M vcl/win/source/app/salinfo.cxx
16 files changed, 172 insertions(+), 142 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I049a38e95342634796eb0e940e2ee8e55193c9d3
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-3-6
Gerrit-Owner: Stephan Bergmann sberg...@redhat.com

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


Re: [PUSHED 3-6][REVIEW 3-6-2 2/3] fdo#55059, use correct cell for broadcasting

2012-09-20 Thread Miklos Vajna
On Wed, Sep 19, 2012 at 01:22:54PM -0400, Kohei Yoshida 
kohei.yosh...@gmail.com wrote:
 Looks good to me.  Pushed to 3-6 with my sign off.
 
 Two more needed for 3-6-2.

Looks reasonable to me.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Is there easy way to find unused .idl files ?

2012-09-20 Thread Matúš Kukan
Hi,

recently I've touched .idl files in
http://cgit.freedesktop.org/libreoffice/core/commit/?id=b7c41f1d1fcabc8f4abc670d5a18546a117b540a
http://cgit.freedesktop.org/libreoffice/core/commit/?id=1f77f1ac177521698ee7c978f218127cb5840046

And I wonder if there is an easy way how to tell, they can be removed.
For files added with gb_UnoApi_add_idlfile_noheader, one probably have
to grep in the code.
But maybe for files in gb_UnoApi_add_idlfile_nohdl, it is possible to
know they are unused if there is no #include foo.hpp ?
And something similar for gb_UnoApi_add_idlfile ?

Is there something better than check each and every .idl file ?
I think we would find few(many?) unused ones.

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


[Libreoffice-commits] .: oox/source

2012-09-20 Thread Libreoffice Gerrit user
 oox/source/export/chartexport.cxx |3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit f3b2fc9be23739cec600b349e6b28b71be9acc3d
Author: Radek Doulik r...@novell.com
Date:   Thu Sep 20 12:46:15 2012 +0200

write only non NaN chart series values (fixes part of n#760997

Change-Id: I7c5bc8b9375a325d4ea78f033f577eff44cfd6e9

diff --git a/oox/source/export/chartexport.cxx 
b/oox/source/export/chartexport.cxx
index b51afcf..6519543 100644
--- a/oox/source/export/chartexport.cxx
+++ b/oox/source/export/chartexport.cxx
@@ -1801,7 +1801,8 @@ void ChartExport::exportSeriesValues( const Reference 
chart2::data::XDataSequen
 FSEND );
 pFS-startElement( FSNS( XML_c, XML_v ),
 FSEND );
-pFS-write( aValues[i] );
+if (aValues[i] == aValues[i])
+pFS-write( aValues[i] );
 pFS-endElement( FSNS( XML_c, XML_v ) );
 pFS-endElement( FSNS( XML_c, XML_pt ) );
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: sot/source

2012-09-20 Thread Libreoffice Gerrit user
 sot/source/sdstor/stgcache.cxx |   67 +
 sot/source/sdstor/stgcache.hxx |   19 ++-
 2 files changed, 52 insertions(+), 34 deletions(-)

New commits:
commit 540ab389f2adb021a95b5ac52c55c97fc5db5c17
Author: Michael Meeks michael.me...@suse.com
Date:   Thu Sep 20 11:52:49 2012 +0100

sot: memory savings to avoid duplicating the whole stream in RAM

re-work LRU cache to really be an 8 entry LRU cache.
only force dirty pages to stay in memory, hopefully huge reads
should now stream through memory.

Change-Id: I0bedc762086f5f02202795568743750aefaaa50b

diff --git a/sot/source/sdstor/stgcache.cxx b/sot/source/sdstor/stgcache.cxx
index 06a0ee0..e787c08 100644
--- a/sot/source/sdstor/stgcache.cxx
+++ b/sot/source/sdstor/stgcache.cxx
@@ -88,14 +88,17 @@ sal_Int32 lcl_GetPageCount( sal_uLong nFileSize, short 
nPageSize )
 }
 
 StgCache::StgCache()
+   : nError( SVSTREAM_OK )
+   , nPages( 0 )
+   , nRef( 0 )
+   , nReplaceIdx( 0 )
+   , maLRUPages( 8 ) // entries in the LRU lookup
+   , nPageSize( 512 )
+   , pStorageStream( NULL )
+   , pStrm( NULL )
+   , bMyStream( sal_False )
+   , bFile( sal_False )
 {
-nRef = 0;
-pStrm = NULL;
-nPageSize = 512;
-nError = SVSTREAM_OK;
-bMyStream = sal_False;
-bFile = sal_False;
-pStorageStream = NULL;
 }
 
 StgCache::~StgCache()
@@ -122,7 +125,7 @@ void StgCache::SetPhysPageSize( short n )
 rtl::Reference StgPage  StgCache::Create( sal_Int32 nPg )
 {
 rtl::Reference StgPage  xElem( StgPage::Create( nPageSize, nPg ) );
-maLRUCache[ xElem-GetPage() ] = xElem;
+maLRUPages[ nReplaceIdx++ % maLRUPages.size() ] = xElem;
 return xElem;
 }
 
@@ -131,8 +134,14 @@ rtl::Reference StgPage  StgCache::Create( sal_Int32 nPg )
 void StgCache::Erase( const rtl::Reference StgPage  xElem )
 {
 OSL_ENSURE( xElem.is(), The pointer should not be NULL! );
-if ( xElem.is() )
-maLRUCache.erase( xElem-GetPage() );
+if ( xElem.is() ) {
+for ( LRUList::iterator it = maLRUPages.begin(); it != 
maLRUPages.end(); it++ ) {
+if ( it-is()  (*it)-GetPage() == xElem-GetPage() ) {
+it-clear();
+break;
+}
+}
+}
 }
 
 // remove all cache elements without flushing them
@@ -140,16 +149,20 @@ void StgCache::Erase( const rtl::Reference StgPage  
xElem )
 void StgCache::Clear()
 {
 maDirtyPages.clear();
-maLRUCache.clear();
+for ( LRUList::iterator it = maLRUPages.begin(); it != maLRUPages.end(); 
it++ )
+it-clear();
 }
 
 // Look for a cached page
 
 rtl::Reference StgPage  StgCache::Find( sal_Int32 nPage )
 {
-IndexToStgPage::iterator aIt = maLRUCache.find( nPage );
-if( aIt != maLRUCache.end() )
-return (*aIt).second;
+for ( LRUList::iterator it = maLRUPages.begin(); it != maLRUPages.end(); 
it++ )
+if ( it-is()  (*it)-GetPage() == nPage )
+return *it;
+IndexToStgPage::iterator it2 = maDirtyPages.find( nPage );
+if ( it2 != maDirtyPages.end() )
+return it2-second;
 return rtl::Reference StgPage ();
 }
 
@@ -199,18 +212,21 @@ rtl::Reference StgPage  StgCache::Copy( sal_Int32 nNew, 
sal_Int32 nOld )
 // continue that tradition.
 sal_Bool StgCache::Commit()
 {
-std::vector StgPage *  aToWrite;
-for ( IndexToStgPage::iterator aIt = maDirtyPages.begin();
-  aIt != maDirtyPages.end(); aIt++ )
-aToWrite.push_back( aIt-second.get() );
-
-std::sort( aToWrite.begin(), aToWrite.end(), StgPage::IsPageGreater );
-for (std::vector StgPage * ::iterator aWr = aToWrite.begin();
- aWr != aToWrite.end(); aWr++)
+if ( Good() ) // otherwise Write does nothing
 {
-const rtl::Reference StgPage  pPage = *aWr;
-if ( !Write( pPage-GetPage(), pPage-GetData(), 1 ) )
-return sal_False;
+std::vector StgPage *  aToWrite;
+for ( IndexToStgPage::iterator aIt = maDirtyPages.begin();
+  aIt != maDirtyPages.end(); aIt++ )
+aToWrite.push_back( aIt-second.get() );
+
+std::sort( aToWrite.begin(), aToWrite.end(), StgPage::IsPageGreater );
+for ( std::vector StgPage * ::iterator aWr = aToWrite.begin();
+  aWr != aToWrite.end(); aWr++)
+{
+const rtl::Reference StgPage  pPage = *aWr;
+if ( !Write( pPage-GetPage(), pPage-GetData(), 1 ) )
+return sal_False;
+}
 }
 
 maDirtyPages.clear();
@@ -259,6 +275,7 @@ void StgCache::SetStrm( UCBStorageStream* pStgStream )
 
 void StgCache::SetDirty( const rtl::Reference StgPage  xPage )
 {
+assert( IsWritable() );
 maDirtyPages[ xPage-GetPage() ] = xPage;
 }
 
diff --git a/sot/source/sdstor/stgcache.hxx b/sot/source/sdstor/stgcache.hxx
index 2687846..e134e25 100644
--- a/sot/source/sdstor/stgcache.hxx
+++ b/sot/source/sdstor/stgcache.hxx
@@ -33,20 +33,21 @@ class StgPage;
 class StgDirEntry;
 class StorageBase;
 

[Libreoffice-commits] .: sw/source

2012-09-20 Thread Libreoffice Gerrit user
 sw/source/core/unocore/unodraw.cxx |   18 --
 1 file changed, 18 deletions(-)

New commits:
commit 21359dc87000c65665e49652d1c255e679817787
Author: Miklos Vajna vmik...@suse.cz
Date:   Thu Sep 20 13:01:04 2012 +0200

sw: drop unused includes in unodraw

Change-Id: I2cae970f532c3441a7ec7c8318987ea343aaf1cd

diff --git a/sw/source/core/unocore/unodraw.cxx 
b/sw/source/core/unocore/unodraw.cxx
index 43546dd..2e37dd6 100644
--- a/sw/source/core/unocore/unodraw.cxx
+++ b/sw/source/core/unocore/unodraw.cxx
@@ -27,17 +27,13 @@
  /
 
 
-#include swtypes.hxx
 #include cmdid.h
 
-#include unomid.h
 #include unodraw.hxx
 #include unocoll.hxx
 #include unoframe.hxx
 #include unoparagraph.hxx
 #include unotextrange.hxx
-#include unoprnms.hxx
-#include editeng/unoprnms.hxx
 #include svx/svditer.hxx
 #include swunohelper.hxx
 #include doc.hxx
@@ -48,9 +44,7 @@
 #include docsh.hxx
 #include unomap.hxx
 #include unoport.hxx
-#include unocrsr.hxx
 #include TextCursorHelper.hxx
-#include swundo.hxx
 #include dflyobj.hxx
 #include ndtxt.hxx
 #include svx/svdview.hxx
@@ -58,7 +52,6 @@
 #include dcontact.hxx
 #include svx/fmglob.hxx
 #include fmtornt.hxx
-#include fmtanchr.hxx
 #include fmtsrnd.hxx
 #include fmtfollowtextflow.hxx
 #include rootfrm.hxx
@@ -66,26 +59,15 @@
 #include editeng/ulspitem.hxx
 #include svx/shapepropertynotifier.hxx
 #include crstate.hxx
-#include osl/mutex.hxx
 #include comphelper/extract.hxx
 #include comphelper/stl_types.hxx
 #include comphelper/makesequence.hxx
-#include comphelper/servicehelper.hxx
 #include svx/scene3d.hxx
-#include com/sun/star/beans/PropertyAttribute.hpp
 #include com/sun/star/drawing/XDrawPageSupplier.hpp
-#include com/sun/star/text/HoriOrientation.hpp
-#include com/sun/star/text/VertOrientation.hpp
-#include basegfx/numeric/ftools.hxx
-#include algorithm
 #include fmtwrapinfluenceonobjpos.hxx
 #include com/sun/star/text/TextContentAnchorType.hpp
-#include basegfx/matrix/b2dhommatrix.hxx
 #include basegfx/matrix/b2dhommatrixtools.hxx
 #include com/sun/star/drawing/PointSequence.hpp
-#include vcl/svapp.hxx
-#include list
-#include iterator
 #include switerator.hxx
 
 using ::rtl::OUString;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: offapi/com offapi/UnoApi_offapi.mk officecfg/Configuration_officecfg.mk officecfg/registry postprocess/packregistry

2012-09-20 Thread Libreoffice Gerrit user
 offapi/UnoApi_offapi.mk|1 
 offapi/com/sun/star/setup/ProductRegistration.idl  |   82 +
 officecfg/Configuration_officecfg.mk   |1 
 officecfg/registry/data/org/openoffice/Office/Jobs.xcu |   39 
 postprocess/packregistry/makefile.mk   |1 
 5 files changed, 124 insertions(+)

New commits:
commit 6751cf5017fca96e6bef8ef2c316f676055eefb6
Author: Matúš Kukan matus.ku...@gmail.com
Date:   Thu Sep 20 12:52:49 2012 +0200

Revert remove remnants of ProductRegistration

This reverts commit 1f77f1ac177521698ee7c978f218127cb5840046
and b590d87b380389371eca94ffb0e6f47f86d1c917.
Do this again when we will know how to get rid of published service
in offapi/type_reference/types.rdb.

diff --git a/offapi/UnoApi_offapi.mk b/offapi/UnoApi_offapi.mk
index 047a2ec..b053b9b 100644
--- a/offapi/UnoApi_offapi.mk
+++ b/offapi/UnoApi_offapi.mk
@@ -1051,6 +1051,7 @@ $(eval $(call 
gb_UnoApi_add_idlfiles_noheader,offapi,offapi/com/sun/star/sdbcx,\
ViewDescriptor \
 ))
 $(eval $(call 
gb_UnoApi_add_idlfiles_noheader,offapi,offapi/com/sun/star/setup,\
+   ProductRegistration \
Setup \
 ))
 $(eval $(call 
gb_UnoApi_add_idlfiles_noheader,offapi,offapi/com/sun/star/sheet,\
diff --git a/offapi/com/sun/star/setup/ProductRegistration.idl 
b/offapi/com/sun/star/setup/ProductRegistration.idl
new file mode 100644
index 000..3dbdf57
--- /dev/null
+++ b/offapi/com/sun/star/setup/ProductRegistration.idl
@@ -0,0 +1,82 @@
+/* -*- 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 __com_sun_star_setup_ProductRegistration_idl__
+#define __com_sun_star_setup_ProductRegistration_idl__
+
+#include com/sun/star/task/JobExecutor.idl
+#include com/sun/star/task/XJob.idl
+
+module com {  module sun {  module star {  module setup {
+
+/** provides a mechanism for registering the product.
+
+pTwo main functionalities are covered by this service:
+ulilemTriggering Online Registration/embr/
+If, by any user interface of your choice, the user tells she wants 
to register
+their product copy, ProductRegistration is the service of your 
choice.
+/ilbr/
+ilemExecuting Registration Requests/embr/
+ProductRegistration allows to remind your users to register, using 
a dialog with various
+choices.
+/il
+/ul/p
+
+@since OOo 1.1.2
+*/
+published service ProductRegistration
+{
+/** allows to trigger certain events.
+
+pThe string argument of method 
scope=com::sun::star::taskXJobExecutor::trigger/method is
+used to determine the type of the event. Currently supported 
events are:br/
+ullibRegistrationRequired/bbr/
+Determines that the registration is required to be 
executed.br/
+This is usually triggered by some user interaction, for 
instance upon choosing a menu item
+called emRegistration/em./p
+/li
+/ul
+/p
+*/
+service com::sun::star::task::JobExecutor;
+
+/** allows the registration request to be scheduled automatically.
+
+pThere are processes which allow to schedule events and jobs, for 
instance there
+can be configuration settings scheduling which operations are to be 
executed upon an program
+startup. Such processes usually require the jobs to be executed to 
support the
+type scope=com::sun::star::taskXJob/type interface./p
+
+pThat's why this service supports the type 
scope=com::sun::star::taskXJob/type, too.br/
+Calling the method 
scope=com::sun::star::taskXJob::execute/method method results in 
displaying a
+dialog which asks the user to register. This is done once per session, 
means you need to restart the
+process which called this./p
+
+pAt the moment, there is no specification about how to use the 
arguments given in the
+method scope=com::sun::star::taskXJob::execute/method method, 
but this may change in future versions of the service
+  

Re: sal/config.g sal/types.h and sal/osl/rtl includes in general

2012-09-20 Thread Jack Leigh

On 20/09/12 11:12, Norbert Thiebaud wrote:

On Thu, Sep 20, 2012 at 4:00 AM, Lubos Lunak l.lu...@suse.cz wrote:

On Wednesday 19 of September 2012, Norbert Thiebaud wrote:

So there is no automatic way to verify that an include set is
'minimal', but there isn't even a way to do that with eyeballing...


  Actually, there possibly is an automatic way to do that:
http://code.google.com/p/include-what-you-use/ . I haven't tried it though.


thanks... looks like exactly what we need... /me will have to build a
clang and build that plugin to try it :-)

norbert


https://bugs.freedesktop.org/show_bug.cgi?id=42949 accelerating compile 
times ... is the bug Michael mentioned.


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


[Libreoffice-commits] .: vcl/inc vcl/unx

2012-09-20 Thread Libreoffice Gerrit user
 vcl/inc/cupsmgr.hxx |2 +-
 vcl/unx/generic/printer/cupsmgr.cxx |3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

New commits:
commit 8e1e27ae6d0751097d276eb3394b57af123e09f3
Author: Michael Meeks michael.me...@suse.com
Date:   Thu Sep 20 12:00:11 2012 +0100

cups: initialize thread running member

Change-Id: Idf8b079857638558075fa6b1fb5f7ea6b3c9f632

diff --git a/vcl/inc/cupsmgr.hxx b/vcl/inc/cupsmgr.hxx
index f3cee19..d2ae11d 100644
--- a/vcl/inc/cupsmgr.hxx
+++ b/vcl/inc/cupsmgr.hxx
@@ -47,7 +47,7 @@ struct FPtrHash
 
 class CUPSManager : public PrinterInfoManager
 {
-boost::unordered_map FILE*, rtl::OString, FPtrHash
m_aSpoolFiles;
+boost::unordered_map FILE*, rtl::OString, FPtrHashm_aSpoolFiles;
 int m_nDests;
 void*   m_pDests;
 boolm_bNewDests;
diff --git a/vcl/unx/generic/printer/cupsmgr.cxx 
b/vcl/unx/generic/printer/cupsmgr.cxx
index f13b774..616385e 100644
--- a/vcl/unx/generic/printer/cupsmgr.cxx
+++ b/vcl/unx/generic/printer/cupsmgr.cxx
@@ -193,7 +193,8 @@ CUPSManager::CUPSManager() :
 PrinterInfoManager( CUPS ),
 m_nDests( 0 ),
 m_pDests( NULL ),
-m_bNewDests( false )
+m_bNewDests( false ),
+m_bPPDThreadRunning( false )
 {
 m_aDestThread = osl_createThread( run_dest_thread_stub, this );
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[REVIEW-3-6] trivial cups thread init issue ...

2012-09-20 Thread Michael Meeks
As attached,

I've noticed a number of these straying in. Did we disable a warning on
un-initialized members for some reason ? I could swear the compiler used
to complain helpfully about this.

Thanks,

Michael.

-- 
michael.me...@suse.com  , Pseudo Engineer, itinerant idiot
From 23c4a5bf342abdddb3df13c1d8d8f6fc36ecd94a Mon Sep 17 00:00:00 2001
From: Michael Meeks michael.me...@suse.com
Date: Thu, 20 Sep 2012 12:00:11 +0100
Subject: [PATCH] cups: initialize thread running member

Change-Id: Idf8b079857638558075fa6b1fb5f7ea6b3c9f632
---
 vcl/inc/cupsmgr.hxx |2 +-
 vcl/unx/generic/printer/cupsmgr.cxx |3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/vcl/inc/cupsmgr.hxx b/vcl/inc/cupsmgr.hxx
index f3cee19..d2ae11d 100644
--- a/vcl/inc/cupsmgr.hxx
+++ b/vcl/inc/cupsmgr.hxx
@@ -47,7 +47,7 @@ struct FPtrHash
 
 class CUPSManager : public PrinterInfoManager
 {
-boost::unordered_map FILE*, rtl::OString, FPtrHashm_aSpoolFiles;
+boost::unordered_map FILE*, rtl::OString, FPtrHashm_aSpoolFiles;
 int m_nDests;
 void*   m_pDests;
 boolm_bNewDests;
diff --git a/vcl/unx/generic/printer/cupsmgr.cxx b/vcl/unx/generic/printer/cupsmgr.cxx
index f13b774..616385e 100644
--- a/vcl/unx/generic/printer/cupsmgr.cxx
+++ b/vcl/unx/generic/printer/cupsmgr.cxx
@@ -193,7 +193,8 @@ CUPSManager::CUPSManager() :
 PrinterInfoManager( CUPS ),
 m_nDests( 0 ),
 m_pDests( NULL ),
-m_bNewDests( false )
+m_bNewDests( false ),
+m_bPPDThreadRunning( false )
 {
 m_aDestThread = osl_createThread( run_dest_thread_stub, this );
 }
-- 
1.7.7

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


Re: LO build can not find external dependencies

2012-09-20 Thread Henrik /KaarPoSoft

On 09/20/12 12:15, Michael Stahl wrote:

On 19/09/12 10:35, Henrik /KaarPoSoft wrote:

Dear all,

I am trying to build version 3.6.2.1 from source on a Linux box.

I want to use headers already on system and system installed versions of
libraries as much as possible, so I pass --with-system-headers
--with-system-libs to ./configure.

The external dependencies are installed in /opt, so I
export CPPFLAGS=-I/opt/include
export LDFLAGS=-L/opt/lib
before running ./autogen.sh

LibreOffice currently has 2 build systems; the new one should respect
these variables, but the old one probably does not, which effectively
makes them useless today.  you will probably have a better time by
setting PATH to /opt/bin and whatever variable is used by pkg-config, as
long as all the externals you have installed there have a foo-config
script or are pkg-config enabled the LO configure script should set
things up the right way.

I already have
export PATH=$PATH:/opt/sbin:/opt/bin
export LD_LIBRARY_PATH=/local/lib:/lib:/opt/lib
export PKG_CONFIG_PATH=/opt/lib/pkgconfig
and LO configuration seems to pick up the dependencies with pkg-config, 
as config_host.mk has a lot of export *_CFLAGS= and export *_LIBS with 
info from pkg-config.

another alternative would be to install a complete baseline (i.e. incl.
libc and everything needed) into some prefix and make use of the
existing GCC --sysroot support, but i've actually never done that in a
configure environment (i just remember that it was used to build OOo at
Sun), and you probably have to fix up some bitrot to get it to actually
work (reading configure it seems you'd need to set SYSBASE var).
I would rather not go into this, as it seems much to complex for this 
simple problem.


Anyway, I do not think it would solve the problem.
It is not gcc which is complaining about the missing libraries, but 
rather the system linker which complains when trying to run some 
executable created by LO.

I also have
export LD_LIBRARY_PATH=/opt/lib

apparently the new build system overwrites this always, guess nobody
ever needed to set this from the outside; you could try to edit various
occurrences in solenv/gbuild to prepend the values if it is already defined.

*Do you have any suggestions as to which files to look into/edit?*

I do not have much experience with the LO codebase, and a grep for 
LD_LIBRARY_PATH returns almost 1000 files...


I am now trying to build with:
./autogen.sh [...lots-of-options...] --with-system-headers 
--with-system-libs CPPFLAGS=-I/opt/include LDFLAGS=-L/opt/lib

export CPPFLAGS=-I/opt/include
export LDFLAGS=-L/opt/lib
make verbose=t CPPFLAGS=-I/opt/include LDFLAGS=-L/opt/lib build
but running into the same problems.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: Branch 'libreoffice-3-6' - desktop/inc desktop/source vcl/aqua vcl/generic vcl/inc vcl/ios vcl/source vcl/win

2012-09-20 Thread Libreoffice Gerrit user
 desktop/inc/app.hxx|   19 ++---
 desktop/source/app/app.cxx |  139 +
 desktop/source/app/desktop.hrc |1 
 desktop/source/app/desktop.src |5 -
 desktop/source/app/sofficemain.cxx |   21 -
 vcl/aqua/source/app/salsys.cxx |   19 +++--
 vcl/generic/app/gensys.cxx |   60 +--
 vcl/inc/aqua/salsys.h  |2 
 vcl/inc/generic/gensys.h   |2 
 vcl/inc/ios/salsys.h   |2 
 vcl/inc/salsys.hxx |7 +
 vcl/inc/win/salsys.h   |2 
 vcl/ios/source/app/salsys.cxx  |   19 +++--
 vcl/source/app/svapp.cxx   |2 
 vcl/source/app/svdata.cxx  |   12 +--
 vcl/win/source/app/salinfo.cxx |2 
 16 files changed, 172 insertions(+), 142 deletions(-)

New commits:
commit 98acef27d06dfa63274ec4ce0ec934fc52e5a15d
Author: Stephan Bergmann sberg...@redhat.com
Date:   Tue Sep 18 12:40:57 2012 +0200

Don't access broken service mgr during bootstrap failure

...so that displaying a (non-translated) error box upon 
BE_UNO_SERVICEMANAGER
works after all.  Augment the error text with an exception message where
appropriate.  This allows to revert fdfb7a3c4b3a89b73ab5546b9620348bc4984d8f
Related fdo#51252: Report uncaught exceptions with MessageBox on Windows 
as
that was to catch and display failures from instantiating the service mgr.

(cherry picked from commit 6bcfa095121c91e8bbc396f5bcf7e95424b9)

Change-Id: I049a38e95342634796eb0e940e2ee8e55193c9d3
Reviewed-on: https://gerrit.libreoffice.org/654
Reviewed-by: Michael Stahl mst...@redhat.com
Tested-by: Michael Stahl mst...@redhat.com

diff --git a/desktop/inc/app.hxx b/desktop/inc/app.hxx
index c830b02..8805c57 100644
--- a/desktop/inc/app.hxx
+++ b/desktop/inc/app.hxx
@@ -107,15 +107,16 @@ class Desktop : public Application
 static ResMgr*  GetDesktopResManager();
 static CommandLineArgs GetCommandLineArgs();
 
-voidHandleBootstrapErrors( BootstrapError );
-voidSetBootstrapError( BootstrapError nError )
+voidHandleBootstrapErrors(
+BootstrapError nError, rtl::OUString const  aMessage );
+voidSetBootstrapError(
+BootstrapError nError, rtl::OUString const  aMessage )
 {
 if ( m_aBootstrapError == BE_OK )
+{
 m_aBootstrapError = nError;
-}
-BootstrapError  GetBootstrapError() const
-{
-return m_aBootstrapError;
+m_aBootstrapErrorMessage = aMessage;
+}
 }
 
 voidSetBootstrapStatus( BootstrapStatus nStatus )
@@ -139,8 +140,6 @@ class Desktop : public Application
 voidSetSplashScreenText( const ::rtl::OUString 
rText );
 voidSetSplashScreenProgress( sal_Int32 );
 
-static void ensureProcessServiceFactory();
-
 private:
 // Bootstrap methods
 static ::com::sun::star::uno::Reference 
::com::sun::star::lang::XMultiServiceFactory  
CreateApplicationServiceManager();
@@ -163,9 +162,6 @@ class Desktop : public Application
 voidHandleBootstrapPathErrors( 
::utl::Bootstrap::Status, const ::rtl::OUString aMsg );
 voidStartSetup( const ::rtl::OUString aParameters 
);
 
-// Get a resource message string securely e.g. if resource cannot be 
retrieved return aFaultBackMsg
-::rtl::OUString GetMsgString( sal_uInt16 nId, const 
::rtl::OUString aFaultBackMsg );
-
 // Create a error message depending on bootstrap failure code and an 
optional file url
 ::rtl::OUString CreateErrorMsgString( 
utl::Bootstrap::FailureCode nFailureCode,
   const ::rtl::OUString 
aFileURL );
@@ -207,6 +203,7 @@ class Desktop : public Application
 boolm_bServicesRegistered;
 sal_uInt16  m_nAppEvents;
 BootstrapError  m_aBootstrapError;
+rtl::OUString   m_aBootstrapErrorMessage;
 BootstrapStatus m_aBootstrapStatus;
 
 std::auto_ptr Lockfile  m_pLockfile;
diff --git a/desktop/source/app/app.cxx b/desktop/source/app/app.cxx
index 5fb653b..ea350ca 100644
--- a/desktop/source/app/app.cxx
+++ b/desktop/source/app/app.cxx
@@ -373,28 +373,34 @@ ResMgr* Desktop::GetDesktopResManager()
 return Desktop::pResMgr;
 }
 
+namespace {
+
 // 
 // Get a message string securely. There is a fallback string if the resource
 // is not available.
 
-OUString Desktop::GetMsgString( sal_uInt16 nId, const OUString 

[PUSHED] Change in core[libreoffice-3-6]: Don't access broken service mgr during bootstrap failure

2012-09-20 Thread Michael Stahl (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/654

Approvals:
  Michael Stahl: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I049a38e95342634796eb0e940e2ee8e55193c9d3
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-3-6
Gerrit-Owner: Stephan Bergmann sberg...@redhat.com
Gerrit-Reviewer: Michael Stahl mst...@redhat.com
Gerrit-Reviewer: Stephan Bergmann sberg...@redhat.com

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


[Libreoffice-commits] .: 4 commits - sc/source stoc/source sw/source xmloff/source

2012-09-20 Thread Libreoffice Gerrit user
 sc/source/ui/unoobj/dispuno.cxx   |4 ++--
 stoc/source/corereflection/crefl.cxx  |5 +
 sw/source/core/text/porlay.cxx|6 +++---
 sw/source/ui/lingu/olmenu.cxx |2 +-
 sw/source/ui/uno/unodispatch.cxx  |   12 ++--
 xmloff/source/style/XMLBackgroundImageContext.cxx |4 ++--
 6 files changed, 19 insertions(+), 14 deletions(-)

New commits:
commit e7c9b468f364fd4796f0ff6fc6b76045ad58b26f
Author: Tor Lillqvist t...@iki.fi
Date:   Thu Sep 20 14:05:17 2012 +0300

Add some static goodness here, too

Change-Id: I7de335925748b2cfe28a01f6975a630081b3172e

diff --git a/sc/source/ui/unoobj/dispuno.cxx b/sc/source/ui/unoobj/dispuno.cxx
index 7d0c2e9..9ca3481 100644
--- a/sc/source/ui/unoobj/dispuno.cxx
+++ b/sc/source/ui/unoobj/dispuno.cxx
@@ -45,8 +45,8 @@ using namespace com::sun::star;
 
 //
 
-const char* cURLInsertColumns = .uno:DataSourceBrowser/InsertColumns; //data 
into text
-const char* cURLDocDataSource = .uno:DataSourceBrowser/DocumentDataSource;
+static const char* cURLInsertColumns = .uno:DataSourceBrowser/InsertColumns; 
//data into text
+static const char* cURLDocDataSource = 
.uno:DataSourceBrowser/DocumentDataSource;
 
 //
 
commit fed4ca185bc2fa29b915e6ea63b0bf7aeec29e5f
Author: Tor Lillqvist t...@iki.fi
Date:   Thu Sep 20 13:56:26 2012 +0300

Sprinkle some staticness to avoid duplicate symbols when DISABLE_DYNLOADING

Change-Id: I9c5203400904902238c1f9a8f01338fdcf2904d8

diff --git a/xmloff/source/style/XMLBackgroundImageContext.cxx 
b/xmloff/source/style/XMLBackgroundImageContext.cxx
index 3c46600..ae1e77b 100644
--- a/xmloff/source/style/XMLBackgroundImageContext.cxx
+++ b/xmloff/source/style/XMLBackgroundImageContext.cxx
@@ -82,14 +82,14 @@ const SvXMLTokenMapEntry* 
lcl_getBGImgAttributesAttrTokenMap()
 
 
 
-SvXMLEnumMapEntry psXML_BrushHoriPos[] =
+static SvXMLEnumMapEntry psXML_BrushHoriPos[] =
 {
 { XML_LEFT, GraphicLocation_LEFT_MIDDLE },
 { XML_RIGHT,GraphicLocation_RIGHT_MIDDLE},
 { XML_TOKEN_INVALID,0   }
 };
 
-SvXMLEnumMapEntry psXML_BrushVertPos[] =
+static SvXMLEnumMapEntry psXML_BrushVertPos[] =
 {
 { XML_TOP,  GraphicLocation_MIDDLE_TOP  },
 { XML_BOTTOM,   GraphicLocation_MIDDLE_BOTTOM   },
commit c672b1f43c64e5f411badc2deb2d39a7c0fe6373
Author: Tor Lillqvist t...@iki.fi
Date:   Thu Sep 20 13:55:38 2012 +0300

No point with component_canUnload when DISABLE_DYNLOADING

Change-Id: I85016d1e70dbb5e499aefad568179800b5d00ab7

diff --git a/stoc/source/corereflection/crefl.cxx 
b/stoc/source/corereflection/crefl.cxx
index 1af1b3a..3357a54 100644
--- a/stoc/source/corereflection/crefl.cxx
+++ b/stoc/source/corereflection/crefl.cxx
@@ -464,11 +464,16 @@ static struct ImplementationEntry g_entries[] =
 
 extern C
 {
+
+#ifndef DISABLE_DYNLOADING
+
 SAL_DLLPUBLIC_EXPORT sal_Bool SAL_CALL component_canUnload( TimeValue *pTime )
 {
 return g_moduleCount.canUnload( g_moduleCount , pTime );
 }
 
+#endif
+
 
//==
 SAL_DLLPUBLIC_EXPORT void * SAL_CALL reflection_component_getFactory(
 const sal_Char * pImplName, void * pServiceManager, void * pRegistryKey )
commit 29b0c6093db08ed959022af7ee53b42b155c7fb5
Author: Tor Lillqvist t...@iki.fi
Date:   Thu Sep 20 13:54:27 2012 +0300

Sprinkle some staticness to avoid duplicate symbols when DISABLE_DYNLOADING

Change-Id: I2d62eddb561baa28e320f7d89a8655d2806d7b10

diff --git a/sw/source/core/text/porlay.cxx b/sw/source/core/text/porlay.cxx
index d03304a..f1205dc 100644
--- a/sw/source/core/text/porlay.cxx
+++ b/sw/source/core/text/porlay.cxx
@@ -99,7 +99,7 @@ sal_Bool isTransparentChar ( xub_Unicode cCh )
  * Checks if cCh + cNectCh builds a ligature (used for Kashidas)
  */
 
-sal_Bool lcl_IsLigature( xub_Unicode cCh, xub_Unicode cNextCh )
+static sal_Bool lcl_IsLigature( xub_Unicode cCh, xub_Unicode cNextCh )
 {
 // Lam + Alef
 return ( isLamChar ( cCh )  isAlefChar ( cNextCh ));
@@ -111,7 +111,7 @@ sal_Bool lcl_IsLigature( xub_Unicode cCh, xub_Unicode 
cNextCh )
  * Checks if cCh is connectable to cPrevCh (used for Kashidas)
  */
 
-sal_Bool lcl_ConnectToPrev( xub_Unicode cCh, xub_Unicode cPrevCh )
+static sal_Bool lcl_ConnectToPrev( xub_Unicode cCh, xub_Unicode cPrevCh )
 {
 const int32_t nJoiningType = u_getIntPropertyValue( cPrevCh, 
UCHAR_JOINING_TYPE );
 sal_Bool bRet = nJoiningType != U_JT_RIGHT_JOINING  nJoiningType != 
U_JT_NON_JOINING;
@@ -126,7 +126,7 @@ sal_Bool lcl_ConnectToPrev( xub_Unicode cCh, 

Re: LO build can not find external dependencies

2012-09-20 Thread Michael Stahl
On 20/09/12 13:12, Henrik /KaarPoSoft wrote:
 On 09/20/12 12:15, Michael Stahl wrote:
 On 19/09/12 10:35, Henrik /KaarPoSoft wrote:
 I also have
 export LD_LIBRARY_PATH=/opt/lib
 apparently the new build system overwrites this always, guess nobody
 ever needed to set this from the outside; you could try to edit various
 occurrences in solenv/gbuild to prepend the values if it is already defined.
 *Do you have any suggestions as to which files to look into/edit?*
 
 I do not have much experience with the LO codebase, and a grep for 
 LD_LIBRARY_PATH returns almost 1000 files...

look in solenv/gbuild, only 11 hits there; one of which to set
gb_Helper_LIBRARY_PATH_VAR , which is used here:

solenv/gbuild/platform/com_GCC_defs.mk:gb_Helper_set_ld_path :=
$(gb_Helper_LIBRARY_PATH_VAR)=$(OUTDIR_FOR_BUILD)/lib
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: javaunohelper/com javaunohelper/Library_juh.mk javaunohelper/Module_javaunohelper.mk

2012-09-20 Thread Libreoffice Gerrit user
 javaunohelper/Library_juh.mk  |   11 ++
 javaunohelper/Module_javaunohelper.mk |9 -
 javaunohelper/com/sun/star/comp/helper/Bootstrap.java |   32 +-
 3 files changed, 50 insertions(+), 2 deletions(-)

New commits:
commit 4b326ea35cb9193dbb530b9445a778d61bf8273d
Author: Tor Lillqvist t...@iki.fi
Date:   Thu Sep 20 14:16:40 2012 +0300

No need for the juh/juhx dance when DISABLE_DYNLOADING

Change-Id: Ifc2bfb278947344d14b855ebf5527b603e333f15

diff --git a/javaunohelper/Library_juh.mk b/javaunohelper/Library_juh.mk
index 4783bd9..1336469 100644
--- a/javaunohelper/Library_juh.mk
+++ b/javaunohelper/Library_juh.mk
@@ -36,8 +36,19 @@ $(eval $(call gb_Library_use_libraries,juh,\
 $(gb_STDLIBS) \
 ))
 
+ifneq ($(DISABLE_DYNLOADING),TRUE)
 $(eval $(call gb_Library_add_exception_objects,juh,\
 javaunohelper/source/preload \
 ))
+else
+# In the DISABLE_DYNLOADING case the juh library is a static archive that gets
+# linked into the single .so, so we can put directly into it the code that in
+# normal cases goes into the juhx library
+$(eval $(call gb_Library_add_exception_objects,juh,\
+javaunohelper/source/bootstrap \
+javaunohelper/source/javaunohelper \
+javaunohelper/source/vm \
+))
+endif
 
 # vim:set shiftwidth=4 softtabstop=4 expandtab:
diff --git a/javaunohelper/Module_javaunohelper.mk 
b/javaunohelper/Module_javaunohelper.mk
index 27c53e4..181e10f 100644
--- a/javaunohelper/Module_javaunohelper.mk
+++ b/javaunohelper/Module_javaunohelper.mk
@@ -28,12 +28,19 @@
 $(eval $(call gb_Module_Module,javaunohelper))
 
 ifneq ($(SOLAR_JAVA),)
+
 $(eval $(call gb_Module_add_targets,javaunohelper,\
 Library_juh \
-Library_juhx \
 Jar_juh \
 Zip_juh \
 ))
+
+ifneq ($(DISABLE_DYNLOADING),TRUE)
+$(eval $(call gb_Module_add_targets,javaunohelper,\
+Library_juhx \
+))
+endif
+
 endif
 
 # vim:set shiftwidth=4 softtabstop=4 expandtab:
diff --git a/javaunohelper/com/sun/star/comp/helper/Bootstrap.java 
b/javaunohelper/com/sun/star/comp/helper/Bootstrap.java
index 120217f..2087aec 100644
--- a/javaunohelper/com/sun/star/comp/helper/Bootstrap.java
+++ b/javaunohelper/com/sun/star/comp/helper/Bootstrap.java
@@ -1,3 +1,5 @@
+// -*- Mode: Java; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*-
+
 /*
  * This file is part of the LibreOffice project.
  *
@@ -191,7 +193,33 @@ public class Bootstrap {
 
 if (! m_loaded_juh)
 {
-NativeLibraryLoader.loadLibrary( Bootstrap.class.getClassLoader(), 
juh );
+if (System.getProperty(java.vendor) == The Android Project)
+{
+// Find out if we are configured with DISABLE_DYNLOADING or
+// not. Try to load the lo-bootstrap shared library which
+// won't exist in the DISABLE_DYNLOADING case. (And which will
+// be already loaded otherwise, so nothing unexpected happens
+// that case.) Yeah, this would be simpler if I just could be
+// bothered to keep a separate branch for DISABLE_DYNLOADING
+// on Android, merging in master periodically, until I know
+// for sure whether it is what I want, or not.
+
+boolean disable_dynloading = false;
+try {
+System.loadLibrary( lo-bootstrap );
+} catch ( UnsatisfiedLinkError e ) {
+disable_dynloading = true;
+}
+
+if (!disable_dynloading)
+{
+NativeLibraryLoader.loadLibrary( 
Bootstrap.class.getClassLoader(), juh );
+}
+}
+else
+{
+NativeLibraryLoader.loadLibrary( 
Bootstrap.class.getClassLoader(), juh );
+}
 m_loaded_juh = true;
 }
 return UnoRuntime.queryInterface(
@@ -318,3 +346,5 @@ public class Bootstrap {
 }.start();
 }
 }
+
+// vim:set shiftwidth=4 softtabstop=4 expandtab:
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: idl/source

2012-09-20 Thread Libreoffice Gerrit user
 idl/source/objects/types.cxx |3 +++
 1 file changed, 3 insertions(+)

New commits:
commit b869b73947659c45c1edc5f9cff80bfbe61e52ac
Author: Tor Lillqvist t...@iki.fi
Date:   Thu Sep 20 14:18:22 2012 +0300

Avoid duplicate symbols when DISABLE_DYNLOADING for Android

Change-Id: Iecd0ed802f6cc715ac77256bb4b49082b5976b4f

diff --git a/idl/source/objects/types.cxx b/idl/source/objects/types.cxx
index 9186857..e6cc629 100644
--- a/idl/source/objects/types.cxx
+++ b/idl/source/objects/types.cxx
@@ -1410,6 +1410,9 @@ void SvMetaType::WriteSfxItem(
 
 // write the implementation part
 rOutStm  #ifdef SFX_TYPEMAP  endl
+ #if defined(DISABLE_DYNLOADING)  defined(ANDROID)  endl
+ __attribute__((__weak__))  endl
+ #endif  endl
  aTypeName.getStr()  aVarName.getStr()
   =   endl;
 rOutStm  '{'  endl
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: editeng/source

2012-09-20 Thread Libreoffice Gerrit user
 editeng/source/editeng/editview.cxx |2 +-
 editeng/source/editeng/impedit2.cxx |2 +-
 editeng/source/editeng/impedit3.cxx |4 ++--
 3 files changed, 4 insertions(+), 4 deletions(-)

New commits:
commit 45bae63f0f56060e2a609ebb2610502a3722dfee
Author: Tor Lillqvist t...@iki.fi
Date:   Thu Sep 20 14:19:46 2012 +0300

Add some static goodness here, too, to avoid duplicate symbols

Change-Id: Ie5e7aecd4f1c0e5b4dda9250ae755bf9210e048f

diff --git a/editeng/source/editeng/editview.cxx 
b/editeng/source/editeng/editview.cxx
index bd13494..9cb8900 100644
--- a/editeng/source/editeng/editview.cxx
+++ b/editeng/source/editeng/editview.cxx
@@ -85,7 +85,7 @@ DBG_NAME( EditView )
 
 
 // From SW = Create common method
-LanguageType lcl_CheckLanguage(
+static LanguageType lcl_CheckLanguage(
 const OUString rText,
 Reference XSpellChecker1  xSpell,
 Reference linguistic2::XLanguageGuessing  xLangGuess,
diff --git a/editeng/source/editeng/impedit2.cxx 
b/editeng/source/editeng/impedit2.cxx
index a85b163..3c9990e 100644
--- a/editeng/source/editeng/impedit2.cxx
+++ b/editeng/source/editeng/impedit2.cxx
@@ -1675,7 +1675,7 @@ sal_Bool ImpEditEngine::IsInputSequenceCheckingRequired( 
sal_Unicode nChar, cons
 return bIsSequenceChecking;
 }
 
- bool lcl_HasStrongLTR ( const String rTxt, xub_StrLen nStart, xub_StrLen 
nEnd )
+static  bool lcl_HasStrongLTR ( const String rTxt, xub_StrLen nStart, 
xub_StrLen nEnd )
  {
  for ( xub_StrLen nCharIdx = nStart; nCharIdx  nEnd; ++nCharIdx )
  {
diff --git a/editeng/source/editeng/impedit3.cxx 
b/editeng/source/editeng/impedit3.cxx
index 027302f..8e20ae0 100644
--- a/editeng/source/editeng/impedit3.cxx
+++ b/editeng/source/editeng/impedit3.cxx
@@ -254,7 +254,7 @@ Point lcl_ImplCalcRotatedPos( Point rPos, Point rOrigin, 
double nSin, double nCo
 return aTranslatedPos;
 }
 
-sal_Bool lcl_IsLigature( xub_Unicode cCh, xub_Unicode cNextCh ) // For 
Kashidas from sw/source/core/text/porlay.txt
+static sal_Bool lcl_IsLigature( xub_Unicode cCh, xub_Unicode cNextCh ) // For 
Kashidas from sw/source/core/text/porlay.txt
 {
 // Lam + Alef
 return ( 0x644 == cCh  0x627 == cNextCh ) ||
@@ -262,7 +262,7 @@ sal_Bool lcl_IsLigature( xub_Unicode cCh, xub_Unicode 
cNextCh ) // For Kashidas
( 0x628 == cCh  0x631 == cNextCh );
 }
 
-sal_Bool lcl_ConnectToPrev( xub_Unicode cCh, xub_Unicode cPrevCh )  // For 
Kashidas from sw/source/core/text/porlay.txt
+static sal_Bool lcl_ConnectToPrev( xub_Unicode cCh, xub_Unicode cPrevCh )  // 
For Kashidas from sw/source/core/text/porlay.txt
 {
 // Alef, Dal, Thal, Reh, Zain, and Waw do not connect to the left
 sal_Bool bRet = 0x627 != cPrevCh  0x62F != cPrevCh  0x630 != cPrevCh 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: Options included in CC/CXX vs. in gb_CFLAGS/gb_CXXFLAGS

2012-09-20 Thread Michael Stahl
On 20/09/12 09:08, Stephan Bergmann wrote:
 On 09/20/2012 08:48 AM, Tor Lillqvist wrote:
 I think there is some fuzziness about what compiler options should be
 part of the CC/CXX variables and what should be in
 gb_CFLAGS/gb_CXXFLAGS.

 In some cases, like my Mac build tree that uses the 10.4 SDK, from
 Xcode 3 installed in /Xcode3, gb_CXXFLAGS contains at least one flag
 that apparently *must* be used for also external projects (well,
 liborcus in particular) to compile successfully.  This is the
 -isysroot from solenv/gbuild/platform/macosx.mk.
 [...]

 In other cases (normal Linuxes?) gb_CXXFLAGS contains just optional
 flags that will in fact hurt an external project that isn't prepared
 for them, like -Werror.

agree, there is some distinction to be made between mandatory flags
that must be used for everything like sysroot and flags that are just
intended for LO code.

 Should the -isysroot flag actually be part ot CC/CXX, not gb_CXXFLAGS?
 Should configure.in and macosx.mk be modified thusly?
 
 I think there has traditionally not been much consistency in this area 
 in OOo.  Originally, I think this was exclusively handled via flags 
 variables (and re-crafting similar such variables for passing to 
 external projects, with values more or less similar to the primary OOo 
 variables).  Much cruft here also stems from the fact that Sun's 
 internal OOo build environment required lots of overrides (as builds on 
 arbitrary machines used a centralized toolchain instead of a local one), 
 and this was typically achieved with ad-hoc fixes instead of a truly 
 principled approach.
 
 That said, your rule of when to put something into CC/CXX vs. 
 gb_CXXFLAGS etc. sounds good to me.

i didn't quite get what tml's rule is exactly :)

but anyway there are basically 2 options for getting mandatory flags
like -isysroot used everywhere: either include them in CC/CXX, or manage
some additional gb_MANDATORY_CFLAGS or whatever that are passed as
CFLAGS=$(gb_MANDATORY_CFLAGS) $(CFLAGS) to all invoked external
configure etc. scripts or makefiles.  i think the CC/CXX apporach is
harder to get wrong.

btw, why doesn't macosx.mk use --sysroot, which is much better than
-isysroot because it mangles the library search path as well?  is the
gcc too old for it?

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


Re: Options included in CC/CXX vs. in gb_CFLAGS/gb_CXXFLAGS

2012-09-20 Thread Tor Lillqvist
 btw, why doesn't macosx.mk use --sysroot, which is much better than
 -isysroot because it mangles the library search path as well?  is the
 gcc too old for it?

man gcc:

   -isysroot dir
   This option is like the --sysroot option, but applies only
to header files, except for Apple's version of GCC, where it applies
to both header files and libraries and effectively replaces the
--sysroot option.  See the --sysroot option for more information.

man clang doesn't mention either option, but presumably they have it
meaning the same as in gcc.

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


[PATCH] docx import: position table using tblInd OR tblCellMar

2012-09-20 Thread Pierre-Eric Pelloux-Prayer (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/655

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/55/655/1

docx import: position table using tblInd OR tblCellMar

Previously cell-margin was unconditionnaly used to compute table position.
Office only does so on top-level tables and if tblInd is missing.

Change-Id: I183647bcf090d5822b3f2e04e097c8bbd277168d
---
M writerfilter/inc/resourcemodel/TableManager.hxx
M writerfilter/source/dmapper/DomainMapperTableHandler.cxx
M writerfilter/source/dmapper/DomainMapperTableHandler.hxx
M writerfilter/source/resourcemodel/resourcemodel.cxx
4 files changed, 27 insertions(+), 7 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I183647bcf090d5822b3f2e04e097c8bbd277168d
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Pierre-Eric Pelloux-Prayer pierre-e...@lanedo.com

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


[PATCH] docx export: if table has no width use w:tblW 'auto' type

2012-09-20 Thread Pierre-Eric Pelloux-Prayer (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/657

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/57/657/1

docx export: if table has no width use w:tblW 'auto' type

Change-Id: I085febf04205181e7c4488bc948f346f92070fe2
---
M sw/source/filter/ww8/docxattributeoutput.cxx
1 file changed, 2 insertions(+), 3 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I085febf04205181e7c4488bc948f346f92070fe2
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Pierre-Eric Pelloux-Prayer pierre-e...@lanedo.com

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


Re: Options included in CC/CXX vs. in gb_CFLAGS/gb_CXXFLAGS

2012-09-20 Thread Christian Lohmaier
Hi Tor, *,

rantIt is pointless to write to the mailinglist for feedback, if you
don't even wait two ours before taking the action you ask to be OK or
not. If you want to do it, just do it, but such alibi posts don't
really make much sense then./rant

On Thu, Sep 20, 2012 at 10:29 AM, Tor Lillqvist t...@iki.fi wrote:
 Should the -isysroot flag actually be part ot CC/CXX, not gb_CXXFLAGS?
 Should configure.in and macosx.mk be modified thusly?

 That's what I did, and reverted my other attempts from this morning to
 get liborcus to build,

external modules traditionally got EXTRA_CDEFS as their CPPFLAGS and
EXTRA_CFLAGS in their CFLAGS.

 and it seems that it was a good move, the
 tinderboxes are getting greener now.

And now the build will break for those who use CC=/path/to/ccache
gcc, won't it?
I don't care personally, as I have it setup using path  symlinks.
(that configure happily doubles to call it with an additional ccache -
doppelt hält besser.)

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


Re: Options included in CC/CXX vs. in gb_CFLAGS/gb_CXXFLAGS

2012-09-20 Thread Tor Lillqvist
 And now the build will break for those who use CC=/path/to/ccache
 gcc, won't it?

Probably, yes. But that might well have happened earlier, too. Living
on the master branch requires some manual adaption now and then. The
alternative would be to make no progress.

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


Re: Failed to compile accessibility module due to gb_Deliver_deliver file is missing.

2012-09-20 Thread Noel Grandin
It looks like you're trying to build individual pieces without doing a 
complete build first.


Before you try hacking on individual modules, you have to first have a 
complete build i.e.


./autogen.sh
make

and let it run to completion. May take several hours.

On 2012-09-20 14:54, LOH KOK HOE wrote:

Hi noel,
When I issue the command

make rsc


I got this error:

make -r -f /home/kokhoe/workspace/LibreOffice/core/Makefile.top rsc
make[1]: Entering directory `/home/kokhoe/workspace/LibreOffice/core'
cd rsc  make -j 2 -rs gb_PARTIALBUILD=T
make[2]: Entering directory `/home/kokhoe/workspace/LibreOffice/core/rsc'
[build DEP] LNK:Executable/rsc
make[2]: Leaving directory `/home/kokhoe/workspace/LibreOffice/core/rsc'
make[2]: Entering directory `/home/kokhoe/workspace/LibreOffice/core/rsc'
[info  ALL] LinkTarget Library/libi18nisolang1gcc3.so not defined: 
Assuming headers to be there!
[info  ALL] LinkTarget Library/libtllo.so not defined: Assuming 
headers to be there!

[build YAC] rsc/source/parser/rscyacc
/home/kokhoe/workspace/LibreOffice/core/solenv/gbuild/Library.mk:54: 
*** gb_Deliver_deliver: file does not exist in solver, and cannot be 
delivered: 
/home/kokhoe/workspace/LibreOffice/core/solver/unxlngx6.pro/lib/libi18nisolang1gcc3.so 
http://unxlngx6.pro/lib/libi18nisolang1gcc3.so. Stop.

make[2]: *** Waiting for unfinished jobs
/home/kokhoe/workspace/LibreOffice/core/rsc/source/parser/rscyacc.y: 
conflicts: 2 shift/reduce

make[2]: Leaving directory `/home/kokhoe/workspace/LibreOffice/core/rsc'
make[1]: *** [rsc] Error 2
make[1]: Leaving directory `/home/kokhoe/workspace/LibreOffice/core'
make: *** [rsc] Error 2

It seems that I was missing the libi18nisolang1gcc3.so file. Any advice?

THanks @!


On Wed, Sep 19, 2012 at 10:04 PM, Noel Grandin n...@peralex.com 
mailto:n...@peralex.com wrote:


Looks like the RSC module build failed.

Try building RSC by itself first (make rsc)


On 2012-09-19 16:01, LOH KOK HOE wrote:

I am trying to compile accessibility module by using following
command

make accessibility


Unfortunately the compilation was fail due to the file named
gb_Deliver_deliver was missing. May I know where can I get the
missing file so that I can complete the compilation.

Here is the full stack trace:


*
*   Running the post download checks.
*

checking build system type... x86_64-unknown-linux-gnu
checking host system type... x86_64-unknown-linux-gnu
cd accessibility  make -j 2 -rs gb_PARTIALBUILD=T
make[2]: Entering directory
`/home/kokhoe/workspace/LibreOffice/core/accessibility'
/home/kokhoe/workspace/LibreOffice/core/solenv/gbuild/Package.mk:44:
*** gb_Deliver_deliver: file does not exist in solver, and cannot
be delivered:
/home/kokhoe/workspace/LibreOffice/core/solver/unxlngx6.pro/bin/rsc
http://unxlngx6.pro/bin/rsc. Stop.
make[2]: Leaving directory
`/home/kokhoe/workspace/LibreOffice/core/accessibility'
make[1]: *** [accessibility] Error 2
make[1]: Leaving directory `/home/kokhoe/workspace/LibreOffice/core'
make: *** [accessibility] Error 2

THanks @!


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






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





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


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


Re: [PATCH] i18npool Add locale data for ebo_CG and tyx_CG

2012-09-20 Thread Jeremy Brown
I hereby declare that:

All my contributions, past and future, to LibreOffice are licensed
under the terms of the MPL / LGPLv3+.

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


Re: Failed to compile accessibility module due to gb_Deliver_deliver file is missing.

2012-09-20 Thread LOH KOK HOE
Sorry noel,
I just want to know the options available for ./autogen.sh, and I had
accidentally type ./autogen.sh help. Thus the next time I issuing the
command ./autogen.sh I have following error. Is there a way to clean it?

*console output*

running ./configure with 'help'
configure: WARNING: you should use --build, --host, --target

*
*   Running LibreOffice build configuration.
*


checking build system type... Invalid configuration `help': machine `help'
not recognized
configure: error: /bin/sh ./config.sub help failed
Error running configure at ./autogen.sh line 187.


THanks @!


On Thu, Sep 20, 2012 at 9:06 PM, Noel Grandin n...@peralex.com wrote:

  It looks like you're trying to build individual pieces without doing a
 complete build first.

 Before you try hacking on individual modules, you have to first have a
 complete build i.e.

 ./autogen.sh
 make

 and let it run to completion. May take several hours.


 On 2012-09-20 14:54, LOH KOK HOE wrote:

 Hi noel,
 When I issue the command

 make rsc


 I got this error:

 make -r -f /home/kokhoe/workspace/LibreOffice/core/Makefile.top rsc
 make[1]: Entering directory `/home/kokhoe/workspace/LibreOffice/core'
 cd rsc  make -j 2 -rs gb_PARTIALBUILD=T
 make[2]: Entering directory `/home/kokhoe/workspace/LibreOffice/core/rsc'
 [build DEP] LNK:Executable/rsc
 make[2]: Leaving directory `/home/kokhoe/workspace/LibreOffice/core/rsc'
 make[2]: Entering directory `/home/kokhoe/workspace/LibreOffice/core/rsc'
 [info  ALL] LinkTarget Library/libi18nisolang1gcc3.so not defined:
 Assuming headers to be there!
 [info  ALL] LinkTarget Library/libtllo.so not defined: Assuming headers to
 be there!
 [build YAC] rsc/source/parser/rscyacc
 /home/kokhoe/workspace/LibreOffice/core/solenv/gbuild/Library.mk:54: ***
 gb_Deliver_deliver: file does not exist in solver, and cannot be delivered:
 /home/kokhoe/workspace/LibreOffice/core/solver/
 unxlngx6.pro/lib/libi18nisolang1gcc3.so.  Stop.
 make[2]: *** Waiting for unfinished jobs
 /home/kokhoe/workspace/LibreOffice/core/rsc/source/parser/rscyacc.y:
 conflicts: 2 shift/reduce
 make[2]: Leaving directory `/home/kokhoe/workspace/LibreOffice/core/rsc'
 make[1]: *** [rsc] Error 2
 make[1]: Leaving directory `/home/kokhoe/workspace/LibreOffice/core'
 make: *** [rsc] Error 2

 It seems that I was missing the libi18nisolang1gcc3.so file. Any advice?

 THanks @!


 On Wed, Sep 19, 2012 at 10:04 PM, Noel Grandin n...@peralex.com wrote:

  Looks like the RSC module build failed.

 Try building RSC by itself first (make rsc)


 On 2012-09-19 16:01, LOH KOK HOE wrote:

  I am trying to compile accessibility module by using following command

 make accessibility


 Unfortunately the compilation was fail due to the file named
 gb_Deliver_deliver was missing. May I know where can I get the missing file
 so that I can complete the compilation.

 Here is the full stack trace:

 
 *
 *   Running the post download checks.
 *
 
 checking build system type... x86_64-unknown-linux-gnu
 checking host system type... x86_64-unknown-linux-gnu
 cd accessibility  make -j 2 -rs gb_PARTIALBUILD=T
 make[2]: Entering directory
 `/home/kokhoe/workspace/LibreOffice/core/accessibility'
 /home/kokhoe/workspace/LibreOffice/core/solenv/gbuild/Package.mk:44: ***
 gb_Deliver_deliver: file does not exist in solver, and cannot be delivered:
 /home/kokhoe/workspace/LibreOffice/core/solver/unxlngx6.pro/bin/rsc.
 Stop.
 make[2]: Leaving directory
 `/home/kokhoe/workspace/LibreOffice/core/accessibility'
 make[1]: *** [accessibility] Error 2
 make[1]: Leaving directory `/home/kokhoe/workspace/LibreOffice/core'
 make: *** [accessibility] Error 2

 THanks @!


  ___
 LibreOffice mailing 
 listLibreOffice@lists.freedesktop.orghttp://lists.freedesktop.org/mailman/listinfo/libreoffice





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






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


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


[PATCH] neon: convert to gbuild

2012-09-20 Thread via Code Review
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/659

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/59/659/1

neon: convert to gbuild

Change-Id: I1cca0e3be7e148c835af26d2c001878b849e8432
---
M Makefile.top
M Module_tail_build.mk
M RepositoryExternal.mk
M RepositoryModule_ooo.mk
M config_host.mk.in
M configure.in
A neon/Library_neon.mk
A neon/Makefile
A neon/Module_neon.mk
A neon/UnpackedTarball_neon.mk
A neon/configs/config.h
D neon/makefile.mk
M neon/neon.patch
D neon/neon_exports_unix.patch
D neon/neon_exports_win.patch
M neon/neon_ne_set_request_flag.patch
M neon/prj/d.lst
D neon/prj/dmake
D neon/version.mk
M ooo.lst.in
M solenv/inc/libs.mk
M tail_build/prj/build.lst
M ucb/prj/build.lst
M xmlsecurity/prj/build.lst
24 files changed, 647 insertions(+), 1,367 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1cca0e3be7e148c835af26d2c001878b849e8432
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Matúš Kukan matus.ku...@gmail.com

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


[PATCH] allow Library to disable hidden visibility

2012-09-20 Thread via Code Review
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/658

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/58/658/1

allow Library to disable hidden visibility

Change-Id: I1f676b7536183f68795595f216ae23610cadb1eb
---
M solenv/gbuild/Library.mk
M solenv/gbuild/LinkTarget.mk
M solenv/gbuild/platform/com_GCC_class.mk
M solenv/gbuild/platform/com_GCC_defs.mk
M solenv/gbuild/platform/macosx.mk
M solenv/gbuild/platform/solaris.mk
M solenv/gbuild/platform/unxgcc.mk
7 files changed, 22 insertions(+), 54 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1f676b7536183f68795595f216ae23610cadb1eb
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Matúš Kukan matus.ku...@gmail.com

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


Re: Failed to compile accessibility module due to gb_Deliver_deliver file is missing.

2012-09-20 Thread Noel Grandin


Maybe you should try reading the documentation?

http://wiki.documentfoundation.org/Development/Native_Build


On 2012-09-20 15:46, LOH KOK HOE wrote:

Sorry noel,
I just want to know the options available for ./autogen.sh, and I had 
accidentally type ./autogen.sh help. Thus the next time I issuing the 
command ./autogen.sh I have following error. Is there a way to clean it?


*_console output_*

running ./configure with 'help'
configure: WARNING: you should use --build, --host, --target

*
*   Running LibreOffice build configuration.
*


checking build system type... Invalid configuration `help': machine 
`help' not recognized

configure: error: /bin/sh ./config.sub help failed
Error running configure at ./autogen.sh line 187.


THanks @!


On Thu, Sep 20, 2012 at 9:06 PM, Noel Grandin n...@peralex.com 
mailto:n...@peralex.com wrote:


It looks like you're trying to build individual pieces without
doing a complete build first.

Before you try hacking on individual modules, you have to first
have a complete build i.e.

./autogen.sh
make

and let it run to completion. May take several hours.


On 2012-09-20 14:54, LOH KOK HOE wrote:

Hi noel,
When I issue the command

make rsc


I got this error:

make -r -f /home/kokhoe/workspace/LibreOffice/core/Makefile.top rsc
make[1]: Entering directory `/home/kokhoe/workspace/LibreOffice/core'
cd rsc  make -j 2 -rs gb_PARTIALBUILD=T
make[2]: Entering directory
`/home/kokhoe/workspace/LibreOffice/core/rsc'
[build DEP] LNK:Executable/rsc
make[2]: Leaving directory
`/home/kokhoe/workspace/LibreOffice/core/rsc'
make[2]: Entering directory
`/home/kokhoe/workspace/LibreOffice/core/rsc'
[info  ALL] LinkTarget Library/libi18nisolang1gcc3.so not
defined: Assuming headers to be there!
[info  ALL] LinkTarget Library/libtllo.so not defined: Assuming
headers to be there!
[build YAC] rsc/source/parser/rscyacc
/home/kokhoe/workspace/LibreOffice/core/solenv/gbuild/Library.mk:54:
*** gb_Deliver_deliver: file does not exist in solver, and cannot
be delivered:

/home/kokhoe/workspace/LibreOffice/core/solver/unxlngx6.pro/lib/libi18nisolang1gcc3.so
http://unxlngx6.pro/lib/libi18nisolang1gcc3.so. Stop.
make[2]: *** Waiting for unfinished jobs
/home/kokhoe/workspace/LibreOffice/core/rsc/source/parser/rscyacc.y:
conflicts: 2 shift/reduce
make[2]: Leaving directory
`/home/kokhoe/workspace/LibreOffice/core/rsc'
make[1]: *** [rsc] Error 2
make[1]: Leaving directory `/home/kokhoe/workspace/LibreOffice/core'
make: *** [rsc] Error 2

It seems that I was missing the libi18nisolang1gcc3.so file. Any
advice?

THanks @!


On Wed, Sep 19, 2012 at 10:04 PM, Noel Grandin n...@peralex.com
mailto:n...@peralex.com wrote:

Looks like the RSC module build failed.

Try building RSC by itself first (make rsc)


On 2012-09-19 16:01, LOH KOK HOE wrote:

I am trying to compile accessibility module by using
following command

make accessibility


Unfortunately the compilation was fail due to the file named
gb_Deliver_deliver was missing. May I know where can I get
the missing file so that I can complete the compilation.

Here is the full stack trace:


*
*   Running the post download checks.
*

checking build system type... x86_64-unknown-linux-gnu
checking host system type... x86_64-unknown-linux-gnu
cd accessibility  make -j 2 -rs gb_PARTIALBUILD=T
make[2]: Entering directory
`/home/kokhoe/workspace/LibreOffice/core/accessibility'
/home/kokhoe/workspace/LibreOffice/core/solenv/gbuild/Package.mk:44:
*** gb_Deliver_deliver: file does not exist in solver, and
cannot be delivered:
/home/kokhoe/workspace/LibreOffice/core/solver/unxlngx6.pro/bin/rsc
http://unxlngx6.pro/bin/rsc. Stop.
make[2]: Leaving directory
`/home/kokhoe/workspace/LibreOffice/core/accessibility'
make[1]: *** [accessibility] Error 2
make[1]: Leaving directory
`/home/kokhoe/workspace/LibreOffice/core'
make: *** [accessibility] Error 2

THanks @!


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






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








Re: How to use internal neon ?

2012-09-20 Thread Matúš Kukan
On 10 September 2012 21:53, Caolán McNamara caol...@redhat.com wrote:
 On Mon, 2012-09-10 at 18:55 +0200, Matúš Kukan wrote:
 Hi,

 I am pondering gbuildizing neon.
 It's built by dmake as dynamic library currently.
 Could it be used as static library ?

 It used to be a static library but got converted to a shared library
 with 386b66d1a91f081b7f5f7673bfe640fa0d6b602c for
 https://issues.apache.org/ooo/show_bug.cgi?id=105418

I see, so it will be a little complicated with exporting symbols.
Currently, there is even .map file used. (probably, I don't know how
dmake works)
I've removed .map files, allowed to turn of -fvisibility=hidden in
https://gerrit.libreoffice.org/#/c/658/
and then tried to convert neon in
https://gerrit.libreoffice.org/#/c/659/
Seems to work for me on Linux.
Is this approach right ?

If anybody could try make ucb.all with MinGW and MacOSX, to see if it
links, that would be nice.
( with --enable-neon of course )

 And why there is --disable-neon option if we can build internal copy ?

 This comes from https://issues.apache.org/ooo/show_bug.cgi?id=61380 as
 a5a236bc116ba38a6668f38bbb924ad333b7fdab

Good to know.

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


[Libreoffice-commits] .: solenv/gbuild

2012-09-20 Thread Libreoffice Gerrit user
 solenv/gbuild/platform/unxgcc.mk |3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

New commits:
commit 4ba760078019fba15f9d0d33a6094960026b9dfb
Author: Stephan Bergmann sberg...@redhat.com
Date:   Thu Sep 20 15:54:23 2012 +0200

Make make debugrun less fancy but more useful

For one, --tui has negative impact on terminal behavior in certain ssh 
scenarios
for me.  Whether and how to use it should be a user's personal choice 
anyway.

For another, you sometimes need to interact with gdb early, so do not
automatically start to run soffice.bin.

Change-Id: I1a1a02f5359aae2a11effa6620aebf01202357f9

diff --git a/solenv/gbuild/platform/unxgcc.mk b/solenv/gbuild/platform/unxgcc.mk
index e6ed9e3..080a3ad 100644
--- a/solenv/gbuild/platform/unxgcc.mk
+++ b/solenv/gbuild/platform/unxgcc.mk
@@ -390,9 +390,8 @@ endef
 define gb_Module_DEBUGRUNCOMMAND
 OFFICESCRIPT=`mktemp`  \
 printf 'if [ -e $(DEVINSTALLDIR)/opt/program/ooenv ]; then . 
$(DEVINSTALLDIR)/opt/program/ooenv; fi\n'  $${OFFICESCRIPT}  \
-printf gdb --tui $(DEVINSTALLDIR)/opt/program/soffice.bin  
$${OFFICESCRIPT}  \
+printf gdb $(DEVINSTALLDIR)/opt/program/soffice.bin  $${OFFICESCRIPT}  \
 printf  -ex \set args --norestore --nologo '--accept=pipe,name=$(USER);urp;' 
-env:UserInstallation=file://$(DEVINSTALLDIR)/\  $${OFFICESCRIPT}  \
-printf  -ex \r\\\n  $${OFFICESCRIPT}  \
 $(SHELL) $${OFFICESCRIPT}  \
 rm $${OFFICESCRIPT}
 endef
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: Failed to compile accessibility module due to gb_Deliver_deliver file is missing.

2012-09-20 Thread LOH KOK HOE
I have already figure it out how to resolve my problem by re-issuing the
./autogen.sh command with new options. BTW, I have read that document
before I start compiling, just that my typing skills is very bad. :p
Anyway, thank you so much on your help.


THanks @!


On Thu, Sep 20, 2012 at 9:58 PM, Noel Grandin n...@peralex.com wrote:


 Maybe you should try reading the documentation?

 http://wiki.documentfoundation.org/Development/Native_Build



 On 2012-09-20 15:46, LOH KOK HOE wrote:

 Sorry noel,
 I just want to know the options available for ./autogen.sh, and I had
 accidentally type ./autogen.sh help. Thus the next time I issuing the
 command ./autogen.sh I have following error. Is there a way to clean it?

 *console output*

 running ./configure with 'help'
 configure: WARNING: you should use --build, --host, --target
 
 *
 *   Running LibreOffice build configuration.
 *
 

 checking build system type... Invalid configuration `help': machine `help'
 not recognized
 configure: error: /bin/sh ./config.sub help failed
 Error running configure at ./autogen.sh line 187.


 THanks @!


 On Thu, Sep 20, 2012 at 9:06 PM, Noel Grandin n...@peralex.com wrote:

  It looks like you're trying to build individual pieces without doing a
 complete build first.

 Before you try hacking on individual modules, you have to first have a
 complete build i.e.

 ./autogen.sh
 make

 and let it run to completion. May take several hours.


 On 2012-09-20 14:54, LOH KOK HOE wrote:

 Hi noel,
 When I issue the command

 make rsc


 I got this error:

 make -r -f /home/kokhoe/workspace/LibreOffice/core/Makefile.top rsc
 make[1]: Entering directory `/home/kokhoe/workspace/LibreOffice/core'
 cd rsc  make -j 2 -rs gb_PARTIALBUILD=T
 make[2]: Entering directory `/home/kokhoe/workspace/LibreOffice/core/rsc'
 [build DEP] LNK:Executable/rsc
 make[2]: Leaving directory `/home/kokhoe/workspace/LibreOffice/core/rsc'
 make[2]: Entering directory `/home/kokhoe/workspace/LibreOffice/core/rsc'
 [info  ALL] LinkTarget Library/libi18nisolang1gcc3.so not defined:
 Assuming headers to be there!
 [info  ALL] LinkTarget Library/libtllo.so not defined: Assuming headers
 to be there!
 [build YAC] rsc/source/parser/rscyacc
 /home/kokhoe/workspace/LibreOffice/core/solenv/gbuild/Library.mk:54: ***
 gb_Deliver_deliver: file does not exist in solver, and cannot be delivered:
 /home/kokhoe/workspace/LibreOffice/core/solver/
 unxlngx6.pro/lib/libi18nisolang1gcc3.so.  Stop.
 make[2]: *** Waiting for unfinished jobs
 /home/kokhoe/workspace/LibreOffice/core/rsc/source/parser/rscyacc.y:
 conflicts: 2 shift/reduce
 make[2]: Leaving directory `/home/kokhoe/workspace/LibreOffice/core/rsc'
 make[1]: *** [rsc] Error 2
 make[1]: Leaving directory `/home/kokhoe/workspace/LibreOffice/core'
 make: *** [rsc] Error 2

 It seems that I was missing the libi18nisolang1gcc3.so file. Any advice?

 THanks @!


 On Wed, Sep 19, 2012 at 10:04 PM, Noel Grandin n...@peralex.com wrote:

  Looks like the RSC module build failed.

 Try building RSC by itself first (make rsc)


 On 2012-09-19 16:01, LOH KOK HOE wrote:

  I am trying to compile accessibility module by using following command

 make accessibility


 Unfortunately the compilation was fail due to the file named
 gb_Deliver_deliver was missing. May I know where can I get the missing file
 so that I can complete the compilation.

 Here is the full stack trace:

 
 *
 *   Running the post download checks.
 *
 
 checking build system type... x86_64-unknown-linux-gnu
 checking host system type... x86_64-unknown-linux-gnu
 cd accessibility  make -j 2 -rs gb_PARTIALBUILD=T
 make[2]: Entering directory
 `/home/kokhoe/workspace/LibreOffice/core/accessibility'
 /home/kokhoe/workspace/LibreOffice/core/solenv/gbuild/Package.mk:44: ***
 gb_Deliver_deliver: file does not exist in solver, and cannot be delivered:
 /home/kokhoe/workspace/LibreOffice/core/solver/unxlngx6.pro/bin/rsc.
 Stop.
 make[2]: Leaving directory
 `/home/kokhoe/workspace/LibreOffice/core/accessibility'
 make[1]: *** [accessibility] Error 2
 make[1]: Leaving directory `/home/kokhoe/workspace/LibreOffice/core'
 make: *** [accessibility] Error 2

 THanks @!


  ___
 LibreOffice mailing 
 listLibreOffice@lists.freedesktop.orghttp://lists.freedesktop.org/mailman/listinfo/libreoffice





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






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






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


___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org

Build error in offapi module

2012-09-20 Thread LOH KOK HOE
I was trying to make a clean build using this command ./autogen.sh and have
encountering a build error in offapi module. I am running out of clue on
how this problem could be solve. Below are the details of the stack trace,
hope you guys can give some pointer.

*Trace extract from build_error.log*
===

log for /home/kokhoe/workspace/LibreOffice/core/offapi/prj
make[2]: *** No rule to make target
`/home/kokhoe/workspace/LibreOffice/core/offapi/com/sun/star/auth/SSOExceptions.idl',
needed by `/home/kokhoe/workspace/LibreOffice/core/workdir/
unxlngx6.pro/UnoApiPartTarget/offapi/com/sun/star/auth/SSOManagerFactory.urd'.
Stop.


*Trace from Terminal output*

=
(60/77) Building module jurt
=
Entering /home/kokhoe/workspace/LibreOffice/core/jurt/prj

gbuild module /home/kokhoe/workspace/LibreOffice/core/jurt: make -f
Makefile -j1 -rs all slowcheck gb_PARTIALBUILD=T
[javac] Note: Some input files use or override a deprecated API.
[javac] Note: Recompile with -Xlint:deprecation for details.
[javac] Note: Some input files use unchecked or unsafe operations.
[javac] Note: Recompile with -Xlint:unchecked for details.
 [copy] Copying 29 files to
/home/kokhoe/workspace/LibreOffice/core/jfreereport/
unxlngx6.pro/misc/build/flow-engine-0.9.4/build/classes

jar:
 [copy] Copying 1 file to
/home/kokhoe/workspace/LibreOffice/core/jfreereport/
unxlngx6.pro/misc/build/flow-engine-0.9.4/build
  [jar] Building jar:
/home/kokhoe/workspace/LibreOffice/core/jfreereport/
unxlngx6.pro/misc/build/flow-engine-0.9.4/build/lib/flow-engine.jar
[build JCS] Jar/jurt

BUILD SUCCESSFUL
Total time: 2 seconds
cp -f ../../
unxlngx6.pro/misc/build/flow-engine-0.9.4/build/lib/flow-engine.jar ../../
unxlngx6.pro/class/flow-engine-0.9.4.jar
jfreereport deliver
Module 'jfreereport' delivered successfully. 13 files copied, 0 files
unchanged
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
[build JAR] jurt
[build C  ]
jurt/source/pipe/com_sun_star_lib_connections_pipe_PipeConnection.c
[build LNK] Library/libjpipe.so
[build ZIP] jurt_src
[build MOD] jurt
[build ALL] top level modules: jurt
[build ALL] loaded modules: jurt

[build CHK] loaded modules: jurt

[build SLC] jurt
[build SLC] loaded modules: jurt
jurt deliver


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


[Libreoffice-commits] .: oox/source

2012-09-20 Thread Libreoffice Gerrit user
 oox/source/vml/vmlshape.cxx |7 ++-
 1 file changed, 6 insertions(+), 1 deletion(-)

New commits:
commit b5a1db961372b3fa66724ee580726c67e7683dbc
Author: Miklos Vajna vmik...@suse.cz
Date:   Thu Sep 20 16:13:51 2012 +0200

n#779834 fix VML import of absolutely positioned image with no left margin

Given that we talk about mm100, changing 0 to 1 is not that harmful, and
it seems later core would crash without at least a small margin.

Change-Id: I913e4a5eeaba2e36b55f2d7d7fb5bdfbad8eae94

diff --git a/oox/source/vml/vmlshape.cxx b/oox/source/vml/vmlshape.cxx
index 0bd3399..fecade9 100644
--- a/oox/source/vml/vmlshape.cxx
+++ b/oox/source/vml/vmlshape.cxx
@@ -172,8 +172,13 @@ Rectangle ShapeType::getAbsRectangle() const
 if ( nHeight == 0 )
 nHeight = 1;
 
+sal_Int32 nLeft = ConversionHelper::decodeMeasureToHmm( rGraphicHelper, 
maTypeModel.maLeft, 0, true, true )
++ ConversionHelper::decodeMeasureToHmm( rGraphicHelper, 
maTypeModel.maMarginLeft, 0, true, true );
+if (nLeft == 0  maTypeModel.maPosition == absolute)
+nLeft = 1;
+
 return Rectangle(
-ConversionHelper::decodeMeasureToHmm( rGraphicHelper, 
maTypeModel.maLeft, 0, true, true ) + ConversionHelper::decodeMeasureToHmm( 
rGraphicHelper, maTypeModel.maMarginLeft, 0, true, true ),
+nLeft,
 ConversionHelper::decodeMeasureToHmm( rGraphicHelper, 
maTypeModel.maTop, 0, false, true ) + ConversionHelper::decodeMeasureToHmm( 
rGraphicHelper, maTypeModel.maMarginTop, 0, false, true ),
 nWidth, nHeight );
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PATCH] fdo#39468: Translated German comments in sw/source/core/text

2012-09-20 Thread via Code Review
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/660

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/60/660/1

fdo#39468: Translated German comments in sw/source/core/text

Change-Id: I53a415e1ec573d342e06d0cf10bad6f8cb95b478
---
M sw/source/core/text/txttab.cxx
M sw/source/core/text/widorp.cxx
2 files changed, 94 insertions(+), 100 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I53a415e1ec573d342e06d0cf10bad6f8cb95b478
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: gerhard öttl lo...@ogersoft.at

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


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

2012-09-20 Thread Libreoffice Gerrit user
 sc/inc/rangenam.hxx |3 +-
 sc/source/core/tool/rangenam.cxx|   36 +---
 sc/source/filter/oox/defnamesbuffer.cxx |   13 +--
 3 files changed, 37 insertions(+), 15 deletions(-)

New commits:
commit 7539a695b801b92d10ff30463a8c08672a258442
Author: Noel Power noel.po...@suse.com
Date:   Thu Sep 20 15:05:00 2012 +0100

xlsx fix cyclic referenced defined names partially fixes bnc#780296 
bnc#781166

Change-Id: Ibad37ac05c608c0211063bfa73931427d48c11f9

diff --git a/sc/inc/rangenam.hxx b/sc/inc/rangenam.hxx
index d370328..64955e1 100644
--- a/sc/inc/rangenam.hxx
+++ b/sc/inc/rangenam.hxx
@@ -87,7 +87,7 @@ private:
 SCCOL   mnMaxCol;
 
 void CompileRangeData( const String rSymbol, bool bSetError );
-
+void InitCode();
 public:
 typedef ::std::mapsal_uInt16, sal_uInt16 IndexMap;
 
@@ -120,6 +120,7 @@ public:
 voidSetIndex( sal_uInt16 nInd ) { nIndex = nInd; }
 sal_uInt16GetIndex() const{ return nIndex; }
 ScTokenArray*   GetCode()   { return pCode; }
+SC_DLLPUBLIC void   SetCode( ScTokenArray );
 const ScTokenArray* GetCode() const { return pCode; }
 SC_DLLPUBLIC sal_uInt16 GetErrCode() const;
 boolHasReferences() const;
diff --git a/sc/source/core/tool/rangenam.cxx b/sc/source/core/tool/rangenam.cxx
index a2f67a2..8a52a71 100644
--- a/sc/source/core/tool/rangenam.cxx
+++ b/sc/source/core/tool/rangenam.cxx
@@ -100,18 +100,7 @@ ScRangeData::ScRangeData( ScDocument* pDok,
 mnMaxRow(-1),
 mnMaxCol(-1)
 {
-if( !pCode-GetCodeError() )
-{
-pCode-Reset();
-FormulaToken* p = pCode-GetNextReference();
-if( p )   // exact one reference at first
-{
-if( p-GetType() == svSingleRef )
-eType = eType | RT_ABSPOS;
-else
-eType = eType | RT_ABSAREA;
-}
-}
+InitCode();
 }
 
 ScRangeData::ScRangeData( ScDocument* pDok,
@@ -648,6 +637,29 @@ void ScRangeData::ValidateTabRefs()
 }
 }
 
+void ScRangeData::SetCode( ScTokenArray rArr )
+{
+::std::auto_ptrScTokenArray pOldCode( pCode); // old pCode will be 
deleted
+pCode = new ScTokenArray( rArr );
+InitCode();
+}
+
+void ScRangeData::InitCode()
+{
+if( !pCode-GetCodeError() )
+{
+pCode-Reset();
+FormulaToken* p = pCode-GetNextReference();
+if( p )   // exact one reference at first
+{
+if( p-GetType() == svSingleRef )
+eType = eType | RT_ABSPOS;
+else
+eType = eType | RT_ABSAREA;
+}
+}
+}
+
 
 extern C
 int SAL_CALL ScRangeData_QsortNameCompare( const void* p1, const void* p2 )
diff --git a/sc/source/filter/oox/defnamesbuffer.cxx 
b/sc/source/filter/oox/defnamesbuffer.cxx
index b31b5fb..6a3a3aa 100644
--- a/sc/source/filter/oox/defnamesbuffer.cxx
+++ b/sc/source/filter/oox/defnamesbuffer.cxx
@@ -382,9 +382,9 @@ void DefinedName::createNameObject( sal_Int32 nIndex )
 
 // create the name and insert it into the document, maCalcName will be 
changed to the resulting name
 if (maModel.mnSheet = 0)
-mpScRangeData = createLocalNamedRangeObject( maCalcName, getTokens(), 
nIndex, nNameFlags, maModel.mnSheet );
+mpScRangeData = createLocalNamedRangeObject( maCalcName, 
ApiTokenSequence(), nIndex, nNameFlags, maModel.mnSheet );
 else
-mpScRangeData = createNamedRangeObject( maCalcName, getTokens(), 
nIndex, nNameFlags );
+mpScRangeData = createNamedRangeObject( maCalcName, 
ApiTokenSequence(), nIndex, nNameFlags );
 mnTokenIndex = nIndex;
 }
 
@@ -426,6 +426,15 @@ DefinedName::getTokens()
 
 void DefinedName::convertFormula()
 {
+// convert and set formula of the defined name
+if ( getFilterType() == FILTER_OOXML )
+{
+ApiTokenSequence aTokens = getTokens();
+ScTokenArray aTokenArray;
+(void)ScTokenConversion::ConvertToTokenArray( this-getScDocument(), 
aTokenArray, aTokens );
+mpScRangeData-SetCode( aTokenArray );
+}
+
 ScTokenArray* pTokenArray = mpScRangeData-GetCode();
 Sequence FormulaToken  aFTokenSeq;
 (void)ScTokenConversion::ConvertToTokenSequence( this-getScDocument(), 
aFTokenSeq, *pTokenArray );
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: Build error in offapi module

2012-09-20 Thread Stephan Bergmann

On 09/20/2012 04:18 PM, LOH KOK HOE wrote:

I was trying to make a clean build


Did you actually do make clean? 
offapi/com/sun/star/auth/SSOExceptions.idl was removed quite a while ago 
on master, so it looks to me like you still have some remains from on 
old build around.


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


Re: How to use internal neon ?

2012-09-20 Thread Stephan Bergmann

On 09/20/2012 04:00 PM, Matúš Kukan wrote:

I see, so it will be a little complicated with exporting symbols.
Currently, there is even .map file used. (probably, I don't know how
dmake works)
I've removed .map files, allowed to turn of -fvisibility=hidden in
https://gerrit.libreoffice.org/#/c/658/
and then tried to convert neon in
https://gerrit.libreoffice.org/#/c/659/
Seems to work for me on Linux.
Is this approach right ?


In case that dramatically increases the amount of symbols exported from 
neon, it might make sense to keep the mapfile, no?


What's the output of nm -D --defined solver/*/lib/libneon,so | wc -l 
for you with those changes?  For me the old one is 280.


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


[Libreoffice-commits] .: helpcontent2/source

2012-09-20 Thread Libreoffice Gerrit user
 helpcontent2/source/auxiliary/sbasic.tree   |   22 +++---
 helpcontent2/source/auxiliary/scalc.tree|   12 ++--
 helpcontent2/source/auxiliary/shared.tree   |   22 +++---
 helpcontent2/source/auxiliary/simpress.tree |2 +-
 helpcontent2/source/auxiliary/swriter.tree  |   16 
 5 files changed, 37 insertions(+), 37 deletions(-)

New commits:
commit c87e800a0da6ae192b74df3f05527ca3cd4588c3
Author: Andras Timar ati...@suse.com
Date:   Thu Sep 20 17:37:09 2012 +0200

update tree files from topic titles

Change-Id: Ia49783c59614c4f01978ba4eef671daf74aeee07

diff --git a/helpcontent2/source/auxiliary/sbasic.tree 
b/helpcontent2/source/auxiliary/sbasic.tree
index 8766b4a..70c5aa3 100644
--- a/helpcontent2/source/auxiliary/sbasic.tree
+++ b/helpcontent2/source/auxiliary/sbasic.tree
@@ -17,11 +17,11 @@
node id=0702 title=Command Reference
 topic id=sbasic/text/sbasic/shared/01020300.xhpUsing Procedures and 
Functions/topic
 topic id=sbasic/text/sbasic/shared/01020500.xhpLibraries, Modules and 
Dialogs/topic
-   node id=070202 title=Runtime Functions, Statements, 
and Operators
+   node id=070202 title=Run-Time Functions, 
Statements, and Operators
 topic id=sbasic/text/sbasic/shared/0301.xhpScreen I/O Functions/topic
 topic id=sbasic/text/sbasic/shared/0302.xhpFile I/O Functions/topic
 topic id=sbasic/text/sbasic/shared/0303.xhpDate and Time 
Functions/topic
-topic id=sbasic/text/sbasic/shared/0305.xhpError Handling 
Functions/topic
+topic id=sbasic/text/sbasic/shared/0305.xhpError-Handling 
Functions/topic
 topic id=sbasic/text/sbasic/shared/0306.xhpLogical Operators/topic
 topic id=sbasic/text/sbasic/shared/0307.xhpMathematical 
Operators/topic
 topic id=sbasic/text/sbasic/shared/0308.xhpNumeric Functions/topic
@@ -31,7 +31,7 @@
 topic id=sbasic/text/sbasic/shared/0312.xhpStrings/topic
 topic id=sbasic/text/sbasic/shared/0313.xhpOther Commands/topic
/node
-   node id=070201 title=List of common Functions, 
Statements, and Operators
+   node id=070201 title=Alphabetic List of Functions, 
Statements, and Operators
 topic id=sbasic/text/sbasic/shared/03080601.xhpAbs Function 
[Runtime]/topic
 topic id=sbasic/text/sbasic/shared/03060100.xhpAND Operator 
[Runtime]/topic
 topic id=sbasic/text/sbasic/shared/03104200.xhpArray Function 
[Runtime]/topic
@@ -90,14 +90,14 @@
 topic id=sbasic/text/sbasic/shared/03050100.xhpErl Function 
[Runtime]/topic
 topic id=sbasic/text/sbasic/shared/03050200.xhpErr Function 
[Runtime]/topic
 topic id=sbasic/text/sbasic/shared/03050300.xhpError Function 
[Runtime]/topic
-topic id=sbasic/text/sbasic/shared/0305.xhpError Handling 
Functions/topic
+topic id=sbasic/text/sbasic/shared/0305.xhpError-Handling 
Functions/topic
 topic id=sbasic/text/sbasic/shared/03090412.xhpExit Statement 
[Runtime]/topic
 topic id=sbasic/text/sbasic/shared/03080201.xhpExp Function 
[Runtime]/topic
-topic id=sbasic/text/sbasic/shared/03020405.xhpFileAttr Function 
[Runtime]/topic
+topic id=sbasic/text/sbasic/shared/03020405.xhpFileAttr-Function 
[Runtime]/topic
 topic id=sbasic/text/sbasic/shared/03020406.xhpFileCopy Statement 
[Runtime]/topic
 topic id=sbasic/text/sbasic/shared/03020407.xhpFileDateTime Function 
[Runtime]/topic
 topic id=sbasic/text/sbasic/shared/03020415.xhpFileExists Function 
[Runtime]/topic
-topic id=sbasic/text/sbasic/shared/03020408.xhpFileLen Function 
[Runtime]/topic
+topic id=sbasic/text/sbasic/shared/03020408.xhpFileLen-Function 
[Runtime]/topic
 topic id=sbasic/text/sbasic/shared/03103800.xhpFindObject Function 
[Runtime]/topic
 topic id=sbasic/text/sbasic/shared/03103900.xhpFindPropertyObject Function 
[Runtime]/topic
 topic id=sbasic/text/sbasic/shared/03080501.xhpFix Function 
[Runtime]/topic
@@ -122,7 +122,7 @@
 topic id=sbasic/text/sbasic/shared/03030201.xhpHour Function 
[Runtime]/topic
 topic id=sbasic/text/sbasic/shared/03090103.xhpIIf Statement 
[Runtime]/topic
 topic id=sbasic/text/sbasic/shared/03090101.xhpIf...Then...Else Statement 
[Runtime]/topic
-topic id=sbasic/text/sbasic/shared/03060300.xhpImp Operator 
[Runtime]/topic
+topic id=sbasic/text/sbasic/shared/03060300.xhpImp-Operator 
[Runtime]/topic
 topic id=sbasic/text/sbasic/shared/03120401.xhpInStr Function 
[Runtime]/topic
 topic id=sbasic/text/sbasic/shared/03010201.xhpInputBox Function 
[Runtime]/topic
 topic id=sbasic/text/sbasic/shared/03020202.xhpInput# Statement 
[Runtime]/topic
@@ -151,12 +151,12 @@
 topic id=sbasic/text/sbasic/shared/03120306.xhpMid Function, Mid Statement 
[Runtime]/topic
 topic id=sbasic/text/sbasic/shared/03030202.xhpMinute Function 
[Runtime]/topic
 topic id=sbasic/text/sbasic/shared/03020411.xhpMkDir Statement 
[Runtime]/topic
-topic id=sbasic/text/sbasic/shared/03070600.xhpMod Operator 
[Runtime]/topic
+topic 

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

2012-09-20 Thread Libreoffice Gerrit user
 oox/source/docprop/ooxmldocpropimport.cxx |   22 +++---
 1 file changed, 15 insertions(+), 7 deletions(-)

New commits:
commit 3ad52a6453ff612409123db29dc74a9071789aab
Author: Muthu Subramanian sumu...@suse.com
Date:   Tue Sep 11 12:44:44 2012 +0530

fdo#54609, fdo#45366: Exception while importing xlsx.

* Handle any exception thrown during document properties
  import. This is not so critical so as to stop the import.
  We anyways check for hasElements().
* Also lclGetRelatedStreams might throw IllegalArgumentException

Change-Id: I5206ff1ab07f2853e6a20aebea3f892460b881ad
Signed-off-by: Kohei Yoshida kohei.yosh...@gmail.com

diff --git a/oox/source/docprop/ooxmldocpropimport.cxx 
b/oox/source/docprop/ooxmldocpropimport.cxx
index ef65173..b80069e 100644
--- a/oox/source/docprop/ooxmldocpropimport.cxx
+++ b/oox/source/docprop/ooxmldocpropimport.cxx
@@ -77,7 +77,7 @@ Reference XInterface  SAL_CALL 
DocumentPropertiesImport_createInstance( const
 
 namespace {
 
-Sequence InputSource  lclGetRelatedStreams( const Reference XStorage  
rxStorage, const OUString rStreamType ) throw (RuntimeException)
+Sequence InputSource  lclGetRelatedStreams( const Reference XStorage  
rxStorage, const OUString rStreamType ) throw (RuntimeException, 
IllegalArgumentException)
 {
 Reference XRelationshipAccess  xRelation( rxStorage, UNO_QUERY_THROW );
 Reference XHierarchicalStorageAccess  xHierarchy( rxStorage, 
UNO_QUERY_THROW );
@@ -147,19 +147,27 @@ void SAL_CALL DocumentPropertiesImport::importProperties(
 const Reference XStorage  rxSource, const Reference 
XDocumentProperties  rxDocumentProperties )
 throw (RuntimeException, IllegalArgumentException, SAXException, 
Exception)
 {
+Sequence InputSource  aCoreStreams;
+Sequence InputSource  aExtStreams;
+Sequence InputSource  aCustomStreams;
+
 if( !mxContext.is() )
 throw RuntimeException();
 
 if( !rxSource.is() || !rxDocumentProperties.is() )
 throw IllegalArgumentException();
 
-Sequence InputSource  aCoreStreams = lclGetRelatedStreams( rxSource, 
CREATE_OFFICEDOC_RELATION_TYPE( metadata/core-properties ) );
-// MS Office seems to have a bug, so we have to do similar handling
-if( !aCoreStreams.hasElements() )
-aCoreStreams = lclGetRelatedStreams( rxSource, 
CREATE_PACKAGE_RELATION_TYPE( metadata/core-properties ) );
+try
+{
+aCoreStreams = lclGetRelatedStreams( rxSource, 
CREATE_OFFICEDOC_RELATION_TYPE( metadata/core-properties ) );
+// MS Office seems to have a bug, so we have to do similar handling
+if( !aCoreStreams.hasElements() )
+aCoreStreams = lclGetRelatedStreams( rxSource, 
CREATE_PACKAGE_RELATION_TYPE( metadata/core-properties ) );
 
-Sequence InputSource  aExtStreams = lclGetRelatedStreams( rxSource, 
CREATE_OFFICEDOC_RELATION_TYPE( extended-properties ) );
-Sequence InputSource  aCustomStreams = lclGetRelatedStreams( rxSource, 
CREATE_OFFICEDOC_RELATION_TYPE( custom-properties ) );
+aExtStreams = lclGetRelatedStreams( rxSource, 
CREATE_OFFICEDOC_RELATION_TYPE( extended-properties ) );
+aCustomStreams = lclGetRelatedStreams( rxSource, 
CREATE_OFFICEDOC_RELATION_TYPE( custom-properties ) );
+}
+catch (const Exception) {}
 
 if( aCoreStreams.hasElements() || aExtStreams.hasElements() || 
aCustomStreams.hasElements() )
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[REVIEW 3-6-2 1/3] Crasher fix for xlsx import

2012-09-20 Thread Kohei Yoshida

I've cherry-picked

https://gerrit.libreoffice.org/gitweb?p=core.git;a=commitdiff;h=3ad52a6453ff612409123db29dc74a9071789aab

Just a minute ago to fix

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

the latter of which is a hard-hack.  The cherry-picking to the 3-6 
branch was done with my sign-off, and we need two more sign-offs to push 
this change into 3-6-2.


Kohei

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


[PATCH] Change in core[libreoffice-3-6]: Add RPATH to liblangtag

2012-09-20 Thread Stephan Bergmann (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/661

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/61/661/1

Add RPATH to liblangtag

...so that it consistenly uses the LO versions of libxml2/glibc libs at runtime.

Change-Id: I89b7fd13a8b5b7e697ec21db74fdf8ef088fdb92
(cherry picked from commit 179a6db61ee30cf776747802f06edeef45fec461)
---
M liblangtag/makefile.mk
1 file changed, 8 insertions(+), 0 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I89b7fd13a8b5b7e697ec21db74fdf8ef088fdb92
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-3-6
Gerrit-Owner: Stephan Bergmann sberg...@redhat.com

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


[PATCH] Change in core[libreoffice-3-6]: Do not call xmlCleanupParser from liblangtag

2012-09-20 Thread Stephan Bergmann (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/662

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/62/662/1

Do not call xmlCleanupParser from liblangtag

Despite its name, it is a cleanup function for libxml2 that must only be called
when the process as a whole no longer needs libxml2.  Calling it from a library
like liblangtag is not appropriate (just like liblangtag does not call
xmlInitParser, either).

In LibreOffice, this caused confusion about pthread keys for thread-local
storage (xmlCleanupParser calls pthread_key_delete, but apparently if libxml2 is
later used by unrelated code in LibreOffice again, it still uses the stale key,
and if that key is now reused via an unrelated pthread_key_create, arbitrary
crashes happen during thread termination in pthread's __nptl_deallocate_tsd).

(cherry picked from commit 5e6c595e9d698caf23ae68d38b9cab30ff9c9b94)

Conflicts:
liblangtag/makefile.mk

Change-Id: I82cef8bc400d48f19eaa94f26d50d4b8589da106
---
A liblangtag/liblangtag-0.2-xmlCleanupParser.patch
M liblangtag/makefile.mk
2 files changed, 21 insertions(+), 0 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I82cef8bc400d48f19eaa94f26d50d4b8589da106
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-3-6
Gerrit-Owner: Stephan Bergmann sberg...@redhat.com

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


Re: [Libreoffice-qa] minutes of ESC call ...

2012-09-20 Thread Bjoern Michaelsen
On Thu, Sep 20, 2012 at 04:24:09PM +0100, Michael Meeks wrote:
   + gerrit - patch abandonment can only be done by owner or gerrit admin 
 (Bjoern)

please drop, nothing actionable there
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [Libreoffice-qa] minutes of ESC call ...

2012-09-20 Thread Bjoern Michaelsen
On Thu, Sep 20, 2012 at 04:24:09PM +0100, Michael Meeks wrote:
 * Present:
   + propose new Difficulty: DifficultyHard

Eek, multiple questions on that:
- What would be the difference to DifficultyInteresting in hard and objective
  terms?
- How do we make sure we would all judge the EasyHack the same way as either
  DifficultyBeginner, DifficultyInteresting and DifficultyHard?

If we cant sufficiently safely make the second answer a yes without
generating a huge wikipage describing the process of judging an EasyHack in an
objective manner, the finegrained classification is worthless as it will be
inconsistent. If we need a huge wikipage, a/ we have already lost b/ we have
hours of bikeshedding on that in front of us.

 AI:   + add/adapt the wiki page / generation scripts (Bjoern)

Rejecting, and invoking you touch it, you own it on Michael. ;)

Best,

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


[Libreoffice-commits] .: 4 commits - oox/inc oox/source sw/source writerfilter/inc writerfilter/source

2012-09-20 Thread Libreoffice Gerrit user
 oox/inc/oox/vml/vmlshape.hxx |1 
 oox/source/token/properties.txt  |1 
 oox/source/vml/vmlshape.cxx  |   19 +++
 oox/source/vml/vmlshapecontext.cxx   |3 +
 sw/source/core/text/porrst.cxx   |   10 +++---
 sw/source/core/text/txtfrm.cxx   |   11 ++
 writerfilter/inc/resourcemodel/TableManager.hxx  |4 +-
 writerfilter/source/dmapper/DomainMapperTableHandler.cxx |   24 +--
 writerfilter/source/dmapper/DomainMapperTableHandler.hxx |2 -
 writerfilter/source/doctok/WW8ResourceModelImpl.cxx  |4 +-
 writerfilter/source/filter/ImportFilter.cxx  |   14 
 writerfilter/source/resourcemodel/resourcemodel.cxx  |4 +-
 12 files changed, 76 insertions(+), 21 deletions(-)

New commits:
commit 53b7f7df0617bcbd7bbef9a34ef53e5097eb16dc
Author: Pierre-Eric Pelloux-Prayer pierre-e...@lanedo.com
Date:   Thu Sep 20 11:49:29 2012 +0200

docx import: position table using tblInd OR tblCellMar

Previously cell-margin was unconditionnaly used to compute table position.
Office only does so on top-level tables and if tblInd is missing.

Change-Id: I183647bcf090d5822b3f2e04e097c8bbd277168d

diff --git a/writerfilter/inc/resourcemodel/TableManager.hxx 
b/writerfilter/inc/resourcemodel/TableManager.hxx
index bb72934..c601adf 100644
--- a/writerfilter/inc/resourcemodel/TableManager.hxx
+++ b/writerfilter/inc/resourcemodel/TableManager.hxx
@@ -62,7 +62,7 @@ public:
 /**
Handle end of table.
  */
-virtual void endTable() = 0;
+virtual void endTable(unsigned int nestedTableLevel) = 0;
 
 /**
Handle start of row.
@@ -994,7 +994,7 @@ void TableManagerT, 
PropertiesPointer::resolveCurrentTable()
 mpTableDataHandler-endRow();
 }
 
-mpTableDataHandler-endTable();
+mpTableDataHandler-endTable(mTableDataStack.size() - 1);
 }
 catch (uno::Exception const e)
 {
diff --git a/writerfilter/source/dmapper/DomainMapperTableHandler.cxx 
b/writerfilter/source/dmapper/DomainMapperTableHandler.cxx
index fbdabd9..dce337d 100644
--- a/writerfilter/source/dmapper/DomainMapperTableHandler.cxx
+++ b/writerfilter/source/dmapper/DomainMapperTableHandler.cxx
@@ -262,6 +262,7 @@ struct WRITERFILTER_DLLPRIVATE TableInfo
 sal_Int32 nRightBorderDistance;
 sal_Int32 nTopBorderDistance;
 sal_Int32 nBottomBorderDistance;
+sal_Int32 nNestLevel;
 PropertyMapPtr pTableDefaults;
 PropertyMapPtr pTableBorders;
 TableStyleSheetEntry* pTableStyle;
@@ -272,6 +273,7 @@ struct WRITERFILTER_DLLPRIVATE TableInfo
 , nRightBorderDistance(DEF_BORDER_DIST)
 , nTopBorderDistance(0)
 , nBottomBorderDistance(0)
+, nNestLevel(0)
 , pTableDefaults(new PropertyMap)
 , pTableBorders(new PropertyMap)
 , pTableStyle(NULL)
@@ -433,7 +435,24 @@ TableStyleSheetEntry * 
DomainMapperTableHandler::endTableGetTableStyle(TableInfo
 lcl_debug_TableBorder(aTableBorder);
 #endif
 
-m_aTableProperties-Insert( PROP_LEFT_MARGIN, false, uno::makeAny( 
nLeftMargin - nGapHalf - rInfo.nLeftBorderDistance));
+// Mimic Office behavior : if tlbInd is defined, use it place table.
+// Otherwise, top-level table's position depends w:tblCellMar 
attribute (but not nested tables)
+if (nLeftMargin)
+{
+m_aTableProperties-Insert( PROP_LEFT_MARGIN, false, uno::makeAny( 
nLeftMargin - nGapHalf));
+}
+else
+{
+// TODO: top-level position depends on w:tblCellMar attribute, not 
w:cellMar
+if (rInfo.nNestLevel  1)
+{
+m_aTableProperties-Insert( PROP_LEFT_MARGIN, false, 
uno::makeAny( - nGapHalf));
+}
+else
+{
+m_aTableProperties-Insert( PROP_LEFT_MARGIN, false, 
uno::makeAny( - nGapHalf - rInfo.nLeftBorderDistance));
+}
+}
 
 m_aTableProperties-getValue( TablePropertyMap::TABLE_WIDTH, 
nTableWidth );
 if( nTableWidth  0 )
@@ -672,13 +691,14 @@ RowPropertyValuesSeq_t 
DomainMapperTableHandler::endTableGetRowProperties()
 return aRowProperties;
 }
 
-void DomainMapperTableHandler::endTable()
+void DomainMapperTableHandler::endTable(unsigned int nestedTableLevel)
 {
 #ifdef DEBUG_DMAPPER_TABLE_HANDLER
 dmapper_logger-startElement(tablehandler.endTable);
 #endif
 
 TableInfo aTableInfo;
+aTableInfo.nNestLevel = nestedTableLevel;
 aTableInfo.pTableStyle = endTableGetTableStyle(aTableInfo);
 //  expands to uno::Sequence Sequence beans::PropertyValues  
 
diff --git a/writerfilter/source/dmapper/DomainMapperTableHandler.hxx 
b/writerfilter/source/dmapper/DomainMapperTableHandler.hxx
index 0ccd71c..ea4c421 100644
--- a/writerfilter/source/dmapper/DomainMapperTableHandler.hxx

Re: How to use internal neon ?

2012-09-20 Thread Matúš Kukan
On 20 September 2012 17:09, Stephan Bergmann sberg...@redhat.com wrote:
 On 09/20/2012 04:00 PM, Matúš Kukan wrote:

 I see, so it will be a little complicated with exporting symbols.
 Currently, there is even .map file used. (probably, I don't know how
 dmake works)
 I've removed .map files, allowed to turn of -fvisibility=hidden in
 https://gerrit.libreoffice.org/#/c/658/
 and then tried to convert neon in
 https://gerrit.libreoffice.org/#/c/659/
 Seems to work for me on Linux.
 Is this approach right ?


 In case that dramatically increases the amount of symbols exported from
 neon, it might make sense to keep the mapfile, no?

Originally, I wanted to keep it.
But current gbuild implementation of gb_Library_set_soversion_script
is linux only anyway
(with, in this case pointless, SOVERSION number)
and it just seemed unnecessary to me. ( maybe I am lazy )

 What's the output of nm -D --defined solver/*/lib/libneon,so | wc -l for
 you with those changes?  For me the old one is 280.

the old one:
nm -D --defined solver/*/lib/libneon.so | wc -l : 285
ll solver/*/lib/libneon.so* : 1732824
the new one, without .map file:
330
1732932
new one, with .map file:
283
1732672
So, it's best, but linux only and the size is almost the same.

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


Re: [REVIEW 3-6-2 1/3] Crasher fix for xlsx import

2012-09-20 Thread Markus Mohrhard
Hey,

2012/9/20 Kohei Yoshida kohei.yosh...@gmail.com:
 I've cherry-picked

 https://gerrit.libreoffice.org/gitweb?p=core.git;a=commitdiff;h=3ad52a6453ff612409123db29dc74a9071789aab

 Just a minute ago to fix

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

 the latter of which is a hard-hack.  The cherry-picking to the 3-6 branch
 was done with my sign-off, and we need two more sign-offs to push this
 change into 3-6-2.


Looks good to me. One more needed for 3-6-2.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: Branch 'libreoffice-3-6' - liblangtag/makefile.mk

2012-09-20 Thread Libreoffice Gerrit user
 liblangtag/makefile.mk |8 
 1 file changed, 8 insertions(+)

New commits:
commit c5f049680130a9af6c97856ea18869738811822b
Author: Stephan Bergmann sberg...@redhat.com
Date:   Thu Sep 20 15:00:50 2012 +0200

Add RPATH to liblangtag

...so that it consistenly uses the LO versions of libxml2/glibc libs at 
runtime.

Change-Id: I89b7fd13a8b5b7e697ec21db74fdf8ef088fdb92
(cherry picked from commit 179a6db61ee30cf776747802f06edeef45fec461)
Reviewed-on: https://gerrit.libreoffice.org/661
Reviewed-by: Michael Stahl mst...@redhat.com
Tested-by: Michael Stahl mst...@redhat.com

diff --git a/liblangtag/makefile.mk b/liblangtag/makefile.mk
index a8efa55..7efbc32 100644
--- a/liblangtag/makefile.mk
+++ b/liblangtag/makefile.mk
@@ -72,6 +72,14 @@ CONFIGURE_ACTION=$(AUGMENT_LIBRARY_PATH) .$/configure
 BUILD_ACTION=$(AUGMENT_LIBRARY_PATH) $(GNUMAKE) -j$(EXTMAXPROCESS)  \
 $(AUGMENT_LIBRARY_PATH) $(GNUMAKE) install
 
+.IF $(SYSTEM_LIBXML)!=YES || $(SYSTEM_GLIB)!=YES
+.IF $(OS)==FREEBSD || $(OS)==LINUX
+CONFIGURE_FLAGS+= \
+ LDFLAGS=-Wl,-z,origin\ 
-Wl,-rpath,\'\$$\$$ORIGIN:\$$\$$ORIGIN/../ure-link/lib\'
+.ELIF $(OS)==SOLARIS
+CONFIGURE_FLAGS+= LDFLAGS=-Wl,-R\'\$$\$$ORIGIN:\$$\$$ORIGIN/../ure-link/lib\'
+.END
+.END
 
 .IF $(GUI)==WNT
 .IF $(COM)==GCC
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: Branch 'libreoffice-3-6' - liblangtag/liblangtag-0.2-xmlCleanupParser.patch liblangtag/makefile.mk

2012-09-20 Thread Libreoffice Gerrit user
 liblangtag/liblangtag-0.2-xmlCleanupParser.patch |   20 
 liblangtag/makefile.mk   |1 +
 2 files changed, 21 insertions(+)

New commits:
commit c63f4962caefc22cbf0fa9cde87f3a49d79f0b85
Author: Stephan Bergmann sberg...@redhat.com
Date:   Thu Sep 20 15:40:30 2012 +0200

Do not call xmlCleanupParser from liblangtag

Despite its name, it is a cleanup function for libxml2 that must only be 
called
when the process as a whole no longer needs libxml2.  Calling it from a 
library
like liblangtag is not appropriate (just like liblangtag does not call
xmlInitParser, either).

In LibreOffice, this caused confusion about pthread keys for thread-local
storage (xmlCleanupParser calls pthread_key_delete, but apparently if 
libxml2 is
later used by unrelated code in LibreOffice again, it still uses the stale 
key,
and if that key is now reused via an unrelated pthread_key_create, arbitrary
crashes happen during thread termination in pthread's 
__nptl_deallocate_tsd).

(cherry picked from commit 5e6c595e9d698caf23ae68d38b9cab30ff9c9b94)

Conflicts:
liblangtag/makefile.mk

Change-Id: I82cef8bc400d48f19eaa94f26d50d4b8589da106
Reviewed-on: https://gerrit.libreoffice.org/662
Reviewed-by: Michael Stahl mst...@redhat.com
Tested-by: Michael Stahl mst...@redhat.com

diff --git a/liblangtag/liblangtag-0.2-xmlCleanupParser.patch 
b/liblangtag/liblangtag-0.2-xmlCleanupParser.patch
new file mode 100644
index 000..3617125
--- /dev/null
+++ b/liblangtag/liblangtag-0.2-xmlCleanupParser.patch
@@ -0,0 +1,20 @@
+--- misc/liblangtag-0.2/liblangtag/lt-xml.c
 misc/build/liblangtag-0.2/liblangtag/lt-xml.c
+@@ -101,8 +101,6 @@ lt_xml_read_subtag_registry(lt_xml_t  *x
+   if (xmlparser)
+   xmlFreeParserCtxt(xmlparser);
+ 
+-  xmlCleanupParser();
+-
+   if (err) {
+   if (error)
+   *error = g_error_copy(err);
+@@ -158,8 +156,6 @@ lt_xml_read_cldr_bcp47(lt_xml_t *xml
+   if (xmlparser)
+   xmlFreeParserCtxt(xmlparser);
+ 
+-  xmlCleanupParser();
+-
+   if (err) {
+   if (error)
+   *error = g_error_copy(err);
diff --git a/liblangtag/makefile.mk b/liblangtag/makefile.mk
index 7efbc32..3272698 100644
--- a/liblangtag/makefile.mk
+++ b/liblangtag/makefile.mk
@@ -37,6 +37,7 @@ PATCH_FILES=liblangtag-0.2-0001-Fix-a-memory-leak.patch
 PATCH_FILES+=liblangtag-0.2-0002-Fix-invalid-memory-access.patch
 PATCH_FILES+=liblangtag-0.2-configure.patch
 PATCH_FILES+=liblangtag-0.2-datadir.patch
+PATCH_FILES+=liblangtag-0.2-xmlCleanupParser.patch
 
 CONFIGURE_DIR=.
 BUILD_DIR=$(CONFIGURE_DIR)
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED] Change in core[libreoffice-3-6]: Add RPATH to liblangtag

2012-09-20 Thread Michael Stahl (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/661

Approvals:
  Michael Stahl: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I89b7fd13a8b5b7e697ec21db74fdf8ef088fdb92
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-3-6
Gerrit-Owner: Stephan Bergmann sberg...@redhat.com
Gerrit-Reviewer: Michael Stahl mst...@redhat.com

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


‘auto_ptr’ is deprecated in sc/source/core/tool/rangenam.cxx

2012-09-20 Thread julien2412
Hello,

WAE is enabled in my autogen.lastrun for master sources. I just updated and
had this during compilation:
/home/julien/compile-libreoffice/libo/sc/source/core/tool/rangenam.cxx: In
member function ‘void ScRangeData::SetCode(ScTokenArray)’:
/home/julien/compile-libreoffice/libo/sc/source/core/tool/rangenam.cxx:642:50:
error: ‘auto_ptr’ is deprecated (declared at
/usr/include/c++/4.7/backward/auto_ptr.h:87)
[-Werror=deprecated-declarations]
cc1plus: all warnings being treated as errors

Here are the lines:
640 void ScRangeData::SetCode( ScTokenArray rArr )
641 {
642 ::std::auto_ptrScTokenArray pOldCode( pCode); // old pCode
will be deleted
643 pCode = new ScTokenArray( rArr );
644 InitCode();
645 }


Could it be replaced by a non deprecated function (I don't know between
std::shared_ptr, std::unique_ptr or boost functions)?

Julien



--
View this message in context: 
http://nabble.documentfoundation.org/auto-ptr-is-deprecated-in-sc-source-core-tool-rangenam-cxx-tp4008439.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


getting started

2012-09-20 Thread Jorge Luis Roque Alvarez
Hi iam just learning how to develop; my first step is to understand the 
examples i already install the sdk from the repository aim currently using 
ubuntu 12.04.1 but when i run make to the bookmarkInsertation(example in java)  
it do not find any library i'm i missing something could anybody help me with 
this. Iam just having this problems with java examples, since i test c++ ex and 
they work ok.

thanks in advanced 

PS: this is a section of the output when running make

make
mkdir -p /usr/lib/libreoffice/sdk//LINUXexample.out/class/JavaTextExamples
/bin/javac  -classpath 
:/usr/lib/libreoffice/sdk//LINUXexample.out/class/JavaTextExamples -d 
/usr/lib/libreoffice/sdk//LINUXexample.out/class/JavaTextExamples 
BookmarkInsertion.java
BookmarkInsertion.java:44: error: package com.sun.star.uno does not exist
import com.sun.star.uno.UnoRuntime;
   ^
BookmarkInsertion.java:78: error: package com.sun.star.text does not exist
public static void markList(com.sun.star.text.XTextDocument xTextDocument,
 ^
BookmarkInsertion.java:108: error: package com.sun.star.text does not exist
public static void InsertBookmark(com.sun.star.text.XTextDocument 
xTextDocument,
   ^
BookmarkInsertion.java:109: error: package com.sun.star.text does not exist
  com.sun.star.text.XTextRange xTextRange,
   ^
BookmarkInsertion.java:149: error: package com.sun.star.text does not exist
com.sun.star.text.XTextDocument xTextDocument, String sSearchString)
 ^
BookmarkInsertion.java:148: error: package com.sun.star.uno does not exist


Best Regards,
Jorge Luis

10mo. ANIVERSARIO DE LA CREACION DE LA UNIVERSIDAD DE LAS CIENCIAS 
INFORMATICAS...
CONECTADOS AL FUTURO, CONECTADOS A LA REVOLUCION

http://www.uci.cu
http://www.facebook.com/universidad.uci
http://www.flickr.com/photos/universidad_uci
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: ‘auto_ptr’ is deprecated in sc/source/core/tool/rangenam.cxx

2012-09-20 Thread Michael Stahl
On 20/09/12 19:23, julien2412 wrote:
 Hello,
 
 WAE is enabled in my autogen.lastrun for master sources. I just updated and
 had this during compilation:
 /home/julien/compile-libreoffice/libo/sc/source/core/tool/rangenam.cxx: In
 member function ‘void ScRangeData::SetCode(ScTokenArray)’:
 /home/julien/compile-libreoffice/libo/sc/source/core/tool/rangenam.cxx:642:50:
 error: ‘auto_ptr’ is deprecated (declared at
 /usr/include/c++/4.7/backward/auto_ptr.h:87)
 [-Werror=deprecated-declarations]
 cc1plus: all warnings being treated as errors
 
 Here are the lines:
 640 void ScRangeData::SetCode( ScTokenArray rArr )
 641 {
 642 ::std::auto_ptrScTokenArray pOldCode( pCode); // old pCode
 will be deleted
 643 pCode = new ScTokenArray( rArr );
 644 InitCode();
 645 }
 
 
 Could it be replaced by a non deprecated function (I don't know between
 std::shared_ptr, std::unique_ptr or boost functions)?

usually can be replaced with boost::scoped_ptr.

but in this case i wonder, what purpose does pOldCode serve?
why isn't pCode some kind of smart pointer?


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


Re: Is there easy way to find unused .idl files ?

2012-09-20 Thread Matúš Kukan
On 20 September 2012 13:01, Michael Stahl mst...@redhat.com wrote:
 And I wonder if there is an easy way how to tell, they can be removed.
 For files added with gb_UnoApi_add_idlfile_noheader, one probably have
 to grep in the code.

 technically all of those are unused because they are mere
 documentation.  but that documentation is still nice to have :)

 well, Noel is currently working on converting these to the less-unused
 new-style services...

So, it seems, he is almost done in oovbaapi :-)
But there are more interfaces like services.
And I have often seen (unused ?) constants in there.
In files without license part like oovbaapi/ooo/vba/access/AcCloseSave.idl

 my opinion is that while there are probably a few more useless relics
 like ProductRegistration, IDL removal requires a case-by-case review anyway.

Well, I guess it's better to suppose they are mostly used and let it be :-)
It's not that long to build udkapi, offapi and oovbaapi.
And this whole UNO .idl thing is quite mysterious to me.

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


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

2012-09-20 Thread Libreoffice Gerrit user
 sw/source/core/text/txttab.cxx |   48 ++--
 sw/source/core/text/widorp.cxx |  152 +++--
 2 files changed, 97 insertions(+), 103 deletions(-)

New commits:
commit 606b57c625aebabd004a4f3e3a2552f3a03551a0
Author: Michael Stahl mst...@redhat.com
Date:   Thu Sep 20 20:28:00 2012 +0200

sw: tweak previous comment translation

Change-Id: Id4a89bd131f37b6844046803aae9b2f6e081af0d

diff --git a/sw/source/core/text/widorp.cxx b/sw/source/core/text/widorp.cxx
index b761ce3..4fe8c41 100644
--- a/sw/source/core/text/widorp.cxx
+++ b/sw/source/core/text/widorp.cxx
@@ -94,10 +94,10 @@ SwTxtFrmBreak::SwTxtFrmBreak( SwTxtFrm *pNewFrm, const 
SwTwips nRst )
 
 /* BP 18.6.93: Widows.
  * In contrast to the first implementation the Widows are not calculated
- * in advance but detected when formating the splitted Follow.
+ * in advance but detected when formating the split Follow.
  * In Master the Widows-calculation is dropped completely
  * (nWidows is manipulated). If the Follow detects that the
- * Widows rule applies it sends a Prepare to its ancestor.
+ * Widows rule applies it sends a Prepare to its predecessor.
  * A special problem is when the Widow rule applies but in Master
  * there are some lines available.
  *
@@ -113,9 +113,9 @@ SwTxtFrmBreak::SwTxtFrmBreak( SwTxtFrm *pNewFrm, const 
SwTwips nRst )
  * One difficulty with Widows and different formats between
  * Master- and Follow-Frame:
  * Example: If the first column is 3cm and the second is 4cm and
- * Widows is set to 3. The decision if the Widows rule matches can not
+ * Widows is set to 3, the decision if the Widows rule matches can not
  * be done until the Follow is formated. Unfortunately this is crucial
- * sto decide if the whole paragraph goes to the next page or not.
+ * to decide if the whole paragraph goes to the next page or not.
  */
 
 sal_Bool SwTxtFrmBreak::IsInside( SwTxtMargin rLine ) const
@@ -167,9 +167,9 @@ sal_Bool SwTxtFrmBreak::IsInside( SwTxtMargin rLine ) const
 // grow the requested area.
 nHeight += pFrm-GrowTst( LONG_MAX );
 
-// The Grow() returnes the height, that the Upper of the TxtFrm
-// would grow the TxtFrm.
-// The TxtFrm itself can grow as much as it want.
+// The Grow() returns the height by which the Upper of the TxtFrm
+// would let the TxtFrm grow.
+// The TxtFrm itself can grow as much as it wants.
 bFit = nHeight = nLineHeight;
 }
 }
@@ -188,7 +188,7 @@ sal_Bool SwTxtFrmBreak::IsBreakNow( SwTxtMargin rLine )
 SWAP_IF_SWAPPED( pFrm )
 
 // bKeep is stronger than IsBreakNow()
-// Is enough space ?
+// Is there enough space ?
 if( bKeep || IsInside( rLine ) )
 bBreak = sal_False;
 else
@@ -255,13 +255,13 @@ WidowsAndOrphans::WidowsAndOrphans( SwTxtFrm *pNewFrm, 
const SwTwips nRst,
 
 if( bKeep )
 {
-// 5652: If pararagraph should not be splited but is larger than
+// 5652: If pararagraph should not be split but is larger than
 // the page, then bKeep is overruled.
 if( bChkKeep  !pFrm-GetPrev()  !pFrm-IsInFtn() 
 pFrm-IsMoveable() 
 ( !pFrm-IsInSct() || pFrm-FindSctFrm()-MoveAllowed(pFrm) ) )
 bKeep = sal_False;
-// Even if Keep is set, the Orphans has to be regarded
+// Even if Keep is set, Orphans has to be respected.
 // e.g. if there are chained frames where a Follow in the last frame
 // receives a Keep, because it is not (forward) movable -
 // nevertheless the paragraph can request lines from the Master
@@ -360,8 +360,8 @@ sal_Bool WidowsAndOrphans::FindBreak( SwTxtFrm *pFrame, 
SwTxtMargin rLine,
 else
 break;
 }
-// Normaly Orphans are not taken into account for HasToFit.
-// But if Dummy-Lines are concerned and they violate the orphans rules
+// Usually Orphans are not taken into account for HasToFit.
+// But if Dummy-Lines are concerned and the Orphans rule is violated
 // we make an exception: We leave behind one Dummyline and take
 // the whole text to the next page/column.
 if( rLine.GetLineNr() = nOldOrphans 
@@ -383,10 +383,10 @@ sal_Bool WidowsAndOrphans::FindBreak( SwTxtFrm *pFrame, 
SwTxtMargin rLine,
  *  WidowsAndOrphans::FindWidows()
  */
 
-/*  FindWidows positiones the SwTxtMargin of the Master to the line where to
+/*  FindWidows positions the SwTxtMargin of the Master to the line where to
  *  break by examining and formatting the Follow.
  *  Returns sal_True if the Widows-rule matches, that means that the
- *  paragraph should be not splitted (keep) !
+ *  paragraph should not be split (keep) !
  */
 
 sal_Bool WidowsAndOrphans::FindWidows( SwTxtFrm *pFrame, SwTxtMargin rLine )
@@ -433,13 

[Libreoffice-commits] .: Branch 'feature/killsdf' - l10ntools/Executable_cfgex.mk l10ntools/Executable_helpex.mk l10ntools/Executable_transex3.mk l10ntools/Executable_ulfex.mk l10ntools/Executable_xrm

2012-09-20 Thread Libreoffice Gerrit user
 l10ntools/Executable_cfgex.mk|1 
 l10ntools/Executable_helpex.mk   |1 
 l10ntools/Executable_transex3.mk |1 
 l10ntools/Executable_ulfex.mk|1 
 l10ntools/Executable_xrmex.mk|1 
 l10ntools/inc/po.hxx |   18 +++
 l10ntools/source/merge.cxx   |  108 -
 l10ntools/source/po.cxx  |  175 ++-
 l10ntools/source/renewpo.cxx |6 -
 solenv/gbuild/AllLangResTarget.mk|   22 +++-
 solenv/gbuild/Configuration.mk   |   19 ++-
 solenv/gbuild/ExtensionTarget.mk |   31 --
 solenv/gbuild/InstallModuleTarget.mk |   11 +-
 solenv/gbuild/TargetLocations.mk |2 
 solenv/inc/rules.mk  |   12 --
 15 files changed, 299 insertions(+), 110 deletions(-)

New commits:
commit 161f4ac95ae3df1daaede762862ea1c3ac4bb399
Author: Zolnai Tamás zolnaitamas2...@gmail.com
Date:   Mon Sep 10 20:31:38 2012 +0200

Use po file for merge 1. step

Make executables to waiting for po as input
Add some function to po files
Rewrite makefiles to use po files for merge
Change KeyId generator not to use xml special
charachters
One filetype remain: properties

Change-Id: If2bd2ae7afc61518c2d838a3cae301ef62dd9761
Reviewed-on: https://gerrit.libreoffice.org/593
Reviewed-by: Michael Stahl mst...@redhat.com
Tested-by: Michael Stahl mst...@redhat.com

diff --git a/l10ntools/Executable_cfgex.mk b/l10ntools/Executable_cfgex.mk
index 7dc2327..9fea979 100644
--- a/l10ntools/Executable_cfgex.mk
+++ b/l10ntools/Executable_cfgex.mk
@@ -33,6 +33,7 @@ $(eval $(call gb_Executable_set_include,cfgex,\
 
 $(eval $(call gb_Executable_use_libraries,cfgex,\
 sal \
+i18nregexp \
 ))
 
 $(eval $(call gb_Executable_add_scanners,cfgex,\
diff --git a/l10ntools/Executable_helpex.mk b/l10ntools/Executable_helpex.mk
index 470bd4c..3569dff 100644
--- a/l10ntools/Executable_helpex.mk
+++ b/l10ntools/Executable_helpex.mk
@@ -33,6 +33,7 @@ $(eval $(call gb_Executable_set_include,helpex,\
 
 $(eval $(call gb_Executable_use_libraries,helpex,\
 sal \
+i18nregexp \
 ))
 
 $(eval $(call gb_Executable_use_external,helpex,expat_utf8))
diff --git a/l10ntools/Executable_transex3.mk b/l10ntools/Executable_transex3.mk
index bc9a852..2d95906 100644
--- a/l10ntools/Executable_transex3.mk
+++ b/l10ntools/Executable_transex3.mk
@@ -33,6 +33,7 @@ $(eval $(call gb_Executable_set_include,transex3,\
 
 $(eval $(call gb_Executable_use_libraries,transex3,\
 sal \
+i18nregexp \
 ))
 
 $(eval $(call gb_Executable_add_scanners,transex3,\
diff --git a/l10ntools/Executable_ulfex.mk b/l10ntools/Executable_ulfex.mk
index e82c9b8..f3fd559 100644
--- a/l10ntools/Executable_ulfex.mk
+++ b/l10ntools/Executable_ulfex.mk
@@ -33,6 +33,7 @@ $(eval $(call gb_Executable_set_include,ulfex,\
 
 $(eval $(call gb_Executable_use_libraries,ulfex,\
 sal \
+i18nregexp \
 ))
 
 $(eval $(call gb_Executable_use_static_libraries,ulfex,\
diff --git a/l10ntools/Executable_xrmex.mk b/l10ntools/Executable_xrmex.mk
index 33255f4..b86dbc4 100644
--- a/l10ntools/Executable_xrmex.mk
+++ b/l10ntools/Executable_xrmex.mk
@@ -33,6 +33,7 @@ $(eval $(call gb_Executable_set_include,xrmex,\
 
 $(eval $(call gb_Executable_use_libraries,xrmex,\
 sal \
+i18nregexp \
 ))
 
 $(eval $(call gb_Executable_add_scanners,xrmex,\
diff --git a/l10ntools/inc/po.hxx b/l10ntools/inc/po.hxx
index b4b18c4..81bafe7 100644
--- a/l10ntools/inc/po.hxx
+++ b/l10ntools/inc/po.hxx
@@ -25,6 +25,7 @@ private:
 OStringm_sUnTransStr;
 OStringm_sTransStr;
 bool   m_bFuzzy;
+bool   m_bNull;
 OStringm_sKeyId;
 
 public:
@@ -38,6 +39,8 @@ public:
 virtual OString getUnTransStr() const   { return m_sUnTransStr; }
 virtual OString getTransStr() const { return m_sTransStr; }
 virtual boolgetFuzzy() const{ return m_bFuzzy; }
+virtual boolisNull() const  { return m_bNull; }
+virtual OString getKeyId() const{ return m_sKeyId; }
 
 virtual voidsetWhiteSpace(const OString rWhiteSpace);
 virtual voidsetExtractCom(const OString rExtractCom);
@@ -68,6 +71,8 @@ private:
 OString m_sResourceType;
 TYPEm_eType;
 OString m_sHelpText;
+
+voidSetMembers();
 public:
 
 PoEntry();
@@ -84,6 +89,8 @@ public:
 OString getUnTransStr() const;
 OString getTransStr() const;
 boolgetFuzzy() const{ return m_aGenPo.getFuzzy(); }
+boolisNull() const  { return m_aGenPo.isNull(); }
+OString getKeyId() const{ return m_aGenPo.getKeyId(); }
 voidsetUnTransStr(const OString rUnTransStr);
 voidsetTransStr(const OString rTransStr);
 voidsetFuzzy(const bool bFuzzy);
@@ -91,6 +98,8 @@ public:
 void 

Re: [Libreoffice-qa] minutes of ESC call ...

2012-09-20 Thread Michael Meeks

On Thu, 2012-09-20 at 18:19 +0200, Bjoern Michaelsen wrote:
 On Thu, Sep 20, 2012 at 04:24:09PM +0100, Michael Meeks wrote:
  * Present:
  + propose new Difficulty: DifficultyHard
 
 Eek, multiple questions on that:
 - What would be the difference to DifficultyInteresting in hard
 and objective terms?

Why does it have to have hard and objective terms ? Most reasonable
people can make a distinction between a beginner problem: ie. something
almost anyone can pickup and do as their very first commit ever, from
something that requires a little experience, from something that takes a
week of work [eg.] :-)

 - How do we make sure we would all judge the EasyHack the same way as either
   DifficultyBeginner, DifficultyInteresting and DifficultyHard?

Is it the end of the world if we cannot ? :-)

 If we cant sufficiently safely make the second answer a yes without
 generating a huge wikipage describing the process of judging an EasyHack in an
 objective manner, the finegrained classification is worthless as it will be
 inconsistent.

A rough measure of Severity / Importance is not intrinsically useless
in my view for it's rough-ness. Not all ladders should have only two
rungs ;-) We could add a rough guideline (like the above) in a paragraph
to the wiki page easily enough if even that is needed.

  If we need a huge wikipage, a/ we have already lost b/ we have
 hours of bikeshedding on that in front of us.

How about avoiding the bikeshedding altogether  just restoring the
original(?) three-way categorisation of difficulty :-)

  AI: + add/adapt the wiki page / generation scripts (Bjoern)
 
 Rejecting, and invoking you touch it, you own it on Michael. ;)

So - how does that magic bugzilla - wiki page generation thing work ?
is that documented somewhere ? I'm happy to do it if you hand-hold me -
though there is a risk that that takes longer.

ATB,

Michael.

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

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


[PUSHED] Change in core[feature/killsdf]: Use po file for merge 1. step

2012-09-20 Thread Michael Stahl (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/593

Approvals:
  Michael Stahl: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: If2bd2ae7afc61518c2d838a3cae301ef62dd9761
Gerrit-PatchSet: 4
Gerrit-Project: core
Gerrit-Branch: feature/killsdf
Gerrit-Owner: Zolnai Tamás zolnaitamas2...@gmail.com
Gerrit-Reviewer: Michael Stahl mst...@redhat.com
Gerrit-Reviewer: Zolnai Tamás zolnaitamas2...@gmail.com

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


  1   2   3   4   >