[Libreoffice-commits] .: writerfilter/source

2013-01-02 Thread Libreoffice Gerrit user
 writerfilter/source/dmapper/DomainMapper.cxx |2 --
 1 file changed, 2 deletions(-)

New commits:
commit 73731b01cd65defdf9b42a9754bede3ba84221d7
Author: Pierre-Eric Pelloux-Prayer pierre-e...@lanedo.com
Date:   Thu Dec 20 15:39:37 2012 +0100

docx import: don't apply complex font size to non-complex font

OOXML spec says:
[szCs] specifies the font size which shall be applied to all
complex script characters in the contents of this run when displayed

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

diff --git a/writerfilter/source/dmapper/DomainMapper.cxx 
b/writerfilter/source/dmapper/DomainMapper.cxx
index dd61686..84f344e 100644
--- a/writerfilter/source/dmapper/DomainMapper.cxx
+++ b/writerfilter/source/dmapper/DomainMapper.cxx
@@ -2130,8 +2130,6 @@ void DomainMapper::sprmWithProps( Sprm rSprm, 
PropertyMapPtr rContext, SprmType
 if( NS_sprm::LN_CHpsBi == nSprmId )
 {
 rContext-Insert( PROP_CHAR_HEIGHT_COMPLEX, true, aVal );
-// Also set Western, but don't overwrite it.
-rContext-Insert( PROP_CHAR_HEIGHT, true, aVal, false );
 }
 else
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED] docx import: don't apply complex font size to non-complex fo...

2013-01-02 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/1454

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I0faf599e38ef5a2e2005bb6f98874639be7d8287
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Pierre-Eric Pelloux-Prayer pierre-e...@lanedo.com
Gerrit-Reviewer: Miklos Vajna vmik...@suse.cz

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


Re: [SOLVED] Re: Question about operator = overloaded in reportdesign module

2013-01-02 Thread Lionel Elie Mamane
On Sun, Dec 30, 2012 at 09:58:30PM +0100, Julien Nabet wrote:
 On 30/12/2012 21:48, Markus Mohrhard wrote:

Cppcheck reported this:
[reportdesign/source/filter/xml/xmlComponent.hxx:37]: (style)
'OXMLComponent::operator=' should return 'OXMLComponent'.
[reportdesign/source/filter/xml/xmlFunction.hxx:41]: (style)
'OXMLFunction::operator=' should return 'OXMLFunction'.
[reportdesign/source/filter/xml/xmlGroup.hxx:38]: (style)
'OXMLGroup::operator=' should return 'OXMLGroup'.
[reportdesign/source/filter/xml/xmlCell.hxx:41]: (style)
'OXMLCell::operator=' should return 'OXMLCell'.

By trying to fix these, I noticed that none of them was implemented. So can
they just be removed, is it another C++ trick, or something obvious I
missed?

You can't remove them. They are declared there to prevent the default
operator= and copy c'tor. If you don't declare them private and leave
out the implementation the compiler will generate a default operator=
(the same is true for the copy c'tor).

 I didn't think that one's want prevent default behaviour without
 overloading

The intent is to *forbid* copying of objects of this type. The
canonical example is a smart pointer that deletes what it points
to when it goes out of scope (when it is destructed). This is a kind
of primitive garbage collector / automatic memory management, that
does not require any collaboration from the managed object (the
pointer pointed to), as opposed to reference counting (which needs the
object to have an integer member for the reference count and a mutex
member to handle changing / reading the reference count).

C++11 has a specific syntax to achieve mostly the same effect more
cleanly:

  OXMLCell operator =(const OXMLCell) = delete;

That's slightly better since it tells the compiler that this class
should have *no* assignment operator. So any code that tries to use it
will have an error message no such operator.

The private + unimplemented trick tells the compiler the operator
exists, but only the class itself is allowed to use it.

So, if some non-member function code tries to use the operator, it
will get an error message like this operator is private, which is
nearly as good as this operator does not exist... if the user knows
of this trick, but is arguably less clear.

If a member class function tries to use that operator, the compiler
will accept it, and produce no error. The error will only come at the
linker stage, something like undefined reference to operator=. Which
could be caused by a forgotten .o in the linker command line rather
than by design, so is a rather poor error message for this situation.

(We cannot yet use C++11 in LibreOffice because not all platforms we
 want to support have good (any?) support for C++11)


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


Re: [REVIEW] domain-mapper import speedup patch ...

2013-01-02 Thread Miklos Vajna
Hi Michael,

On Fri, Dec 21, 2012 at 04:55:44PM +, Michael Meeks 
michael.me...@suse.com wrote:
   I attach a prototype patch. It passes make check and make slowcheck in
 sw/ (not tried subsequentcheck).

That makes a lot of sense, we also do something similar for section
properties in
http://opengrok.libreoffice.org/xref/core/writerfilter/source/dmapper/PropertyMap.cxx#1062

 +// FIXME: we should have some nice way of merging ranges surely ?
 +aWhichPairs.push_back(pEntry-nWID);
 +aWhichPairs.push_back(pEntry-nWID);
 +}
 +aEntries.push_back(pEntry);

I'm not sure if we have something like that -- Björn? :-)

Miklos


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


[PATCH] Paren fix for Ubu 10.04 build (and fdo#58417 ?)

2013-01-02 Thread LeMoyne Castle (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1531

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/31/1531/1

Paren fix for Ubu 10.04 build (and fdo#58417 ?)

The form: OuterCtor( InnerCtor(array[i]).method() )
with explicit ctor(s), unused optional arg on ctor+method
-- error 'parameter can not have variably modified type'
gcc version 4.4.3 (Ubuntu 4.4.3-4ubuntu5.1).
Min fix: force precedence/associativity in type resolution.
fdo#58417 BTs inside 1 of these ctor calls.

Change-Id: I4d8b979df0fc827758b1e9a336031671425486ea
---
M linguistic/source/lngsvcmgr.cxx
1 file changed, 7 insertions(+), 7 deletions(-)



diff --git a/linguistic/source/lngsvcmgr.cxx b/linguistic/source/lngsvcmgr.cxx
index 4f68407..a8e9763 100644
--- a/linguistic/source/lngsvcmgr.cxx
+++ b/linguistic/source/lngsvcmgr.cxx
@@ -744,7 +744,7 @@
 const OUString *pNodeName = aNodeNames.getConstArray();
 for (i = 0;  i  nNodeNames;  ++i)
 {
-Locale aLocale( LanguageTag( pNodeName[i]).getLocale() );
+Locale aLocale( (LanguageTag(pNodeName[i])).getLocale() );
 Sequence OUString  aCfgSvcs( getConfiguredServices( aService, 
aLocale ));
 Sequence OUString  aAvailSvcs( getAvailableServices( aService, 
aLocale ));
 
@@ -763,7 +763,7 @@
 const Locale *pAvailLocale = aAvailLocales.getConstArray();
 for (i = 0;  i  nAvailLocales;  ++i)
 {
-OUString aCfgLocaleStr( LanguageTag( pAvailLocale[i] ).getBcp47() 
);
+OUString aCfgLocaleStr( (LanguageTag(pAvailLocale[i])).getBcp47() 
);
 
 Sequence OUString  aAvailSvcs( getAvailableServices( aService, 
pAvailLocale[i] ));
 
@@ -863,7 +863,7 @@
 if (0 == rName.compareTo( aSpellCheckerList, 
aSpellCheckerList.getLength() ))
 {
 // delete old cached data, needs to be acquired new on demand
-   clearSvcInfoArray(pAvailSpellSvcs);
+clearSvcInfoArray(pAvailSpellSvcs);
 
 OUString aNode( aSpellCheckerList );
 if (lcl_SeqHasString( aSpellCheckerListEntries, aKeyText ))
@@ -888,7 +888,7 @@
 else if (0 == rName.compareTo( aGrammarCheckerList, 
aGrammarCheckerList.getLength() ))
 {
 // delete old cached data, needs to be acquired new on demand
-   clearSvcInfoArray(pAvailGrammarSvcs);
+clearSvcInfoArray(pAvailGrammarSvcs);
 
 OUString aNode( aGrammarCheckerList );
 if (lcl_SeqHasString( aGrammarCheckerListEntries, aKeyText ))
@@ -916,7 +916,7 @@
 else if (0 == rName.compareTo( aHyphenatorList, 
aHyphenatorList.getLength() ))
 {
 // delete old cached data, needs to be acquired new on demand
-   clearSvcInfoArray(pAvailHyphSvcs);
+clearSvcInfoArray(pAvailHyphSvcs);
 
 OUString aNode( aHyphenatorList );
 if (lcl_SeqHasString( aHyphenatorListEntries, aKeyText ))
@@ -941,7 +941,7 @@
 else if (0 == rName.compareTo( aThesaurusList, 
aThesaurusList.getLength() ))
 {
 // delete old cached data, needs to be acquired new on demand
-   clearSvcInfoArray(pAvailThesSvcs);
+clearSvcInfoArray(pAvailThesSvcs);
 
 OUString aNode( aThesaurusList );
 if (lcl_SeqHasString( aThesaurusListEntries, aKeyText ))
@@ -1840,7 +1840,7 @@
 aCfgAny = aSvcImplNames;
 DBG_ASSERT( aCfgAny.hasValue(), missing value for 'Any' type );
 
-OUString aCfgLocaleStr( LanguageTag( pLocale[i] ).getBcp47() );
+OUString aCfgLocaleStr( (LanguageTag(pLocale[i])).getBcp47() );
 pValue-Value = aCfgAny;
 pValue-Name  = aNodeName;
 pValue-Name += OUString::valueOf( (sal_Unicode) '/' );

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4d8b979df0fc827758b1e9a336031671425486ea
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: LeMoyne Castle lemoyne.cas...@gmail.com

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


Re: linux / solaris vs other *nix file handling in sal/osl/unx/file.c

2013-01-02 Thread Riccardo Magliocchetti

Il 01/01/2013 14:36, Riccardo Magliocchetti ha scritto:

Hello,

does anyone know why linux and solaris are using pread / pwrite syscalls
while other unixes are using read + lseek? The code is at least
FileHandle_Impl::writeAt which AFAIU is the base for the osl_* file
functions.


[snip]

If the idea is sound i can cook a patch, but i'd like to have some
feedback from *BSD / MacOS X people that will compile the new code path.


Patch attached if anyone on these plaforms want to give it a try.

thanks,
riccardo
From b574788b119f62e0446eb66d78cf99c21f1ca58f Mon Sep 17 00:00:00 2001
From: Riccardo Magliocchetti riccardo.magliocche...@gmail.com
Date: Wed, 2 Jan 2013 11:54:08 +0100
Subject: [PATCH] sal: use pread / pwrite on every *nix

pread / pwrite don't look as a Linux and Solaris privilege.

Change-Id: Ifb2e88445d4064c13a406007bfd523ae0caa38e5
---
 sal/osl/unx/file.cxx |   36 
 1 file changed, 36 deletions(-)

diff --git a/sal/osl/unx/file.cxx b/sal/osl/unx/file.cxx
index 2f1a99a..d1840a3 100644
--- a/sal/osl/unx/file.cxx
+++ b/sal/osl/unx/file.cxx
@@ -379,8 +379,6 @@ oslFileError FileHandle_Impl::readAt (
 return osl_File_E_None;
 }
 
-#if defined(LINUX) || defined(SOLARIS)
-
 ssize_t nBytes = ::pread (m_fd, pBuffer, nBytesRequested, nOffset);
 if ((-1 == nBytes)  (EOVERFLOW == errno))
 {
@@ -393,22 +391,6 @@ oslFileError FileHandle_Impl::readAt (
 if (-1 == nBytes)
 return oslTranslateFileError (OSL_FET_ERROR, errno);
 
-#else /* !(LINUX || SOLARIS) */
-
-if (nOffset != m_offset)
-{
-if (-1 == ::lseek (m_fd, nOffset, SEEK_SET))
-return oslTranslateFileError (OSL_FET_ERROR, errno);
-m_offset = nOffset;
-}
-
-ssize_t nBytes = ::read (m_fd, pBuffer, nBytesRequested);
-if (-1 == nBytes)
-return oslTranslateFileError (OSL_FET_ERROR, errno);
-m_offset += nBytes;
-
-#endif /* !(LINUX || SOLARIS) */
-
 OSL_FILE_TRACE(FileHandle_Impl::readAt(%d, %lld, %ld), m_fd, nOffset, nBytes);
 *pBytesRead = nBytes;
 return osl_File_E_None;
@@ -428,28 +410,10 @@ oslFileError FileHandle_Impl::writeAt (
 if (!(m_state  STATE_WRITEABLE))
 return osl_File_E_BADF;
 
-#if defined(LINUX) || defined(SOLARIS)
-
 ssize_t nBytes = ::pwrite (m_fd, pBuffer, nBytesToWrite, nOffset);
 if (-1 == nBytes)
 return oslTranslateFileError (OSL_FET_ERROR, errno);
 
-#else /* !(LINUX || SOLARIS) */
-
-if (nOffset != m_offset)
-{
-if (-1 == ::lseek (m_fd, nOffset, SEEK_SET))
-return oslTranslateFileError (OSL_FET_ERROR, errno);
-m_offset = nOffset;
-}
-
-ssize_t nBytes = ::write (m_fd, pBuffer, nBytesToWrite);
-if (-1 == nBytes)
-return oslTranslateFileError (OSL_FET_ERROR, errno);
-m_offset += nBytes;
-
-#endif /* !(LINUX || SOLARIS) */
-
 OSL_FILE_TRACE(FileHandle_Impl::writeAt(%d, %lld, %ld), m_fd, nOffset, nBytes);
 m_size = std::max (m_size, sal::static_int_cast sal_uInt64 (nOffset + nBytes));
 
-- 
1.7.10.4

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


[Libreoffice-commits] .: linguistic/source

2013-01-02 Thread Libreoffice Gerrit user
 linguistic/source/lngsvcmgr.cxx |   14 +++---
 1 file changed, 7 insertions(+), 7 deletions(-)

New commits:
commit 252136551f2c032b62f9650a06389f2b4fe6e6c1
Author: LeMoyne Castle lemoyne.cas...@gmail.com
Date:   Tue Jan 1 12:47:35 2013 -0700

Paren fix for Ubu 10.04 build (and fdo#58417 ?)

The form: OuterCtor( InnerCtor(array[i]).method() )
with explicit ctor(s), unused optional arg on ctor+method
-- error 'parameter can not have variably modified type'
gcc version 4.4.3 (Ubuntu 4.4.3-4ubuntu5.1).
Min fix: force precedence/associativity in type resolution.
fdo#58417 BTs inside 1 of these ctor calls.

Change-Id: I4d8b979df0fc827758b1e9a336031671425486ea
Reviewed-on: https://gerrit.libreoffice.org/1531
Reviewed-by: Tor Lillqvist t...@iki.fi
Tested-by: Tor Lillqvist t...@iki.fi

diff --git a/linguistic/source/lngsvcmgr.cxx b/linguistic/source/lngsvcmgr.cxx
index 4f68407..a8e9763 100644
--- a/linguistic/source/lngsvcmgr.cxx
+++ b/linguistic/source/lngsvcmgr.cxx
@@ -744,7 +744,7 @@ void LngSvcMgr::UpdateAll()
 const OUString *pNodeName = aNodeNames.getConstArray();
 for (i = 0;  i  nNodeNames;  ++i)
 {
-Locale aLocale( LanguageTag( pNodeName[i]).getLocale() );
+Locale aLocale( (LanguageTag(pNodeName[i])).getLocale() );
 Sequence OUString  aCfgSvcs( getConfiguredServices( aService, 
aLocale ));
 Sequence OUString  aAvailSvcs( getAvailableServices( aService, 
aLocale ));
 
@@ -763,7 +763,7 @@ void LngSvcMgr::UpdateAll()
 const Locale *pAvailLocale = aAvailLocales.getConstArray();
 for (i = 0;  i  nAvailLocales;  ++i)
 {
-OUString aCfgLocaleStr( LanguageTag( pAvailLocale[i] ).getBcp47() 
);
+OUString aCfgLocaleStr( (LanguageTag(pAvailLocale[i])).getBcp47() 
);
 
 Sequence OUString  aAvailSvcs( getAvailableServices( aService, 
pAvailLocale[i] ));
 
@@ -863,7 +863,7 @@ void LngSvcMgr::Notify( const uno::Sequence OUString  
rPropertyNames )
 if (0 == rName.compareTo( aSpellCheckerList, 
aSpellCheckerList.getLength() ))
 {
 // delete old cached data, needs to be acquired new on demand
-   clearSvcInfoArray(pAvailSpellSvcs);
+clearSvcInfoArray(pAvailSpellSvcs);
 
 OUString aNode( aSpellCheckerList );
 if (lcl_SeqHasString( aSpellCheckerListEntries, aKeyText ))
@@ -888,7 +888,7 @@ void LngSvcMgr::Notify( const uno::Sequence OUString  
rPropertyNames )
 else if (0 == rName.compareTo( aGrammarCheckerList, 
aGrammarCheckerList.getLength() ))
 {
 // delete old cached data, needs to be acquired new on demand
-   clearSvcInfoArray(pAvailGrammarSvcs);
+clearSvcInfoArray(pAvailGrammarSvcs);
 
 OUString aNode( aGrammarCheckerList );
 if (lcl_SeqHasString( aGrammarCheckerListEntries, aKeyText ))
@@ -916,7 +916,7 @@ void LngSvcMgr::Notify( const uno::Sequence OUString  
rPropertyNames )
 else if (0 == rName.compareTo( aHyphenatorList, 
aHyphenatorList.getLength() ))
 {
 // delete old cached data, needs to be acquired new on demand
-   clearSvcInfoArray(pAvailHyphSvcs);
+clearSvcInfoArray(pAvailHyphSvcs);
 
 OUString aNode( aHyphenatorList );
 if (lcl_SeqHasString( aHyphenatorListEntries, aKeyText ))
@@ -941,7 +941,7 @@ void LngSvcMgr::Notify( const uno::Sequence OUString  
rPropertyNames )
 else if (0 == rName.compareTo( aThesaurusList, 
aThesaurusList.getLength() ))
 {
 // delete old cached data, needs to be acquired new on demand
-   clearSvcInfoArray(pAvailThesSvcs);
+clearSvcInfoArray(pAvailThesSvcs);
 
 OUString aNode( aThesaurusList );
 if (lcl_SeqHasString( aThesaurusListEntries, aKeyText ))
@@ -1840,7 +1840,7 @@ sal_Bool LngSvcMgr::SaveCfgSvcs( const String 
rServiceName )
 aCfgAny = aSvcImplNames;
 DBG_ASSERT( aCfgAny.hasValue(), missing value for 'Any' type );
 
-OUString aCfgLocaleStr( LanguageTag( pLocale[i] ).getBcp47() );
+OUString aCfgLocaleStr( (LanguageTag(pLocale[i])).getBcp47() );
 pValue-Value = aCfgAny;
 pValue-Name  = aNodeName;
 pValue-Name += OUString::valueOf( (sal_Unicode) '/' );
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED] Paren fix for Ubu 10.04 build (and fdo#58417 ?)

2013-01-02 Thread Tor Lillqvist (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/1531

Approvals:
  Tor Lillqvist: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4d8b979df0fc827758b1e9a336031671425486ea
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: LeMoyne Castle lemoyne.cas...@gmail.com
Gerrit-Reviewer: Tor Lillqvist t...@iki.fi

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


[PATCH] fix 'DEBUGCPPUNIT=TRUE make subsequenttest'

2013-01-02 Thread Noel Grandin (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1532

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/32/1532/1

fix 'DEBUGCPPUNIT=TRUE make subsequenttest'

When running a unit test under GDB, we do not want to prefix the
cppunittester executable with the LD_LIBRARY_PATH, because that has
already happened before the gdb --args part of the command line.

Change-Id: If3f81ba3fc3e5260142d7e9c2d4a78e9ca63382c
---
M solenv/gbuild/CppunitTest.mk
1 file changed, 7 insertions(+), 1 deletion(-)



diff --git a/solenv/gbuild/CppunitTest.mk b/solenv/gbuild/CppunitTest.mk
index 9c145b6..47c8687 100644
--- a/solenv/gbuild/CppunitTest.mk
+++ b/solenv/gbuild/CppunitTest.mk
@@ -53,7 +53,13 @@
 # DBGSV_ERROR_OUT = in non-product builds, ensure that tools-based assertions 
do not pop up as message box, but are routed to the shell
 ifneq ($(CROSS_COMPILING),YES)
 gb_CppunitTest_CPPTESTDEPS := $(call 
gb_Executable_get_runtime_dependencies,cppunit/cppunittester)
-gb_CppunitTest_CPPTESTCOMMAND := $(call 
gb_Executable_get_command,cppunit/cppunittester)
+ifeq ($(strip $(DEBUGCPPUNIT)),TRUE)
+gb_CppunitTest_CPPTESTCOMMAND := $(call 
gb_Executable_get_target_for_build,cppunit/cppunittester)
+else ifneq ($(strip $(GDBCPPUNITTRACE)),)
+gb_CppunitTest_CPPTESTCOMMAND := $(call 
gb_Executable_get_target_for_build,cppunit/cppunittester)
+else
+gb_CppunitTest_CPPTESTCOMMAND := $(gb_Helper_set_ld_path) $(call 
gb_Executable_get_target_for_build,cppunit/cppunittester)
+endif
 endif
 
 gb_CppunitTest__get_linktargetname = CppunitTest/$(call 
gb_CppunitTest_get_filename,$(1))

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If3f81ba3fc3e5260142d7e9c2d4a78e9ca63382c
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Noel Grandin noelgran...@gmail.com

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


[Libreoffice-commits] .: solenv/gbuild

2013-01-02 Thread Libreoffice Gerrit user
 solenv/gbuild/CppunitTest.mk |8 +++-
 1 file changed, 7 insertions(+), 1 deletion(-)

New commits:
commit 265feeb31ffc08cd4ccbe48398cd41d9cf1e2a25
Author: Noel Grandin n...@peralex.com
Date:   Wed Jan 2 13:32:09 2013 +0200

fix 'DEBUGCPPUNIT=TRUE make subsequenttest'

When running a unit test under GDB, we do not want to prefix the
cppunittester executable with the LD_LIBRARY_PATH, because that has
already happened before the gdb --args part of the command line.

Change-Id: If3f81ba3fc3e5260142d7e9c2d4a78e9ca63382c
Reviewed-on: https://gerrit.libreoffice.org/1532
Reviewed-by: Tor Lillqvist t...@iki.fi
Tested-by: Tor Lillqvist t...@iki.fi

diff --git a/solenv/gbuild/CppunitTest.mk b/solenv/gbuild/CppunitTest.mk
index 9c145b6..47c8687 100644
--- a/solenv/gbuild/CppunitTest.mk
+++ b/solenv/gbuild/CppunitTest.mk
@@ -53,7 +53,13 @@ endif
 # DBGSV_ERROR_OUT = in non-product builds, ensure that tools-based assertions 
do not pop up as message box, but are routed to the shell
 ifneq ($(CROSS_COMPILING),YES)
 gb_CppunitTest_CPPTESTDEPS := $(call 
gb_Executable_get_runtime_dependencies,cppunit/cppunittester)
-gb_CppunitTest_CPPTESTCOMMAND := $(call 
gb_Executable_get_command,cppunit/cppunittester)
+ifeq ($(strip $(DEBUGCPPUNIT)),TRUE)
+gb_CppunitTest_CPPTESTCOMMAND := $(call 
gb_Executable_get_target_for_build,cppunit/cppunittester)
+else ifneq ($(strip $(GDBCPPUNITTRACE)),)
+gb_CppunitTest_CPPTESTCOMMAND := $(call 
gb_Executable_get_target_for_build,cppunit/cppunittester)
+else
+gb_CppunitTest_CPPTESTCOMMAND := $(gb_Helper_set_ld_path) $(call 
gb_Executable_get_target_for_build,cppunit/cppunittester)
+endif
 endif
 
 gb_CppunitTest__get_linktargetname = CppunitTest/$(call 
gb_CppunitTest_get_filename,$(1))
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: configure.ac

2013-01-02 Thread Libreoffice Gerrit user
 configure.ac |6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

New commits:
commit 8620f8b51dcc2c9f50364d42e756fd8b11bdc1e3
Author: Luboš Luňák l.lu...@suse.cz
Date:   Wed Jan 2 13:09:00 2013 +0100

fix use of MINGW_SYSROOT

This got broken in a084ea60680372efb7998f7369c9fc99eb85c50a.

diff --git a/configure.ac b/configure.ac
index 68723f4..1e6327a 100644
--- a/configure.ac
+++ b/configure.ac
@@ -7529,9 +7529,9 @@ if test $enable_python = system; then
 dnl How to find out the cross-compilation Python installation path?
 dnl Let's hardocode what we know for different distributions for now...
 for python_version in 2.6; do
-if test -f 
${MINGW_SYSROOT}/include/python${python_version}/Python.h; then
-
PYTHON_CFLAGS=-I${MINGW_SYSROOT}/include/python$python_version
-PYTHON_LIBS=-L${MINGW_SYSROOT}lib -lpython$python_version 
$python_libs
+if test -f 
${MINGW_SYSROOT}/mingw/include/python${python_version}/Python.h; then
+
PYTHON_CFLAGS=-I${MINGW_SYSROOT}/mingw/include/python$python_version
+PYTHON_LIBS=-L${MINGW_SYSROOT}/mingw/lib 
-lpython$python_version $python_libs
 AC_MSG_CHECKING([for python.exe])
 AS_IF([test -f $MINGW_SYSROOT/mingw/bin/python.exe],
   [AC_MSG_RESULT([$MINGW_SYSROOT/mingw/bin/python.exe])
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PUSHED] fix 'DEBUGCPPUNIT=TRUE make subsequenttest'

2013-01-02 Thread Tor Lillqvist (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/1532

Approvals:
  Tor Lillqvist: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: If3f81ba3fc3e5260142d7e9c2d4a78e9ca63382c
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Noel Grandin noelgran...@gmail.com
Gerrit-Reviewer: Tor Lillqvist t...@iki.fi

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


[Bug 54157] LibreOffice 3.7/4.0 most annoying bugs

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=54157

--- Comment #66 from mike.h...@onepoyle.net ---
Would like to suggest adding https://bugs.freedesktop.org/show_bug.cgi?id=58943

This regression results in significant visual differences where a user has
depended on derived formats for calculated cells. The intention in 4.0.0.0B2 is
obviously to follow what happened in 3.6.n.n as the expected derived formats do
appear when sheet layout is changed - see issue report for details

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


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

2013-01-02 Thread Libreoffice Gerrit user
 vcl/win/source/gdi/winlayout.cxx |9 +
 1 file changed, 9 insertions(+)

New commits:
commit 49dcb4794c838d5f1cf61060f216fc67f36c1618
Author: Michael Stahl mst...@redhat.com
Date:   Sat Dec 22 01:53:09 2012 +0100

fdo#47553: UniscribeLayout: adjust mnSubStringMin

... to actual minimum so that the fix for fdo#33090 in
UniscribeLayout::GetNextGlyphs subtracts exactly the right number.

The original value of mnSubStringMin is guessed and may be up to 8
smaller than the actual start of the first VisualItem.

For the bugdoc it happens that sometimes it's too small by 8 and then
the wrong entries in mpGlyphs2Chars are initialized and then apparently
there are single glyphs that represent multiple characters and so
GetNextGlyphs returns a -1 character which is invalid.

 -GetNextGlyphs dir 1 36 mnSubStringMin 28
 -GetNextGlyphs g2c #1 [8] = 36
 -GetNextGlyphs g2c #1 [9] = 37
 -GetNextGlyphs g2c #1 [10] = 38
 -GetNextGlyphs g2c #1 [11] = 39
 -GetNextGlyphs g2c #1 [12] = 40
 -GetNextGlyphs g2c #2 [4] = 40
 -GetNextGlyphs g2c #2 [3] = 39
 -GetNextGlyphs g2c #2 [1] = 38
 -GetNextGlyphs g2c #2 [1] = 37
 -GetNextGlyphs g2c #2 [0] = 36
...
 -GetNextGlyphs init nCharPos -1
 -GetNextGlyphs g2c [2]  nCharPos -1
 -GetNextGlyphs set pCharPosAry -1
layout[0]-GetNextGlyphs 768,1024 a1800 c-1 0

Change-Id: Ie33ec797a412aa898bec3f4e8f97b88dcfed4d11
(cherry picked from commit cec68bceba9aa1e984d74897fcd7bf4db702d14b)
Reviewed-on: https://gerrit.libreoffice.org/1467
Reviewed-by: Miklos Vajna vmik...@suse.cz
Tested-by: Miklos Vajna vmik...@suse.cz

diff --git a/vcl/win/source/gdi/winlayout.cxx b/vcl/win/source/gdi/winlayout.cxx
index 1df1a75..4ebdd06 100644
--- a/vcl/win/source/gdi/winlayout.cxx
+++ b/vcl/win/source/gdi/winlayout.cxx
@@ -1339,6 +1339,12 @@ bool UniscribeLayout::LayoutText( ImplLayoutArgs rArgs )
 {
 for( int j = rVisualItem.mnMinCharPos; j  
rVisualItem.mnEndCharPos; ++j )
 mpLogClusters[j] = sal::static_int_castWORD(~0U);
+if (rArgs.mnMinCharPos = rVisualItem.mnEndCharPos)
+{   // fdo#47553 adjust guessed min (maybe up to -8 off) to
+// actual min so it can be used properly in GetNextGlyphs
+assert(mnSubStringMin = rVisualItem.mnEndCharPos);
+mnSubStringMin = rVisualItem.mnEndCharPos;
+}
 continue;
 }
 
@@ -1793,7 +1799,10 @@ int UniscribeLayout::GetNextGlyphs( int nLen, 
sal_GlyphId* pGlyphs, Point rPos,
 int nGlyphWidth = pGlyphWidths[ nStart ];
 int nCharPos = -1;// no need to determine charpos
 if( mpGlyphs2Chars )  // unless explicitly requested+provided
+{
 nCharPos = mpGlyphs2Chars[ nStart ];
+assert(-1 != nCharPos);
+}
 
 // inject kashida glyphs if needed
 if( !mbDisableGlyphInjection
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PATCH] Stop disabling graphite on non-WINNT and non-Linux platforms

2013-01-02 Thread Francois Tigeot
configure.ac checks if the operating system family is Linux or WINNT and
refuses to enable graphite support if this is not the case.

I'm using graphite on DragonFly and FreeBSD ports also enable it with a local
patch; the Linux or WINNT check should be removed.

Patch vs -master attached.

-- 
Francois Tigeot
From 9d033067bb7854c19b36bc21cba230eaa3d5815c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fran=C3=A7ois=20Tigeot?= ftig...@wolfpond.org
Date: Wed, 2 Jan 2013 13:06:11 +0100
Subject: [PATCH] configure: graphite can be used on non-Linux, non-WINNT
 platforms

Change-Id: I9e4b041273c07e39824e173ee06f125d2fbf937c
---
 configure.ac | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/configure.ac b/configure.ac
index 68723f4..9eefb03 100644
--- a/configure.ac
+++ b/configure.ac
@@ -8445,7 +8445,7 @@ dnl Graphite
 dnl ===
 
 AC_MSG_CHECKING([whether to enable graphite support])
-if test $_os = WINNT -o $_os = Linux  test $enable_graphite =  
-o $enable_graphite != no; then
+if test $enable_graphite =  -o $enable_graphite != no; then
 AC_MSG_RESULT([yes])
 ENABLE_GRAPHITE=TRUE
 AC_DEFINE(ENABLE_GRAPHITE)
-- 
1.7.12

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


[PUSHED] Change in core[libreoffice-3-6]: fdo#47553: UniscribeLayout: adjust mnSubStringMin

2013-01-02 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/1467

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie33ec797a412aa898bec3f4e8f97b88dcfed4d11
Gerrit-PatchSet: 2
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


[libreoffice-dev] - ScalcBT.cxx C++ using new uno bootsrap is slow

2013-01-02 Thread Rai, Neeraj
Hi ,

Hope this is the right forum for the question below. If not, please point me in 
the right direction.

I have written a new version of Scalc.java in C++, this time using UNO 
bootstrap mentioned in examples/DevelopersGuide/Components/CppComponents.
The last version I wrote used examples/cpp/DocumentLoader and 
examples/java/Scalc.java. It expected soffice listening on a port and was too 
slow.

Even this version is not as fast as I would have expected, and I would like to 
know what may be done to speed it up.
The 500x12 cell updated 120 times in 60sec. I used gettimeofday to measure time 
for 3 cases writing same amount of data :
A) write 5000 rows of 12 cols   (~60sec)
B) write 5000x12 entries to a single cell   (~40sec)
C) write 10 rows of 50 cols, 120 times  (~60sec)
If I read a file with 100,000x12 cell from disk, it takes 20 seconds - so I am 
hopeful that this example can be speeded up.

The box is running RHEL 6.3, 2 cpu with 4 cores and 16GB of RAM.

The attached files can be dropped into any cpp example dir and built. The 
makefile is copied from DocumentLoaded.
Any pointers/googlable keywords/web links would be highly appreciated.

Thanks
Neeraj


Makefile
Description: Makefile


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


Re: Addin exceptions in SvStream

2013-01-02 Thread Jack Leigh

On 30/12/12 15:41, Marc-André Laverdière wrote:

@Lubos. I started writing a plug-in. I have a problem with compiling,
however. It somehow doesn't find the headers on my system.
This is the autogen.sh I use (I tried with and without --includedir,
and --includedir=/usr/)
./autogen.sh CC=clang CXX=clang++ --includedir=/usr/include/
--enable-compiler-plugins

I get this in the output:
checking clang/AST/RecursiveASTVisitor.h usability... no
checking clang/AST/RecursiveASTVisitor.h presence... no
checking for clang/AST/RecursiveASTVisitor.h... no
configure: error: Cannot find Clang headers to build compiler plugins.

Yet...
$ ls /usr/include/clang/AST/RecursiveASTVisitor.h
/usr/include/clang/AST/RecursiveASTVisitor.h

I am on Linux Mint 14. Any clue what is going on?


IIRC it needs llvm headers that *buntu puts in /usr/lib eg. 
/usr/lib/llvm-3.0/include


Didn't know how I should patch the configure file tho

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


Re: Questions about function lcl_GetMergeRange sc/source/core/data/fillinfo.cxx

2013-01-02 Thread Julien Nabet

On 29/12/2012 19:31, Markus Mohrhard wrote:

Hey,


In  function lcl_GetMergeRange sc/source/core/data/fillinfo.cxx, what's the
use to process bHOver variable in while (bVOver) ?

It is unused. Just copy pasted code from the bHOver case.

Ok, here's the change:
https://gerrit.libreoffice.org/gitweb?p=core.git;a=commitdiff;h=26a0979711f89915508359a9804765d934631ec2



I thought about these too:
- declaring nOverlap before the 2 loops, it would avoid to declare it each
time in the else of both loops.
- bVOver treatment is useless in first loop, only last loop is relevant


Please don't. These two steps fall under micro-optimization. A
compiler is in the end much better in optimizing the code than we can
do manually so unless profiling shows that we have a performance
problem we should write clean and easy code.

Variables should be defined as late as possible and trying to optimise
bVOver will make the code much more complicated for most likely no
measurable gain. If you really want to improve the performance the
right way is to take a slow operation and use callgrind to get an idea
where to optimise at much higher level than these assignments.

Ok then

Thank you Markus for your support!

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


RE: [libreoffice-dev] - ScalcBT.cxx C++ using new uno bootsrap is slow

2013-01-02 Thread Rai, Neeraj
Hi Guys,

I tried to modified my original slow example to use array updates 
(setDataArray) based on the example in the wiki.
http://wiki.openoffice.org/wiki/Calc/API/Programming
When I call setDataArray(arr1), the program terminates with error on screen 
***Error couldn't get sheet
The relevant portion of code is pasted below. I get CellRangeData. I prepare a 
sequence of sequence and call cellrangedata-setDataArray(SofS)

Would someone be able to point out what is wrong here?

Thanks
Neeraj
 portion of attached code giving me problems:

void fillLines (Reference XSpreadsheetDocument  myDoc)
{
printf(Fill the lines\n);

Reference  XSpreadsheets  xSheets = myDoc-getSheets() ;
Reference  XIndexAccess  oIndexSheets ( xSheets, UNO_QUERY);
Any any = oIndexSheets-getByIndex(0);
Reference  XSpreadsheet  xSheet ( any, UNO_QUERY);
//Reference  XCellRange  xCellRange = xSheet-getCellRangeByPosition 
(0, 0, X, Y);
//Reference  XCellRangeData  xCellRangeData (xCellRange, UNO_QUERY);

Reference XModel  rSpreadsheetModel (myDoc, UNO_QUERY);
Reference XInterface  rInterface = 
rSpreadsheetModel-getCurrentSelection();
Reference XCellRange  xCellRange(rInterface,UNO_QUERY);
Reference XCellRangeData xCellRangeData(xCellRange, UNO_QUERY);

Sequence  Sequence  Any   arr1 (X+1);
Sequence  Any   arr2 (Y+1);

for (int ii=0; ii  120; ++ii)
{
for (int jj=0; jj  X; ++jj)
{
for (int kk=0; kk  Y; ++kk)
{
int val = jj+kk+ii+2;
arr2[kk+1] = val;
}
arr1[jj+1] = arr2;
}
printf([%d] b4 set data array\n, ii);
xCellRangeData-setDataArray(arr1);
printf([%d] set data array\n, ii);
}


//***
}


-Original Message-
From: Rai, Neeraj [ICG-MKTS]
Sent: Friday, December 28, 2012 3:56 PM
To: 'libreoffice@lists.freedesktop.org'
Subject: [libreoffice-dev] - ScalcBT.cxx C++ using new uno bootsrap is slow

Hi ,

Hope this is the right forum for the question below. If not, please point me in 
the right direction.

I have written a new version of Scalc.java in C++, this time using UNO 
bootstrap mentioned in examples/DevelopersGuide/Components/CppComponents.
The last version I wrote used examples/cpp/DocumentLoader and 
examples/java/Scalc.java. It expected soffice listening on a port and was too 
slow.

Even this version is not as fast as I would have expected, and I would like to 
know what may be done to speed it up.
The 500x12 cell updated 120 times in 60sec. I used gettimeofday to measure time 
for 3 cases writing same amount of data :
A) write 5000 rows of 12 cols   (~60sec)
B) write 5000x12 entries to a single cell   (~40sec)
C) write 10 rows of 50 cols, 120 times  (~60sec)
If I read a file with 100,000x12 cell from disk, it takes 20 seconds - so I am 
hopeful that this example can be speeded up.

The box is running RHEL 6.3, 2 cpu with 4 cores and 16GB of RAM.

The attached files can be dropped into any cpp example dir and built. The 
makefile is copied from DocumentLoaded.
Any pointers/googlable keywords/web links would be highly appreciated.

Thanks
Neeraj


Makefile
Description: Makefile


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


Re: linux / solaris vs other *nix file handling in sal/osl/unx/file.c

2013-01-02 Thread Francois Tigeot
Hi,

On Wed, Jan 02, 2013 at 12:03:16PM +0100, Riccardo Magliocchetti wrote:
 Il 01/01/2013 14:36, Riccardo Magliocchetti ha scritto:
 
 does anyone know why linux and solaris are using pread / pwrite syscalls
 while other unixes are using read + lseek? The code is at least
 FileHandle_Impl::writeAt which AFAIU is the base for the osl_* file
 functions.
 
 Patch attached if anyone on these plaforms want to give it a try.

I tried a patched out LibreOffice-3.6.4 on DragonFly; it worked like a charm.
Reading and opening files even seemed a bit faster.

There is absolutely no justification for the linux+solaris check. HPUX
support has been removed along with OS/2 and other dead or dying platforms
anyway.

I encountered many #if defined(OPERATING_SYSTEM_FOO) checks while porting
LibreOffice to a new platform and can't help to think they were badly written
in the first place.
They should have checked for particular features if anything, not operating
system names.

configure.ac contains at least one similar bug; I'll send patches in another
thread.

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


Wrong platform check for Xinerama support

2013-01-02 Thread Francois Tigeot
configure.in / configure.ac also contains some fishy platform checks in
the Xinerama support check:

...
elif test $_os = Linux -o $_os = FreeBSD; then
if test $x_libraries = default_x_libraries; then
XINERAMALIB=`$PKG_CONFIG --variable=libdir xinerama`
...

I'm not too sure of why this line was added but Linux and FreeBSD are not
the only operating systems which can use Xorg.

Maybe this should be changed to something like this ?

elif test $_os != WINNT; then

I'm reluctant to add all current Unix operating systems using Xorg to the
elif line. Does someone know what was the intended goal for this check ?

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


[Libreoffice-commits] .: solenv/gbuild

2013-01-02 Thread Libreoffice Gerrit user
 solenv/gbuild/UI.mk |   34 +-
 1 file changed, 29 insertions(+), 5 deletions(-)

New commits:
commit 76091d814ca084ba66fcb9db8bc4565c47ee16a5
Author: David Tardon dtar...@redhat.com
Date:   Wed Jan 2 13:51:20 2013 +0100

only package .ui translations that can be produced

That means that there is corresponding .po file for given language.

As a slight optimization, do not even try to run uiex if _no_
translation would be produced.

Change-Id: I901e88b20edfb55e4f8bc661bacf8218b603bdcb

diff --git a/solenv/gbuild/UI.mk b/solenv/gbuild/UI.mk
index 16624813..5e28464 100644
--- a/solenv/gbuild/UI.mk
+++ b/solenv/gbuild/UI.mk
@@ -148,6 +148,34 @@ $(call 
gb_UI__package_uifile,$(1),$(1)_ui_localized,res/$(3)/$(notdir $(2)),$(2)
 
 endef
 
+# gb_UI__add_translations_impl target uifile langs
+define gb_UI__add_translations_impl
+$(call gb_UILocalizeTarget_UILocalizeTarget,$(2))
+$(call gb_UI_get_target,$(1)) : $(call gb_UILocalizeTarget_get_target,$(2))
+$(call gb_UI_get_clean_target,$(1)) : $(call 
gb_UILocalizeTarget_get_clean_target,$(2))
+$(call gb_Package_get_preparation_target,$(1)_ui_localized) : $(call 
gb_UILocalizeTarget_get_target,$(2))
+$(foreach lang,$(3),$(call gb_UI__add_uifile_for_lang,$(1),$(2),$(lang)))
+
+endef
+
+# gb_UI__add_translations target uifile langs
+define gb_UI__add_translations
+$(if $(strip $(3)),$(call gb_UI__add_translations_impl,$(1),$(2),$(3)))
+
+endef
+
+# Adds translations for languages that have corresponding .po file
+#
+# gb_UI__add_uifile_translations target uifile
+define gb_UI__add_uifile_translations
+$(call gb_UI__add_translations,$(1),$(2),\
+   $(foreach lang,$(gb_UI_LANGS),\
+   $(if $(wildcard $(gb_POLOCATION)/$(lang)/$(patsubst %/,%,$(dir 
$(2))).po),$(lang)) \
+   ) \
+)
+
+endef
+
 # Adds .ui file to the package
 #
 # The file is relative to $(SRCDIR) and without extension.
@@ -157,11 +185,7 @@ define gb_UI_add_uifile
 $(call gb_UI__add_uifile,$(1),$(2))
 
 ifneq ($(gb_UI_LANGS),)
-$(call gb_UILocalizeTarget_UILocalizeTarget,$(2))
-$(call gb_UI_get_target,$(1)) : $(call gb_UILocalizeTarget_get_target,$(2))
-$(call gb_UI_get_clean_target,$(1)) : $(call 
gb_UILocalizeTarget_get_clean_target,$(2))
-$(call gb_Package_get_preparation_target,$(1)_ui_localized) : $(call 
gb_UILocalizeTarget_get_target,$(2))
-$(foreach lang,$(gb_UI_LANGS),$(call 
gb_UI__add_uifile_for_lang,$(1),$(2),$(lang)))
+$(call gb_UI__add_uifile_translations,$(1),$(2))
 endif
 
 endef
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: Branch 'libreoffice-4-0' - solenv/gbuild

2013-01-02 Thread Libreoffice Gerrit user
 solenv/gbuild/UI.mk |   34 +-
 1 file changed, 29 insertions(+), 5 deletions(-)

New commits:
commit afa240330b51fb3607cfa6390aeb4e085cd1c525
Author: David Tardon dtar...@redhat.com
Date:   Wed Jan 2 13:51:20 2013 +0100

only package .ui translations that can be produced

That means that there is corresponding .po file for given language.

As a slight optimization, do not even try to run uiex if _no_
translation would be produced.

Change-Id: I901e88b20edfb55e4f8bc661bacf8218b603bdcb
(cherry picked from commit 76091d814ca084ba66fcb9db8bc4565c47ee16a5)

Signed-off-by: David Tardon dtar...@redhat.com

diff --git a/solenv/gbuild/UI.mk b/solenv/gbuild/UI.mk
index 00572b0..f6f4e00 100644
--- a/solenv/gbuild/UI.mk
+++ b/solenv/gbuild/UI.mk
@@ -147,6 +147,34 @@ $(call 
gb_UI__package_uifile,$(1),$(1)_ui_localized,res/$(3)/$(notdir $(2)),$(2)
 
 endef
 
+# gb_UI__add_translations_impl target uifile langs
+define gb_UI__add_translations_impl
+$(call gb_UILocalizeTarget_UILocalizeTarget,$(2))
+$(call gb_UI_get_target,$(1)) : $(call gb_UILocalizeTarget_get_target,$(2))
+$(call gb_UI_get_clean_target,$(1)) : $(call 
gb_UILocalizeTarget_get_clean_target,$(2))
+$(call gb_Package_get_preparation_target,$(1)_ui_localized) : $(call 
gb_UILocalizeTarget_get_target,$(2))
+$(foreach lang,$(3),$(call gb_UI__add_uifile_for_lang,$(1),$(2),$(lang)))
+
+endef
+
+# gb_UI__add_translations target uifile langs
+define gb_UI__add_translations
+$(if $(strip $(3)),$(call gb_UI__add_translations_impl,$(1),$(2),$(3)))
+
+endef
+
+# Adds translations for languages that have corresponding .po file
+#
+# gb_UI__add_uifile_translations target uifile
+define gb_UI__add_uifile_translations
+$(call gb_UI__add_translations,$(1),$(2),\
+   $(foreach lang,$(gb_UI_LANGS),\
+   $(if $(wildcard $(gb_POLOCATION)/$(lang)/$(patsubst %/,%,$(dir 
$(2))).po),$(lang)) \
+   ) \
+)
+
+endef
+
 # Adds .ui file to the package
 #
 # The file is relative to $(SRCDIR) and without extension.
@@ -156,11 +184,7 @@ define gb_UI_add_uifile
 $(call gb_UI__add_uifile,$(1),$(2))
 
 ifneq ($(gb_UI_LANGS),)
-$(call gb_UILocalizeTarget_UILocalizeTarget,$(2))
-$(call gb_UI_get_target,$(1)) : $(call gb_UILocalizeTarget_get_target,$(2))
-$(call gb_UI_get_clean_target,$(1)) : $(call 
gb_UILocalizeTarget_get_clean_target,$(2))
-$(call gb_Package_get_preparation_target,$(1)_ui_localized) : $(call 
gb_UILocalizeTarget_get_target,$(2))
-$(foreach lang,$(gb_UI_LANGS),$(call 
gb_UI__add_uifile_for_lang,$(1),$(2),$(lang)))
+$(call gb_UI__add_uifile_translations,$(1),$(2))
 endif
 
 endef
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: [PATCH] Stop disabling graphite on non-WINNT and non-Linux platforms

2013-01-02 Thread Riccardo Magliocchetti

Il 02/01/2013 13:22, Francois Tigeot ha scritto:

configure.ac checks if the operating system family is Linux or WINNT and
refuses to enable graphite support if this is not the case.

I'm using graphite on DragonFly and FreeBSD ports also enable it with a local
patch; the Linux or WINNT check should be removed.

Patch vs -master attached.


+if test $enable_graphite =  -o $enable_graphite != no; then

Isn't the second check enough? What about MacOS X / Android? I suppose 
both don't want graphite.


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


[Bug 54157] LibreOffice 3.7/4.0 most annoying bugs

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=54157

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

   What|Removed |Added

 Depends on||58943

--- Comment #67 from Petr Mladek pmla...@suse.cz ---
(In reply to comment #66)
 Would like to suggest adding
 https://bugs.freedesktop.org/show_bug.cgi?id=58943
 
 This regression results in significant visual differences where a user has
 depended on derived formats for calculated cells. The intention in 4.0.0.0B2
 is obviously to follow what happened in 3.6.n.n as the expected derived
 formats do appear when sheet layout is changed - see issue report for details

Yup, it sounds like an important regression.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: svx/source

2013-01-02 Thread Libreoffice Gerrit user
 svx/source/items/pageitem.cxx |   17 -
 1 file changed, 8 insertions(+), 9 deletions(-)

New commits:
commit ce9deff347ebdb1ab6faced894e8de9979eabb0e
Author: Werner Koerner wk66...@gmail.com
Date:   Sat Dec 29 12:56:02 2012 +0100

Change if-statements to switch

The bit-values do overlap, 3rd and 4th case were never reached.

Change-Id: Iaaf39a11811c4e28c34260a7eab7fa0476165d1b
Reviewed-on: https://gerrit.libreoffice.org/1497
Reviewed-by: Tomáš Chvátal tchva...@suse.cz
Tested-by: Tomáš Chvátal tchva...@suse.cz

diff --git a/svx/source/items/pageitem.cxx b/svx/source/items/pageitem.cxx
index e7062ee..67d491d 100644
--- a/svx/source/items/pageitem.cxx
+++ b/svx/source/items/pageitem.cxx
@@ -84,15 +84,14 @@ int SvxPageItem::operator==( const SfxPoolItem rAttr ) 
const
 
 inline XubString GetUsageText( const sal_uInt16 eU )
 {
-if ( eU  SVX_PAGE_LEFT )
-return SVX_RESSTR(RID_SVXITEMS_PAGE_USAGE_LEFT);
-if ( eU  SVX_PAGE_RIGHT )
-return SVX_RESSTR(RID_SVXITEMS_PAGE_USAGE_RIGHT);
-if ( eU  SVX_PAGE_ALL )
-return SVX_RESSTR(RID_SVXITEMS_PAGE_USAGE_ALL);
-if ( eU  SVX_PAGE_MIRROR )
-return SVX_RESSTR(RID_SVXITEMS_PAGE_USAGE_MIRROR);
-return String();
+switch( eU  0x000f )
+{
+case SVX_PAGE_LEFT  : return SVX_RESSTR(RID_SVXITEMS_PAGE_USAGE_LEFT);
+case SVX_PAGE_RIGHT : return SVX_RESSTR(RID_SVXITEMS_PAGE_USAGE_RIGHT);
+case SVX_PAGE_ALL   : return SVX_RESSTR(RID_SVXITEMS_PAGE_USAGE_ALL);
+case SVX_PAGE_MIRROR: return 
SVX_RESSTR(RID_SVXITEMS_PAGE_USAGE_MIRROR);
+default:  return String();
+}
 }
 
 //
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Bug 54157] LibreOffice 3.7/4.0 most annoying bugs

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=54157

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

   What|Removed |Added

 Depends on||58863

--- Comment #68 from Petr Mladek pmla...@suse.cz ---
(In reply to comment #64)
 Add Bug 58863 - FILEOPEN .xlsx: IF condition not calculated: very basic
 function, additional functions might be affected.

Really added ;-)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[PUSHED] Change if-statements to switch

2013-01-02 Thread 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/1497

Approvals:
  Tomáš Chvátal: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iaaf39a11811c4e28c34260a7eab7fa0476165d1b
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Werner Körner wk66...@gmail.com
Gerrit-Reviewer: Tomáš Chvátal tchva...@suse.cz

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


Re: linux / solaris vs other *nix file handling in sal/osl/unx/file.c

2013-01-02 Thread Riccardo Magliocchetti

Hi,

Il 02/01/2013 12:39, Francois Tigeot ha scritto:

Hi,

On Wed, Jan 02, 2013 at 12:03:16PM +0100, Riccardo Magliocchetti wrote:

Il 01/01/2013 14:36, Riccardo Magliocchetti ha scritto:


does anyone know why linux and solaris are using pread / pwrite syscalls
while other unixes are using read + lseek? The code is at least
FileHandle_Impl::writeAt which AFAIU is the base for the osl_* file
functions.


Patch attached if anyone on these plaforms want to give it a try.


I tried a patched out LibreOffice-3.6.4 on DragonFly; it worked like a charm.
Reading and opening files even seemed a bit faster.


Nice, thanks! Now if only a MacOS X guy shows up...

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


[Libreoffice-commits] .: config_host.mk.in

2013-01-02 Thread Libreoffice Gerrit user
 config_host.mk.in |1 -
 1 file changed, 1 deletion(-)

New commits:
commit ca3aba4cdf766cf33b0dbbe495c9e6f57f069822
Author: Tor Lillqvist t...@iki.fi
Date:   Wed Jan 2 15:59:00 2013 +0200

DBGHELP_DLL is set in download.lst included at the end

Change-Id: I5465c9176b0b53f9a148fe50373f38c71f179746

diff --git a/config_host.mk.in b/config_host.mk.in
index 08f6a85..f99e3e4 100644
--- a/config_host.mk.in
+++ b/config_host.mk.in
@@ -88,7 +88,6 @@ export CXX_X64_BINARY=@CXX_X64_BINARY@
 @x_CXXFLAGS@ export CXXFLAGS=@CXXFLAGS@
 export CXXFLAGS_CXX11=@CXXFLAGS_CXX11@
 export DATADIR=@DATADIR@
-export DBGHELP_DLL=@DBGHELP_DLL@
 export DBUSMENUGTK_CFLAGS=$(gb_SPACE)@DBUSMENUGTK_CFLAGS@
 export DBUSMENUGTK_LIBS=$(gb_SPACE)@DBUSMENUGTK_LIBS@
 export DBUS_CFLAGS=$(gb_SPACE)@DBUS_CFLAGS@
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Bug 44446] LibreOffice 3.6 most annoying bugs

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=6

Joren De Cuyper joren.libreoff...@telenet.be changed:

   What|Removed |Added

 CC||joren.libreoffice@telenet.b
   ||e
 Depends on||54264

--- Comment #191 from Joren De Cuyper joren.libreoff...@telenet.be ---
I add Bug 54264 - FILEOPEN: LO cannot open more than one file at a time from
Mac Finder because of the regression with 3.5 branch. It's quite annoying when
you need to open all files one by one, when you try to open multiple documents
at once.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] .: Branch 'libreoffice-4-0' - sc/source

2013-01-02 Thread Libreoffice Gerrit user
 sc/source/ui/docshell/docfunc.cxx |2 ++
 1 file changed, 2 insertions(+)

New commits:
commit fbbbcdde585636aea0db1741a0a0ddcc7e0358f3
Author: Julien Nabet serval2...@yahoo.fr
Date:   Tue Jan 1 16:23:16 2013 +0100

fdo#47466 FORMATTING: Autoformat empty rows causes app to crash

Thank you John LeMoyne Castle for your support

http://nabble.documentfoundation.org/Wrong-indentation-which-leads-to-segfault-in-sc-source-ui-docshell-docfunc-cxx-td4026726.html
However, the weird thing is I reproduced the crash with master sources but 
didn't with 4.0 branch nor with 3.5.4.2 Debian packages

Change-Id: Ia5366f479a1066106551b77b5a6315fb78e1bf7d

diff --git a/sc/source/ui/docshell/docfunc.cxx 
b/sc/source/ui/docshell/docfunc.cxx
index abcd66d..bbd6c96 100644
--- a/sc/source/ui/docshell/docfunc.cxx
+++ b/sc/source/ui/docshell/docfunc.cxx
@@ -3739,10 +3739,12 @@ bool ScDocFunc::AutoFormat( const ScRange rRange, 
const ScMarkData* pTabMark,
 
 ScMarkData::iterator itr = aMark.begin(), itrEnd = aMark.end();
 for (; itr != itrEnd  *itr  nTabCount; ++itr)
+{
 SetWidthOrHeight( sal_True, 1,nCols, *itr, SC_SIZE_VISOPT, 
STD_EXTRA_WIDTH, false, sal_True);
 SetWidthOrHeight( false,1,nRows, *itr, SC_SIZE_VISOPT, 0, 
false, false);
 rDocShell.PostPaint( 0,0,*itr, MAXCOL,MAXROW,*itr,
 PAINT_GRID | PAINT_LEFT | PAINT_TOP );
+}
 }
 else
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: Branch 'distro/suse/suse-3.6' - sc/source

2013-01-02 Thread Libreoffice Gerrit user
 sc/source/ui/docshell/docfunc.cxx |2 ++
 1 file changed, 2 insertions(+)

New commits:
commit 10536938a1f96c985b6deb6683ff1e1b64d040e7
Author: Julien Nabet serval2...@yahoo.fr
Date:   Tue Jan 1 16:23:16 2013 +0100

fdo#47466 FORMATTING: Autoformat empty rows causes app to crash

Thank you John LeMoyne Castle for your support

http://nabble.documentfoundation.org/Wrong-indentation-which-leads-to-segfault-in-sc-source-ui-docshell-docfunc-cxx-td4026726.html
However, the weird thing is I reproduced the crash with master sources but 
didn't with 4.0 branch nor with 3.5.4.2 Debian packages

Change-Id: Ia5366f479a1066106551b77b5a6315fb78e1bf7d

diff --git a/sc/source/ui/docshell/docfunc.cxx 
b/sc/source/ui/docshell/docfunc.cxx
index c3cb2ca..2ed13d6 100644
--- a/sc/source/ui/docshell/docfunc.cxx
+++ b/sc/source/ui/docshell/docfunc.cxx
@@ -3734,10 +3734,12 @@ bool ScDocFunc::AutoFormat( const ScRange rRange, 
const ScMarkData* pTabMark,
 
 ScMarkData::iterator itr = aMark.begin(), itrEnd = aMark.end();
 for (; itr != itrEnd  *itr  nTabCount; ++itr)
+{
 SetWidthOrHeight( sal_True, 1,nCols, *itr, SC_SIZE_VISOPT, 
STD_EXTRA_WIDTH, false, sal_True);
 SetWidthOrHeight( false,1,nRows, *itr, SC_SIZE_VISOPT, 0, 
false, false);
 rDocShell.PostPaint( 0,0,*itr, MAXCOL,MAXROW,*itr,
 PAINT_GRID | PAINT_LEFT | PAINT_TOP );
+}
 }
 else
 {
___
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

2013-01-02 Thread Libreoffice Gerrit user
 sc/source/ui/docshell/docfunc.cxx |2 ++
 1 file changed, 2 insertions(+)

New commits:
commit 5fe2b0eee8058a64745a7a51fdd2ac5e102300ee
Author: Julien Nabet serval2...@yahoo.fr
Date:   Tue Jan 1 16:23:16 2013 +0100

fdo#47466 FORMATTING: Autoformat empty rows causes app to crash

Thank you John LeMoyne Castle for your support

http://nabble.documentfoundation.org/Wrong-indentation-which-leads-to-segfault-in-sc-source-ui-docshell-docfunc-cxx-td4026726.html
However, the weird thing is I reproduced the crash with master sources but 
didn't with 4.0 branch nor with 3.5.4.2 Debian packages

Change-Id: Ia5366f479a1066106551b77b5a6315fb78e1bf7d

diff --git a/sc/source/ui/docshell/docfunc.cxx 
b/sc/source/ui/docshell/docfunc.cxx
index c3cb2ca..2ed13d6 100644
--- a/sc/source/ui/docshell/docfunc.cxx
+++ b/sc/source/ui/docshell/docfunc.cxx
@@ -3734,10 +3734,12 @@ bool ScDocFunc::AutoFormat( const ScRange rRange, 
const ScMarkData* pTabMark,
 
 ScMarkData::iterator itr = aMark.begin(), itrEnd = aMark.end();
 for (; itr != itrEnd  *itr  nTabCount; ++itr)
+{
 SetWidthOrHeight( sal_True, 1,nCols, *itr, SC_SIZE_VISOPT, 
STD_EXTRA_WIDTH, false, sal_True);
 SetWidthOrHeight( false,1,nRows, *itr, SC_SIZE_VISOPT, 0, 
false, false);
 rDocShell.PostPaint( 0,0,*itr, MAXCOL,MAXROW,*itr,
 PAINT_GRID | PAINT_LEFT | PAINT_TOP );
+}
 }
 else
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: [PUSHED: 4.0, 3.6] fdo#47466 FORMATTING: Autoformat empty rows causes app to crash

2013-01-02 Thread Petr Mladek
julien2412 píše v Út 01. 01. 2013 v 10:37 -0800:
 Hello,
 
 I don't reproduce the problem with 4.0 sources (I don't know why) but I can
 reproduce it with 3.6 sources.

I guess that you are just lucky with 4.0. It probably modifies an
allocated memory, so it does not crash but it might cause another funny
random effects anywhere else.

 Anyway, would it be possible to cherry-pick
 f843850ee9994653673ef5591aae875d7fb22fed ?
 (it's just some 2 lacking braces)

Yup, it makes sense = cherry picked into both 3-6 and 4-0 branches, see
http://cgit.freedesktop.org/libreoffice/core/commit/?h=libreoffice-3-6id=5fe2b0eee8058a64745a7a51fdd2ac5e102300ee
http://cgit.freedesktop.org/libreoffice/core/commit/?h=libreoffice-4-0id=fbbbcdde585636aea0db1741a0a0ddcc7e0358f3


Best Regards,
Petr

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


[Libreoffice-commits] .: configure.ac external/Package_mingw_dlls.mk m4/mingw.m4

2013-01-02 Thread Libreoffice Gerrit user
 configure.ac   |   14 +++---
 external/Package_mingw_dlls.mk |2 +-
 m4/mingw.m4|2 +-
 3 files changed, 9 insertions(+), 9 deletions(-)

New commits:
commit dc277bc6a1c357fe725db2c567d3e1cf16fb7806
Author: Luboš Luňák l.lu...@suse.cz
Date:   Wed Jan 2 17:04:34 2013 +0100

use MINGW_SYSROOT consistently

Restore all cases to expect /mingw/ included in the path.

diff --git a/configure.ac b/configure.ac
index 1e6327a..59e20c0 100644
--- a/configure.ac
+++ b/configure.ac
@@ -5315,8 +5315,8 @@ dnl 
===
 if test $WITH_MINGW = yes; then
 AC_MSG_CHECKING([for MinGW sysroot])
 sysroot=`$CC -print-sysroot`
-AS_IF([test -d $sysroot],
-  [MINGW_SYSROOT=$sysroot
+AS_IF([test -d $sysroot/mingw],
+  [MINGW_SYSROOT=$sysroot/mingw
AC_MSG_RESULT([$MINGW_SYSROOT])],
   [AC_MSG_RESULT([not found])
AC_MSG_ERROR([cannot determine MinGW sysroot])])
@@ -7529,12 +7529,12 @@ if test $enable_python = system; then
 dnl How to find out the cross-compilation Python installation path?
 dnl Let's hardocode what we know for different distributions for now...
 for python_version in 2.6; do
-if test -f 
${MINGW_SYSROOT}/mingw/include/python${python_version}/Python.h; then
-
PYTHON_CFLAGS=-I${MINGW_SYSROOT}/mingw/include/python$python_version
-PYTHON_LIBS=-L${MINGW_SYSROOT}/mingw/lib 
-lpython$python_version $python_libs
+if test -f 
${MINGW_SYSROOT}/include/python${python_version}/Python.h; then
+
PYTHON_CFLAGS=-I${MINGW_SYSROOT}/include/python$python_version
+PYTHON_LIBS=-L${MINGW_SYSROOT}/lib -lpython$python_version 
$python_libs
 AC_MSG_CHECKING([for python.exe])
-AS_IF([test -f $MINGW_SYSROOT/mingw/bin/python.exe],
-  [AC_MSG_RESULT([$MINGW_SYSROOT/mingw/bin/python.exe])
+AS_IF([test -f $MINGW_SYSROOT/bin/python.exe],
+  [AC_MSG_RESULT([$MINGW_SYSROOT/bin/python.exe])
MINGW_PYTHON_EXE=python.exe],
   [AC_MSG_RESULT([not found])
AC_MSG_ERROR([could not find python.exe])])
diff --git a/external/Package_mingw_dlls.mk b/external/Package_mingw_dlls.mk
index 6b613d1..50a8cda 100644
--- a/external/Package_mingw_dlls.mk
+++ b/external/Package_mingw_dlls.mk
@@ -7,7 +7,7 @@
 # file, You can obtain one at http://mozilla.org/MPL/2.0/.
 #
 
-$(eval $(call gb_Package_Package,mingw_dlls,$(MINGW_SYSROOT)/mingw/bin))
+$(eval $(call gb_Package_Package,mingw_dlls,$(MINGW_SYSROOT)/bin))
 
 $(eval $(call gb_Package_add_files,mingw_dlls,bin,\
 $(MINGW_BOOST_DATE_TIME_DLL) \
diff --git a/m4/mingw.m4 b/m4/mingw.m4
index eb2dadb..d7aecbd 100644
--- a/m4/mingw.m4
+++ b/m4/mingw.m4
@@ -47,7 +47,7 @@ AC_DEFUN([libo_MINGW_CHECK_DLL],
 [AC_ARG_VAR([MINGW_][$1][_DLL],[output variable containing the found dll 
name])dnl
 
 if test -n $WITH_MINGW; then
-_libo_mingw_dlldir=[$MINGW_SYSROOT]/mingw/bin
+_libo_mingw_dlldir=[$MINGW_SYSROOT]/bin
 _libo_mingw_dllname=
 AC_MSG_CHECKING([for $2 dll])
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: Wrong indentation which leads to segfault in sc/source/ui/docshell/docfunc.cxx

2013-01-02 Thread Lubos Lunak
On Tuesday 01 of January 2013, John LeMoyne Castle wrote:
 Julien,

 Looks to me like a very nice catch ...
...
 The next version (2011.07.05)
 http://opengrok.libreoffice.org/xref/core/sc/source/ui/docshell/docfunc.cxx
?r=2b88f6d32f572792597ccbb15276b9db52db7d10 is a commit labelled
 quot;change remaining manual loops to
 ScMarkData::iterator quot; and the braces disappear on just one of the
 loops.  Because the braces were once there and, especially because the else
 block has ~ same action within the braces, I say that they should
 definitely come back.  Even though, I am not at all clear what this code is
 doing (whilst autoformatting selected cells in Calc)... this is an obvious
 oopsie within a greater code cleanup that should be corrected.
...
 I note that, for all its type mushiness, BASIC will not allow this kind of
 mistake.  The then-clause must either be all on one line (can do IF test
 THEN StmtA : StmtB : StmtC) and a multi-line then-clause must be terminated
 with End If or Else.

http://tinderbox.libreoffice.org/cgi-bin/gunzip.cgi?tree=MASTERfull-log=1356992414.21678#20458
 :

20458 In file included from stdin:1:
20459 
/home/tinderbox/clang-master-build/sc/source/ui/docshell/docfunc.cxx:3743:17: 
warning: statement aligned as second statement in for body but not in a 
statement block [loplugin]
20460 SetWidthOrHeight( false,1,nRows, *itr, 
SC_SIZE_VISOPT, 0, false, false);
20461 ^
20462 
/home/tinderbox/clang-master-build/sc/source/ui/docshell/docfunc.cxx:3742:17: 
note: for body statement is here [loplugin]
20463 SetWidthOrHeight( sal_True, 1,nCols, *itr, 
SC_SIZE_VISOPT, STD_EXTRA_WIDTH, false, sal_True);
20464 ^
20465 1 warning generated.

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


[Libreoffice-commits] .: config_host.mk.in configure.ac libxmlsec/xmlsec1-configure.patch nss/ExternalPackage_nss.mk nss/ExternalProject_nss.mk nss/Makefile nss/makefile.mk nss/Module_nss.mk nss/nss-c

2013-01-02 Thread Libreoffice Gerrit user
 RepositoryExternal.mk|   10 -
 config_host.mk.in|3 
 configure.ac |6 
 libxmlsec/xmlsec1-configure.patch|   21 ++
 nss/ExternalPackage_nss.mk   |   78 ++
 nss/ExternalProject_nss.mk   |   80 ++
 nss/Makefile |7 
 nss/Module_nss.mk|   22 ++
 nss/UnpackedTarball_nss.mk   |   24 +++
 nss/makefile.mk  |  269 ---
 nss/nss-config.in|  147 +++
 nss/nss-config.patch |  150 ---
 nss/nss.mingw.patch  |  128 
 nss/nss.patch.mingw  |  128 
 nss/prj/build.lst|3 
 nss/prj/d.lst|   34 
 openldap/ExternalProject_openldap.mk |4 
 17 files changed, 523 insertions(+), 591 deletions(-)

New commits:
commit c003d25d24786073229d0f8e545b2863c27fa282
Author: Peter Foley pefol...@verizon.net
Date:   Tue Jan 1 09:39:36 2013 -0500

convert nss to gbuild

Change-Id: I59edc4c437abccc201823d97f1cbec230d273b05
Reviewed-on: https://gerrit.libreoffice.org/1529
Reviewed-by: Luboš Luňák l.lu...@suse.cz
Tested-by: Luboš Luňák l.lu...@suse.cz

diff --git a/RepositoryExternal.mk b/RepositoryExternal.mk
index cb63693..0b3fc76 100644
--- a/RepositoryExternal.mk
+++ b/RepositoryExternal.mk
@@ -2206,9 +2206,7 @@ endif # GUIBASE=unx
 
 ifeq (,$(filter DESKTOP,$(BUILD_TYPE)))
 
-define gb_LinkTarget__use_nss3
-
-endef
+gb_LinkTarget__use_nss3:=
 
 else
 
@@ -2245,10 +2243,11 @@ $(eval $(call 
gb_Helper_register_libraries,PLAINLIBS_OOO,\
 ))
 
 define gb_LinkTarget__use_nss3
+$(call gb_LinkTarget_use_package,$(1),nss)
 $(call gb_LinkTarget_set_include,$(1),\
$$(INCLUDE) \
-   -I$(OUTDIR)/inc/mozilla/nspr \
-   -I$(OUTDIR)/inc/mozilla/nss \
+   -I$(call gb_UnpackedTarball_get_dir,nss)/mozilla/dist/public/nss \
+   -I$(call gb_UnpackedTarball_get_dir,nss)/mozilla/dist/out/include \
 )
 
 $(call gb_LinkTarget_use_libraries,$(1),\
@@ -2260,6 +2259,7 @@ $(call gb_LinkTarget_use_libraries,$(1),\
 endef
 
 define gb_LinkTarget__use_plc4
+$(call gb_LinkTarget_use_package,$(1),nss)
 $(call gb_LinkTarget_use_libraries,$(1),\
 plc4 \
 )
diff --git a/config_host.mk.in b/config_host.mk.in
index f99e3e4..f99205d 100644
--- a/config_host.mk.in
+++ b/config_host.mk.in
@@ -404,6 +404,9 @@ export NEON_LIBS=$(gb_SPACE)@NEON_LIBS@
 export NEON_VERSION=@NEON_VERSION@
 export NM=@NM@
 export nodep=@nodep@
+export NSS_MAJOR=@NSS_MAJOR@
+export NSS_MINOR=@NSS_MINOR@
+export NSS_PATCH=@NSS_PATCH@
 export NSS_CFLAGS=$(gb_SPACE)@NSS_CFLAGS@
 export NSS_LIBS=$(gb_SPACE)@NSS_LIBS@
 export NSSBUILDTOOLS=@NSSBUILDTOOLS@
diff --git a/configure.ac b/configure.ac
index 59e20c0..dbcaf66 100644
--- a/configure.ac
+++ b/configure.ac
@@ -8255,6 +8255,9 @@ if test $with_system_nss = yes; then
 libo_MINGW_CHECK_DLL([SSL3], [ssl3])
 else
 SYSTEM_NSS=NO
+   NSS_MAJOR=3
+   NSS_MINOR=13
+   NSS_PATCH=5
 BUILD_TYPE=$BUILD_TYPE NSS
 AC_MSG_RESULT([internal])
 if test $build_os = cygwin; then
@@ -8275,6 +8278,9 @@ from 
http://ftp.mozilla.org/pub/mozilla.org/mozilla/libraries/win32])
 fi
 fi # system nss
 AC_SUBST(SYSTEM_NSS)
+AC_SUBST(NSS_MAJOR)
+AC_SUBST(NSS_MINOR)
+AC_SUBST(NSS_PATCH)
 AC_SUBST(NSS_CFLAGS)
 AC_SUBST(NSS_LIBS)
 AC_SUBST(NSSBUILDTOOLS)
diff --git a/libxmlsec/xmlsec1-configure.patch 
b/libxmlsec/xmlsec1-configure.patch
index ead9050..2db0426 100644
--- a/libxmlsec/xmlsec1-configure.patch
+++ b/libxmlsec/xmlsec1-configure.patch
@@ -97,10 +97,29 @@
 -ac_nss_lib_dir=/usr/lib /usr/lib64 /usr/local/lib 
/usr/lib/$ac_mozilla_name /usr/local/lib/$ac_mozilla_name
 -ac_nss_inc_dir=/usr/include /usr/include/mozilla /usr/local/include 
/usr/local/include/mozilla /usr/include/$ac_mozilla_name 
/usr/local/include/$ac_mozilla_name
 +ac_nss_lib_dir=${SOLARVERSION}/${INPATH}/lib${UPDMINOREXT}
-+ac_nss_inc_dir=${SOLARVERSION}/${INPATH}/inc${UPDMINOREXT}/mozilla
++ac_nss_inc_dir=${WORKDIR}/UnpackedTarball/nss/mozilla/dist/out/include 
${WORKDIR}/UnpackedTarball/nss/mozilla/dist/public
  
  AC_MSG_CHECKING(for nspr libraries = $NSPR_MIN_VERSION)
  NSPR_INCLUDES_FOUND=no
+@@ -637,15 +660,15 @@
+   NSPR_PRINIT_H=$with_nspr/include/prinit.h
+ else
+   for dir in $ac_nss_inc_dir ; do
+-  if test -f $dir/nspr/prinit.h ; then
++  if test -f $dir/prinit.h ; then
+   dnl do not add -I/usr/include because compiler does it anyway
+   if test z$dir = z/usr/include ; then
+   NSPR_CFLAGS=
+   else
+-  NSPR_CFLAGS=-I$dir/nspr
++  NSPR_CFLAGS=-I$dir
+   fi
+   NSPR_INCLUDES_FOUND=yes
+-  NSPR_PRINIT_H=$dir/nspr/prinit.h
++  

[PUSHED] convert nss to gbuild

2013-01-02 Thread 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/1529

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I59edc4c437abccc201823d97f1cbec230d273b05
Gerrit-PatchSet: 2
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Peter Foley pefol...@verizon.net
Gerrit-Reviewer: Luboš Luňák l.lu...@suse.cz

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


Re: [Libreoffice-commits] .: 2 commits - pyuno/CustomTarget_python_shell.mk pyuno/CustomTarget_pyversion.mk pyuno/Executable_python_wrapper.mk pyuno/Module_pyuno.mk pyuno/zipcore

2013-01-02 Thread Lubos Lunak
On Tuesday 01 of January 2013, Libreoffice Gerrit user wrote:
 commit 085e0adf53baff298059980a54758d81b08bb059
 Author: David Tardon dtar...@redhat.com
 Date:   Tue Jan 1 11:47:56 2013 +0100

 just pass the define through -D

 I am constantly amazed at the creativity of the original makefile
 writers. An extra header file, processed by sed, rather then adding one
 item to CDEFS? Really?

 Please use config_xxx.h files rather than -D flags whenever possible (which, 
interestingly, is an extra header file, processed by sed, rather than adding 
one item to CDEFS).

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


[Libreoffice-commits] .: nss/ExternalPackage_nss.mk

2013-01-02 Thread Libreoffice Gerrit user
 nss/ExternalPackage_nss.mk |   21 +++--
 1 file changed, 19 insertions(+), 2 deletions(-)

New commits:
commit e912ea714ec40c5046d1c1425c1e58fd6e689df1
Author: Luboš Luňák l.lu...@suse.cz
Date:   Wed Jan 2 18:06:18 2013 +0100

try to fix nss on macosx

diff --git a/nss/ExternalPackage_nss.mk b/nss/ExternalPackage_nss.mk
index ae28d98..2d2a9f1 100644
--- a/nss/ExternalPackage_nss.mk
+++ b/nss/ExternalPackage_nss.mk
@@ -16,7 +16,24 @@ $(eval $(call gb_ExternalPackage_add_files,nss,bin,\
config/nss-config \
 ))
 
-ifeq ($(OS),WNT)
+ifeq ($(OS),MACOSX)
+$(eval $(call gb_ExternalPackage_add_files,nss,lib,\
+   mozilla/dist/out/lib/libcrmf.a \
+   mozilla/dist/out/lib/libfreebl3.dylib \
+   mozilla/dist/out/lib/libnspr4.dylib \
+   mozilla/dist/out/lib/libnss3.dylib \
+   mozilla/dist/out/lib/libnssckbi.dylib \
+   mozilla/dist/out/lib/libnssdbm3.dylib \
+   mozilla/dist/out/lib/libnsssysinit.dylib \
+   mozilla/dist/out/lib/libnssutil3.dylib \
+   mozilla/dist/out/lib/libplc4.dylib \
+   mozilla/dist/out/lib/libplds4.dylib \
+   mozilla/dist/out/lib/libsmime3.dylib \
+   mozilla/dist/out/lib/libsoftokn3.dylib \
+   mozilla/dist/out/lib/libsqlite3.dylib \
+   mozilla/dist/out/lib/libssl3.dylib \
+))
+else ifeq ($(OS),WNT)
 ifeq ($(COM),MSC)
 $(eval $(call gb_ExternalPackage_add_files,nss,lib,\
mozilla/dist/out/lib/nspr4.lib \
@@ -56,7 +73,7 @@ $(eval $(call gb_ExternalPackage_add_files,nss,bin,\
mozilla/dist/out/lib/sqlite3.dll \
mozilla/dist/out/lib/ssl3.dll \
 ))
-else # OS!=WNT
+else # OS!=WNT/MACOSX
 $(eval $(call gb_ExternalPackage_add_files,nss,lib,\
mozilla/dist/out/lib/libcrmf.a \
mozilla/dist/out/lib/libfreebl3.so \
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: sal/inc

2013-01-02 Thread Libreoffice Gerrit user
 sal/inc/sal/log-areas.dox |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 49fc6c6b85e4902dc9b776d82ec17ea7c01a8dd1
Author: Miklos Vajna vmik...@suse.cz
Date:   Wed Jan 2 18:07:52 2013 +0100

sal: missing jvmfwk in log-areas

diff --git a/sal/inc/sal/log-areas.dox b/sal/inc/sal/log-areas.dox
index c167267..428c7fb 100644
--- a/sal/inc/sal/log-areas.dox
+++ b/sal/inc/sal/log-areas.dox
@@ -221,6 +221,7 @@ certain functionality.
 @li @c editeng
 @li @c framework
 @li @c helpcompiler
+@li @c jvmfwk
 @li @c linguistic
 @li @c oox
 @li @c rsc
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: configure.ac

2013-01-02 Thread Libreoffice Gerrit user
 configure.ac |   23 ++-
 1 file changed, 22 insertions(+), 1 deletion(-)

New commits:
commit 2210bade74626de68e51adb9759b81f62d7c0b01
Author: Tor Lillqvist t...@iki.fi
Date:   Wed Jan 2 15:18:11 2013 +0200

Check also for a (self-built) 64-bit NDK tool-chain

Also make the --with-android-ndk-toolchain-version option mandatory. 
Otherwise
it would be possible to leave it out and that would appear to work in the
configure phase but in fact mix stuff up when using a self-built 64-bit
tool-chain merged into a r8b,c,d NDK.

Sigh, maybe we shouldn't even try to have the configury generic enough to 
work
with any NDK version (which probably it doesn't anyway), but explicitly 
accept
only a small set of NDKs, like r8b,c,d (which probably are the only that we
work with anyway).

Change-Id: Iecdf3a2d841e7b29598228ae49f48eca9640a82c

diff --git a/configure.ac b/configure.ac
index dbcaf66..af1b786 100644
--- a/configure.ac
+++ b/configure.ac
@@ -154,7 +154,24 @@ if test -n $with_android_ndk; then
 android_cpu=x86
 fi
 
-ANDROID_ABI_PREBUILT_BIN=`echo 
$ANDROID_NDK_HOME/toolchains/$android_cpu*-*$with_android_ndk_toolchain_version/prebuilt/*/bin`
+# Check if there is a (self-built) 64-bit tool-chain
+toolchain_system='*'
+if test $build_os = linux-gnu; then
+case $build_cpu in
+x86_64)
+case `echo 
$ANDROID_NDK_HOME/toolchains/$android_cpu*-*$with_android_ndk_toolchain_version/prebuilt/linux-x86_64/bin`
 in
+*/bin|*/bin\ */bin*)
+toolchain_system=linux-x86_64
+;;
+esac
+;;
+i?86|x86)
+toolchain_system=linux-x86
+;;
+esac
+fi
+
+ANDROID_ABI_PREBUILT_BIN=`echo 
$ANDROID_NDK_HOME/toolchains/$android_cpu*-*$with_android_ndk_toolchain_version/prebuilt/$toolchain_system/bin`
 # Check if there are several toolchain versions
 case $ANDROID_ABI_PREBUILT_BIN in
 */bin\ */bin*)
@@ -489,6 +506,10 @@ linux-android*)
 AC_MSG_ERROR([the --with-android-ndk option is mandatory])
 fi
 
+if test -z $with_android_ndk_toolchain_version; then
+AC_MSG_ERROR([the --with-android-ndk-toolchain-version option is 
mandatory])
+fi
+
 if test -z $with_android_sdk; then
 AC_MSG_ERROR([the --with-android-sdk option is mandatory])
 fi
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: nss/ExternalPackage_nss.mk

2013-01-02 Thread Libreoffice Gerrit user
 nss/ExternalPackage_nss.mk |1 -
 1 file changed, 1 deletion(-)

New commits:
commit 3b1102ad193d55b98c7c72e706173353c35bb1b6
Author: Luboš Luňák l.lu...@suse.cz
Date:   Wed Jan 2 18:25:31 2013 +0100

looks like libnsssysinit is linux-only

diff --git a/nss/ExternalPackage_nss.mk b/nss/ExternalPackage_nss.mk
index 2d2a9f1..ce19727 100644
--- a/nss/ExternalPackage_nss.mk
+++ b/nss/ExternalPackage_nss.mk
@@ -24,7 +24,6 @@ $(eval $(call gb_ExternalPackage_add_files,nss,lib,\
mozilla/dist/out/lib/libnss3.dylib \
mozilla/dist/out/lib/libnssckbi.dylib \
mozilla/dist/out/lib/libnssdbm3.dylib \
-   mozilla/dist/out/lib/libnsssysinit.dylib \
mozilla/dist/out/lib/libnssutil3.dylib \
mozilla/dist/out/lib/libplc4.dylib \
mozilla/dist/out/lib/libplds4.dylib \
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: config_host.mk.in configure.ac i18npool/CustomTarget_breakiterator.mk i18npool/qa i18npool/source

2013-01-02 Thread Libreoffice Gerrit user
 config_host.mk.in   |2 ++
 configure.ac|8 
 i18npool/CustomTarget_breakiterator.mk  |   18 +-
 i18npool/qa/cppunit/test_breakiterator.cxx  |7 +++
 i18npool/source/breakiterator/data/line.txt |3 ++-
 5 files changed, 36 insertions(+), 2 deletions(-)

New commits:
commit 79a3c9b186534c5e3adde2266d3e7fd6527b11b9
Author: Eike Rathke er...@redhat.com
Date:   Wed Jan 2 19:03:51 2013 +0100

partly revert 92a9b7780c6e13a4da3b12794342edbc4c09ef51 for ICU  49

Re-enable build with ICU 4.6 and 4.8
ICU versions prior to 49 don't know Conditional_Japanese_Starter and
Hebrew_Letter

Also, the change in i18npool/CustomTarget_breakiterator.mk

- -e s#\[:LineBreak = Close_Punctuation:\]#\[ \[:LineBreak = 
Close_Parenthesis:\]\]# \

with i18npool/source/breakiterator/data/line.txt

-$CL = [:LineBreak = Close_Punctuation:] ;
+$CL = [:LineBreak = Close_Parenthesis:];

did not produce equivalent results. Instead use

$CP = [:LineBreak =  Close_Parenthesis:];
$CL = [[:LineBreak =  Close_Punctuation:] $CP];

Change-Id: I14fc14319ea34f23393264560452a79bb49fc3a7

diff --git a/config_host.mk.in b/config_host.mk.in
index f99205d..70f8c4e 100644
--- a/config_host.mk.in
+++ b/config_host.mk.in
@@ -251,6 +251,8 @@ export ICU_MAJOR=@ICU_MAJOR@
 export ICU_MICRO=@ICU_MICRO@
 export ICU_MINOR=@ICU_MINOR@
 export ICU_RECLASSIFIED_PREPEND_SET_EMPTY=@ICU_RECLASSIFIED_PREPEND_SET_EMPTY@
+export 
ICU_RECLASSIFIED_CONDITIONAL_JAPANESE_STARTER=@ICU_RECLASSIFIED_CONDITIONAL_JAPANESE_STARTER@
+export ICU_RECLASSIFIED_HEBREW_LETTER=@ICU_RECLASSIFIED_HEBREW_LETTER@
 export ILIB=@ILIB@
 export INPATH=@INPATH@
 export INPATH_FOR_BUILD=@INPATH_FOR_BUILD@
diff --git a/configure.ac b/configure.ac
index af1b786..f3e69ac 100644
--- a/configure.ac
+++ b/configure.ac
@@ -8381,6 +8381,8 @@ ICU_MAJOR=49
 ICU_MINOR=1
 ICU_MICRO=1
 ICU_RECLASSIFIED_PREPEND_SET_EMPTY=YES
+ICU_RECLASSIFIED_CONDITIONAL_JAPANESE_STARTER=YES
+ICU_RECLASSIFIED_HEBREW_LETTER=YES
 AC_MSG_CHECKING([which icu to use])
 if test $with_system_icu = yes; then
 AC_MSG_RESULT([external])
@@ -8442,8 +8444,12 @@ You can use --with-system-icu-for-build=force to use it 
anyway.])
 fi
 if test $ICU_MAJOR -ge 49; then
 ICU_RECLASSIFIED_PREPEND_SET_EMPTY=YES
+ICU_RECLASSIFIED_CONDITIONAL_JAPANESE_STARTER=YES
+ICU_RECLASSIFIED_HEBREW_LETTER=YES
 else
 ICU_RECLASSIFIED_PREPEND_SET_EMPTY=NO
+ICU_RECLASSIFIED_CONDITIONAL_JAPANESE_STARTER=NO
+ICU_RECLASSIFIED_HEBREW_LETTER=NO
 fi
 fi
 
@@ -8463,6 +8469,8 @@ AC_SUBST(ICU_MAJOR)
 AC_SUBST(ICU_MINOR)
 AC_SUBST(ICU_MICRO)
 AC_SUBST(ICU_RECLASSIFIED_PREPEND_SET_EMPTY)
+AC_SUBST(ICU_RECLASSIFIED_CONDITIONAL_JAPANESE_STARTER)
+AC_SUBST(ICU_RECLASSIFIED_HEBREW_LETTER)
 AC_SUBST([MINGW_ICUDATA_DLL])
 AC_SUBST([MINGW_ICUI18N_DLL])
 AC_SUBST([MINGW_ICUUC_DLL])
diff --git a/i18npool/CustomTarget_breakiterator.mk 
b/i18npool/CustomTarget_breakiterator.mk
index a8fe3bd..da3c224 100644
--- a/i18npool/CustomTarget_breakiterator.mk
+++ b/i18npool/CustomTarget_breakiterator.mk
@@ -88,13 +88,29 @@ $(i18npool_BIDIR)/%.brk : $(i18npool_BIDIR)/%.txt $(call 
gb_ExternalExecutable_g
$(call gb_ExternalExecutable_get_command,genbrk) -r $ -o $@ 
$(if $(findstring s,$(MAKEFLAGS)), /dev/null))
 
 # fdo#31271 ) reclassified in more recent Unicode Standards / ICU 4.4
-# * Prepend set empty as of Unicode Version 6.1 / ICU 4.9, which bails out if 
used.
+# * Prepend set empty as of Unicode Version 6.1 / ICU 49, which bails out if 
used.
 #   NOTE: strips every line with _word_ 'Prepend', including $Prepend
+# * Conditional_Japanese_Starter does not exist in ICU  49, which bail out if 
used.
+# * Hebrew_Letter does not exist in ICU  49, which bail out if used.
 #   NOTE: I sincerely hope there is a better way to avoid problems than this 
abominable
 #   sed substitution...
 $(i18npool_BIDIR)/%.txt : \
$(SRCDIR)/i18npool/source/breakiterator/data/%.txt | 
$(i18npool_BIDIR)/.dir
sed -e ': dummy' \
+   $(if $(filter-out 
YES,$(ICU_RECLASSIFIED_CONDITIONAL_JAPANESE_STARTER)),\
+   -e '/\[:LineBreak =  Conditional_Japanese_Starter:\]/d' 
\
+   -e 's# $$CJ##' \
+   ) \
+   $(if $(filter-out YES,$(ICU_RECLASSIFIED_HEBREW_LETTER)),\
+   -e '/\[:LineBreak =  Hebrew_Letter:\]/d' \
+   -e '/^$$HLcm =/d' \
+   -e '/^$$HLcm $$NUcm;/d' \
+   -e '/^$$NUcm $$HLcm;/d' \
+   -e '/^$$HL $$CM+;/d' \
+   -e 's# | $$HL\(cm\)\?##g' \
+   -e 's#$$HLcm ##g' \
+   -e 's# $$HL##g' \
+   ) \
$(if $(filter 

Re: build failure with ICU 4.8

2013-01-02 Thread Eike Rathke
Hi Ivan,

On Monday, 2012-12-31 12:20:13 +0400, Ivan Timofeev wrote:

 ICU 4.8 does not know Conditional_Japanese_Starter (only ICU 49 does), I
 think we should put that conditional stuff from
 http://cgit.freedesktop.org/libreoffice/core/commit/?id=92a9b7780c6e13a4da3b12794342edbc4c09ef51
 back.

Partly reverted with
http://cgit.freedesktop.org/libreoffice/core/commit/?id=79a3c9b186534c5e3adde2266d3e7fd6527b11b9

Builds with system ICU 4.6 again.

  Eike

-- 
LibreOffice Calc developer. Number formatter stricken i18n transpositionizer.
GnuPG key 0x293C05FD : 997A 4C60 CE41 0149 0DB3  9E96 2F1A D073 293C 05FD
Support the FSFE, care about Free Software! https://fsfe.org/support/?erack


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


[Libreoffice-commits] .: 2 commits - configure.ac python3/ExternalProject_python3.mk

2013-01-02 Thread Libreoffice Gerrit user
 configure.ac   |4 +++-
 python3/ExternalProject_python3.mk |2 +-
 2 files changed, 4 insertions(+), 2 deletions(-)

New commits:
commit b24f82fda9558a8b07a6df9ac898d0e166113a10
Author: Tor Lillqvist t...@iki.fi
Date:   Wed Jan 2 20:53:59 2013 +0200

Can't build the NPAPI plugin stuff as 64-bit for OS X

It uses QuickTime and Carbon.

diff --git a/configure.ac b/configure.ac
index f3e69ac..d5ed79d 100644
--- a/configure.ac
+++ b/configure.ac
@@ -8508,7 +8508,9 @@ dnl 
===
 dnl Check for NPAPI interface to plug browser plugins into LibreOffice 
documents
 dnl ===
 AC_MSG_CHECKING([whether to plug browser plugins into LibreOffice documents])
-if test $_os != Android -a $_os != iOS
+# Obviously no such thing on iOS or Android. Also not possible when building 
+# 64-bit OS X code as the plugin code uses QuickTime and Carbon.
+if test $_os != Android -a $_os != iOS -a \( $_os != Darwin -o 
$BITNESS_OVERRIDE =  \)
 then
 AC_MSG_RESULT([yes])
 ENABLE_NPAPI_FROM_BROWSER=YES
commit 0f6f38b5d7288dd71a26040aee04b1ca4f6e0bcf
Author: Tor Lillqvist t...@iki.fi
Date:   Wed Jan 2 17:51:37 2013 +0200

Fix 64-bit OS X build: don't try any universalness

diff --git a/python3/ExternalProject_python3.mk 
b/python3/ExternalProject_python3.mk
index e2ea53a..a2dbb09 100644
--- a/python3/ExternalProject_python3.mk
+++ b/python3/ExternalProject_python3.mk
@@ -74,7 +74,7 @@ $(call gb_ExternalProject_get_state_target,python3,build) :
$(if $(filter AIX,$(OS)),--disable-ipv6 --with-threads) \
$(if $(filter WNT-GCC,$(OS)-$(COM)),--with-threads 
ac_cv_printf_zd_format=no) \
$(if $(filter MACOSX,$(OS)), \
-   --enable-universalsdk=$(MACOSX_SDK_PATH) 
--with-universal-archs=32-bit 
--enable-framework=/@__OOO 
--with-framework-name=LibreOfficePython, \
+   $(if $(filter INTEL 
POWERPC,$(CPUNAME)),--enable-universalsdk=$(MACOSX_SDK_PATH) 
--with-universal-archs=32-bit) 
--enable-framework=/@__OOO 
--with-framework-name=LibreOfficePython, \
--enable-shared \
) \
CC=$(strip $(CC) \
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


First steps finding regression bug(s)

2013-01-02 Thread Joren

Hi,

I'm not that familiar with the source code of LibreOffice. The last past 
weeks I'm helping the QA-team.


I try now to take a little step forward and try to comprehend a bit of 
the source code. I would like to help core developers point at 
regressions (find the erroneous commit). In first instance just point to 
it, maybe in a later stage try to fix it on my own.


Said that I'm now intrigued by bug/regression 
https://bugs.freedesktop.org/show_bug.cgi?id=54264 (I'm a Mac OSX user).

Quote (Roman Eisele):

If I select two or more files which should open with LibreOffice (e.g., .odt 
or .rtf or .xls files) in the Finder and press Command + O or drag both/all files on 
the LibreOffice application icon, LibreOffice does not open the files (as expected),
but shows an alert which says path of 1st documentpath of 2nd document does not 
exist.. 

Following the alert, I can search for the string 'does not exists.'. I 
found it here: 
http://opengrok.libreoffice.org/xref/core/fpicker/source/office/iodlg.src#286 
:

String RID_FILEOPEN_INVALIDFOLDER
{
Text [ en-US ] = $name$ does not exist.;
};

So my search goes further ... by what is this string triggered/where is 
it defined? 
http://opengrok.libreoffice.org/xref/core/fpicker/source/office/OfficeFilePicker.hrc#39 
:


#define RID_FILEOPEN_INVALIDFOLDER  (RID_FPICKER_START+23)

Now I need to search for 'RID_FPICKER_START' ... and I found it here: 
http://opengrok.libreoffice.org/xref/core/svl/inc/svl/solar.hrc#63


#define RID_FPICKER_START   (RID_LIB_START+6370)

and on line 36 of that file (solar.hrc)

#define RID_LIB_START   1


Ok... Now I found this ... Can I 'conclude' that my string is at 
'position'  16393 (=23 + 6370 + 1). That's 0x4009 in hex notation. I 
searched for 0x4009 and 16393; but I can't find anything relevant where 
this could be triggered. On top of 'solar.hrc' (line 27 and 28) there is 
a function:


#define CREATERESMGR_NAME( Name )   #Name
#define CREATERESMGR( Name )ResMgr::CreateResMgr( CREATERESMGR_NAME( 
Name ) )


As far I can comprehend I guess ResMgr::CreateResMgr (found here 
http://opengrok.libreoffice.org/xref/core/tools/source/rc/resmgr.cxx#CreateResMgr) 
is the function that chooses the correct string... And there I'm stuck I 
think :s. Because the bug is a regression bug 3.5 vs 3.6, and the source 
code of 3.6 is frozen on Week 23, Jun 4 - Jun 10, 2012 I compared the 
current version of resmgr.cxx with a version at the beginning of 2012 
(to be sure not overlook something)... But I couldn't find something. 
Otherwise with this information I still can't find the code that 
triggers the string.


1) Is this information useful for a developer? (should I add that 
information to the bug report?)

2) Am I'm doing it right?

Thanks in advance,
Joren



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


[Libreoffice-commits] .: sal/inc sw/qa sw/source

2013-01-02 Thread Libreoffice Gerrit user
 sal/inc/sal/log-areas.dox|1 +
 sw/qa/extras/rtfimport/rtfimport.cxx |1 -
 sw/source/core/crsr/crstrvl.cxx  |2 +-
 sw/source/core/unocore/unostyle.cxx  |2 +-
 sw/source/ui/app/docsh2.cxx  |1 -
 sw/source/ui/ribbar/workctrl.cxx |4 +---
 6 files changed, 4 insertions(+), 7 deletions(-)

New commits:
commit 8275c8fc33b63aa40a0a8a8c215f32b9986edab6
Author: Miklos Vajna vmik...@suse.cz
Date:   Wed Jan 2 19:55:34 2013 +0100

sw: fix loplugin warnings

Change-Id: I090a51c112c960b8cc9b208bc7378a5aa3754552

diff --git a/sal/inc/sal/log-areas.dox b/sal/inc/sal/log-areas.dox
index 428c7fb..5232778 100644
--- a/sal/inc/sal/log-areas.dox
+++ b/sal/inc/sal/log-areas.dox
@@ -182,6 +182,7 @@ certain functionality.
 @section Writer
 
 @li @c sw
+@li @c sw.level2
 @li @c sw.core - Writer core
 @li @c sw.rtf - .rtf export filter
 @li @c sw.uno - Writer UNO interfaces
diff --git a/sw/qa/extras/rtfimport/rtfimport.cxx 
b/sw/qa/extras/rtfimport/rtfimport.cxx
index be8c34d..88e6797 100644
--- a/sw/qa/extras/rtfimport/rtfimport.cxx
+++ b/sw/qa/extras/rtfimport/rtfimport.cxx
@@ -527,7 +527,6 @@ void Test::testFdo48356()
 uno::Referencetext::XTextDocument xTextDocument(mxComponent, 
uno::UNO_QUERY);
 uno::Referencecontainer::XEnumerationAccess 
xParaEnumAccess(xTextDocument-getText(), uno::UNO_QUERY);
 uno::Referencecontainer::XEnumeration xParaEnum = 
xParaEnumAccess-createEnumeration();
-OUStringBuffer aBuf;
 int i = 0;
 while (xParaEnum-hasMoreElements())
 {
diff --git a/sw/source/core/crsr/crstrvl.cxx b/sw/source/core/crsr/crstrvl.cxx
index 5229fc9..040e998 100644
--- a/sw/source/core/crsr/crstrvl.cxx
+++ b/sw/source/core/crsr/crstrvl.cxx
@@ -1377,7 +1377,7 @@ sal_Bool SwCrsrShell::GetContentAtPos( const Point rPt,
 rCntntAtPos.eCntntAtPos = 
SwContentAtPos::SW_TABLEBOXVALUE;
 else
 #endif
-((SwTblBoxFormula*)pItem)-PtrToBoxNm( 
pTblNd-GetTable() );
+((SwTblBoxFormula*)pItem)-PtrToBoxNm( 
pTblNd-GetTable() );
 
 bRet = sal_True;
 if( bSetCrsr )
diff --git a/sw/source/core/unocore/unostyle.cxx 
b/sw/source/core/unocore/unostyle.cxx
index 71e7d72..7dc7524 100644
--- a/sw/source/core/unocore/unostyle.cxx
+++ b/sw/source/core/unocore/unostyle.cxx
@@ -873,7 +873,7 @@ void SwXStyleFamily::insertByName(const OUString rName, 
const uno::Any rElemen
 #if OSL_DEBUG_LEVEL  1
 SfxStyleSheetBase rNewBase =
 #endif
-pBasePool-Make(sStyleName, eFamily, nMask);
+pBasePool-Make(sStyleName, eFamily, nMask);
 pNewStyle-SetDoc(pDocShell-GetDoc(), pBasePool);
 pNewStyle-SetStyleName(sStyleName);
 String sParentStyleName(pNewStyle-GetParentStyleName());
diff --git a/sw/source/ui/app/docsh2.cxx b/sw/source/ui/app/docsh2.cxx
index f3cbb08..58bbd6d 100644
--- a/sw/source/ui/app/docsh2.cxx
+++ b/sw/source/ui/app/docsh2.cxx
@@ -1691,7 +1691,6 @@ sal_uLong SwDocShell::LoadStylesFromFile( const String 
rURL,
 
 // Create a URL from filename
 INetURLObject aURLObj( rURL );
-String sURL( aURLObj.GetMainURL( INetURLObject::NO_DECODE ) );
 
 // Set filter:
 String 
sFactory(rtl::OUString::createFromAscii(SwDocShell::Factory().GetShortName()));
diff --git a/sw/source/ui/ribbar/workctrl.cxx b/sw/source/ui/ribbar/workctrl.cxx
index 4e21205..41088d1 100644
--- a/sw/source/ui/ribbar/workctrl.cxx
+++ b/sw/source/ui/ribbar/workctrl.cxx
@@ -208,13 +208,12 @@ SfxPopupWindow* SwTbxAutoTextCtrl::CreatePopupWindow()
 {
 // Gruppenname mit Pfad-Extension besorgen
 String sTitle;
-String sGroupName = pGlossaryList-GetGroupName(i - 1, 
sal_False, sTitle);
 sal_uInt16 nBlockCount = pGlossaryList-GetBlockCount(i 
-1);
 if(nBlockCount)
 {
 sal_uInt16 nIndex = 100 * (i);
 // aber ohne extension einfuegen
-pPopup-InsertItem( i, 
sTitle);//sGroupName.GetToken(0, GLOS_DELIM));
+pPopup-InsertItem( i, sTitle);
 PopupMenu* pSub = new PopupMenu;
 pSub-SetSelectHdl(aLnk);
 pPopup-SetPopupMenu(i, pSub);
@@ -306,7 +305,6 @@ IMPL_LINK(SwTbxAutoTextCtrl, PopupHdl, PopupMenu*, pMenu)
 SwGlossaryList* pGlossaryList = ::GetGlossaryList();
 String sShortName;
 String sGroup = pGlossaryList-GetGroupName(nBlock - 1, sal_False);
-String sLongName(pGlossaryList-GetBlockName(nBlock - 1, nId - (100 * 
nBlock) - 1, sShortName));
 
 SwGlossaryHdl* pGlosHdl = pView-GetGlosHdl();
 SwAbstractDialogFactory* pFact = SwAbstractDialogFactory::Create();

[Libreoffice-commits] .: sc/qa

2013-01-02 Thread Libreoffice Gerrit user
 sc/qa/unit/ucalc.cxx |   34 ++
 1 file changed, 34 insertions(+)

New commits:
commit c1f9efa4ac8a406cf10f3b5585f45ee0114621e4
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Wed Jan 2 14:07:09 2013 -0500

fdo#56278: Write a unit test for this.

The reported bug description was very concise and unit-testable.  We
should turn it into a real test.

Change-Id: I7383316e8fcf799a59b1c32a1722fd4b6b224d1c

diff --git a/sc/qa/unit/ucalc.cxx b/sc/qa/unit/ucalc.cxx
index 55b8b9c..4b7e4fd 100644
--- a/sc/qa/unit/ucalc.cxx
+++ b/sc/qa/unit/ucalc.cxx
@@ -121,7 +121,16 @@ public:
  */
 void testSheetsFunc();
 void testVolatileFunc();
+
+/**
+ * Basic test for formula dependency tracking.
+ */
 void testFormulaDepTracking();
+
+/**
+ * Another test for formula dependency tracking, inspired by fdo#56278.
+ */
+void testFormulaDepTracking2();
 void testFuncParam();
 void testNamedRange();
 void testCSV();
@@ -248,6 +257,7 @@ public:
 CPPUNIT_TEST(testSheetsFunc);
 CPPUNIT_TEST(testVolatileFunc);
 CPPUNIT_TEST(testFormulaDepTracking);
+CPPUNIT_TEST(testFormulaDepTracking2);
 CPPUNIT_TEST(testFuncParam);
 CPPUNIT_TEST(testNamedRange);
 CPPUNIT_TEST(testCSV);
@@ -1223,6 +1233,30 @@ void Test::testFormulaDepTracking()
 m_pDoc-DeleteTab(0);
 }
 
+void Test::testFormulaDepTracking2()
+{
+CPPUNIT_ASSERT_MESSAGE (failed to insert sheet, m_pDoc-InsertTab (0, 
foo));
+
+AutoCalcSwitch aACSwitch(m_pDoc, true); // turn on auto calculation.
+
+double val = 2.0;
+m_pDoc-SetValue(0, 0, 0, val);
+val = 4.0;
+m_pDoc-SetValue(1, 0, 0, val);
+val = 5.0;
+m_pDoc-SetValue(0, 1, 0, val);
+m_pDoc-SetString(2, 0, 0, =A1/B1);
+m_pDoc-SetString(1, 1, 0, =B1*C1);
+
+CPPUNIT_ASSERT_EQUAL(2.0, m_pDoc-GetValue(1, 1, 0)); // B2 should equal 2.
+
+clearRange(m_pDoc, ScAddress(2, 0, 0)); // Delete C1.
+
+CPPUNIT_ASSERT_EQUAL(0.0, m_pDoc-GetValue(1, 1, 0)); // B2 should now 
equal 0.
+
+m_pDoc-DeleteTab(0);
+}
+
 void Test::testFuncParam()
 {
 rtl::OUString aTabName(foo);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PATCH] fdo#40465 fix to maintain focus whilst zooming

2013-01-02 Thread Winfried Donkers (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1533

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/33/1533/1

fdo#40465 fix to maintain focus whilst zooming

Change-Id: I3fab5d62d8f479a5efe242693d2fbe33375206f8
---
M dictionaries
M helpcontent2
M sw/source/ui/uiview/viewmdi.cxx
3 files changed, 3 insertions(+), 0 deletions(-)



diff --git a/dictionaries b/dictionaries
index f0c914a..1595bf1 16
--- a/dictionaries
+++ b/dictionaries
-Subproject commit f0c914a43e7e6540300da25c935a77aebb672094
+Subproject commit 1595bf11e295d96123fcb327ba9919307aa5b127
diff --git a/helpcontent2 b/helpcontent2
index d5d84f0..1909df0 16
--- a/helpcontent2
+++ b/helpcontent2
-Subproject commit d5d84f0ec4584e32147eeab355d0ab73e7dd9172
+Subproject commit 1909df07cbd54bf753514cc6dc4137b7b69af63c
diff --git a/sw/source/ui/uiview/viewmdi.cxx b/sw/source/ui/uiview/viewmdi.cxx
index 286435e..17a1ac5 100644
--- a/sw/source/ui/uiview/viewmdi.cxx
+++ b/sw/source/ui/uiview/viewmdi.cxx
@@ -65,6 +65,9 @@
 void SwView::SetZoom( SvxZoomType eZoomType, short nFactor, sal_Bool bViewOnly 
)
 {
 _SetZoom( GetEditWin().GetOutputSizePixel(), eZoomType, nFactor, bViewOnly 
);
+
+//fdo40465 force the cursor to stay in view whilst zooming
+pWrtShell-ShowCrsr();
 }
 
 void SwView::_SetZoom( const Size rEditSize, SvxZoomType eZoomType,

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3fab5d62d8f479a5efe242693d2fbe33375206f8
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Winfried Donkers o...@dci-electronics.nl

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


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

2013-01-02 Thread Libreoffice Gerrit user
 sc/qa/unit/ucalc.cxx |   10 ++
 sc/source/core/tool/scmatrix.cxx |2 ++
 2 files changed, 12 insertions(+)

New commits:
commit 79f1ef44e77074d8f5a1d32e0447118e5b9c4e70
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Wed Jan 2 15:49:24 2013 -0500

fdo#58539: Resizing matrix should also resize the flag storage too.

Or else resizing and then putting empty elements may crash.  The flag
storage is used only for empty elements.

Change-Id: I1ada8f795a01336af185e6180bc03247c44472ba

diff --git a/sc/qa/unit/ucalc.cxx b/sc/qa/unit/ucalc.cxx
index 4b7e4fd..6a8af5d 100644
--- a/sc/qa/unit/ucalc.cxx
+++ b/sc/qa/unit/ucalc.cxx
@@ -1548,6 +1548,16 @@ void Test::testMatrix()
 pMat-PutString(aStr, 8, 2);
 pMat-PutEmptyPath(8, 11);
 checkMatrixElementsPartiallyFilledEmptyMatrix(*pMat);
+
+// Test resizing.
+pMat = new ScMatrix(0, 0);
+pMat-Resize(2, 2, 1.5);
+pMat-PutEmpty(1, 1);
+
+CPPUNIT_ASSERT_EQUAL(1.5, pMat-GetDouble(0, 0));
+CPPUNIT_ASSERT_EQUAL(1.5, pMat-GetDouble(0, 1));
+CPPUNIT_ASSERT_EQUAL(1.5, pMat-GetDouble(1, 0));
+CPPUNIT_ASSERT_MESSAGE(PutEmpty() call failed., pMat-IsEmpty(1, 1));
 }
 
 void Test::testEnterMixedMatrix()
diff --git a/sc/source/core/tool/scmatrix.cxx b/sc/source/core/tool/scmatrix.cxx
index 66dcb2f..0a92ffc 100644
--- a/sc/source/core/tool/scmatrix.cxx
+++ b/sc/source/core/tool/scmatrix.cxx
@@ -418,11 +418,13 @@ bool ScMatrixImpl::IsImmutable() const
 void ScMatrixImpl::Resize(SCSIZE nC, SCSIZE nR)
 {
 maMat.resize(nR, nC);
+maMatFlag.resize(nR, nC);
 }
 
 void ScMatrixImpl::Resize(SCSIZE nC, SCSIZE nR, double fVal)
 {
 maMat.resize(nR, nC, fVal);
+maMatFlag.resize(nR, nC);
 }
 
 void ScMatrixImpl::SetErrorInterpreter( ScInterpreter* p)
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: nss/ExternalPackage_nss.mk

2013-01-02 Thread Libreoffice Gerrit user
 nss/ExternalPackage_nss.mk |8 ++--
 1 file changed, 6 insertions(+), 2 deletions(-)

New commits:
commit ebaaefcbf34d037aa8091ffbef3b7f8adc7a9ac5
Author: Luboš Luňák l.lu...@suse.cz
Date:   Wed Jan 2 21:56:30 2013 +0100

try to sort out libsqlite3 handling in nss

diff --git a/nss/ExternalPackage_nss.mk b/nss/ExternalPackage_nss.mk
index ce19727..6f106af 100644
--- a/nss/ExternalPackage_nss.mk
+++ b/nss/ExternalPackage_nss.mk
@@ -29,9 +29,11 @@ $(eval $(call gb_ExternalPackage_add_files,nss,lib,\
mozilla/dist/out/lib/libplds4.dylib \
mozilla/dist/out/lib/libsmime3.dylib \
mozilla/dist/out/lib/libsoftokn3.dylib \
-   mozilla/dist/out/lib/libsqlite3.dylib \
mozilla/dist/out/lib/libssl3.dylib \
 ))
+$(eval $(call gb_ExternalPackage_add_files,nss,lib/sqlite,\
+   $(if $(filter 1060 1070 
1080,$(MAC_OS_X_VERSION_MIN_REQUIRED)),,mozilla/dist/out/lib/libsqlite3.dylib) \
+))
 else ifeq ($(OS),WNT)
 ifeq ($(COM),MSC)
 $(eval $(call gb_ExternalPackage_add_files,nss,lib,\
@@ -86,9 +88,11 @@ $(eval $(call gb_ExternalPackage_add_files,nss,lib,\
mozilla/dist/out/lib/libplds4.so \
mozilla/dist/out/lib/libsmime3.so \
mozilla/dist/out/lib/libsoftokn3.so \
-   mozilla/dist/out/lib/libsqlite3.so \
mozilla/dist/out/lib/libssl3.so \
 ))
+$(eval $(call gb_ExternalPackage_add_files,nss,lib/sqlite,\
+   mozilla/dist/out/lib/libsqlite3.so \
+))
 endif
 
 # vim: set noet sw=4 ts=4:
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: sc/source

2013-01-02 Thread Libreoffice Gerrit user
 sc/source/ui/unoobj/dapiuno.cxx |   12 ++--
 1 file changed, 10 insertions(+), 2 deletions(-)

New commits:
commit c27e334661b62897a81a450ea9188f80e053261e
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Wed Jan 2 16:03:35 2013 -0500

fdo#58539: Avoid throwing exception not specified in the signature.

This should fix the crasher with the fdo#45266 document.

Change-Id: I41cf02f211e289b85c31b2d2d60e0c4d849b7e8e

diff --git a/sc/source/ui/unoobj/dapiuno.cxx b/sc/source/ui/unoobj/dapiuno.cxx
index feb8b4c..07c2693 100644
--- a/sc/source/ui/unoobj/dapiuno.cxx
+++ b/sc/source/ui/unoobj/dapiuno.cxx
@@ -2697,8 +2697,16 @@ Reference XDataPilotField  SAL_CALL 
ScDataPilotFieldObj::createNameGroup( cons
 Reference XNameAccess  xFields(mrParent.getDataPilotFields(), 
UNO_QUERY);
 if (xFields.is())
 {
-xRet.set(xFields-getByName(sNewDim), UNO_QUERY);
-OSL_ENSURE(xRet.is(), there is a name, so there should be also a 
field);
+try
+{
+xRet.set(xFields-getByName(sNewDim), UNO_QUERY);
+OSL_ENSURE(xRet.is(), there is a name, so there should be 
also a field);
+}
+catch (const container::NoSuchElementException)
+{
+// Avoid throwing exception that's not specified in the method 
signature.
+throw RuntimeException();
+}
 }
 }
 return xRet;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


How to set the temp directory when building LibreOffice ?

2013-01-02 Thread Jean-Baptiste Faure
Hi all,

When building LibreOffice, a lot of files are created in the standard
temp directory (/tmp for me under Ubuntu). Many of these files are not
removed when the build has been completed with success.

I think it would be cleaner if these temporary files was created in a
subdirectory of /tmp. Is there an option to set the temp directory ? I
did not find such an option in the options list of autogen.sh.

Best regards.
JBF
-- 
Seuls des formats ouverts peuvent assurer la pérennité de vos documents.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[PATCH] Get rid of nsinstall hackery

2013-01-02 Thread Peter Foley (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1534

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/34/1534/1

Get rid of nsinstall hackery

This removes the need for using NSS Build Tools on windows.
It also removes the nees to build nss for the build system while cross
compiling.

Change-Id: I13c9fdb575223f2940d3e4eda00e77ba9158f2b7
---
M Makefile.in
M config_host.mk.in
M configure.ac
M nss/ExternalProject_nss.mk
M nss/Module_nss.mk
M nss/nsinstall.py
6 files changed, 9 insertions(+), 35 deletions(-)



diff --git a/Makefile.in b/Makefile.in
index 2a56db0..0b54e3b 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -298,13 +298,6 @@
$(GNUMAKE) gb_Side=build icu
 #
cd cross_toolset  $(GNUMAKE) -j $(PARALLELISM) $(GMAKE_OPTIONS)
-#
-ifneq (,$(filter DESKTOP,$(BUILD_TYPE)))
-ifneq (WNT,$(OS))
-# We need to build nss for nsinstall... See NSINSTALL=... in nss/makefile.mk
-   $(GNUMAKE) gb_Side=build nss
-endif
-endif
 
 #
 # Install
diff --git a/config_host.mk.in b/config_host.mk.in
index 70f8c4e..1c98744 100644
--- a/config_host.mk.in
+++ b/config_host.mk.in
@@ -411,7 +411,6 @@
 export NSS_PATCH=@NSS_PATCH@
 export NSS_CFLAGS=$(gb_SPACE)@NSS_CFLAGS@
 export NSS_LIBS=$(gb_SPACE)@NSS_LIBS@
-export NSSBUILDTOOLS=@NSSBUILDTOOLS@
 export NUMBERTEXT_EXTENSION_PACK=@NUMBERTEXT_EXTENSION_PACK@
 @x_OBJCFLAGS@ export OBJCFLAGS=@OBJCFLAGS@
 @x_OBJCXXFLAGS@ export OBJCXXFLAGS=@OBJCXXFLAGS@
diff --git a/configure.ac b/configure.ac
index c60ae01..4b32e6b 100644
--- a/configure.ac
+++ b/configure.ac
@@ -8281,22 +8281,6 @@
NSS_PATCH=5
 BUILD_TYPE=$BUILD_TYPE NSS
 AC_MSG_RESULT([internal])
-if test $build_os = cygwin; then
-AC_MSG_CHECKING([for nss build tooling]) # coming from mozilla
-if test -z $NSSBUILDTOOLS; then
-AC_MSG_ERROR([nss build tooling not provided
-Use the --with-nss-build-tools option after installing the tools obtained
-from http://ftp.mozilla.org/pub/mozilla.org/mozilla/libraries/win32])
-else
-if test \( $WITH_MINGW = yes -a  ! -d $NSSBUILDTOOLS \) \
--o ! -d $NSSBUILDTOOLS/moztools \
--o ! -d $NSSBUILDTOOLS/msys ; then
-AC_MSG_ERROR([nss build tooling incomplete!])
-else
-AC_MSG_RESULT([ok])
-fi
-fi
-fi
 fi # system nss
 AC_SUBST(SYSTEM_NSS)
 AC_SUBST(NSS_MAJOR)
@@ -8304,7 +8288,6 @@
 AC_SUBST(NSS_PATCH)
 AC_SUBST(NSS_CFLAGS)
 AC_SUBST(NSS_LIBS)
-AC_SUBST(NSSBUILDTOOLS)
 AC_SUBST(MINGW_SMIME3_DLL)
 
 dnl ===
diff --git a/nss/ExternalProject_nss.mk b/nss/ExternalProject_nss.mk
index 330a6e6..7ddbcae 100644
--- a/nss/ExternalProject_nss.mk
+++ b/nss/ExternalProject_nss.mk
@@ -34,15 +34,15 @@
 
 ifeq ($(OS),WNT)
 ifeq ($(COM),MSC)
-$(call gb_ExternalProject_get_state_target,nss,build): $(call 
gb_ExternalProject_get_state_target,nss,configure)
+$(call gb_ExternalProject_get_state_target,nss,build): $(call 
gb_ExternalProject_get_state_target,nss,configure) $(call 
gb_ExternalExecutable_get_dependencies,python)
cd $(EXTERNAL_WORKDIR)/mozilla/security/nss \
 $(if $(debug),,BUILD_OPT=1) \
MOZ_MSVCVERSION=9 OS_TARGET=WIN95 \
-   PATH=$(NSSBUILDTOOLS)/msys/bin:$(NSSBUILDTOOLS)/moztools/bin:$(PATH) \
$(if $(filter X,$(CPU)),USE_64=1) \
LIB=$(ILIB) \
XCFLAGS=$(SOLARINC) \
$(MAKE) -j1 nss_build_all RC=rc.exe $(SOLARINC) \
+   NSINSTALL='$(call gb_ExternalExecutable_get_command,python) 
$(SRCDIR)/nss/nsinstall.py' \
 touch $@
 
 
@@ -53,7 +53,7 @@
CXX=$(CXX) $(if $(filter YES,$(MINGW_SHARED_GCCLIB)),-shared-libgcc) \
OS_LIBS=-ladvapi32 -lws2_32 -lmwsock -lwinm $(if $(filter 
YES,$(MINGW_SHARED_GXXLIB)),$(MINGW_SHARED_LIBSTDCPP)) \
OS_TARGET=WINNT RC=$(WINDRES) OS_RELEASE=5.0 \
-   PATH=$(NSSBUILDTOOLS)/bin:$(PATH) IMPORT_LIB_SUFFIX=dll.a \
+   IMPORT_LIB_SUFFIX=dll.a \
NSPR_CONFIGURE_OPTS=--build=$(BUILD_PLATFORM) --host=$(HOST_PLATFORM) 
--enable-shared --disable-static \
NSINSTALL=$(PYTHON_FOR_BUILD) $(SRCDIR)/nss/nsinstall.py \
$(MAKE) -j1 nss_build_all \
@@ -69,10 +69,9 @@
$(if $(filter SOLARIS,$(OS)),NS_USE_GCC=1) \
$(if $(filter YES,$(CROSS_COMPILING)),\
$(if $(filter MACOSXP,$(OS)$(CPU)),CPU_ARCH=ppc) \
-   NSINSTALL=$(subst $(INPATH),$(INPATH_FOR_BUILD),\
-   $(call 
gb_UnpackedTarball_get_dir,nss)/mozilla/security/coreconf/nsinstall/out/nsinstall))
 \
+   NSINSTALL=$(PYTHON_FOR_BUILD) $(SRCDIR)/nss/nsinstall.py) \
NSDISTMODE=copy \
-   $(MAKE) -j1 $(if $(filter 
build,$(gb_Side)),build_coreconf,nss_build_all) \
+   $(MAKE) -j1 nss_build_all \
 touch $@
 
 endif
diff --git a/nss/Module_nss.mk b/nss/Module_nss.mk
index 5455fc0..c1619ff 100644
--- a/nss/Module_nss.mk
+++ b/nss/Module_nss.mk
@@ -13,7 

[Libreoffice-commits] .: 2 commits - distro-configs/LibreOfficeMinGW.conf sc/Library_scfilt.mk

2013-01-02 Thread Libreoffice Gerrit user
 distro-configs/LibreOfficeMinGW.conf |1 +
 sc/Library_scfilt.mk |2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

New commits:
commit 669c27ca0575e046b37db7ecc0a3d24ed2001a62
Author: Luboš Luňák l.lu...@suse.cz
Date:   Thu Jan 3 00:39:02 2013 +0100

system-zlib for mingw

diff --git a/distro-configs/LibreOfficeMinGW.conf 
b/distro-configs/LibreOfficeMinGW.conf
index 039ce78..e14e993 100644
--- a/distro-configs/LibreOfficeMinGW.conf
+++ b/distro-configs/LibreOfficeMinGW.conf
@@ -18,6 +18,7 @@
 --with-system-openssl
 --with-system-poppler
 --with-system-redland
+--with-system-zlib
 --without-junit
 --without-myspell-dicts
 --disable-activex
commit ec51a95936499cdab17e5b892cc8abb648fc710d
Author: Luboš Luňák l.lu...@suse.cz
Date:   Thu Jan 3 00:38:09 2013 +0100

libs should be listed after the libs that require them

diff --git a/sc/Library_scfilt.mk b/sc/Library_scfilt.mk
index bdb99ca..3901091 100644
--- a/sc/Library_scfilt.mk
+++ b/sc/Library_scfilt.mk
@@ -65,8 +65,8 @@ $(eval $(call gb_Library_use_libraries,scfilt,\
 ))
 
 $(eval $(call gb_Library_use_externals,scfilt,\
-   zlib \
orcus \
+   zlib \
 ))
 
 $(eval $(call gb_Library_add_exception_objects,scfilt,\
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: sc/source

2013-01-02 Thread Libreoffice Gerrit user
 sc/source/filter/xml/xmlsubti.cxx |2 ++
 sc/source/filter/xml/xmlsubti.hxx |2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)

New commits:
commit fa1b00095011def26cc291e74948817632e3545d
Author: Markus Mohrhard markus.mohrh...@googlemail.com
Date:   Thu Jan 3 01:41:45 2013 +0100

don't overflow column number during import, fdo#58539

This should fix the crash with kde#245919

Change-Id: I7c3af01e27233d2f94d5585247c59e7d5b6ea8ca

diff --git a/sc/source/filter/xml/xmlsubti.cxx 
b/sc/source/filter/xml/xmlsubti.cxx
index 2dddb90..1950e3c 100644
--- a/sc/source/filter/xml/xmlsubti.cxx
+++ b/sc/source/filter/xml/xmlsubti.cxx
@@ -220,6 +220,8 @@ void ScMyTables::AddColStyle(const sal_Int32 nRepeat, const 
rtl::OUString rCell
 {
 rImport.GetStylesImportHelper()-AddColumnStyle(rCellStyleName, 
nCurrentColCount, nRepeat);
 nCurrentColCount += nRepeat;
+SAL_WARN_IF(nCurrentColCount  MAXCOL, sc, more columns than fit into 
SCCOL);
+nCurrentColCount = std::minsal_Int32( nCurrentColCount, MAXCOL );
 }
 
 uno::Reference drawing::XDrawPage  ScMyTables::GetCurrentXDrawPage()
diff --git a/sc/source/filter/xml/xmlsubti.hxx 
b/sc/source/filter/xml/xmlsubti.hxx
index 2cdea77..b770771 100644
--- a/sc/source/filter/xml/xmlsubti.hxx
+++ b/sc/source/filter/xml/xmlsubti.hxx
@@ -87,7 +87,7 @@ public:
 ScXMLTabProtectionData GetCurrentProtectionData() { return 
maProtectionData; }
 rtl::OUString   GetCurrentSheetName() const { return 
sCurrentSheetName; }
 SCTAB   GetCurrentSheet() const { return 
(maCurrentCellPos.Tab() = 0) ? maCurrentCellPos.Tab() : 0; }
-SCCOL   GetCurrentColCount() const { return 
nCurrentColCount; }
+SCCOL   GetCurrentColCount() const { return 
std::minsal_Int32(nCurrentColCount, MAXCOL); }
 SCROW   GetCurrentRow() const { return 
(maCurrentCellPos.Row() = 0) ? maCurrentCellPos.Row() : 0; }
 ::com::sun::star::uno::Reference ::com::sun::star::sheet::XSpreadsheet 
 GetCurrentXSheet() const { return 
xCurrentSheet; }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: nss/ExternalPackage_nss.mk

2013-01-02 Thread Libreoffice Gerrit user
 nss/ExternalPackage_nss.mk |2 ++
 1 file changed, 2 insertions(+)

New commits:
commit 96e7e3564b473944218ea2f5440c70399e978e14
Author: Luboš Luňák l.lu...@suse.cz
Date:   Thu Jan 3 02:00:58 2013 +0100

add missing nss windows libs

diff --git a/nss/ExternalPackage_nss.mk b/nss/ExternalPackage_nss.mk
index 6f106af..7c8ffdb 100644
--- a/nss/ExternalPackage_nss.mk
+++ b/nss/ExternalPackage_nss.mk
@@ -63,8 +63,10 @@ $(eval $(call gb_ExternalPackage_add_files,nss,lib,\
 ))
 endif
 $(eval $(call gb_ExternalPackage_add_files,nss,bin,\
+   mozilla/dist/out/lib/freebl3.dll \
mozilla/dist/out/lib/nspr4.dll \
mozilla/dist/out/lib/nss3.dll \
+   mozilla/dist/out/lib/nssckbi.dll \
mozilla/dist/out/lib/nssdbm3.dll \
mozilla/dist/out/lib/nssutil3.dll \
mozilla/dist/out/lib/plc4.dll \
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: How to set the temp directory when building LibreOffice ?

2013-01-02 Thread Ruslan Kabatsayev
Hi,

I've just sent a message on similar topic :)

On Thu, Jan 3, 2013 at 1:11 AM, Jean-Baptiste Faure
jbf.fa...@sud-ouest.org wrote:
 Hi all,

 When building LibreOffice, a lot of files are created in the standard
 temp directory (/tmp for me under Ubuntu). Many of these files are not
 removed when the build has been completed with success.

 I think it would be cleaner if these temporary files was created in a
 subdirectory of /tmp. Is there an option to set the temp directory ? I
 did not find such an option in the options list of autogen.sh.

If anything, you can always use standard TMPDIR environment variable.


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


[Libreoffice-commits] .: xmloff/source

2013-01-02 Thread Libreoffice Gerrit user
 xmloff/source/style/xmlnumfi.cxx |4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

New commits:
commit 5693934456f917861af7cb02734578b049a46940
Author: Markus Mohrhard markus.mohrh...@googlemail.com
Date:   Thu Jan 3 02:18:25 2013 +0100

limit the number of imported digits, fdo#58539

This should fix the crash with gnome#627420.

Change-Id: Ibfff498282dc1c6fe9124ced645392107ef8829f

diff --git a/xmloff/source/style/xmlnumfi.cxx b/xmloff/source/style/xmlnumfi.cxx
index 7b0799c..8b57734 100644
--- a/xmloff/source/style/xmlnumfi.cxx
+++ b/xmloff/source/style/xmlnumfi.cxx
@@ -916,6 +916,8 @@ static void lcl_EnquoteIfNecessary( rtl::OUStringBuffer 
rContent, const SvXMLNu
 //  SvXMLNumFmtElementContext
 //
 
+const sal_Int32 MAX_SECOND_DIGITS = 20; // fdo#58539  gnome#627420: limit 
number of digits during import
+
 SvXMLNumFmtElementContext::SvXMLNumFmtElementContext( SvXMLImport rImport,
 sal_uInt16 nPrfx, const rtl::OUString 
rLName,
 SvXMLNumFormatContext rParentContext, 
sal_uInt16 nNewType,
@@ -948,7 +950,7 @@ SvXMLNumFmtElementContext::SvXMLNumFmtElementContext( 
SvXMLImport rImport,
 {
 case XML_TOK_ELEM_ATTR_DECIMAL_PLACES:
 if (::sax::Converter::convertNumber( nAttrVal, sValue, 0 ))
-aNumInfo.nDecimals = nAttrVal;
+aNumInfo.nDecimals = std::minsal_Int32(nAttrVal, 
MAX_SECOND_DIGITS);
 break;
 case XML_TOK_ELEM_ATTR_MIN_INTEGER_DIGITS:
 if (::sax::Converter::convertNumber( nAttrVal, sValue, 0 ))
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: nss/nss.windows.patch nss/UnpackedTarball_nss.mk

2013-01-02 Thread Libreoffice Gerrit user
 nss/UnpackedTarball_nss.mk |1 +
 nss/nss.windows.patch  |   11 +++
 2 files changed, 12 insertions(+)

New commits:
commit dfbcb34441fed0d38e7c86f32d659ba8ab52640f
Author: Luboš Luňák l.lu...@suse.cz
Date:   Thu Jan 3 02:35:40 2013 +0100

force nss build to pass windows path to cl.exe

Not sure what's wrong exactly, but on one tinderbox cl fails because
of unknown argument (unix path to the source file). Work it around
by explicitly converting the path to windows path.

diff --git a/nss/UnpackedTarball_nss.mk b/nss/UnpackedTarball_nss.mk
index 6e9ad7b..44b9d14 100644
--- a/nss/UnpackedTarball_nss.mk
+++ b/nss/UnpackedTarball_nss.mk
@@ -16,6 +16,7 @@ $(eval $(call gb_UnpackedTarball_add_patches,nss,\
nss/nss.aix.patch \
nss/nss-3.13.5-zlib-werror.patch \
$(if $(filter MACOSX,$(OS)),nss/nss_macosx.patch) \
+   $(if $(filter WNTMSC,$(OS)$(COM)),nss/nss.windows.patch) \
$(if $(filter WNTGCC,$(OS)$(COM)),nss/nspr-4.9-build.patch \
nss/nss-3.13.3-build.patch \
nss/nss.mingw.patch) \
diff --git a/nss/nss.windows.patch b/nss/nss.windows.patch
new file mode 100644
index 000..935943b
--- /dev/null
+++ b/nss/nss.windows.patch
@@ -0,0 +1,11 @@
+--- misc/nss-3.13.5/mozilla/nsprpub/config/rules.mk2008-12-03 
00:24:39.0 +0100
 misc/build/nss-3.13.5/mozilla/nsprpub/config/rules.mk  2009-11-27 
13:36:22.662753328 +0100
+@@ -415,7 +415,7 @@
+ 
+ ifdef NEED_ABSOLUTE_PATH
+ # The quotes allow absolute paths to contain spaces.
+-pr_abspath = $(if $(findstring :,$(1)),$(1),$(if $(filter 
/%,$(1)),$(1),$(CURDIR)/$(1)))
++pr_abspath = $(if $(findstring :,$(1)),$(1),$(if $(filter /%,$(shell cygpath 
-m $(1))),$(1),$(shell cygpath -m $(CURDIR)/$(1
+ endif
+ 
+ $(OBJDIR)/%.$(OBJ_SUFFIX): %.cpp
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: nss/nss.windows.patch

2013-01-02 Thread Libreoffice Gerrit user
 nss/nss.windows.patch |   11 +++
 1 file changed, 11 insertions(+)

New commits:
commit 58e76a64b46791406277d574deecb081f230b910
Author: Luboš Luňák l.lu...@suse.cz
Date:   Thu Jan 3 02:52:50 2013 +0100

one more place for forcing windows path in nss build

diff --git a/nss/nss.windows.patch b/nss/nss.windows.patch
index 935943b..0d4ce1b 100644
--- a/nss/nss.windows.patch
+++ b/nss/nss.windows.patch
@@ -9,3 +9,14 @@
  endif
  
  $(OBJDIR)/%.$(OBJ_SUFFIX): %.cpp
+--- misc/nss-3.13.5/mozilla/security/coreconf/rules.mk 2008-12-03 
00:24:39.0 +0100
 misc/build/nss-3.13.5/mozilla/security/coreconf/rules.mk   2009-11-27 
13:36:22.662753328 +0100
+@@ -411,7 +411,7 @@
+ endif
+ 
+ # The quotes allow absolute paths to contain spaces.
+-core_abspath = $(if $(findstring :,$(1)),$(1),$(if $(filter 
/%,$(1)),$(1),$(PWD)/$(1)))
++core_abspath = $(if $(findstring :,$(1)),$(1),$(if $(filter /%,$(shell 
cygpath -m $(1))),$(1),$(shell cygpath -m $(PWD)/$(1
+ 
+ $(OBJDIR)/$(PROG_PREFIX)%$(OBJ_SUFFIX): %.c
+   @$(MAKE_OBJDIR)
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: Branch 'libreoffice-4-0' - 3 commits - sc/qa sc/source

2013-01-02 Thread Libreoffice Gerrit user
 sc/qa/unit/ucalc.cxx |   44 +++
 sc/source/core/tool/scmatrix.cxx |2 +
 sc/source/ui/unoobj/dapiuno.cxx  |   12 --
 3 files changed, 56 insertions(+), 2 deletions(-)

New commits:
commit b060b43f093dce23222fd99375b1c6bd433703d9
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Wed Jan 2 16:03:35 2013 -0500

fdo#58539: Avoid throwing exception not specified in the signature.

This should fix the crasher with the fdo#45266 document.

Change-Id: I41cf02f211e289b85c31b2d2d60e0c4d849b7e8e

diff --git a/sc/source/ui/unoobj/dapiuno.cxx b/sc/source/ui/unoobj/dapiuno.cxx
index feb8b4c..07c2693 100644
--- a/sc/source/ui/unoobj/dapiuno.cxx
+++ b/sc/source/ui/unoobj/dapiuno.cxx
@@ -2697,8 +2697,16 @@ Reference XDataPilotField  SAL_CALL 
ScDataPilotFieldObj::createNameGroup( cons
 Reference XNameAccess  xFields(mrParent.getDataPilotFields(), 
UNO_QUERY);
 if (xFields.is())
 {
-xRet.set(xFields-getByName(sNewDim), UNO_QUERY);
-OSL_ENSURE(xRet.is(), there is a name, so there should be also a 
field);
+try
+{
+xRet.set(xFields-getByName(sNewDim), UNO_QUERY);
+OSL_ENSURE(xRet.is(), there is a name, so there should be 
also a field);
+}
+catch (const container::NoSuchElementException)
+{
+// Avoid throwing exception that's not specified in the method 
signature.
+throw RuntimeException();
+}
 }
 }
 return xRet;
commit 30285360e5d1fbb14bb6bf54e55a3a9f9b7619e7
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Wed Jan 2 15:49:24 2013 -0500

fdo#58539: Resizing matrix should also resize the flag storage too.

Or else resizing and then putting empty elements may crash.  The flag
storage is used only for empty elements.

Change-Id: I1ada8f795a01336af185e6180bc03247c44472ba

diff --git a/sc/qa/unit/ucalc.cxx b/sc/qa/unit/ucalc.cxx
index 4b7e4fd..6a8af5d 100644
--- a/sc/qa/unit/ucalc.cxx
+++ b/sc/qa/unit/ucalc.cxx
@@ -1548,6 +1548,16 @@ void Test::testMatrix()
 pMat-PutString(aStr, 8, 2);
 pMat-PutEmptyPath(8, 11);
 checkMatrixElementsPartiallyFilledEmptyMatrix(*pMat);
+
+// Test resizing.
+pMat = new ScMatrix(0, 0);
+pMat-Resize(2, 2, 1.5);
+pMat-PutEmpty(1, 1);
+
+CPPUNIT_ASSERT_EQUAL(1.5, pMat-GetDouble(0, 0));
+CPPUNIT_ASSERT_EQUAL(1.5, pMat-GetDouble(0, 1));
+CPPUNIT_ASSERT_EQUAL(1.5, pMat-GetDouble(1, 0));
+CPPUNIT_ASSERT_MESSAGE(PutEmpty() call failed., pMat-IsEmpty(1, 1));
 }
 
 void Test::testEnterMixedMatrix()
diff --git a/sc/source/core/tool/scmatrix.cxx b/sc/source/core/tool/scmatrix.cxx
index 66dcb2f..0a92ffc 100644
--- a/sc/source/core/tool/scmatrix.cxx
+++ b/sc/source/core/tool/scmatrix.cxx
@@ -418,11 +418,13 @@ bool ScMatrixImpl::IsImmutable() const
 void ScMatrixImpl::Resize(SCSIZE nC, SCSIZE nR)
 {
 maMat.resize(nR, nC);
+maMatFlag.resize(nR, nC);
 }
 
 void ScMatrixImpl::Resize(SCSIZE nC, SCSIZE nR, double fVal)
 {
 maMat.resize(nR, nC, fVal);
+maMatFlag.resize(nR, nC);
 }
 
 void ScMatrixImpl::SetErrorInterpreter( ScInterpreter* p)
commit e2b91f39f7162e031c07235a60bfe04f26fee53a
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Wed Jan 2 14:07:09 2013 -0500

fdo#56278: Write a unit test for this.

The reported bug description was very concise and unit-testable.  We
should turn it into a real test.

Change-Id: I7383316e8fcf799a59b1c32a1722fd4b6b224d1c

diff --git a/sc/qa/unit/ucalc.cxx b/sc/qa/unit/ucalc.cxx
index 55b8b9c..4b7e4fd 100644
--- a/sc/qa/unit/ucalc.cxx
+++ b/sc/qa/unit/ucalc.cxx
@@ -121,7 +121,16 @@ public:
  */
 void testSheetsFunc();
 void testVolatileFunc();
+
+/**
+ * Basic test for formula dependency tracking.
+ */
 void testFormulaDepTracking();
+
+/**
+ * Another test for formula dependency tracking, inspired by fdo#56278.
+ */
+void testFormulaDepTracking2();
 void testFuncParam();
 void testNamedRange();
 void testCSV();
@@ -248,6 +257,7 @@ public:
 CPPUNIT_TEST(testSheetsFunc);
 CPPUNIT_TEST(testVolatileFunc);
 CPPUNIT_TEST(testFormulaDepTracking);
+CPPUNIT_TEST(testFormulaDepTracking2);
 CPPUNIT_TEST(testFuncParam);
 CPPUNIT_TEST(testNamedRange);
 CPPUNIT_TEST(testCSV);
@@ -1223,6 +1233,30 @@ void Test::testFormulaDepTracking()
 m_pDoc-DeleteTab(0);
 }
 
+void Test::testFormulaDepTracking2()
+{
+CPPUNIT_ASSERT_MESSAGE (failed to insert sheet, m_pDoc-InsertTab (0, 
foo));
+
+AutoCalcSwitch aACSwitch(m_pDoc, true); // turn on auto calculation.
+
+double val = 2.0;
+m_pDoc-SetValue(0, 0, 0, val);
+val = 4.0;
+m_pDoc-SetValue(1, 0, 0, val);
+val = 5.0;
+m_pDoc-SetValue(0, 1, 0, val);
+m_pDoc-SetString(2, 0, 0, =A1/B1);
+m_pDoc-SetString(1, 1, 0, =B1*C1);
+
+

Re: [PATCH] Get rid of nsinstall hackery

2013-01-02 Thread Mat M

Hi Peter,

Two caveats I want to submit :
what happens if --disable-python is here ?
And this thread [1] has never come to a solution about ssl API, but your  
change may conflict with one of the scenarios if the python version is not  
enough.



[1] http://lists.freedesktop.org/archives/libreoffice/2012-May/031198.html

--
Regards
Mat M

Le Thu, 03 Jan 2013 00:25:21 +0100, Peter Foley (via Code Review)  
ger...@gerrit.libreoffice.org a écrit:



Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/1534

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core  
refs/changes/34/1534/1


Get rid of nsinstall hackery

This removes the need for using NSS Build Tools on windows.
It also removes the nees to build nss for the build system while cross
compiling.

Change-Id: I13c9fdb575223f2940d3e4eda00e77ba9158f2b7
---
M Makefile.in
M config_host.mk.in
M configure.ac
M nss/ExternalProject_nss.mk
M nss/Module_nss.mk
M nss/nsinstall.py
6 files changed, 9 insertions(+), 35 deletions(-)



diff --git a/Makefile.in b/Makefile.in
index 2a56db0..0b54e3b 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -298,13 +298,6 @@
$(GNUMAKE) gb_Side=build icu
 #
cd cross_toolset  $(GNUMAKE) -j $(PARALLELISM) $(GMAKE_OPTIONS)
-#
-ifneq (,$(filter DESKTOP,$(BUILD_TYPE)))
-ifneq (WNT,$(OS))
-# We need to build nss for nsinstall... See NSINSTALL=... in  
nss/makefile.mk

-   $(GNUMAKE) gb_Side=build nss
-endif
-endif
#
 # Install
diff --git a/config_host.mk.in b/config_host.mk.in
index 70f8c4e..1c98744 100644
--- a/config_host.mk.in
+++ b/config_host.mk.in
@@ -411,7 +411,6 @@
 export NSS_PATCH=@NSS_PATCH@
 export NSS_CFLAGS=$(gb_SPACE)@NSS_CFLAGS@
 export NSS_LIBS=$(gb_SPACE)@NSS_LIBS@
-export NSSBUILDTOOLS=@NSSBUILDTOOLS@
 export NUMBERTEXT_EXTENSION_PACK=@NUMBERTEXT_EXTENSION_PACK@
 @x_OBJCFLAGS@ export OBJCFLAGS=@OBJCFLAGS@
 @x_OBJCXXFLAGS@ export OBJCXXFLAGS=@OBJCXXFLAGS@
diff --git a/configure.ac b/configure.ac
index c60ae01..4b32e6b 100644
--- a/configure.ac
+++ b/configure.ac
@@ -8281,22 +8281,6 @@
NSS_PATCH=5
 BUILD_TYPE=$BUILD_TYPE NSS
 AC_MSG_RESULT([internal])
-if test $build_os = cygwin; then
-AC_MSG_CHECKING([for nss build tooling]) # coming from mozilla
-if test -z $NSSBUILDTOOLS; then
-AC_MSG_ERROR([nss build tooling not provided
-Use the --with-nss-build-tools option after installing the tools  
obtained

-from http://ftp.mozilla.org/pub/mozilla.org/mozilla/libraries/win32])
-else
-if test \( $WITH_MINGW = yes -a  ! -d $NSSBUILDTOOLS  
\) \

--o ! -d $NSSBUILDTOOLS/moztools \
--o ! -d $NSSBUILDTOOLS/msys ; then
-AC_MSG_ERROR([nss build tooling incomplete!])
-else
-AC_MSG_RESULT([ok])
-fi
-fi
-fi
 fi # system nss
 AC_SUBST(SYSTEM_NSS)
 AC_SUBST(NSS_MAJOR)
@@ -8304,7 +8288,6 @@
 AC_SUBST(NSS_PATCH)
 AC_SUBST(NSS_CFLAGS)
 AC_SUBST(NSS_LIBS)
-AC_SUBST(NSSBUILDTOOLS)
 AC_SUBST(MINGW_SMIME3_DLL)
dnl ===
diff --git a/nss/ExternalProject_nss.mk b/nss/ExternalProject_nss.mk
index 330a6e6..7ddbcae 100644
--- a/nss/ExternalProject_nss.mk
+++ b/nss/ExternalProject_nss.mk
@@ -34,15 +34,15 @@
ifeq ($(OS),WNT)
 ifeq ($(COM),MSC)
-$(call gb_ExternalProject_get_state_target,nss,build): $(call  
gb_ExternalProject_get_state_target,nss,configure)
+$(call gb_ExternalProject_get_state_target,nss,build): $(call  
gb_ExternalProject_get_state_target,nss,configure) $(call  
gb_ExternalExecutable_get_dependencies,python)

cd $(EXTERNAL_WORKDIR)/mozilla/security/nss \
 $(if $(debug),,BUILD_OPT=1) \
MOZ_MSVCVERSION=9 OS_TARGET=WIN95 \
-	PATH=$(NSSBUILDTOOLS)/msys/bin:$(NSSBUILDTOOLS)/moztools/bin:$(PATH)  
\

$(if $(filter X,$(CPU)),USE_64=1) \
LIB=$(ILIB) \
XCFLAGS=$(SOLARINC) \
$(MAKE) -j1 nss_build_all RC=rc.exe $(SOLARINC) \
+	NSINSTALL='$(call gb_ExternalExecutable_get_command,python)  
$(SRCDIR)/nss/nsinstall.py' \

 touch $@
@@ -53,7 +53,7 @@
 	CXX=$(CXX) $(if $(filter YES,$(MINGW_SHARED_GCCLIB)),-shared-libgcc)  
\
 	OS_LIBS=-ladvapi32 -lws2_32 -lmwsock -lwinm $(if $(filter  
YES,$(MINGW_SHARED_GXXLIB)),$(MINGW_SHARED_LIBSTDCPP)) \

OS_TARGET=WINNT RC=$(WINDRES) OS_RELEASE=5.0 \
-   PATH=$(NSSBUILDTOOLS)/bin:$(PATH) IMPORT_LIB_SUFFIX=dll.a \
+   IMPORT_LIB_SUFFIX=dll.a \
 	NSPR_CONFIGURE_OPTS=--build=$(BUILD_PLATFORM) --host=$(HOST_PLATFORM)  
--enable-shared --disable-static \

NSINSTALL=$(PYTHON_FOR_BUILD) $(SRCDIR)/nss/nsinstall.py \
$(MAKE) -j1 nss_build_all \
@@ -69,10 +69,9 @@
$(if $(filter SOLARIS,$(OS)),NS_USE_GCC=1) \
$(if $(filter YES,$(CROSS_COMPILING)),\
$(if $(filter MACOSXP,$(OS)$(CPU)),CPU_ARCH=ppc) \
-   NSINSTALL=$(subst $(INPATH),$(INPATH_FOR_BUILD),\
-	$(call  

[Libreoffice-commits] .: sc/source

2013-01-02 Thread Libreoffice Gerrit user
 sc/source/filter/oox/defnamesbuffer.cxx |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 7a8a3e46978b3397ca8aba38917bd5dbc808c9b8
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Wed Jan 2 23:56:43 2013 -0500

fdo#58539: Properly initialize a member pointer value.

Or else it would crash when loading the doc from fdo#55174.

Change-Id: I21eb77f97dcab467c185dacaf9f00753bb4fc18c

diff --git a/sc/source/filter/oox/defnamesbuffer.cxx 
b/sc/source/filter/oox/defnamesbuffer.cxx
index d735ac4..488b7e3 100644
--- a/sc/source/filter/oox/defnamesbuffer.cxx
+++ b/sc/source/filter/oox/defnamesbuffer.cxx
@@ -285,6 +285,7 @@ ApiTokenSequence DefinedNameBase::importBiffFormula( 
sal_Int16 nBaseSheet, BiffI
 
 DefinedName::DefinedName( const WorkbookHelper rHelper ) :
 DefinedNameBase( rHelper ),
+mpScRangeData(NULL),
 mnTokenIndex( -1 ),
 mcBuiltinId( BIFF_DEFNAME_UNKNOWN ),
 mnFmlaSize( 0 )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: sc/source

2013-01-02 Thread Libreoffice Gerrit user
 sc/source/filter/excel/xichart.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 329f3cd02546dfe58bf00f6ad3b71bc84a8d4320
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Thu Jan 3 00:27:08 2013 -0500

fdo#58539: Check for mxValueLink being NULL before dereferencing.

Or else Calc would crash while loading the xls document from
gnome519788.

As an aside, this xls document appears to be corrupt. Trying to
open it in Excel (XP and 2007) causes Excel to offer to repair it,
while fails in both versions.  Excel XP is somehow able to open it
with some content preserved. No such luck with Excel 2007.

Change-Id: I04616a4c926862461a2efdd99ccabe36122d6825

diff --git a/sc/source/filter/excel/xichart.cxx 
b/sc/source/filter/excel/xichart.cxx
index 1aa9d45..8310deb 100644
--- a/sc/source/filter/excel/xichart.cxx
+++ b/sc/source/filter/excel/xichart.cxx
@@ -2057,7 +2057,7 @@ Reference XDataSeries  
XclImpChSeries::CreateDataSeries() const
 aSeriesProp.SetBoolProperty( EXC_CHPROP_VARYCOLORSBY, 
rTypeInfo.meTypeCateg == EXC_CHTYPECATEG_PIE );
 #endif
 // #i91271# always set area formatting for every point in pie/doughnut 
charts
-if( mxSeriesFmt  ((bVarPointFmt  mxSeriesFmt-IsAutoArea()) || 
(rTypeInfo.meTypeCateg == EXC_CHTYPECATEG_PIE)) )
+if (mxSeriesFmt  mxValueLink  ((bVarPointFmt  
mxSeriesFmt-IsAutoArea()) || (rTypeInfo.meTypeCateg == EXC_CHTYPECATEG_PIE)))
 {
 for( sal_uInt16 nPointIdx = 0, nPointCount = 
mxValueLink-GetCellCount(); nPointIdx  nPointCount; ++nPointIdx )
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] .: Branch 'libreoffice-4-0' - 2 commits - sc/source

2013-01-02 Thread Libreoffice Gerrit user
 sc/source/filter/excel/xichart.cxx  |2 +-
 sc/source/filter/oox/defnamesbuffer.cxx |1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

New commits:
commit f204cd9b52ec7af3e0d20f5c88f09737b38b9e7d
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Thu Jan 3 00:27:08 2013 -0500

fdo#58539: Check for mxValueLink being NULL before dereferencing.

Or else Calc would crash while loading the xls document from
gnome519788.

As an aside, this xls document appears to be corrupt. Trying to
open it in Excel (XP and 2007) causes Excel to offer to repair it,
while fails in both versions.  Excel XP is somehow able to open it
with some content preserved. No such luck with Excel 2007.

Change-Id: I04616a4c926862461a2efdd99ccabe36122d6825

diff --git a/sc/source/filter/excel/xichart.cxx 
b/sc/source/filter/excel/xichart.cxx
index 1aa9d45..8310deb 100644
--- a/sc/source/filter/excel/xichart.cxx
+++ b/sc/source/filter/excel/xichart.cxx
@@ -2057,7 +2057,7 @@ Reference XDataSeries  
XclImpChSeries::CreateDataSeries() const
 aSeriesProp.SetBoolProperty( EXC_CHPROP_VARYCOLORSBY, 
rTypeInfo.meTypeCateg == EXC_CHTYPECATEG_PIE );
 #endif
 // #i91271# always set area formatting for every point in pie/doughnut 
charts
-if( mxSeriesFmt  ((bVarPointFmt  mxSeriesFmt-IsAutoArea()) || 
(rTypeInfo.meTypeCateg == EXC_CHTYPECATEG_PIE)) )
+if (mxSeriesFmt  mxValueLink  ((bVarPointFmt  
mxSeriesFmt-IsAutoArea()) || (rTypeInfo.meTypeCateg == EXC_CHTYPECATEG_PIE)))
 {
 for( sal_uInt16 nPointIdx = 0, nPointCount = 
mxValueLink-GetCellCount(); nPointIdx  nPointCount; ++nPointIdx )
 {
commit eee87edf551f7a9f8c4da4267e5710edafd1725d
Author: Kohei Yoshida kohei.yosh...@gmail.com
Date:   Wed Jan 2 23:56:43 2013 -0500

fdo#58539: Properly initialize a member pointer value.

Or else it would crash when loading the doc from fdo#55174.

Change-Id: I21eb77f97dcab467c185dacaf9f00753bb4fc18c

diff --git a/sc/source/filter/oox/defnamesbuffer.cxx 
b/sc/source/filter/oox/defnamesbuffer.cxx
index d735ac4..488b7e3 100644
--- a/sc/source/filter/oox/defnamesbuffer.cxx
+++ b/sc/source/filter/oox/defnamesbuffer.cxx
@@ -285,6 +285,7 @@ ApiTokenSequence DefinedNameBase::importBiffFormula( 
sal_Int16 nBaseSheet, BiffI
 
 DefinedName::DefinedName( const WorkbookHelper rHelper ) :
 DefinedNameBase( rHelper ),
+mpScRangeData(NULL),
 mnTokenIndex( -1 ),
 mcBuiltinId( BIFF_DEFNAME_UNKNOWN ),
 mnFmlaSize( 0 )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: Wrong platform check for Xinerama support

2013-01-02 Thread Lionel Elie Mamane
On Wed, Jan 02, 2013 at 01:50:33PM +0100, Francois Tigeot wrote:
 configure.in / configure.ac also contains some fishy platform checks in
 the Xinerama support check:

 ...
 elif test $_os = Linux -o $_os = FreeBSD; then
 if test $x_libraries = default_x_libraries; then
 XINERAMALIB=`$PKG_CONFIG --variable=libdir xinerama`
 ...

 I'm not too sure of why this line was added but Linux and FreeBSD are not
 the only operating systems which can use Xorg.

We should probably have a generic test for traditional Unix-like
with X11, which is probably the intended meaning of many Linux or
Linux or FreeBSD tests. Also in reference to your previous thread
(graphite was reserved for Linux and Windows NT).

Something like this early in configure.ac:

if test $_os = Linux -o $_os = FreeBSD -o $_os = MirBSD -o $_os = 
OpenBSD #etc, etc;
   UNIX_X11=true || POSIX_OS=true
fi

and then test UNIX_X11 (or POSIX_OS). So when we add support for
yet-another of these Unix/POSIX/SuSvN variants, there is not that much
to change.

As gravy, this way, if someone for whatever reason ever wants to
compile e.g. for MacOS X using the X11 server (instead of native Aqua
/ Carbon / I lost track), they can set UNIX_X11 and have a good start
:)


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


Re: Wrong platform check for Xinerama support

2013-01-02 Thread Tor Lillqvist
 if test $_os = Linux -o $_os = FreeBSD -o $_os = MirBSD -o $_os 
 = OpenBSD #etc, etc;
UNIX_X11=true || POSIX_OS=true
 fi

 and then test UNIX_X11 (or POSIX_OS).

I doubt we need any separate flag for POSIXness, because basically
anything that isn't Windows is POSIX, surely. Actually, POSIX or not
is a bit too wide a test; after all the non-desktop OSes (iOS and
Android) have most of the normal POSIX system calls, but one sure is
not supposed/allowed to for instance fork() in an app on them.

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


Re: Wrong platform check for Xinerama support

2013-01-02 Thread Lionel Elie Mamane
On Thu, Jan 03, 2013 at 09:12:12AM +0200, Tor Lillqvist wrote:

 if test $_os = Linux -o $_os = FreeBSD -o $_os = MirBSD -o 
 $_os = OpenBSD #etc, etc;
UNIX_X11=true || POSIX_OS=true
 fi

 and then test UNIX_X11 (or POSIX_OS).

 I doubt we need any separate flag for POSIXness, because basically
 anything that isn't Windows is POSIX, surely.

And even Windows NT has a POSIX layer... OK, then POSIX_OS is not
the right name choice for what I want this variable to mean. UNIX_X11
is maybe the better name.

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


Re: Wrong platform check for Xinerama support

2013-01-02 Thread Noel Grandin

The XINE check from here:

http://xine.cvs.sourceforge.net/viewvc/xine/xine-ui/configure.ac?revision=1.134view=markup

looks like this:

dnl
dnl Checks for Xinerama extension
dnl
AC_ARG_ENABLE([xinerama],
   AS_HELP_STRING([--disable-xinerama], [Don't build support for 
xinerama (default: check)]))

if test x$enable_xinerama != xno; then
  PKG_CHECK_MODULES([XINERAMA], [xinerama], [ac_have_xinerama=yes], [
AC_CHECK_LIB(Xinerama, XineramaQueryExtension,
   XINERAMA_LIBS=-lXinerama
   ac_have_xinerama=yes,,
   $X_LIBS $X_PRE_LIBS -lXext $X_EXTRA_LIBS)
  ])

  if test x$ac_have_xinerama = xyes; then
 X_PRE_LIBS=$X_PRE_LIBS $XINERAMA_LIBS
 AC_DEFINE(HAVE_XINERAMA,,[Define this if you have libXinerama 
installed])

  fi
fi
AM_CONDITIONAL(HAVE_XINERAMA, test x$ac_have_xinerama = xyes)

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


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


Re: Wrong platform check for Xinerama support

2013-01-02 Thread Tor Lillqvist
 And even Windows NT has a POSIX layer...

You mean subsystem. (It's not as if there was a POSIX layer hiding
below the Win32 layer or something.) Nowadays it is called the
Subsystem for Unix applications (SUA), but LibreOffice has nothing
to do with that, and can't have. SUA can be completely ignored here.

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


[Libreoffice-bugs] [Bug 40907] FILEOPEN - concurrent access to AFP shared files impossible - total lock on file

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=40907

--- Comment #13 from fabien.mic...@hespul.org ---
For information,
we have paid someone to work on this issue. We hope it will be resolved soon.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 58085] =Wert! after 2242 rows

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=58085

Rainer Bielefeld libreoff...@bielefeldundbuss.de changed:

   What|Removed |Added

 Status|NEW |NEEDINFO
   Severity|trivial |normal
   Priority|highest |medium
 CC||LibreOffice@bielefeldundbus
   ||s.de
  Component|Libreoffice |Spreadsheet

--- Comment #1 from Rainer Bielefeld libreoff...@bielefeldundbuss.de ---
A trivial problem with highest importance? Strange!

NOT [Reproducible] with parallel installation of  LOdev  4.0.0.0.alpha1   - 
ENGLISH UI / German Locale  [Build ID: dec8fe)]  {tinderbox: @6, pull time
2012-11-13 06:07:28} on German WIN7 Home Premium (64bit) with separate /4 User
Profile for Master Branch

@reporter:
Thank you for your report – unfortunately important information is missing.
May be hints on http://wiki.documentfoundation.org/BugReport will help you to
find out what information will be useful to reproduce your problem? If you
believe that that  is really sophisticated please as for Help on a user mailing
list
Please:
- Write a meaningful Summary describing exactly what the problem is
  Really =Wert!? Not #Wert!?
- Attach a sample document (not only screenshot) or refer to an existing 
  sample document in an other Bug with a link; to attach a file to this 
  bug report, just click on Add an attachment right on this page.
- Attach screenshots with comments if you believe that that might explain the 
  problem better than a text comment. Best way is to insert your screenshots
  into a DRAW document and to add comments that explain what you want to show
  (attachment 68877, attachment 68490)
- Contribute a document related step by step instruction containing every 
  key press and every mouse click how to reproduce your problem 
  (similar to example in Bug 43431)
– if possible contribute an instruction how to create a sample document 
  from the scratch
- add information 
  -- what EXACTLY is unexpected
  -- and WHY do you believe it's unexpected (cite Help or Documentation!)
  -- concerning your PC
  -- concerning your OS (Version, Distribution, Language)
  -- concerning your LibO localization (UI language, Locale setting)
  –- Libo settings that might be related to your problems 
  -- how you launch LibO and how you opened the sample document
  –- Whether your problem persists when you renamed your user profile 
 before you launch LibO (please see
 https://wiki.documentfoundation.org/UserProfile#User_profile_location)
  -- everything else crossing your mind after you read linked texts

Even if you can not provide all demanded information, every little new
information might bring the breakthrough.

May be you want to test https://www.libreoffice.org/get-help/bug/ for
submitting bug reports? You reach that Bug Submission Assistant via LibO menu
'Help - Feedback / Bug Report'. The assistant will collect and insert such
information automatically.

Please submit Bug reports with status UNCONFIRMED if your are not absolutely
sure that you contributed all required background information, that the problem
will be reproducible with information you can provide or that your enhancement
request will be accepted! Thank you!

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 54165] Unable to localize key names

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=54165

--- Comment #6 from jmontane j...@montane.cat ---
Created attachment 72375
  -- https://bugs.freedesktop.org/attachment.cgi?id=72375action=edit
Screenshots of several programs

Screenshots showing shortcut names, using Catalan UI in all programs but with
and Spanish keyboard.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 54165] Unable to localize key names

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=54165

--- Comment #7 from jmontane j...@montane.cat ---
I've done some searching,

Adobe Reader, Chrome, Firefox, MS Office, Thunderbird and Windows Live localize
key name according language UI of program.

7-zip, IE Explorer, PDFCreator, SumatraPDF and WinRAR shows key name in
English, *not* as provided by keyboard layout

LibreOffice, OpenOffice and Keyboard (embedded program of Windows) shows key
names as provides by keyboard layout.

More on, according to bug 50415, in Linux LibO the key names are showed by
language UI, why doing diffent in Windows LibO?

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 58939] New: FORMATTING: Working with vertical text [Writer, Calc]

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=58939

  Priority: medium
Bug ID: 58939
  Assignee: libreoffice-bugs@lists.freedesktop.org
   Summary: FORMATTING: Working with vertical text [Writer, Calc]
  Severity: normal
Classification: Unclassified
OS: All
  Reporter: minnigalie...@yandex.ru
  Hardware: All
Status: UNCONFIRMED
   Version: unspecified
 Component: Spreadsheet
   Product: LibreOffice

Once again trying to work with vertical text in a Calc document and the tables
Writer decided to ask other people from the Russian community of Ubuntu, what
they think on this issue. Convinced that no one I work LO vertical text seems
uncomfortable and not completed.

Some points that require further development:
1. Vertical text direction should set the same way in Calc and Writer.
2. The transition to a new line in a given direction should be carried out
according to Enter.
3. Should remove the horizontal cursor with the height of the length of the
vertical text, when you put it in the beginning of the line of vertical text.
4. Should do understandable navigation cursors in vertical text.

Dear developers, just try to work with vertical text in Calc and Writer and you
will see many small troubles that make a difficult to work with office!

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 40907] FILEOPEN - concurrent access to AFP shared files impossible - total lock on file

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=40907

--- Comment #14 from Roman Eisele b...@eikota.de ---
(In reply to comment #13)
 For information,
 we have paid someone to work on this issue. We hope it will be resolved soon.

Thank you very much!

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 58888] EDITING: Copying and Inserting Lines

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=5

Rainer Bielefeld libreoff...@bielefeldundbuss.de changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEEDINFO
 CC||LibreOffice@bielefeldundbus
   ||s.de
Summary|EDITING: Verb Bad Usability |EDITING: Copying and
   |Problem - Copying and   |Inserting Lines
   |Inserting Lines |
 Ever confirmed|0   |1

--- Comment #2 from Rainer Bielefeld libreoff...@bielefeldundbuss.de ---
Not a valid bug report, I have no idea what reporter thinks what a typical
lline item is and what reporter's problem might be.

@reporter:
Thank you for your report – unfortunately important information is missing.
May be hints on http://wiki.documentfoundation.org/BugReport will help you to
find out what information will be useful to reproduce your problem? If you
believe that that  is really sophisticated please as for Help on a user mailing
list
Please:
- Write a meaningful Summary describing exactly what the problem is
- Attach a sample document (not only screenshot) or refer to an existing 
  sample document in an other Bug with a link; to attach a file to this 
  bug report, just click on Add an attachment right on this page.
- Attach screenshots with comments if you believe that that might explain the 
  problem better than a text comment. Best way is to insert your screenshots
  into a DRAW document and to add comments that explain what you want to show
  (attachment 68877, attachment 68490)
- Contribute a document related step by step instruction containing every 
  key press and every mouse click how to reproduce your problem 
  (similar to example in Bug 43431)
– if possible contribute an instruction how to create a sample document 
  from the scratch
- add information 
  -- what EXACTLY is unexpected
  -- and WHY do you believe it's unexpected (cite Help or Documentation!)
  -- concerning your PC 
  -- concerning your OS (Version, Distribution, Language)
  -- concerning your localization (UI language, Locale setting)
  –- Libo settings that might be related to your problems
  -- how you launch LibO and how you opened the sample document
  –- Whether your problem persists when you renamed your user profile 
 before you launch LibO (please see
 https://wiki.documentfoundation.org/UserProfile#User_profile_location)
  –- If you can contribute an AOOo Issue that might be useful
  -- everything else crossing your mind after you read linked texts

Even if you can not provide all demanded information, every little new
information might bring the breakthrough.

May be you want to test https://www.libreoffice.org/get-help/bug/ for
submitting bug reports? You reach that Bug Submission Assistant via LibO menu
'Help - Feedback / Bug Report'. The assistant will collect and insert such
information automatically.

May be you can test http://wiki.documentfoundation.org/BugReport_Details for
submitting bug reports? 

Please submit Bug reports with status UNCONFIRMED if your are not absolutely
sure that you contributed all required background information, that the problem
will be reproducible with information you can provide or that your enhancement
request will be accepted! Thank you!

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 56963] FILEOPEN, EDITING: Stability degrades with large document

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=56963

--- Comment #12 from David genericinet+libreoffice@gmail.com ---
Have you tried version 3.4.6 with the display options off?  I haven't actually
tried seeing what happens in later versions with my document and this error
because of another bug which is preventing me from upgrading. Another thing
I've found that worked until I closed the document and re-opened it was to
create a new document and insert the old one using the File command from the
insert menu.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 58889] EDITING: Inputting Data ???

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=58889

Rainer Bielefeld libreoff...@bielefeldundbuss.de changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEEDINFO
   Severity|normal  |enhancement
 CC||LibreOffice@bielefeldundbus
   ||s.de
Summary|EDITING: Usability Problem  |EDITING: Inputting Data ???
   |- Inputting Data|
 Ever confirmed|0   |1

--- Comment #1 from Rainer Bielefeld libreoff...@bielefeldundbuss.de ---
I can't imagine what the problem might be.

@Kim:
please follow my advice in Bug 5 !

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 50928] EDITING: Copying and Inserting Lines - needs redesigning

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=50928

Rainer Bielefeld libreoff...@bielefeldundbuss.de changed:

   What|Removed |Added

 Status|NEW |NEEDINFO
 CC||LibreOffice@bielefeldundbus
   ||s.de

--- Comment #5 from Rainer Bielefeld libreoff...@bielefeldundbuss.de ---
I can't imagine what the problem might be.

BTW, If I want to proceed reporter's example in comment 4, I
1. Click row heading 1 to select first row
2. control+c
3. Click row heading 2 to select second row
4. shift + click on row heading 42
5. Click 'Insert row' Icon
6. control+v for paste
all new lines are filled

That's simple, and there exist many alternatives (record a macro, ...)
Currently I can't see any urgency for a redesign.

@Kim:
please follow my advice in Bug 5!

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 58750] FORMATTING: Crash when Format Columns

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=58750

tommy27 ba...@quipo.it changed:

   What|Removed |Added

 CC||ba...@quipo.it

--- Comment #3 from tommy27 ba...@quipo.it ---
NOT reproducible on Windows Vista 64bit using:
Version 4.0.0.0.beta2+ (Build ID: 350ae8294a8df78403fd8cdce56b9aeb8178e13)

@manj_k

do you still reproduce it with current LibO 4.0 beta2?
if not we shall close this bug report

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 58750] FORMATTING: Crash when Format Columns

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=58750

--- Comment #4 from tommy27 ba...@quipo.it ---
oooppsss... sorry... did not notice that the bug refers to master (future 4.1)

however my test shows that the bug is not present in the 4.0 branch

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 50929] BUGZILLAASSISTANT: You Need a Sign Off button

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=50929

Rainer Bielefeld libreoff...@bielefeldundbuss.de changed:

   What|Removed |Added

 Status|NEEDINFO|RESOLVED
 Resolution|--- |DUPLICATE
 CC||LibreOffice@bielefeldundbus
   ||s.de
Summary|: You Need a My Bugs button |BUGZILLAASSISTANT: You Need
   |and a Sign Off button   |a Sign Off button

--- Comment #7 from Rainer Bielefeld libreoff...@bielefeldundbuss.de ---
I think the Sign off button request is a DUP of Bug 41627 -
BUGZILLAASSISTANT: No possibility to log out. I reduce this bug to that
problem

My Bugs can be found easily on the Bugzilla page you can reach after you
reported a bug. The BSA is mainly for newcomers who do not have (many) My
bugs, so currently I can't see evidence for that request.

@Kim:
Only one issue per report! 
http://wiki.documentfoundation.org/BugReport#General_information item 4
Your reports are much too imprecise, please follow my advice in Bug 5!
Please feel free to submit a new report for the My Bugs request if you can
contribute evidence why the link in Bugzilla is not sufficient.

*** This bug has been marked as a duplicate of bug 41627 ***

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 41627] BUGZILLAASSISTANT: No possibility to log out

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=41627

Rainer Bielefeld libreoff...@bielefeldundbuss.de changed:

   What|Removed |Added

 CC||libreoffic...@wrldcomp.com

--- Comment #2 from Rainer Bielefeld libreoff...@bielefeldundbuss.de ---
*** Bug 50929 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 50932] EDITING: Double Thickness Cell Borders are Not Displayed

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=50932

Rainer Bielefeld libreoff...@bielefeldundbuss.de changed:

   What|Removed |Added

 Resolution|FIXED   |WORKSFORME
 CC||LibreOffice@bielefeldundbus
   ||s.de

--- Comment #3 from Rainer Bielefeld libreoff...@bielefeldundbuss.de ---
@Kim:
Please contribute precise info, but do not touch the status picker. With what
version did you see the improvement?

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 50933] EDITING: Zooming Happening Automatically

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=50933

Rainer Bielefeld libreoff...@bielefeldundbuss.de changed:

   What|Removed |Added

 Status|NEEDINFO|RESOLVED
 Resolution|--- |WORKSFORME
 CC||LibreOffice@bielefeldundbus
   ||s.de

--- Comment #4 from Rainer Bielefeld libreoff...@bielefeldundbuss.de ---
@Kim:
I am pretty sure that you accidently did a mouse zoom. I also have this problem
from time to time using a Laptop with touch pad.
Please feel free to reopen this bug if you find evidence that we have a real
LibO bug here.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 57626] Macro assigned to print event does not execute

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57626

LeMoyne Castle lemoyne.cas...@gmail.com changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |DUPLICATE

--- Comment #7 from LeMoyne Castle lemoyne.cas...@gmail.com ---
Dup of Bug 58269 which has patch merged to master that  reinstates  Print event
registration code.

*** This bug has been marked as a duplicate of bug 58269 ***

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 58816] Macros: basic script / module error

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=58816

LeMoyne Castle lemoyne.cas...@gmail.com changed:

   What|Removed |Added

   See Also|https://bugs.freedesktop.or |
   |g/show_bug.cgi?id=58269 |

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 58541] BASIC: BASIC runtime error '423' OnSheetActivate

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=58541

LeMoyne Castle lemoyne.cas...@gmail.com changed:

   What|Removed |Added

   See Also|https://bugs.freedesktop.or |
   |g/show_bug.cgi?id=58269 |

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 58750] FORMATTING: Crash when Format Columns

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=58750

--- Comment #5 from manj_k courrier.oou.fr@googlemail.com ---
(In reply to comment #4)
 [...]
 however my test shows that the bug is not present in the 4.0 branch

Yes, I know.  ;)
Please, could you test it with the current master build?

My latest test:
Reproducible (on WinXP 32b) with
LibO-Dev_4.1.0.0.alpha0+
tinderbox: Win-x86@6 MASTER
pull time 2012-12-31 19:23:06
core:dd9655868e60af0f67de715f1f04aa0989b6b58b

The bug occurred for the first time on 2012-12-21, worked well with master
build 2012-12-20 (see Description).

[ See also: bug 58749 · bug 58751 · bug 58752 · bug 58760 ]

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 58913] Crash when creating a form with Wizard - Linux JRE 7

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=58913

--- Comment #7 from rob...@familiegrosskopf.de ---
(In reply to comment #6)
 Created attachment 72367 [details]
 bt + console logs on master

Hello Julien,

I couldn't see anything about JRE. Which Java-version do you use?
I can't confirm any crash with the form wizard, LO  4.0.0.0.beta2+ (28.12.2012)
on a Linux-rpm-32bit-system. LO works right with Java 1.6.0_29.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 58940] New: CONFIGURATION: Can not change paths used by LibreOffice

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=58940

  Priority: medium
Bug ID: 58940
  Assignee: libreoffice-bugs@lists.freedesktop.org
   Summary: CONFIGURATION: Can not change paths used by
LibreOffice
  Severity: normal
Classification: Unclassified
OS: Linux (All)
  Reporter: brucey...@live.com
  Hardware: Other
Whiteboard: BSA
Status: UNCONFIRMED
   Version: 3.6.4.3 release
 Component: Writer
   Product: LibreOffice

Problem description: 

Can not change the paths used by LO in Linux Mint 14

Steps to reproduce:
1. Tools  Options
2. LibreOffice  Paths
3. Select 'My Documents'  Edit
4. Select new folder, OK

Current behavior:

Path reverts to /home/username/

Expected behavior:

New path should save
Operating System: Linux (Other)
Version: 3.6.4.3 release

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 58921] FORMATTING: New cells conditional format missing after save and load file

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=58921

Francisco Lloret fcollo...@terra.es changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
 Ever confirmed|0   |1

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 58921] FORMATTING: New cells conditional format missing after save and load file

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=58921

--- Comment #1 from Francisco Lloret fcollo...@terra.es ---
3.5.5.3 version works OK.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 58417] Libreoffice 4.0 crashes in LanguageTag constructor at start on Ubuntu 12.10 64bit

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=58417

--- Comment #12 from LeMoyne Castle lemoyne.cas...@gmail.com ---
I submitted the min fix patch to fix a compile error I was having on Ubu 10.04
with gcc-4.4.3   Compile time type resolution was failing inside a complex
nested constructor call.  It is possible that the errors here are from a
similar compiler problem that doesn't error out at compile time, but writes
junk and errors at run-time.  A bit of a shot in the dark, but the BTs here go
right through at least one of the patched lines.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 58941] New: Support optional smartfont features

2013-01-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=58941

  Priority: medium
Bug ID: 58941
  Assignee: libreoffice-bugs@lists.freedesktop.org
   Summary: Support optional smartfont features
  Severity: enhancement
Classification: Unclassified
OS: All
  Reporter: g.duff...@gmail.com
  Hardware: All
Status: UNCONFIRMED
   Version: 4.0.0.0.beta2
 Component: UI
   Product: LibreOffice

LO should provide an interface to enable optional smartfont features like liga,
sups etc. and also to disable optional features that are enabled by default
like calt etc.
Ideally this interface covers all three smartfont technologies (Opentype,
Graphite and AAT).

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


  1   2   3   >