LibreOffice Gerrit News for core on 2014-05-20

2014-05-20 Thread gerrit
Moin!

* Open changes on master for project core changed in the last 25 hours:

 First time contributors doing great things! 
+ fdo#43157 - Clean up OSL_ASSERT, DBG_ASSERT
  in https://gerrit.libreoffice.org/9400 from Mukhiddin Yusupov
  about module sal
+ fdo#39631: Add SAL_LIKELY, SAL_UNLIKELY, SAL_HOT and SAT COLD macros.
  in https://gerrit.libreoffice.org/9407 from Michal Strnad
  about module include
+ WIP: fdo#33980 Preserve selection across all slide sorters.
  in https://gerrit.libreoffice.org/6633 from Andrzej Hunt
  about module sd
+ WIP fdo#72987 Use firebird backup format for .odb
  in https://gerrit.libreoffice.org/7299 from Andrzej Hunt
  about module connectivity
+ FDO#74886: Data labels are not preserved for Column Chart in RT file
  in https://gerrit.libreoffice.org/8098 from Umesh Kadam
  about module chart2, oox
+ FDO#75205 : Charts - Up Down bars are getting added to Line Chart.
  in https://gerrit.libreoffice.org/8360 from Umesh Kadam
  about module chart2, include, offapi, oox, xmloff
+ fdo#51525 Fix duplicate Paste Special entries in Calc.
  in https://gerrit.libreoffice.org/7784 from Andrzej Hunt
  about module officecfg
+ care for Java 1.5 support in external packages
  in https://gerrit.libreoffice.org/9312 from Douglas Mencken
  about module build, external, reportbuilder, swext
+ fdo#78663 : Corrupt: 2007 WordArt whithin a Drawing tool
  in https://gerrit.libreoffice.org/9392 from Rajashri Udhoji
  about module sw
 End of freshness 

+ fdo#78833 Improve AutoTexts for all languages (mainly FN Autotext)
  in https://gerrit.libreoffice.org/9408 from Laurent BP
  about module extras
+ upgrade to python-3.3.5
  in https://gerrit.libreoffice.org/9311 from Thomas Arnhold
  about module build, external
+ new method INetURLObject::getData() for data urls
  in https://gerrit.libreoffice.org/8737 from Christina Roßmanith
  about module include, sw, tools
+ fdo#75757 Remove inheritance to std::map from PropertyMap
  in https://gerrit.libreoffice.org/8656 from Krisztian Pinter
  about module writerfilter
+ fdo#71984 fix first line indentation in footnotes for MS Word documents
  in https://gerrit.libreoffice.org/8160 from Andras Timar
  about module filter, sw, writerfilter
+ enhance pass-by-ref plugin to detect large arguments
  in https://gerrit.libreoffice.org/9403 from Noel Grandin
  about module bridges, codemaker, compilerplugins, cui, dbaccess, include, sc, 
sd, sfx2, stoc, sw, tools, unotools, uui, vcl, writerfilter


* Merged changes on master for project core changed in the last 25 hours:

+ libavmedia.so needs to link to librt under Linux
  in https://gerrit.libreoffice.org/9402 from Jeroen Nijhof
+ fdo78474:Corruption a:graphicData has no info for the image.
  in https://gerrit.libreoffice.org/9363 from Sourav Mahajan


* Abandoned changes on master for project core changed in the last 25 hours:

+ Remove visual noise from ios
  in https://gerrit.libreoffice.org/8274 from Alexander Wilms


* Open changes needing tweaks, but being untouched for more than a week:

+ fdo#60338: Saved files have incorrect permissions on linux
  in https://gerrit.libreoffice.org/9287 from Ulrich Kitzinger
+ Lots of changes to Tango icons
  in https://gerrit.libreoffice.org/7987 from Miroslav Mazel
+ fdo#57950: Replace chained O(U)StringBuffer::append() with operator+
  in https://gerrit.libreoffice.org/8699 from Tharindu Lakmal
+ Refactored swooxmlexport tests cases. Added function to check contents.
  in https://gerrit.libreoffice.org/8390 from Nikhil Walvekar
+ fdo#77121 Header / Footer positions not preserved after RT
  in https://gerrit.libreoffice.org/9235 from Priyanka Gaikwad
+ fdo#77716 : Paragraph spacing is not preserved after RT.
  in https://gerrit.libreoffice.org/9197 from Tushar Bende
+ WIP: Converting ExtensionUpdate dialog for new format
  in https://gerrit.libreoffice.org/8506 from Rodolfo Ribeiro Gomes


Best,

Your friendly LibreOffice Gerrit Digest Mailer

Note: The bot generating this message can be found and improved here:
   
https://gerrit.libreoffice.org/gitweb?p=dev-tools.git;a=blob;f=gerritbot/send-daily-digest
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: do NOT immediately remove files from dev-downloads.libreoffice.org on update of externals!

2014-05-20 Thread Stephan Bergmann

On 05/19/2014 09:45 PM, Bjoern Michaelsen wrote:

please do NOT remove files from dev-downloads when updating externals, keep
those files around for at least 12 months! Otherwise you will break bibisect
building hard (apart from also breaking people on local branches).
So please keep whatever tarball was ever used in a LibreOffice build on the
server for at least 12 months.


Where does the value 12 come from?

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


Compilation error

2014-05-20 Thread Mihály Palenik
Hello!

I have just downloaded the source code and make but I got compilation
error. Why? I didn't modify anything.

warn:linguistic:1011:1:linguistic/source/lngsvcmgr.cxx:489: no extension
manager - should fire on mobile only
warn:chart2:1011:1:chart2/source/view/main/ShapeFactory.cxx:2132: Exception
caught. Type: N3com3sun4star3uno9ExceptionE, Message:
warn:vcl.window:1011:1:vcl/source/window/window.cxx:6869: ignoring
DeploymentException component context fails to supply service
'com.sun.star.datatransfer.clipboard.SystemClipboard' of type
'com.sun.star.datatransfer.clipboard.XSystemClipboard'
warn:sfx.control:1011:1:sfx2/source/control/dispatch.cxx:1505: Childwindow
slot missing: 10365
warn:sfx.control:1011:1:sfx2/source/control/dispatch.cxx:1505: Childwindow
slot missing: 10365
Segmentation fault (core dumped)

No core file identified in directory
/home/freetank/Hasznal/LibreOfficeCore/workdir/CppunitTest/chart2_export.test.core
To show backtraces for crashes during test execution,
enable core files with:

   ulimit -c unlimited

warn:linguistic:1027:1:linguistic/source/lngsvcmgr.cxx:489: no extension
manager - should fire on mobile only
warn:chart2:1027:1:chart2/source/view/main/ShapeFactory.cxx:2132: Exception
caught. Type: N3com3sun4star3uno9ExceptionE, Message:
warn:unotools.misc:1027:1:unotools/source/misc/mediadescriptor.cxx:736:
caught Exception an error occurred during file opening while opening
file:///home/freetank/Hasznal/LibreOfficeCore/workdir/unittest/user/config/standard.sod
warn:vcl.window:1027:1:vcl/source/window/window.cxx:6869: ignoring
DeploymentException component context fails to supply service
'com.sun.star.datatransfer.clipboard.SystemClipboard' of type
'com.sun.star.datatransfer.clipboard.XSystemClipboard'
warn:sfx.control:1027:1:sfx2/source/control/dispatch.cxx:1505: Childwindow
slot missing: 10365
warn:sfx.control:1027:1:sfx2/source/control/dispatch.cxx:1505: Childwindow
slot missing: 10365
Segmentation fault (core dumped)

No core file identified in directory
/home/freetank/Hasznal/LibreOfficeCore/workdir/CppunitTest/chart2_import.test.core
To show backtraces for crashes during test execution,
enable core files with:

   ulimit -c unlimited


Error: a unit test failed, please do one of:

export DEBUGCPPUNIT=TRUE# for exception catching
export CPPUNITTRACE=gdb --args# for interactive debugging on Linux
export CPPUNITTRACE=\[full path to devenv.exe]\ /debugexe # for
interactive debugging in Visual Studio
export VALGRIND=memcheck# for memory checking

and retry using: make CppunitTest_chart2_export


Error: a unit test failed, please do one of:

export DEBUGCPPUNIT=TRUE# for exception catching
export CPPUNITTRACE=gdb --args# for interactive debugging on Linux
export CPPUNITTRACE=\[full path to devenv.exe]\ /debugexe # for
interactive debugging in Visual Studio
export VALGRIND=memcheck# for memory checking

and retry using: make CppunitTest_chart2_import

make[1]: ***
[/home/freetank/Hasznal/LibreOfficeCore/workdir/CppunitTest/chart2_export.test]
Error 1
make[1]: *** Waiting for unfinished jobs
make[1]: ***
[/home/freetank/Hasznal/LibreOfficeCore/workdir/CppunitTest/chart2_import.test]
Error 1
make: *** [build] Error 2
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


master build fails in Chart2

2014-05-20 Thread Jean-Baptiste Faure
Hi,

Since several days my master build fails with the same error :

 [build CXX] chart2/source/controller/dialogs/dlg_NumberFormat.cxx
 In file included from 
 .../chart2/source/controller/dialogs/dlg_InsertLegend.cxx:21:0:
 .../chart2/source/controller/dialogs/res_LegendPosition.hxx:32:7: error: 
 redefinition of ‘class chart::LegendPositionResources’
  class LegendPositionResources
^
 In file included from 
 .../chart2/source/controller/inc/dlg_InsertLegend.hxx:29:0,
  from 
 .../chart2/source/controller/dialogs/dlg_InsertLegend.cxx:20:
 .../chart2/source/controller/inc/res_LegendPosition.hxx:32:7: error: previous 
 definition of ‘class chart::LegendPositionResources’
  class LegendPositionResources
^
 make[1]: *** 
 [.../workdir/CxxObject/chart2/source/controller/dialogs/dlg_InsertLegend.o] 
 Erreur 1
 make[1]: *** Attente des tâches non terminées
 make: *** [build] Erreur 2

Each time I build the master, I do a complete rebuild (make disclean -
./autogen.sh  make). As the Linux TB seem to not have the same problem,
I guess it is on my side, but I do not know how to fix that.

I build the master on Ubuntu 14.04 x86-64 with gcc 4.8. No problem when
building LO 4.2.5.0.0+.

Any advice is welcome. :-)

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


Re: do NOT immediately remove files from dev-downloads.libreoffice.org on update of externals!

2014-05-20 Thread David Tardon
Hi,

On Tue, May 20, 2014 at 08:56:08AM +0200, Stephan Bergmann wrote:
 On 05/19/2014 09:45 PM, Bjoern Michaelsen wrote:
 please do NOT remove files from dev-downloads when updating externals, keep
 those files around for at least 12 months! Otherwise you will break bibisect
 building hard (apart from also breaking people on local branches).
 So please keep whatever tarball was ever used in a LibreOffice build on the
 server for at least 12 months.
 
 Where does the value 12 come from?

I am curious about that too :-) Personally I would say do not ever
remove tarballs from dev-www!

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


[Libreoffice-commits] core.git: external/glew

2014-05-20 Thread Tor Lillqvist
 external/glew/UnpackedTarball_glew.mk |4 
 external/glew/glew-debug.patch.1  |   12 
 2 files changed, 16 insertions(+)

New commits:
commit f3a46244a07b76d633ccc7f83da3fa502612e978
Author: Tor Lillqvist t...@collabora.com
Date:   Tue May 20 10:07:43 2014 +0300

Make it actually work to build GLEW for debugging

The check in the GLEW Makefile looked at the whole $(MAKECMDGOALS),
which in our case was the string 'glew.lib debug', not just 'debug'.

Change-Id: I639395c944e8f2b443cbb55b451a4036b2c9d3bb

diff --git a/external/glew/UnpackedTarball_glew.mk 
b/external/glew/UnpackedTarball_glew.mk
index 1b100bc..64cbc5c 100644
--- a/external/glew/UnpackedTarball_glew.mk
+++ b/external/glew/UnpackedTarball_glew.mk
@@ -48,4 +48,8 @@ $(eval $(call gb_UnpackedTarball_add_patches,glew,\
 ))
 endif
 
+$(eval $(call gb_UnpackedTarball_add_patches,glew,\
+   external/glew/glew-debug.patch.1 \
+))
+
 # vim: set noet sw=4 ts=4:
diff --git a/external/glew/glew-debug.patch.1 b/external/glew/glew-debug.patch.1
new file mode 100644
index 000..c6e30a4
--- /dev/null
+++ b/external/glew/glew-debug.patch.1
@@ -0,0 +1,12 @@
+-*- Mode: Diff -*-
+--- glew/Makefile
 glew/Makefile
+@@ -65,7 +65,7 @@
+ RM  ?= rm -f
+ LN  ?= ln -sf
+ 
+-ifeq ($(MAKECMDGOALS), debug)
++ifneq (,$(filter debug,$(MAKECMDGOALS)))
+ OPT = -g
+ else
+ OPT = $(POPT)
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: vcl/source

2014-05-20 Thread Stephan Bergmann
 vcl/source/opengl/OpenGLContext.cxx |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 498c314861f0913a5b31ee29efc38aad12c3a781
Author: Stephan Bergmann sberg...@redhat.com
Date:   Tue May 20 09:41:59 2014 +0200

Hack to for now work around dpy == null crashing CppunitTest_chart2_*

...since 2fc4f5cd750cc3899ffc6b2f831edcb01e7773b4 Register OpenGL windows 
to
ScGridWindow upon file load.  Should be revisited and fixed properly.

Change-Id: I5a3d08cb750fd9ec7c98d5b85250eaabd1c7a8b2

diff --git a/vcl/source/opengl/OpenGLContext.cxx 
b/vcl/source/opengl/OpenGLContext.cxx
index 6bb7c10..20d9acd 100644
--- a/vcl/source/opengl/OpenGLContext.cxx
+++ b/vcl/source/opengl/OpenGLContext.cxx
@@ -390,7 +390,7 @@ bool OpenGLContext::ImplInit()
 #elif defined( ANDROID )
 
 #elif defined( UNX )
-m_aGLWin.ctx = glXCreateContext(m_aGLWin.dpy,
+m_aGLWin.ctx = m_aGLWin.dpy == 0 ? 0 : glXCreateContext(m_aGLWin.dpy,
  m_aGLWin.vi,
  0,
  GL_TRUE);
@@ -690,7 +690,7 @@ SystemWindowData OpenGLContext::generateWinData(Window* 
pParent)
 
 Display *dpy = reinterpret_castDisplay*(sysData-pDisplay);
 
-if( !glXQueryExtension( dpy, NULL, NULL ) )
+if( dpy == 0 || !glXQueryExtension( dpy, NULL, NULL ) )
 return aWinData;
 
 XLIB_Window win = sysData-aWindow;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: do NOT immediately remove files from dev-downloads.libreoffice.org on update of externals!

2014-05-20 Thread Bjoern Michaelsen
On Tue, May 20, 2014 at 09:18:54AM +0200, David Tardon wrote:
 On Tue, May 20, 2014 at 08:56:08AM +0200, Stephan Bergmann wrote:
  On 05/19/2014 09:45 PM, Bjoern Michaelsen wrote:
  please do NOT remove files from dev-downloads when updating externals, keep
  those files around for at least 12 months! Otherwise you will break 
  bibisect
  building hard (apart from also breaking people on local branches).
  So please keep whatever tarball was ever used in a LibreOffice build on the
  server for at least 12 months.
  
  Where does the value 12 come from?
 
 I am curious about that too :-) Personally I would say do not ever
 remove tarballs from dev-www!

Well, 12 months was just a minimum. I have not issue with using never
instead. ;)

12 months was just a rough estimate of two major releases which would be good
enough for bibisect at least.

Best,

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


[Libreoffice-commits] core.git: cppuhelper/source desktop/unx hwpfilter/source rsc/source sal/osl sal/qa

2014-05-20 Thread Arnaud Versini
 cppuhelper/source/findsofficepath.c |   12 -
 desktop/unx/source/splashx.c|2 
 hwpfilter/source/grammar.cxx|   73 
 rsc/source/rscpp/cpp3.c |3 -
 sal/osl/unx/socket.c|8 ---
 sal/qa/osl/process/osl_Thread.cxx   |5 --
 sal/qa/rtl/ostring/rtl_str.cxx  |   14 ++
 7 files changed, 53 insertions(+), 64 deletions(-)

New commits:
commit cd11bc699ac50af4f560ed5f2e5e7903de0898b8
Author: Arnaud Versini arnaud.vers...@gmail.com
Date:   Fri May 9 10:38:04 2014 +0200

C string usage improvment

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

diff --git a/cppuhelper/source/findsofficepath.c 
b/cppuhelper/source/findsofficepath.c
index 1bd7368..903b91b 100644
--- a/cppuhelper/source/findsofficepath.c
+++ b/cppuhelper/source/findsofficepath.c
@@ -135,8 +135,10 @@ static char* platformSpecific()
 env = getenv( PATHVARNAME );
 if (env == NULL)
 return NULL;
-str = (char*) malloc( strlen( env ) + 1 );
-strcpy( str, env );
+
+str = strdup( env );
+if (str == NULL)
+return NULL;
 
 /* get the tokens separated by ':' */
 dir = strtok( str, PATHSEPARATOR );
@@ -145,6 +147,12 @@ static char* platformSpecific()
 {
 /* construct soffice file path */
 file = (char*) malloc( strlen( dir ) + strlen( APPENDIX ) + 1 );
+if (file == NULL)
+{
+free(str);
+return NULL;
+}
+
 strcpy( file, dir );
 strcat( file, APPENDIX );
 
diff --git a/desktop/unx/source/splashx.c b/desktop/unx/source/splashx.c
index 24b2152..288656c 100644
--- a/desktop/unx/source/splashx.c
+++ b/desktop/unx/source/splashx.c
@@ -582,7 +582,7 @@ static void splash_load_image( struct splash* splash, 
rtl_uString* pUAppPath )
 pLang = ustr_to_str (pLoc-Language);
 pCountry = ustr_to_str (pLoc-Country);
 
-nLocSize = strlen (pLang-buffer) + strlen (pCountry-buffer) + 8;
+nLocSize = strlen (pLang-buffer) + strlen (pCountry-buffer) + 3;
 pLocale = malloc (nLocSize);
 pLocale[0] = '-';
 strcpy (pLocale + 1, pLang-buffer);
diff --git a/hwpfilter/source/grammar.cxx b/hwpfilter/source/grammar.cxx
index be4b591..fccd75f 100644
--- a/hwpfilter/source/grammar.cxx
+++ b/hwpfilter/source/grammar.cxx
@@ -44,13 +44,6 @@ Node *top=0L;
 
 int Node::count = 0;
 
-#define allocValue(x,y) \
-x = (char *)malloc( strlen(y) +1 ); \
-strcpy(x,y)
-
-#define allocChar(x,y) \
-x = (char *)malloc(2); \
-sprintf(x,%c,y)
 #ifdef PARSE_DEBUG
 #define debug printf
 #else
@@ -836,55 +829,55 @@ yyreduce:
   switch (yyn) {
 
 case 1:
-{ yyval.ptr = new Node(ID_IDENTIFIER); allocValue(yyval.ptr-value , 
yyvsp[0].str); debug(Identifier : %s\n,yyvsp[0].str); 
nodelist.push_back(yyval.ptr); ;
+{ yyval.ptr = new Node(ID_IDENTIFIER); yyval.ptr-value = 
strdup(yyvsp[0].str); debug(Identifier : %s\n,yyvsp[0].str); 
nodelist.push_back(yyval.ptr); ;
 break;}
 case 2:
-{ yyval.ptr = new Node(ID_IDENTIFIER); allocValue(yyval.ptr-value , 
yyvsp[0].str); debug(Identifier : %s\n,yyvsp[0].str); 
nodelist.push_back(yyval.ptr); ;
+{ yyval.ptr = new Node(ID_IDENTIFIER); yyval.ptr-value = 
strdup(yyvsp[0].str); debug(Identifier : %s\n,yyvsp[0].str); 
nodelist.push_back(yyval.ptr); ;
 break;}
 case 3:
-{ yyval.ptr = new Node(ID_IDENTIFIER); allocValue(yyval.ptr-value , 
yyvsp[0].str); debug(Identifier : %s\n,yyvsp[0].str); 
nodelist.push_back(yyval.ptr); ;
+{ yyval.ptr = new Node(ID_IDENTIFIER); yyval.ptr-value = 
strdup(yyvsp[0].str); debug(Identifier : %s\n,yyvsp[0].str); 
nodelist.push_back(yyval.ptr); ;
 break;}
 case 4:
-{ yyval.ptr = new Node(ID_IDENTIFIER); allocValue(yyval.ptr-value , 
yyvsp[0].str); debug(Identifier : %s\n,yyvsp[0].str); 
nodelist.push_back(yyval.ptr); ;
+{ yyval.ptr = new Node(ID_IDENTIFIER); yyval.ptr-value = 
strdup(yyvsp[0].str); debug(Identifier : %s\n,yyvsp[0].str); 
nodelist.push_back(yyval.ptr); ;
 break;}
 case 5:
-{ yyval.ptr = new Node(ID_IDENTIFIER); allocValue(yyval.ptr-value , 
yyvsp[0].str); debug(Identifier : %s\n,yyvsp[0].str); 
nodelist.push_back(yyval.ptr); ;
+{ yyval.ptr = new Node(ID_IDENTIFIER); yyval.ptr-value = 
strdup(yyvsp[0].str); debug(Identifier : %s\n,yyvsp[0].str); 
nodelist.push_back(yyval.ptr); ;
 break;}
 case 6:
-{ yyval.ptr = new Node(ID_IDENTIFIER); allocValue(yyval.ptr-value , 
yyvsp[0].str); debug(Identifier : %s\n,yyvsp[0].str); 
nodelist.push_back(yyval.ptr); ;
+{ yyval.ptr = new Node(ID_IDENTIFIER); yyval.ptr-value = 
strdup(yyvsp[0].str); debug(Identifier : %s\n,yyvsp[0].str); 
nodelist.push_back(yyval.ptr); ;
 break;}
 case 7:
-{ yyval.ptr = new Node(ID_IDENTIFIER); allocValue(yyval.ptr-value , 
yyvsp[0].str); debug(Identifier : %s\n,yyvsp[0].str); 
nodelist.push_back(yyval.ptr); ;
+{ yyval.ptr = new Node(ID_IDENTIFIER); yyval.ptr-value = 
strdup(yyvsp[0].str); debug(Identifier : 

[Libreoffice-commits] core.git: sal/osl sal/qa

2014-05-20 Thread Stephan Bergmann
 sal/osl/unx/socket.c   |4 ++--
 sal/qa/rtl/ostring/rtl_str.cxx |   10 +-
 2 files changed, 7 insertions(+), 7 deletions(-)

New commits:
commit fe39041f98b1abb3a23b43f399963487fe5b66b9
Author: Stephan Bergmann sberg...@redhat.com
Date:   Tue May 20 10:03:18 2014 +0200

Remove unnecessary casts

Change-Id: Iafc7727a22d657dfb6bc97ed767907cce3679364

diff --git a/sal/osl/unx/socket.c b/sal/osl/unx/socket.c
index 052ace9..d11297f 100644
--- a/sal/osl/unx/socket.c
+++ b/sal/osl/unx/socket.c
@@ -790,7 +790,7 @@ static oslHostAddr _osl_hostentToHostAddr (const struct 
hostent *he)
 
 if (_osl_isFullQualifiedDomainName(he-h_name))
 {
-cn= (sal_Char *)strdup(he-h_name);
+cn= strdup(he-h_name);
 OSL_ASSERT(cn);
 if (cn == NULL)
 return ((oslHostAddr)NULL);
@@ -887,7 +887,7 @@ oslHostAddr SAL_CALL osl_psz_createHostAddr (
 if ((pszHostname == NULL) || (pAddr == NULL))
 return ((oslHostAddr)NULL);
 
-cn = (sal_Char *) strdup(pszHostname);
+cn = strdup(pszHostname);
 OSL_ASSERT(cn);
 if (cn == NULL)
 return ((oslHostAddr)NULL);
diff --git a/sal/qa/rtl/ostring/rtl_str.cxx b/sal/qa/rtl/ostring/rtl_str.cxx
index 759d30e..4ba2962 100644
--- a/sal/qa/rtl/ostring/rtl_str.cxx
+++ b/sal/qa/rtl/ostring/rtl_str.cxx
@@ -711,7 +711,7 @@ namespace rtl_str
 void trim_WithLength_001()
 {
 char const *pStr =   trim this;
-sal_Char *pStr2 = (sal_Char*)strdup(pStr);
+sal_Char *pStr2 = strdup(pStr);
 if (pStr2)
 {
 rtl_str_trim_WithLength( pStr2, 2 );
@@ -724,7 +724,7 @@ namespace rtl_str
 void trim_WithLength_002()
 {
 char const *pStr = trim this;
-sal_Char *pStr2 = (sal_Char*)strdup(pStr);
+sal_Char *pStr2 = strdup(pStr);
 if (pStr2)
 {
 rtl_str_trim_WithLength( pStr2, 5 );
@@ -737,7 +737,7 @@ namespace rtl_str
 void trim_WithLength_003()
 {
 char const *pStr =  trim   this;
-sal_Char *pStr2 = (sal_Char*)strdup(pStr);
+sal_Char *pStr2 = strdup(pStr);
 if (pStr2)
 {
 strcpy(pStr2, pStr);
@@ -751,7 +751,7 @@ namespace rtl_str
 void trim_WithLength_004()
 {
 char const *pStr = \r\n\t \n\rtrim  \n this;
-sal_Char *pStr2 = (sal_Char*)strdup(pStr);
+sal_Char *pStr2 = strdup(pStr);
 if (pStr2)
 {
 rtl_str_trim_WithLength( pStr2, 17 );
@@ -764,7 +764,7 @@ namespace rtl_str
 void trim_WithLength_005()
 {
 char const *pStr = \r\n\t \n\rtrim \t this \n\r\t\t ;
-sal_Char *pStr2 = (sal_Char*)strdup(pStr);
+sal_Char *pStr2 = strdup(pStr);
 if (pStr2)
 {
 rtl_str_trim_WithLength( pStr2, strlen(pStr2) );
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: master build fails in Chart2

2014-05-20 Thread Stephan Bergmann

On 05/20/2014 08:24 AM, Jean-Baptiste Faure wrote:

[build CXX] chart2/source/controller/dialogs/dlg_NumberFormat.cxx
In file included from 
.../chart2/source/controller/dialogs/dlg_InsertLegend.cxx:21:0:
.../chart2/source/controller/dialogs/res_LegendPosition.hxx:32:7: error: 
redefinition of ‘class chart::LegendPositionResources’
  class LegendPositionResources
^
In file included from 
.../chart2/source/controller/inc/dlg_InsertLegend.hxx:29:0,
  from 
.../chart2/source/controller/dialogs/dlg_InsertLegend.cxx:20:
.../chart2/source/controller/inc/res_LegendPosition.hxx:32:7: error: previous 
definition of ‘class chart::LegendPositionResources’
  class LegendPositionResources
^
make[1]: *** 
[.../workdir/CxxObject/chart2/source/controller/dialogs/dlg_InsertLegend.o] 
Erreur 1
make[1]: *** Attente des tâches non terminées
make: *** [build] Erreur 2


Do you have a leftover 
chart2/source/controller/dialogs/res_LegendPosition.hxx that should have 
been removed long ago by 
http://cgit.freedesktop.org/libreoffice/core/commit/?id=27eb33d753eedfa4215eaf8325bc0ee9baeb7417 
remove unnecessary empty dtor, etc.?


Stephan

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


[Libreoffice-commits] core.git: vcl/source

2014-05-20 Thread Tor Lillqvist
 vcl/source/opengl/OpenGLContext.cxx |9 +
 1 file changed, 9 insertions(+)

New commits:
commit c3ed5f6d3348fa2b6551d0d942632d951555a01d
Author: Tor Lillqvist t...@collabora.com
Date:   Tue May 20 10:57:44 2014 +0300

Fail early in OpenGLContext::ImplInit() on platforms where not yet 
implemented

So far this code apparently has not been invoked except in the very
special use cases for which it has been developed, and which
presumably aren't such that one would come across in normal
LibreOffice use.

But now it gets invoked unconditionally, in some unit tests even, and
letting it call glewInit(), which calls glGetString(), without any
OpenGL context, leads to a crash. So instead, explicitly fail early on
OS X (and iOS and Android).

Change-Id: Ia4547cc1ddff1aaa0190fcdb69506ad444214b7a

diff --git a/vcl/source/opengl/OpenGLContext.cxx 
b/vcl/source/opengl/OpenGLContext.cxx
index 20d9acd..0ce85cb 100644
--- a/vcl/source/opengl/OpenGLContext.cxx
+++ b/vcl/source/opengl/OpenGLContext.cxx
@@ -385,10 +385,19 @@ bool OpenGLContext::ImplInit()
 m_aGLWin.hDC = GetDC(m_aGLWin.hWnd);
 #elif defined( MACOSX )
 
+SAL_INFO(vcl.opengl, OpenGLContext not implemented yet for OS X);
+return false;
+
 #elif defined( IOS )
 
+SAL_INFO(vcl.opengl, OpenGLContext not implemented yet for iOS);
+return false;
+
 #elif defined( ANDROID )
 
+SAL_INFO(vcl.opengl, OpenGLContext not implemented yet for Android);
+return false;
+
 #elif defined( UNX )
 m_aGLWin.ctx = m_aGLWin.dpy == 0 ? 0 : glXCreateContext(m_aGLWin.dpy,
  m_aGLWin.vi,
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: Compilation error

2014-05-20 Thread Stephan Bergmann

On 05/20/2014 08:58 AM, Mihály Palenik wrote:

make[1]: ***
[/home/freetank/Hasznal/LibreOfficeCore/workdir/CppunitTest/chart2_export.test]
Error 1
make[1]: *** Waiting for unfinished jobs
make[1]: ***
[/home/freetank/Hasznal/LibreOfficeCore/workdir/CppunitTest/chart2_import.test]
Error 1
make: *** [build] Error 2


http://cgit.freedesktop.org/libreoffice/core/commit/?id=498c314861f0913a5b31ee29efc38aad12c3a781 
Hack to for now work around dpy == null crashing CppunitTest_chart2_* 
should hopefully work around that for now.


Stephan

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


[Libreoffice-commits] core.git: Branch 'feature/chart-3d-chart2' - 3 commits - avmedia/source external/collada2gltf

2014-05-20 Thread Jan Holesovsky
 avmedia/source/framework/modeltools.cxx  |   11 
++
 external/collada2gltf/write-only-.dae-file-which-we-need.patch.0 |2 -
 2 files changed, 8 insertions(+), 5 deletions(-)

New commits:
commit 7110b392942a73094cecbabb77a44c684d2cd1fb
Author: Jan Holesovsky ke...@collabora.com
Date:   Tue May 20 09:37:47 2014 +0200

collada2gltf: Ignore case of the extension.

Change-Id: If7f39a1a0537c561373137e6a9dcdb22b48037f0

diff --git a/avmedia/source/framework/modeltools.cxx 
b/avmedia/source/framework/modeltools.cxx
index 2366dc7..1adef63 100644
--- a/avmedia/source/framework/modeltools.cxx
+++ b/avmedia/source/framework/modeltools.cxx
@@ -140,7 +140,9 @@ bool Embed3DModel( const uno::Referenceframe::XModel 
xModel,
 {
 OUString sSource = rSourceURL;
 #ifdef ENABLE_COLLADA2GLTF
-if (rSourceURL.endsWith(.dae) || rSourceURL.endsWith(.kmz))
+const bool bIsDAE = rSourceURL.endsWithIgnoreAsciiCase(.dae);
+const bool bIsKMZ = rSourceURL.endsWithIgnoreAsciiCase(.kmz);
+if (bIsDAE || bIsKMZ)
 {
 OUString sName;
 
::utl::LocalFileHelper::ConvertPhysicalNameToURL(::utl::TempFile::CreateTempName(),
 sName);
@@ -152,7 +154,7 @@ bool Embed3DModel( const uno::Referenceframe::XModel 
xModel,
 std::shared_ptr GLTF::GLTFAsset asset(new GLTF::GLTFAsset());
 asset-setInputFilePath(sSourcePath);
 
-if (rSourceURL.endsWith(.kmz))
+if (bIsKMZ)
 {
 std::string strDaeFilePath = 
GLTF::Kmz2Collada()(asset-getInputFilePath());
 if (strDaeFilePath == )
commit 95adbe9fbd156ed0227aca98a3fe7ac9664f09b8
Author: Matúš Kukan matus.ku...@collabora.com
Date:   Tue May 20 02:48:43 2014 +0200

fix .kmz support on Windows; no idea why this int was unsigned

Change-Id: I08fa1c3956b48bcd65edb74add010fafa17bfe82

diff --git a/external/collada2gltf/write-only-.dae-file-which-we-need.patch.0 
b/external/collada2gltf/write-only-.dae-file-which-we-need.patch.0
index 68980e2..56ff909 100644
--- a/external/collada2gltf/write-only-.dae-file-which-we-need.patch.0
+++ b/external/collada2gltf/write-only-.dae-file-which-we-need.patch.0
@@ -74,7 +74,7 @@ index d193784..3b5bdfc 100644
 +continue;
  
 -FILE *fp = fopen(strFilePath.c_str(),wb);
-+unsigned int pos = strlen(pFileName) - 1;
++int pos = strlen(pFileName) - 1;
 +while (pos = 0) {
 +if (pFileName[pos] == '\\' || pFileName[pos] == '/')
 +break;
commit e1fff3d529aa3bb0a8b559d642cc2f5f55416d8d
Author: Matúš Kukan matus.ku...@collabora.com
Date:   Tue May 20 02:34:38 2014 +0200

collada2gltf: output bundle path needs to be in url format

Otherwise it does not work on Windows.

It's strange though that input path has to be converted from url to
native path format. At least that was my observation.

Change-Id: Ia8d9e9719d927cab2ae4d928eb8066cab4588281

diff --git a/avmedia/source/framework/modeltools.cxx 
b/avmedia/source/framework/modeltools.cxx
index 9b48552..2366dc7 100644
--- a/avmedia/source/framework/modeltools.cxx
+++ b/avmedia/source/framework/modeltools.cxx
@@ -142,7 +142,8 @@ bool Embed3DModel( const uno::Referenceframe::XModel 
xModel,
 #ifdef ENABLE_COLLADA2GLTF
 if (rSourceURL.endsWith(.dae) || rSourceURL.endsWith(.kmz))
 {
-OUString sName = ::utl::TempFile::CreateTempName();
+OUString sName;
+
::utl::LocalFileHelper::ConvertPhysicalNameToURL(::utl::TempFile::CreateTempName(),
 sName);
 // remove .tmp extension
 sName = sName.copy(0, sName.getLength() - 4);
 const INetURLObject aSourceURLObj(rSourceURL);
@@ -162,7 +163,7 @@ bool Embed3DModel( const uno::Referenceframe::XModel 
xModel,
 GLTF::COLLADA2GLTFWriter writer(asset);
 writer.write();
 // Path to the .json file created by COLLADA2GLTFWriter
-::utl::LocalFileHelper::ConvertPhysicalNameToURL(sName + / + 
GetFilename(sName) + .json, sSource);
+sSource = sName + / + GetFilename(sName) + .json;
 }
 #endif
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: sw/qa sw/source

2014-05-20 Thread sushil_shinde
 sw/qa/extras/ooxmlexport/data/fdo78654.docx  |binary
 sw/qa/extras/ooxmlexport/ooxmlexport.cxx |   11 +++
 sw/source/filter/ww8/docxattributeoutput.cxx |2 ++
 3 files changed, 13 insertions(+)

New commits:
commit d2a19aad658cc0e8939c16b5137c15727c3cdff9
Author: sushil_shinde sushil.shi...@synerzip.com
Date:   Thu May 15 19:22:12 2014 +0530

fdo#78654 LO was exporting corrupt docx file.

   If docx file contains multiple hyperlinks in one paragraph
   and one of the hyperlink contains PAGEREF field. End tag
   for PAGEREF field was not exported hence it was causing
   corruption in exported file.
   Fixed export part of field end tags in case of PAGEREF.
   Added unit test.

Change-Id: I18a3625213bf7269e3f441859a27acc38cf6ff45
Reviewed-on: https://gerrit.libreoffice.org/9364
Reviewed-by: Miklos Vajna vmik...@collabora.co.uk
Tested-by: Miklos Vajna vmik...@collabora.co.uk

diff --git a/sw/qa/extras/ooxmlexport/data/fdo78654.docx 
b/sw/qa/extras/ooxmlexport/data/fdo78654.docx
new file mode 100644
index 000..81d7bd0
Binary files /dev/null and b/sw/qa/extras/ooxmlexport/data/fdo78654.docx differ
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
index 67ccaae..7f3a5b7 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
@@ -3370,6 +3370,17 @@ DECLARE_OOXMLEXPORT_TEST(testFDO78384,fdo78384.docx)
 assertXPath(pXmlDoc, 
/w:document/w:body/w:p[1]/w:r[1]/w:rPr/w:rFonts,ascii,Wingdings);
 }
 
+DECLARE_OOXMLEXPORT_TEST(testFDO78654 , fdo78654.docx)
+{
+xmlDocPtr pXmlDoc = parseExport(word/document.xml);
+if (!pXmlDoc)
+return;
+// In case of two Hyperlink tags in one paragraph and one of them
+// contains PAGEREF field then field end tag was missing from hyperlink.
+assertXPath ( pXmlDoc, 
/w:document/w:body/w:p[2]/w:hyperlink[3]/w:r[5]/w:fldChar, fldCharType, 
end );
+}
+
+
 DECLARE_OOXMLEXPORT_TEST(testfdo78599,fdo78599.docx)
 {
  xmlDocPtr pXmlDoc = parseExport(word/document.xml);
diff --git a/sw/source/filter/ww8/docxattributeoutput.cxx 
b/sw/source/filter/ww8/docxattributeoutput.cxx
index 4c6c7d0..19807f4 100644
--- a/sw/source/filter/ww8/docxattributeoutput.cxx
+++ b/sw/source/filter/ww8/docxattributeoutput.cxx
@@ -823,6 +823,8 @@ void DocxAttributeOutput::EndRun()
 FSNS( XML_w, XML_fldCharType ), separate,
 FSEND );
 m_pSerializer-endElementNS( XML_w, XML_r );
+// At start of every PAGEREF field m_endPageRef value should be 
true.
+m_endPageRef = true;
 }
 
 m_pSerializer-startElementNS( XML_w, XML_r, FSEND );
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: svl/source

2014-05-20 Thread Eike Rathke
 svl/source/numbers/zforfind.cxx |   85 +++-
 1 file changed, 59 insertions(+), 26 deletions(-)

New commits:
commit 43624d9370c4384f71c6b13fed900eaec222cf64
Author: Eike Rathke er...@redhat.com
Date:   Tue May 20 10:28:58 2014 +0200

resolved fdo#34724 Jan1 or 1Jan without separating character is not date

Change-Id: I2cf02a26b81fa634c842df455de669f1c60241bc

diff --git a/svl/source/numbers/zforfind.cxx b/svl/source/numbers/zforfind.cxx
index 6d6b9d5..fef69c5 100644
--- a/svl/source/numbers/zforfind.cxx
+++ b/svl/source/numbers/zforfind.cxx
@@ -2054,16 +2054,25 @@ bool ImpSvNumberInputScan::ScanStartString( const 
OUString rString,
 }
 else
 {
-nMonth = GetMonth(rString, nPos);
-if ( nMonth )// month (Jan 1)?
+const sal_Int32 nMonthStart = nPos;
+short nTempMonth = GetMonth(rString, nPos);
+if ( nTempMonth )// month (Jan 1)?
 {
-eScannedType = NUMBERFORMAT_DATE;   // !!! it IS a date !!!
-nMonthPos = 1;  // month at the beginning
-if ( nMonth  0 )
+if (nPos  rString.getLength()) // Jan1 without separator 
is not a date
 {
-SkipChar( '.', rString, nPos ); // abbreviated
+eScannedType = NUMBERFORMAT_DATE;   // !!! it IS a date !!!
+nMonth = nTempMonth;
+nMonthPos = 1;  // month at the beginning
+if ( nMonth  0 )
+{
+SkipChar( '.', rString, nPos ); // abbreviated
+}
+SkipBlanks(rString, nPos);
+}
+else
+{
+nPos = nMonthStart; // rewind month
 }
-SkipBlanks(rString, nPos);
 }
 else
 {
@@ -2089,15 +2098,23 @@ bool ImpSvNumberInputScan::ScanStartString( const 
OUString rString,
 SkipString( 
pFormatter-GetLocaleData()-getLongDateDayOfWeekSep(), rString, nPos );
 }
 SkipBlanks(rString, nPos);
-nMonth = GetMonth(rString, nPos);
-if ( nMonth ) // month (Jan 1)?
+nTempMonth = GetMonth(rString, nPos);
+if ( nTempMonth ) // month (Jan 1)?
 {
-nMonthPos = 1; // month a the beginning
-if ( nMonth  0 )
+if (nPos  rString.getLength()) // Jan1 
without separator is not a date
 {
-SkipChar( '.', rString, nPos ); // abbreviated
+nMonth = nTempMonth;
+nMonthPos = 1; // month a the beginning
+if ( nMonth  0 )
+{
+SkipChar( '.', rString, nPos ); // abbreviated
+}
+SkipBlanks(rString, nPos);
+}
+else
+{
+nPos = nMonthStart; // rewind month
 }
-SkipBlanks(rString, nPos);
 }
 }
 if (!nMonth)
@@ -2282,6 +2299,7 @@ bool ImpSvNumberInputScan::ScanMidString( const OUString 
rString,
 }
 }
 
+const sal_Int32 nMonthStart = nPos;
 short nTempMonth = GetMonth(rString, nPos); // month in the middle (10 
Jan 94)
 if (nTempMonth)
 {
@@ -2294,15 +2312,22 @@ bool ImpSvNumberInputScan::ScanMidString( const 
OUString rString,
 {
 return MatchedReturn();
 }
-eScannedType = NUMBERFORMAT_DATE;   // !!! it IS a date
-nMonth = nTempMonth;
-nMonthPos = 2;  // month in the middle
-if ( nMonth  0 )
+if (nMonthStart  0  nPos  rString.getLength())  // 10Jan or Jan94 
without separator are not dates
+{
+eScannedType = NUMBERFORMAT_DATE;   // !!! it IS a date
+nMonth = nTempMonth;
+nMonthPos = 2;  // month in the middle
+if ( nMonth  0 )
+{
+SkipChar( '.', rString, nPos ); // abbreviated
+}
+SkipString( pLoc-getLongDateMonthSep(), rString, nPos );
+SkipBlanks(rString, nPos);
+}
+else
 {
-SkipChar( '.', rString, nPos ); // abbreviated
+nPos = nMonthStart; // rewind month
 }
-SkipString( pLoc-getLongDateMonthSep(), rString, nPos );
-SkipBlanks(rString, nPos);
 }
 
 if ( SkipChar('E', rString, nPos) ||// 10E, 10e, 10,Ee
@@ -2630,6 +2655,7 @@ bool 

[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-4.2' - xmloff/source

2014-05-20 Thread Michael Stahl
 xmloff/source/text/txtfldi.cxx |   15 ++-
 1 file changed, 10 insertions(+), 5 deletions(-)

New commits:
commit 6a66844b611dce6d3c7962b5f31b02cbc5d2cc9c
Author: Michael Stahl mst...@redhat.com
Date:   Sat Feb 15 00:24:26 2014 +0100

fdo#74524: ODF import: fix unhandled exception in annotation import 
cp#175

Crash with unhandled UnknownPropertyException when trying to get Name
property from a PageNumber field.

(cherry picked from commit cf26d7614364176742ae0303efb4499a37ae04c8)

Change-Id: I501cc138caba034c85630519014b94d783ab8f39

diff --git a/xmloff/source/text/txtfldi.cxx b/xmloff/source/text/txtfldi.cxx
index 9d62939..349cd37 100644
--- a/xmloff/source/text/txtfldi.cxx
+++ b/xmloff/source/text/txtfldi.cxx
@@ -3735,12 +3735,17 @@ void XMLAnnotationImportContext::EndElement()
 while (xFields-hasMoreElements())
 {
 uno::Referencebeans::XPropertySet 
xCurrField(xFields-nextElement(), uno::UNO_QUERY);
-OUString aFieldName;
-xCurrField-getPropertyValue(sPropertyName) = aFieldName;
-if ( aFieldName == aName )
+uno::Referencebeans::XPropertySetInfo const xInfo(
+xCurrField-getPropertySetInfo());
+if (xInfo-hasPropertyByName(sPropertyName))
 {
-xPrevField.set( xCurrField, uno::UNO_QUERY );
-break;
+OUString aFieldName;
+xCurrField-getPropertyValue(sPropertyName) = 
aFieldName;
+if (aFieldName == aName)
+{
+xPrevField.set( xCurrField, uno::UNO_QUERY );
+break;
+}
 }
 }
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: 2 commits - svl/source sw/qa sw/source

2014-05-20 Thread Ravindra Vidhate
 svl/source/numbers/zforfind.cxx  |   20 +---
 sw/qa/extras/ooxmlexport/data/fdo78659.docx  |binary
 sw/qa/extras/ooxmlexport/ooxmlexport.cxx |9 +
 sw/source/filter/ww8/docxattributeoutput.cxx |2 +-
 4 files changed, 27 insertions(+), 4 deletions(-)

New commits:
commit 1870ed0e4f63e9c58476e7dcd160e04eb866e0f5
Author: Ravindra Vidhate ravindra.vidh...@synerzip.com
Date:   Thu May 15 16:12:23 2014 +0530

fdo#78659:FIle geting corrupted after Roundtrip due to fieldchar.

In a hyperlink, the field with fieldchartype=end is getting addeded 
eventhough there is no begin for that field.

Change-Id: I50df6efc63fc9a66b146d4a861ae4bd654b4a3c6
Reviewed-on: https://gerrit.libreoffice.org/9368
Tested-by: Miklos Vajna vmik...@collabora.co.uk
Reviewed-by: Miklos Vajna vmik...@collabora.co.uk

diff --git a/sw/qa/extras/ooxmlexport/data/fdo78659.docx 
b/sw/qa/extras/ooxmlexport/data/fdo78659.docx
new file mode 100644
index 000..55d6d08
Binary files /dev/null and b/sw/qa/extras/ooxmlexport/data/fdo78659.docx differ
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
index 7f3a5b7..f70f58d 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
@@ -3362,6 +3362,15 @@ DECLARE_OOXMLEXPORT_TEST(testFDO78284, fdo78284.docx)
 image/png);
 }
 
+DECLARE_OOXMLEXPORT_TEST(testFDO78659, fdo78659.docx)
+{
+xmlDocPtr pXmlDoc = parseExport(word/document.xml);
+if (!pXmlDoc)
+return;
+
+assertXPath(pXmlDoc, 
/w:document[1]/w:body[1]/w:tbl[8]/w:tr[6]/w:tc[1]/w:p[1]/w:hyperlink[1]/w:r[3]/w:fldChar[1],
 0);
+}
+
 DECLARE_OOXMLEXPORT_TEST(testFDO78384,fdo78384.docx)
 {
 xmlDocPtr pXmlDoc = parseExport(word/document.xml);
diff --git a/sw/source/filter/ww8/docxattributeoutput.cxx 
b/sw/source/filter/ww8/docxattributeoutput.cxx
index 19807f4..6608149 100644
--- a/sw/source/filter/ww8/docxattributeoutput.cxx
+++ b/sw/source/filter/ww8/docxattributeoutput.cxx
@@ -1866,7 +1866,7 @@ bool DocxAttributeOutput::StartURL( const OUString rUrl, 
const OUString rTarge
 bool DocxAttributeOutput::EndURL()
 {
 m_closeHyperlinkInThisRun = true;
-if(m_hyperLinkAnchor !=   m_hyperLinkAnchor.startsWith(_Toc))
+if(m_startedHyperlink  m_hyperLinkAnchor !=   
m_hyperLinkAnchor.startsWith(_Toc))
 {
 m_endPageRef = true;
 }
commit 76531b80ea49e9665c628a2e5f8b82280ef80704
Author: Eike Rathke er...@redhat.com
Date:   Tue May 20 10:49:02 2014 +0200

take a shortcut on common separating ASCII characters

Change-Id: I991801b9a0b7be6024612c8f29262f2ebd8e6a64

diff --git a/svl/source/numbers/zforfind.cxx b/svl/source/numbers/zforfind.cxx
index fef69c5..0385970 100644
--- a/svl/source/numbers/zforfind.cxx
+++ b/svl/source/numbers/zforfind.cxx
@@ -459,14 +459,28 @@ bool ImpSvNumberInputScan::StringContainsWord( const 
OUString rWhat,
  * form? */
 
 // Check simple ASCII first before invoking i18n or anything else.
-if (rtl::isAsciiAlphanumeric( rString[nPos] ))
+const sal_Unicode c = rString[nPos];
+
+// Common separating ASCII characters in date context.
+switch (c)
+{
+case ' ':
+case '-':
+case '.':
+case '/':
+return true;
+default:
+;   // nothing
+}
+
+if (rtl::isAsciiAlphanumeric( c ))
 return false;   // Alpha or numeric is not word gap.
 
 sal_Int32 nIndex = nPos;
-const sal_uInt32 c = rString.iterateCodePoints( nIndex);
+const sal_uInt32 uc = rString.iterateCodePoints( nIndex);
 if (nPos+1  nIndex)
 return true;// Surrogate, assume these to be new words.
-(void)c;
+(void)uc;
 
 const sal_Int32 nType = pFormatter-GetCharClass()-getCharacterType( 
rString, nPos);
 using namespace ::com::sun::star::i18n;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: sw/qa

2014-05-20 Thread Miklos Vajna
 sw/qa/extras/ooxmlexport/data/fdo78659.docx |binary
 sw/qa/extras/ooxmlexport/ooxmlexport.cxx|2 +-
 2 files changed, 1 insertion(+), 1 deletion(-)

New commits:
commit e4740dbecfce958c2c707d8cc92e6dbe52f4b71b
Author: Miklos Vajna vmik...@collabora.co.uk
Date:   Tue May 20 10:41:06 2014 +0200

fdo#78659 make testcase minimal

Change-Id: I393b413528685276bffd35fb7fe5ce8c742314b7

diff --git a/sw/qa/extras/ooxmlexport/data/fdo78659.docx 
b/sw/qa/extras/ooxmlexport/data/fdo78659.docx
index 55d6d08..cf08003 100644
Binary files a/sw/qa/extras/ooxmlexport/data/fdo78659.docx and 
b/sw/qa/extras/ooxmlexport/data/fdo78659.docx differ
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
index f70f58d..90021c5 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
@@ -3368,7 +3368,7 @@ DECLARE_OOXMLEXPORT_TEST(testFDO78659, fdo78659.docx)
 if (!pXmlDoc)
 return;
 
-assertXPath(pXmlDoc, 
/w:document[1]/w:body[1]/w:tbl[8]/w:tr[6]/w:tc[1]/w:p[1]/w:hyperlink[1]/w:r[3]/w:fldChar[1],
 0);
+assertXPath(pXmlDoc, 
/w:document[1]/w:body[1]/w:tbl[1]/w:tr[1]/w:tc[1]/w:p[1]/w:hyperlink[1]/w:r[3]/w:fldChar[1],
 0);
 }
 
 DECLARE_OOXMLEXPORT_TEST(testFDO78384,fdo78384.docx)
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Bug 58187

2014-05-20 Thread Ulrich Kitzinger
Hello,

I'd like to try to work on bug 
https://bugs.freedesktop.org/show_bug.cgi?id=58187 (In the Writer Navigator an 
expanded tree is collapsed when you edit an entry). 
Does anybody have any hint for me, which code fragments to look at? For 
example, what's the type of the tree view or the tree items in the navigator? 
Up to now all tries to find the corresponding collapse method have been 
fruitless.

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


Newbie question

2014-05-20 Thread Mukhiddin

Hello,

I have recently committed my first patches for te bug 43157. These were 
my first experience in open source programming, so I chose easy task. 
Then I gor review saying that the workben submodule files where I made 
change are not built. I also noticed that. I waned to know how can I 
respond to the review.


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


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

2014-05-20 Thread Julien Nabet
 sc/source/ui/app/inputhdl.cxx |3 +++
 1 file changed, 3 insertions(+)

New commits:
commit bb8acc29e4ad4fb8245b2ed47e60c6cac641ddb1
Author: Julien Nabet serval2...@yahoo.fr
Date:   Tue May 20 07:33:00 2014 +0200

Resolves: fdo#78838 past-the-end iterator

Inserting data in pColumnData invalidated miAutoPosColumn
See https://bugs.freedesktop.org/show_bug.cgi?id=78838#c6

Cherry-picked from 1df54514d7a1ebb1b4fa16cca386a34e7c41c490

Change-Id: Iece423cd8d14f5ae7bf382583c0369895a00f168
Reviewed-on: https://gerrit.libreoffice.org/9409
Reviewed-by: Eike Rathke er...@redhat.com
Tested-by: Eike Rathke er...@redhat.com

diff --git a/sc/source/ui/app/inputhdl.cxx b/sc/source/ui/app/inputhdl.cxx
index d964312..ed5e352 100644
--- a/sc/source/ui/app/inputhdl.cxx
+++ b/sc/source/ui/app/inputhdl.cxx
@@ -1536,7 +1536,10 @@ void ScInputHandler::GetColData()
 pDoc-GetDataEntries(
 aCursorPos.Col(), aCursorPos.Row(), aCursorPos.Tab(), true, 
aEntries, true);
 if (!aEntries.empty())
+{
 pColumnData-insert(aEntries.begin(), aEntries.end());
+miAutoPosColumn = pColumnData-end();
+}
 }
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: Bug 58187

2014-05-20 Thread Noel Grandin

On 2014-05-20 10:55, Ulrich Kitzinger wrote:


I'd like to try to work on bug https://bugs.freedesktop.org/show_bug.cgi?id=58187 (In the Writer Navigator an expanded 
tree is collapsed when you edit an entry).
Does anybody have any hint for me, which code fragments to look at? For example, what's the type of the tree view or 
the tree items in the navigator? Up to now all tries to find the corresponding collapse method have been fruitless.



That sounds like it's an SvTreeList control.

Grepping through sw/ shows half a dozen places that use an SvTreeList control, but I wouldn't know which one is the 
responsible party.


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


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


[Libreoffice-commits] core.git: oox/source sw/qa

2014-05-20 Thread Dinesh Patil
 oox/source/drawingml/shape.cxx  |6 ++
 sw/qa/extras/ooxmlexport/data/fdo78658.docx |binary
 sw/qa/extras/ooxmlexport/ooxmlexport.cxx|9 +
 3 files changed, 15 insertions(+)

New commits:
commit df973e6e950591c0c36f2761e43ec23b9964
Author: Dinesh Patil dinesh.pa...@synerzip.com
Date:   Fri May 16 16:05:47 2014 +0530

fdo#78658 Corruption: lockedCanvas tag is missing from RT file

While export, locked Canvas is missing and drawing is
exported inside textbox. However a locked Canvas has to be exported
inside a text-box and drawing should fall under locked Canvas
for the RT file to work in MS Office 2010.

Reviewed on:
https://gerrit.libreoffice.org/9377

Change-Id: Iea2c411302b552db18527001cc4c6d0290085afe

diff --git a/oox/source/drawingml/shape.cxx b/oox/source/drawingml/shape.cxx
index 33c846c..ca42d4a 100644
--- a/oox/source/drawingml/shape.cxx
+++ b/oox/source/drawingml/shape.cxx
@@ -962,6 +962,12 @@ Reference XShape  Shape::createAndInsert(
 }
 }
 
+else if( mbLockedCanvas )
+{
+//If we have aServiceName as com.sun.star.drawing.GroupShape and 
lockedCanvas
+putPropertyToGrabBag( LockedCanvas, Any( true ) );
+}
+
 // These can have a custom geometry, so position should be set here,
 // after creation but before custom shape handling, using the position
 // we got from the caller.
diff --git a/sw/qa/extras/ooxmlexport/data/fdo78658.docx 
b/sw/qa/extras/ooxmlexport/data/fdo78658.docx
new file mode 100644
index 000..1c0782e
Binary files /dev/null and b/sw/qa/extras/ooxmlexport/data/fdo78658.docx differ
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
index 90021c5..f69396f 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
@@ -2984,6 +2984,15 @@ DECLARE_OOXMLEXPORT_TEST(testFdo76016, fdo76016.docx)
 assertXPath(pXmlDoc, 
//a:graphic/a:graphicData/wps:wsp/wps:spPr/a:prstGeom/a:avLst/a:gd[2], 
name, adj2);
 }
 
+DECLARE_OOXMLEXPORT_TEST(testLockedCanvas, fdo78658.docx)
+{
+xmlDocPtr pXmlDoc = parseExport(word/document.xml);
+if (!pXmlDoc)
+return;
+// Checking for lockedCanvas tag
+assertXPath(pXmlDoc, 
/w:document[1]/w:body[1]/w:p[1]/w:r[2]/mc:AlternateContent[1]/mc:Choice[1]/w:drawing[1]/wp:anchor[1]/a:graphic[1]/a:graphicData[1]/wps:wsp[1]/wps:txbx[1]/w:txbxContent[1]/w:p[1]/w:r[1]/mc:AlternateContent[1]/mc:Choice[1]/w:drawing[1]/wp:inline[1]/a:graphic[1]/a:graphicData[1]/lc:lockedCanvas[1],
 1);
+}
+
 DECLARE_OOXMLEXPORT_TEST(testFileWithInvalidImageLink, 
FileWithInvalidImageLink.docx)
 {
 /* In case if the original file has an image whose link is
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: sc/source

2014-05-20 Thread Eike Rathke
 sc/source/ui/app/inputhdl.cxx |8 ++--
 1 file changed, 2 insertions(+), 6 deletions(-)

New commits:
commit 23e9f36a21f7daa6175f53fd244677b9c2e2a660
Author: Eike Rathke er...@redhat.com
Date:   Tue May 20 11:38:46 2014 +0200

simplify and ensure valid miAutoPosColumn in all cases, fdo#78838 related

Unconditionally set miAutoPosColumn at the end of all operations so it
is also valid in case of pColumnData-clear() and no entries inserted
after pDoc-GetDataEntries().

Change-Id: I689632f696091fd9ce8e93c06d7918e5eaf05ab4

diff --git a/sc/source/ui/app/inputhdl.cxx b/sc/source/ui/app/inputhdl.cxx
index af767b9..b9e1b90 100644
--- a/sc/source/ui/app/inputhdl.cxx
+++ b/sc/source/ui/app/inputhdl.cxx
@@ -1409,19 +1409,15 @@ void ScInputHandler::GetColData()
 if ( pColumnData )
 pColumnData-clear();
 else
-{
 pColumnData = new ScTypedCaseStrSet;
-miAutoPosColumn = pColumnData-end();
-}
 
 std::vectorScTypedStrData aEntries;
 pDoc-GetDataEntries(
 aCursorPos.Col(), aCursorPos.Row(), aCursorPos.Tab(), true, 
aEntries, true);
 if (!aEntries.empty())
-{
 pColumnData-insert(aEntries.begin(), aEntries.end());
-miAutoPosColumn = pColumnData-end();
-}
+
+miAutoPosColumn = pColumnData-end();
 }
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2014-05-20 Thread Eike Rathke
 sc/source/ui/app/inputhdl.cxx |8 ++--
 1 file changed, 2 insertions(+), 6 deletions(-)

New commits:
commit 51a868729b5adfa402095f5c08fa889bddcd1b34
Author: Eike Rathke er...@redhat.com
Date:   Tue May 20 11:38:46 2014 +0200

simplify and ensure valid miAutoPosColumn in all cases, fdo#78838 related

Unconditionally set miAutoPosColumn at the end of all operations so it
is also valid in case of pColumnData-clear() and no entries inserted
after pDoc-GetDataEntries().

Change-Id: I689632f696091fd9ce8e93c06d7918e5eaf05ab4
(cherry picked from commit 23e9f36a21f7daa6175f53fd244677b9c2e2a660)
Reviewed-on: https://gerrit.libreoffice.org/9412
Tested-by: Markus Mohrhard markus.mohrh...@googlemail.com
Reviewed-by: Markus Mohrhard markus.mohrh...@googlemail.com

diff --git a/sc/source/ui/app/inputhdl.cxx b/sc/source/ui/app/inputhdl.cxx
index ed5e352..0041f3c 100644
--- a/sc/source/ui/app/inputhdl.cxx
+++ b/sc/source/ui/app/inputhdl.cxx
@@ -1527,19 +1527,15 @@ void ScInputHandler::GetColData()
 if ( pColumnData )
 pColumnData-clear();
 else
-{
 pColumnData = new ScTypedCaseStrSet;
-miAutoPosColumn = pColumnData-end();
-}
 
 std::vectorScTypedStrData aEntries;
 pDoc-GetDataEntries(
 aCursorPos.Col(), aCursorPos.Row(), aCursorPos.Tab(), true, 
aEntries, true);
 if (!aEntries.empty())
-{
 pColumnData-insert(aEntries.begin(), aEntries.end());
-miAutoPosColumn = pColumnData-end();
-}
+
+miAutoPosColumn = pColumnData-end();
 }
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Bug 45789] automatic row height in reports

2014-05-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=45789

--- Comment #18 from Lionel Elie Mamane lio...@mamane.lu ---
Michal? Any news on your work on this bug? Any question? Do you still intend to
work on it?

-- 
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] core.git: chart2/qa oox/source

2014-05-20 Thread Bisal Singh Nayal
 chart2/qa/extras/chart2export.cxx   |   40 
++
 chart2/qa/extras/data/docx/fdo78290_Combination_Chart_Marker_x.docx |binary
 chart2/qa/extras/data/docx/fdo78290_Line_Chart_Marker_x.docx|binary
 chart2/qa/extras/data/docx/fdo78290_Scatter_Chart_Marker_x.docx |binary
 oox/source/export/chartexport.cxx   |5 -
 5 files changed, 43 insertions(+), 2 deletions(-)

New commits:
commit ee0bb265c90a135afa53c0fa6a586996e67640fa
Author: Bisal Singh Nayal bisal.na...@synerzip.com
Date:   Wed May 7 20:03:11 2014 +0530

fdo#78290 : The File gets corrupted when saved in LO

Problem Description: Docx file containing a chart (line chart / scatter 
chart)which has used a builtin
marker'x' gets corrupted when we save it in LO.The reason was that while 
exporting LO was writing the
marker information 'x' as 'X' which MS Word doesn't 
recognize.‒c:markerc:symbol val=X /. Also
the size of the marker was coming 1 less than the actual value. Ex: if size 
is 7 then it was being
written as  6.

Solution: During export I have made changes so that now LO writes 'x' in 
the tag information
‒c:marker c:symbol val=x /. Now the size of the marker is also 
being correctly exported.

Change-Id: I26b747f9576625bf3acb941322ae418a0bbc6b64
Reviewed-on: https://gerrit.libreoffice.org/9273
Tested-by: Miklos Vajna vmik...@collabora.co.uk
Reviewed-by: Miklos Vajna vmik...@collabora.co.uk
Tested-by: Markus Mohrhard markus.mohrh...@googlemail.com
Reviewed-by: Markus Mohrhard markus.mohrh...@googlemail.com

diff --git a/chart2/qa/extras/chart2export.cxx 
b/chart2/qa/extras/chart2export.cxx
index 30a252a..c75b5c2 100644
--- a/chart2/qa/extras/chart2export.cxx
+++ b/chart2/qa/extras/chart2export.cxx
@@ -57,6 +57,9 @@ public:
 void testGapWidthXLSX();
 void testSmoothedLines();
 void testLabelStringODS();
+void testfdo78290_Line_Chart_Marker_x();
+void testfdo78290_Scatter_Chart_Marker_x();
+void testfdo78290_Combination_Chart_Marker_x();
 
 CPPUNIT_TEST_SUITE(Chart2ExportTest);
 CPPUNIT_TEST(test);
@@ -86,6 +89,9 @@ public:
 CPPUNIT_TEST(testGapWidthXLSX);
 CPPUNIT_TEST(testSmoothedLines);
 CPPUNIT_TEST(testLabelStringODS);
+CPPUNIT_TEST(testfdo78290_Line_Chart_Marker_x);
+CPPUNIT_TEST(testfdo78290_Scatter_Chart_Marker_x);
+CPPUNIT_TEST(testfdo78290_Combination_Chart_Marker_x);
 CPPUNIT_TEST_SUITE_END();
 
 protected:
@@ -629,6 +635,40 @@ void Chart2ExportTest::testFdo74115WallBitmapFill()
 assertXPath(pXmlDoc, /c:chartSpace/c:chart/c:plotArea/c:spPr/a:blipFill);
 }
 
+//The below test case tests the built in marker 'x' for Office 2010 in Line 
charts
+
+void Chart2ExportTest::testfdo78290_Line_Chart_Marker_x()
+{
+load(/chart2/qa/extras/data/docx/, fdo78290_Line_Chart_Marker_x.docx);
+xmlDocPtr pXmlDoc = parseExport(word/charts/chart, Office Open XML 
Text);
+CPPUNIT_ASSERT(pXmlDoc);
+assertXPath(pXmlDoc, 
/c:chartSpace[1]/c:chart[1]/c:plotArea[1]/c:lineChart[1]/c:ser[1]/c:marker[1]/c:symbol[1],val,x);
+assertXPath(pXmlDoc, 
/c:chartSpace[1]/c:chart[1]/c:plotArea[1]/c:lineChart[1]/c:ser[1]/c:marker[1]/c:size[1],val,7);
+}
+
+// We can also use the built in marker 'x' in scatter chart, hence writing the 
test case for the same.
+
+void Chart2ExportTest::testfdo78290_Scatter_Chart_Marker_x()
+{
+load(/chart2/qa/extras/data/docx/, 
fdo78290_Scatter_Chart_Marker_x.docx);
+xmlDocPtr pXmlDoc = parseExport(word/charts/chart, Office Open XML 
Text);
+CPPUNIT_ASSERT(pXmlDoc);
+assertXPath(pXmlDoc, 
/c:chartSpace[1]/c:chart[1]/c:plotArea[1]/c:scatterChart[1]/c:ser[1]/c:marker[1]/c:symbol[1],val,x);
+assertXPath(pXmlDoc, 
/c:chartSpace[1]/c:chart[1]/c:plotArea[1]/c:scatterChart[1]/c:ser[1]/c:marker[1]/c:size[1],val,7);
+}
+
+// Also in a combination of charts like a column chart and line chart, we can 
use the built in marker 'x'
+// for the line chart too. hence put a test case for the combination chart 
also.
+
+void Chart2ExportTest::testfdo78290_Combination_Chart_Marker_x()
+{
+load(/chart2/qa/extras/data/docx/, 
fdo78290_Combination_Chart_Marker_x.docx);
+xmlDocPtr pXmlDoc = parseExport(word/charts/chart, Office Open XML 
Text);
+CPPUNIT_ASSERT(pXmlDoc);
+assertXPath(pXmlDoc, 
/c:chartSpace[1]/c:chart[1]/c:plotArea[1]/c:lineChart[1]/c:ser[1]/c:marker[1]/c:symbol[1],val,x);
+assertXPath(pXmlDoc, 
/c:chartSpace[1]/c:chart[1]/c:plotArea[1]/c:lineChart[1]/c:ser[1]/c:marker[1]/c:size[1],val,7);
+}
+
 void Chart2ExportTest::testBarChartRotation()
 {
 load (/chart2/qa/extras/data/docx/, barChartRotation.docx);
diff --git 
a/chart2/qa/extras/data/docx/fdo78290_Combination_Chart_Marker_x.docx 
b/chart2/qa/extras/data/docx/fdo78290_Combination_Chart_Marker_x.docx
new file mode 100644
index 000..ee2489d
Binary files /dev/null and 

Re: master build fails in Chart2

2014-05-20 Thread Christian Lohmaier
Hi *,

On Tue, May 20, 2014 at 10:08 AM, Stephan Bergmann sberg...@redhat.com wrote:
 On 05/20/2014 08:24 AM, Jean-Baptiste Faure wrote:
 .../chart2/source/controller/dialogs/res_LegendPosition.hxx:32:7: error:
 redefinition of ‘class chart::LegendPositionResources’
 [...]

 Do you have a leftover
 chart2/source/controller/dialogs/res_LegendPosition.hxx that should have
 been removed long ago by
 http://cgit.freedesktop.org/libreoffice/core/commit/?id=27eb33d753eedfa4215eaf8325bc0ee9baeb7417
 remove unnecessary empty dtor, etc.?

git clean -dxn

will show you all the stuff you have in your tree that's not managed
by git (and thus probably shouldn't be there when you do the build in
a separate directory). with -f instead of -n, git will remove those
obsolete files for you.

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


Re: master build fails in Chart2

2014-05-20 Thread Jean-Baptiste Faure
Le 20/05/2014 10:08, Stephan Bergmann a écrit :
 On 05/20/2014 08:24 AM, Jean-Baptiste Faure wrote:
 [build CXX] chart2/source/controller/dialogs/dlg_NumberFormat.cxx
 In file included from
 .../chart2/source/controller/dialogs/dlg_InsertLegend.cxx:21:0:
 .../chart2/source/controller/dialogs/res_LegendPosition.hxx:32:7:
 error: redefinition of ‘class chart::LegendPositionResources’
   class LegendPositionResources
 ^
 In file included from
 .../chart2/source/controller/inc/dlg_InsertLegend.hxx:29:0,
   from
 .../chart2/source/controller/dialogs/dlg_InsertLegend.cxx:20:
 .../chart2/source/controller/inc/res_LegendPosition.hxx:32:7: error:
 previous definition of ‘class chart::LegendPositionResources’
   class LegendPositionResources
 ^
 make[1]: ***
 [.../workdir/CxxObject/chart2/source/controller/dialogs/dlg_InsertLegend.o]
 Erreur 1
 make[1]: *** Attente des tâches non terminées
 make: *** [build] Erreur 2
 
 Do you have a leftover
 chart2/source/controller/dialogs/res_LegendPosition.hxx that should have
 been removed long ago by
 http://cgit.freedesktop.org/libreoffice/core/commit/?id=27eb33d753eedfa4215eaf8325bc0ee9baeb7417
 remove unnecessary empty dtor, etc.?

Indeed I still have this file. I wonder why that worked until now.

I removed the file and updated my local copy. Build in progress.

Thank you very much.
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


Re: Newbie question

2014-05-20 Thread Christian Lohmaier
Hi Mukhiddin,

On Tue, May 20, 2014 at 11:12 AM, Mukhiddin ymukhid...@gmail.com wrote:
 Hello,

 I have recently committed my first patches for te bug 43157. These were my
 first experience in open source programming, so I chose easy task. Then I
 gor review saying that the workben submodule files where I made change are
 not built. I also noticed that. I waned to know how can I respond to the
 review.

When you are logged-in to gerrit, you will get a Reply button at the
top, above the commit-message.

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


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

2014-05-20 Thread Caolán McNamara
 sd/source/ui/func/fuparagr.cxx |4 -
 sd/source/ui/view/drtxtob.cxx  |  134 +
 2 files changed, 73 insertions(+), 65 deletions(-)

New commits:
commit 27fc60874795849b43903f27076e9dcafb958c76
Author: Caolán McNamara caol...@redhat.com
Date:   Tue May 20 12:37:42 2014 +0100

Related: fdo#78151 disable promote or demote outline levels in master view

it doesn't make sense in master view to do that

Change-Id: I838422f93131b8a92745a624738c5be3a7ed3459

diff --git a/sd/source/ui/view/drtxtob.cxx b/sd/source/ui/view/drtxtob.cxx
index 73852d0..ec24fc9 100644
--- a/sd/source/ui/view/drtxtob.cxx
+++ b/sd/source/ui/view/drtxtob.cxx
@@ -253,88 +253,96 @@ void TextObjectBar::GetAttrState( SfxItemSet rSet )
 bool bDisableRight= true;
 bool bDisableUp   = true;
 bool bDisableDown = true;
-OutlinerView* pOLV = mpView-GetTextEditOutlinerView();
 
-if (mpView-ISA(OutlineView))
-{
-pOLV = static_castOutlineView*(mpView)-GetViewByWindow(
-mpViewShell-GetActiveWindow());
-}
+//fdo#78151 it doesn't make sense to promote or demote outline 
levels in master view.
+const DrawViewShell* pDrawViewShell = dynamic_cast 
DrawViewShell* (mpViewShell);
+const bool bInMasterView = pDrawViewShell  
pDrawViewShell-GetEditMode() == EM_MASTERPAGE;
 
-bool bOutlineViewSh = mpViewShell-ISA(OutlineViewShell);
-
-if (pOLV 
-( pOLV-GetOutliner()-GetMode() == 
OUTLINERMODE_OUTLINEOBJECT || bOutlineViewSh ) )
+if (!bInMasterView)
 {
-// Outliner at outline-mode
-::Outliner* pOutl = pOLV-GetOutliner();
+OutlinerView* pOLV = mpView-GetTextEditOutlinerView();
 
-std::vectorParagraph* aSelList;
-pOLV-CreateSelectionList(aSelList);
-Paragraph* pPara = aSelList.empty() ? NULL : 
*(aSelList.begin());
+if (mpView-ISA(OutlineView))
+{
+pOLV = 
static_castOutlineView*(mpView)-GetViewByWindow(
+mpViewShell-GetActiveWindow());
+}
 
-// find out if we are a OutlineView
-bool bIsOutlineView(OUTLINERMODE_OUTLINEVIEW == 
pOLV-GetOutliner()-GetMode());
+bool bOutlineViewSh = mpViewShell-ISA(OutlineViewShell);
 
-// This is ONLY for OutlineViews
-if(bIsOutlineView)
+if (pOLV 
+( pOLV-GetOutliner()-GetMode() == 
OUTLINERMODE_OUTLINEOBJECT || bOutlineViewSh ) )
 {
-// allow move up if position is 2 or greater OR it
-// is a title object (and thus depth==1)
-if(pOutl-GetAbsPos(pPara)  1 || ( 
pOutl-HasParaFlag(pPara,PARAFLAG_ISPAGE)  pOutl-GetAbsPos(pPara)  0 ) )
+// Outliner at outline-mode
+::Outliner* pOutl = pOLV-GetOutliner();
+
+std::vectorParagraph* aSelList;
+pOLV-CreateSelectionList(aSelList);
+Paragraph* pPara = aSelList.empty() ? NULL : 
*(aSelList.begin());
+
+// find out if we are a OutlineView
+bool bIsOutlineView(OUTLINERMODE_OUTLINEVIEW == 
pOLV-GetOutliner()-GetMode());
+
+// This is ONLY for OutlineViews
+if(bIsOutlineView)
 {
-// not at top
-bDisableUp = false;
+// allow move up if position is 2 or greater OR it
+// is a title object (and thus depth==1)
+if(pOutl-GetAbsPos(pPara)  1 || ( 
pOutl-HasParaFlag(pPara,PARAFLAG_ISPAGE)  pOutl-GetAbsPos(pPara)  0 ) )
+{
+// not at top
+bDisableUp = false;
+}
 }
-}
-else
-{
-// old behaviour for OUTLINERMODE_OUTLINEOBJECT
-if(pOutl-GetAbsPos(pPara)  0)
+else
 {
-// not at top
-bDisableUp = false;
+// old behaviour for OUTLINERMODE_OUTLINEOBJECT
+if(pOutl-GetAbsPos(pPara)  0)
+{
+// not at top
+bDisableUp = false;
+  

[Libreoffice-commits] core.git: Branch 'libreoffice-4-2' - package/source

2014-05-20 Thread Stephan Bergmann
 package/source/xstor/xstorage.cxx |   14 ++
 1 file changed, 10 insertions(+), 4 deletions(-)

New commits:
commit 164e10869f91b75db03b09f53e1467098b3e6b93
Author: Stephan Bergmann sberg...@redhat.com
Date:   Tue May 20 11:12:40 2014 +0200

fdo#78949 Handle (SAXParse-)Exception that can't pass getAllRelationships

...like it was already done in the similar else branch of
ReadRelInfoIfNecessary.  The call site at GetAllRelationshipsIfAny will
translate an m_nRelInfoStatus that remained RELINFO_NOT_INIT into an
IOException.

Change-Id: I79c601acd23b3b0020c7b4de4c20b0282524bce1
(cherry picked from commit 423921b085de43f53e42f957889dd96378d2c3c4)
Reviewed-on: https://gerrit.libreoffice.org/9411
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/package/source/xstor/xstorage.cxx 
b/package/source/xstor/xstorage.cxx
index 9e2ffb7..8775561 100644
--- a/package/source/xstor/xstorage.cxx
+++ b/package/source/xstor/xstorage.cxx
@@ -572,13 +572,19 @@ void OStorage_Impl::ReadRelInfoIfNecessary()
 {
 // Init from original stream
 uno::Reference io::XInputStream  xRelInfoStream = 
GetRelInfoStreamForName( OUString() );
-if ( xRelInfoStream.is() )
-m_aRelInfo = 
::comphelper::OFOPXMLHelper::ReadRelationsInfoSequence(
+try
+{
+if ( xRelInfoStream.is() )
+m_aRelInfo = 
::comphelper::OFOPXMLHelper::ReadRelationsInfoSequence(
 xRelInfoStream,
 _rels/.rels,
 m_xContext );
-
-m_nRelInfoStatus = RELINFO_READ;
+m_nRelInfoStatus = RELINFO_READ;
+}
+catch (css::uno::Exception  e)
+{
+SAL_INFO(package.xstor, caught Exception   e.Message);
+}
 }
 else if ( m_nRelInfoStatus == RELINFO_CHANGED_STREAM )
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: sdext/source

2014-05-20 Thread Vort
 sdext/source/pdfimport/inc/contentsink.hxx|   18 ++---
 sdext/source/pdfimport/test/tests.cxx |3 +
 sdext/source/pdfimport/tree/pdfiprocessor.cxx |   16 ++--
 sdext/source/pdfimport/tree/pdfiprocessor.hxx |3 +
 sdext/source/pdfimport/wrapper/wrapper.cxx|   31 +---
 sdext/source/pdfimport/xpdfwrapper/pdfioutdev_gpl.cxx |   33 ++
 6 files changed, 57 insertions(+), 47 deletions(-)

New commits:
commit be7ec1384e85360179f5645d9775c5e6605d5a0f
Author: Vort vv...@yandex.ru
Date:   Tue May 20 09:05:33 2014 +0300

fdo#78689 PDF Import: get font's ascent value from different source

Change-Id: I19018d25ef53bbea225bb5a9ef806ce5c1b4adc7
Reviewed-on: https://gerrit.libreoffice.org/9410
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/sdext/source/pdfimport/inc/contentsink.hxx 
b/sdext/source/pdfimport/inc/contentsink.hxx
index 93ae5ad..d4fb3b1 100644
--- a/sdext/source/pdfimport/inc/contentsink.hxx
+++ b/sdext/source/pdfimport/inc/contentsink.hxx
@@ -52,13 +52,15 @@ namespace pdfi
 bool isItalic_,
 bool isUnderline_,
 bool isOutline_,
-double   size_ ) :
+double   size_,
+double   ascent_) :
 familyName(familyName_),
 isBold(isBold_),
 isItalic(isItalic_),
 isUnderline(isUnderline_),
 isOutline(isOutline_),
-size(size_)
+size(size_),
+ascent(ascent_)
 {}
 
 FontAttributes() :
@@ -67,7 +69,8 @@ namespace pdfi
 isItalic(false),
 isUnderline(false),
 isOutline(false),
-size(0.0)
+size(0.0),
+ascent(1.0)
 {}
 
 OUString familyName;
@@ -76,6 +79,7 @@ namespace pdfi
 boolisUnderline;
 boolisOutline;
 double  size; // device pixel
+double  ascent;
 
 bool operator==(const FontAttributes rFont) const
 {
@@ -84,7 +88,8 @@ namespace pdfi
 !isItalic == !rFont.isItalic 
 !isUnderline == !rFont.isUnderline 
 !isOutline == !rFont.isOutline 
-size == rFont.size;
+size == rFont.size 
+ascent == rFont.ascent;
 }
 };
 
@@ -135,9 +140,10 @@ namespace pdfi
 virtual void intersectEoClip(const css::uno::Reference
css::rendering::XPolyPolygon2D  
rPath) = 0;
 
-virtual void drawGlyphs( const OUString   
rGlyphs,
+virtual void drawGlyphs( const OUString rGlyphs,
  const css::geometry::RealRectangle2D rRect,
- const css::geometry::Matrix2D
rFontMatrix ) = 0;
+ const css::geometry::Matrix2D
rFontMatrix,
+ double fontSize) = 0;
 
 /// issued when a sequence of associated glyphs is drawn
 virtual void endText() = 0;
diff --git a/sdext/source/pdfimport/test/tests.cxx 
b/sdext/source/pdfimport/test/tests.cxx
index aa11ea6..8b47c0a 100644
--- a/sdext/source/pdfimport/test/tests.cxx
+++ b/sdext/source/pdfimport/test/tests.cxx
@@ -346,7 +346,8 @@ namespace
 
 virtual void drawGlyphs( const OUString rGlyphs,
  const geometry::RealRectangle2D /*rRect*/,
- const geometry::Matrix2D
/*rFontMatrix*/ ) SAL_OVERRIDE
+ const geometry::Matrix2D
/*rFontMatrix*/,
+ double /*fontSize*/) SAL_OVERRIDE
 {
 m_aTextOut.append(rGlyphs);
 }
diff --git a/sdext/source/pdfimport/tree/pdfiprocessor.cxx 
b/sdext/source/pdfimport/tree/pdfiprocessor.cxx
index 487ba20..3a670a6 100644
--- a/sdext/source/pdfimport/tree/pdfiprocessor.cxx
+++ b/sdext/source/pdfimport/tree/pdfiprocessor.cxx
@@ -277,14 +277,20 @@ void PDFIProcessor::processGlyphLine()
 
 void PDFIProcessor::drawGlyphs( const OUString rGlyphs,
 const geometry::RealRectangle2D rRect,
-const geometry::Matrix2DrFontMatrix )
+const geometry::Matrix2DrFontMatrix,
+double fontSize)
 {
+double ascent = getFont(getCurrentContext().FontId).ascent;
+
+double ascentdx = rFontMatrix.m01 * ascent * fontSize;
+double ascentdy = rFontMatrix.m11 * ascent * fontSize;
+
 basegfx::B2DHomMatrix totalTextMatrix1(
- 

[Libreoffice-commits] core.git: Branch 'libreoffice-4-2' - sfx2/source

2014-05-20 Thread Michael Stahl
 sfx2/source/dialog/templdlg.cxx |   13 ++---
 sfx2/source/inc/templdgi.hxx|2 +-
 2 files changed, 11 insertions(+), 4 deletions(-)

New commits:
commit 2fd89dd60366caa64ed8b31b588e72dd639104bd
Author: Michael Stahl mst...@redhat.com
Date:   Mon May 19 22:03:18 2014 +0200

fdo#78536: sfx2: template dialog: make the am-i-dead-yet check reentrant

In the fine tradition of our enterprise-grade code base, this lets the
check in SfxCommonTemplateDialog_Impl::Execute_Impl() do an early
return if the this instance has signalled its untimely demise even if
there are multiple calls of Execute_Impl() on the stack; extending
the scheme with further epicycles to make it work if the Execute_Impl()
calls happen on 2 different threads is left as an exercise for the
interested reader.

(regression from bcc52fb2ebdf77a1cc089857775fd742085b45b6)

(cherry picked from commit 838c580fdba8b1c72bc854d101d302be0e3861d5)

Conflicts:
sfx2/source/dialog/templdlg.cxx

Change-Id: I1d9f0d82bf6d9f5da8f9bd2be57d01937d71fd26
Reviewed-on: https://gerrit.libreoffice.org/9406
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/sfx2/source/dialog/templdlg.cxx b/sfx2/source/dialog/templdlg.cxx
index cbd856f..11b430c 100644
--- a/sfx2/source/dialog/templdlg.cxx
+++ b/sfx2/source/dialog/templdlg.cxx
@@ -100,20 +100,22 @@ class SfxCommonTemplateDialog_Impl::DeletionWatcher : 
private boost::noncopyable
 public:
 explicit DeletionWatcher(SfxCommonTemplateDialog_Impl rDialog)
 : m_pDialog(rDialog)
+, m_pPrevious(m_pDialog-impl_setDeletionWatcher(this))
 {
-m_pDialog-impl_setDeletionWatcher(this);
 }
 
 ~DeletionWatcher()
 {
 if (m_pDialog)
-m_pDialog-impl_setDeletionWatcher(0);
+m_pDialog-impl_setDeletionWatcher(m_pPrevious);
 }
 
 // Signal that the dialog was deleted
 void signal()
 {
 m_pDialog = 0;
+if (m_pPrevious)
+m_pPrevious-signal();
 }
 
 // Return true if the dialog was deleted
@@ -124,6 +126,7 @@ public:
 
 private:
 SfxCommonTemplateDialog_Impl* m_pDialog;
+DeletionWatcher *const m_pPrevious; /// let's add more epicycles!
 };
 
 // Re-direct functions
@@ -1015,9 +1018,13 @@ void SfxCommonTemplateDialog_Impl::impl_clear()
 DELETEZ( m_pStyleFamiliesId );
 }
 
-void SfxCommonTemplateDialog_Impl::impl_setDeletionWatcher(DeletionWatcher* 
pNewWatcher)
+SfxCommonTemplateDialog_Impl::DeletionWatcher *
+SfxCommonTemplateDialog_Impl::impl_setDeletionWatcher(
+DeletionWatcher *const pNewWatcher)
 {
+DeletionWatcher *const pRet(m_pDeletionWatcher);
 m_pDeletionWatcher = pNewWatcher;
+return pRet;
 }
 
 //-
diff --git a/sfx2/source/inc/templdgi.hxx b/sfx2/source/inc/templdgi.hxx
index 0011aed..844acdb 100644
--- a/sfx2/source/inc/templdgi.hxx
+++ b/sfx2/source/inc/templdgi.hxx
@@ -111,7 +111,7 @@ private:
 voidReadResource();
 voidClearResource();
 void impl_clear();
-void impl_setDeletionWatcher(DeletionWatcher* pNewWatcher);
+DeletionWatcher * impl_setDeletionWatcher(DeletionWatcher* pNewWatcher);
 
 protected:
 #define MAX_FAMILIES5
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-4.2' - 2 commits - sw/inc sw/source xmloff/source

2014-05-20 Thread Oliver-Rainer Wittmann
 sw/inc/IDocumentMarkAccess.hxx |2 
 sw/inc/doc.hxx |   27 +++-
 sw/source/core/crsr/annotationmark.cxx |1 
 sw/source/core/doc/docbm.cxx   |  189 -
 sw/source/core/doc/doccorr.cxx |   20 +--
 sw/source/core/undo/undobj.cxx |   13 +-
 sw/source/core/unocore/unoobj.cxx  |   72 +++-
 sw/source/ui/docvw/SidebarWin.cxx  |   25 +++-
 sw/source/ui/shells/textfld.cxx|3 
 sw/source/ui/wrtsh/wrtsh2.cxx  |   21 ++-
 xmloff/source/text/txtfldi.cxx |4 
 11 files changed, 246 insertions(+), 131 deletions(-)

New commits:
commit afbf61890a4d5e846c4aa09380d8b641d99ceefd
Author: Oliver-Rainer Wittmann o...@apache.org
Date:   Thu Jan 16 11:47:18 2014 +

Resolves: cp#179 i#124030 do not delete UNO mark which are not 
expanded...

and only touch the start of the given range.

(cherry picked from commit ae295f7d009842cdceb50c4daffe948ede2b4b88)

Conflicts:
sw/source/core/doc/docbm.cxx

(cherry picked from commit 4b6b51182a9a600805729313278eb707c6ebfb14)

Change-Id: Id7317eeb8e9c063c9d8b30bca97ed0afee3ec8c5

diff --git a/sw/source/core/doc/docbm.cxx b/sw/source/core/doc/docbm.cxx
index 856bbc6..3deaccc 100644
--- a/sw/source/core/doc/docbm.cxx
+++ b/sw/source/core/doc/docbm.cxx
@@ -50,7 +50,6 @@
 #include edimp.hxx
 #include stdio.h
 
-using namespace ::std;
 using namespace ::boost;
 using namespace ::sw::mark;
 
@@ -58,12 +57,19 @@ namespace
 {
 static bool lcl_GreaterThan( const SwPosition rPos, const SwNodeIndex 
rNdIdx, const SwIndex* pIdx )
 {
-return pIdx ? ( rPos.nNode  rNdIdx || ( rPos.nNode == rNdIdx  
rPos.nContent = pIdx-GetIndex() )) : rPos.nNode = rNdIdx;
+return pIdx != NULL
+   ? ( rPos.nNode  rNdIdx
+   || ( rPos.nNode == rNdIdx
+ rPos.nContent = pIdx-GetIndex() ) )
+   : rPos.nNode = rNdIdx;
 }
 
 static bool lcl_Lower( const SwPosition rPos, const SwNodeIndex rNdIdx, 
const SwIndex* pIdx )
 {
-return rPos.nNode  rNdIdx || ( pIdx  rPos.nNode == rNdIdx  
rPos.nContent  pIdx-GetIndex() );
+return rPos.nNode  rNdIdx
+   || ( pIdx != NULL
+ rPos.nNode == rNdIdx
+ rPos.nContent  pIdx-GetIndex() );
 }
 
 static bool lcl_MarkOrderingByStart(const IDocumentMarkAccess::pMark_t 
rpFirst,
@@ -91,9 +97,9 @@ namespace
 }
 
 SAL_WNODEPRECATED_DECLARATIONS_PUSH
-static inline auto_ptrSwPosition lcl_PositionFromCntntNode(SwCntntNode * 
const pCntntNode, const bool bAtEnd=false)
+static inline ::std::auto_ptrSwPosition 
lcl_PositionFromCntntNode(SwCntntNode * const pCntntNode, const bool 
bAtEnd=false)
 {
-auto_ptrSwPosition pResult(new SwPosition(*pCntntNode));
+::std::auto_ptrSwPosition pResult(new SwPosition(*pCntntNode));
 pResult-nContent.Assign(pCntntNode, bAtEnd ? pCntntNode-Len() : 0);
 return pResult;
 }
@@ -104,7 +110,7 @@ namespace
 // else set it to the end of the node before rStt
 // else set it to the CntntNode of the Pos outside the Range
 SAL_WNODEPRECATED_DECLARATIONS_PUSH
-static inline auto_ptrSwPosition lcl_FindExpelPosition(const 
SwNodeIndex rStt,
+static inline ::std::auto_ptrSwPosition lcl_FindExpelPosition(const 
SwNodeIndex rStt,
 const SwNodeIndex rEnd,
 const SwPosition rOtherPosition)
 {
@@ -118,7 +124,7 @@ namespace
 pNode = rStt.GetNodes().GoPrevious(aStt), bAtEnd = true;
 if(pNode)
 return lcl_PositionFromCntntNode(pNode, bAtEnd);
-return auto_ptrSwPosition(new SwPosition(rOtherPosition));
+return ::std::auto_ptrSwPosition(new SwPosition(rOtherPosition));
 }
 SAL_WNODEPRECATED_DECLARATIONS_POP
 
@@ -149,7 +155,7 @@ namespace
 rMarks.begin(),
 pCandidatesEnd,
 back_inserter(vCandidates),
-boost::bind(logical_notbool(), boost::bind(IMark::EndsBefore, 
_1, rPos)));
+boost::bind( ::std::logical_notbool(), boost::bind( 
IMark::EndsBefore, _1, rPos ) ) );
 // no candidate left = we are in front of the first mark or there are 
none
 if(!vCandidates.size()) return NULL;
 // return the highest (last) candidate using mark end ordering
@@ -630,8 +636,8 @@ namespace sw { namespace mark
 const SwIndex* pSttIdx,
 const SwIndex* pEndIdx )
 {
-vectorconst_iterator_t vMarksToDelete;
-bool isSortingNeeded = false;
+::std::vectorconst_iterator_t vMarksToDelete;
+bool bIsSortingNeeded = false;
 
 // copy all bookmarks in the move area to a vector storing all 
position data as offset
 // reassignment is performed after the move
@@ -646,61 +652,77 @@ namespace sw { namespace mark
 
 ::sw::mark::MarkBase* pMark = dynamic_cast 

[Libreoffice-commits] core.git: sdext/Library_pdfimport.mk

2014-05-20 Thread Caolán McNamara
 sdext/Library_pdfimport.mk |1 +
 1 file changed, 1 insertion(+)

New commits:
commit a092418f300f15fd1e2203b6933cfb1f69c107ad
Author: Caolán McNamara caol...@redhat.com
Date:   Tue May 20 13:19:56 2014 +0100

we need fontconfig now

Change-Id: I0482f613d7c908488e7d9f84f891bd521e91cc15

diff --git a/sdext/Library_pdfimport.mk b/sdext/Library_pdfimport.mk
index 5e1d41d..2c98ed7 100644
--- a/sdext/Library_pdfimport.mk
+++ b/sdext/Library_pdfimport.mk
@@ -32,6 +32,7 @@ $(eval $(call gb_Library_use_static_libraries,pdfimport,\
 
 $(eval $(call gb_Library_use_externals,pdfimport,\
boost_headers \
+$(if $(filter-out WNT MACOSX,$(OS)),fontconfig) \
 zlib \
 poppler \
 ))
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-4-2' - sw/source

2014-05-20 Thread Eike Rathke
 sw/source/filter/ww8/writerwordglue.cxx |  104 
 1 file changed, 92 insertions(+), 12 deletions(-)

New commits:
commit 300d27b0ae4665001ec30b47af2142f6c253f370
Author: Eike Rathke er...@redhat.com
Date:   Mon May 19 21:43:06 2014 +0200

resolved fdo#66620 keywords occur in unquoted context only

Change-Id: I4de41e5b66f1a856a786c521ca4ef2eec14c7499
(cherry picked from commit 0d361388060741fe8e2f1ba059fba95707bdc233)
Reviewed-on: https://gerrit.libreoffice.org/9405
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/sw/source/filter/ww8/writerwordglue.cxx 
b/sw/source/filter/ww8/writerwordglue.cxx
index 7fc6061..056268e 100644
--- a/sw/source/filter/ww8/writerwordglue.cxx
+++ b/sw/source/filter/ww8/writerwordglue.cxx
@@ -745,6 +745,83 @@ namespace sw
 return nDT;
 }
 
+
+/** Find cFind in rParams if not embedded in  double quotes.
+Will NOT find '\\' or ''.
+ */
+sal_Int32 findUnquoted( const OUString rParams, sal_Unicode cFind, 
sal_Int32 nFromPos )
+{
+const sal_Int32 nLen = rParams.getLength();
+if (nFromPos  0 || nLen = nFromPos)
+return -1;
+for (sal_Int32 nI = nFromPos; nI  nLen; ++nI)
+{
+const sal_Unicode c = rParams[nI];
+if (c == '\\')
+++nI;
+else if (c == '\')
+{
+++nI;
+// While not at the end and not at an unescaped end quote
+while (nI  nLen)
+{
+if (rParams[nI] == '\'  rParams[nI-1] != '\\')
+break;
+++nI;
+}
+}
+else //normal unquoted section
+{
+if (c == cFind)
+return nI;
+}
+}
+return -1;
+}
+
+/** Find all rFind in rParams if not embedded in  double quotes and
+replace with rReplace. Will NOT find '\\' or ''.
+ */
+bool replaceUnquoted( OUString rParams, const OUString rFind, const 
OUString rReplace )
+{
+bool bReplaced = false;
+if (rFind.isEmpty())
+return bReplaced;
+const sal_Unicode cFirst = rFind[0];
+
+sal_Int32 nLen = rParams.getLength();
+for (sal_Int32 nI = 0; nI  nLen; ++nI)
+{
+const sal_Unicode c = rParams[nI];
+if (rParams[nI] == '\\')
+++nI;
+else if (rParams[nI] == '\')
+{
+++nI;
+// While not at the end and not at an unescaped end quote
+while (nI  nLen)
+{
+if (rParams[nI] == '\'  rParams[nI-1] != '\\')
+break;
+++nI;
+}
+}
+else //normal unquoted section
+{
+if (c == cFirst  rParams.match( rFind, nI))
+{
+const sal_Int32 nFindLen = rFind.getLength();
+const sal_Int32 nDiff = rReplace.getLength() - 
nFindLen;
+rParams.replaceAt( nI, nFindLen, rReplace);
+nI += nFindLen + nDiff - 1;
+nLen += nDiff;
+bReplaced = true;
+}
+}
+}
+return bReplaced;
+}
+
 sal_uLong MSDateTimeFormatToSwFormat(OUString rParams,
 SvNumberFormatter *pFormatter, sal_uInt16 rLang, bool bHijri,
 sal_uInt16 nDocLang)
@@ -756,36 +833,39 @@ namespace sw
 
 SwapQuotesInField(rParams);
 
-// Force to Japanese when finding one of 'geaE'
-bool bForceJapanese = (-1 != rParams.indexOf('g')
-|| -1 != rParams.indexOf('e') || -1 != rParams.indexOf('E') );
-if ( bForceJapanese )
-{
-rParams = rParams.replaceAll( ee,  ).replaceAll( EE, 
 );
-}
+// Force to Japanese when finding one of 'geE'.
+// XXX This actually may not be correct, all era keywords could be
+// used in other locales as well. I just don't know about Word. But
+// this is how it was for 10 years..
+bool bForceJapanese = (-1 != findUnquoted( rParams, 'g', 0));
+// XXX Why replace? The number formatter does handle them and this
+// effectively changes from Gengou to Gregorian calendar. Legacy
+// because it wasn't supported a decade ago and now moot? Or is
+// that a Word specialty?
+   

Re: do NOT immediately remove files from dev-downloads.libreoffice.org on update of externals!

2014-05-20 Thread Michael Stahl
On 20/05/14 09:54, Bjoern Michaelsen wrote:
 On Tue, May 20, 2014 at 09:18:54AM +0200, David Tardon wrote:
 On Tue, May 20, 2014 at 08:56:08AM +0200, Stephan Bergmann wrote:
 On 05/19/2014 09:45 PM, Bjoern Michaelsen wrote:
 please do NOT remove files from dev-downloads when updating externals, keep
 those files around for at least 12 months! Otherwise you will break 
 bibisect
 building hard (apart from also breaking people on local branches).
 So please keep whatever tarball was ever used in a LibreOffice build on the
 server for at least 12 months.

 Where does the value 12 come from?

 I am curious about that too :-) Personally I would say do not ever
 remove tarballs from dev-www!
 
 Well, 12 months was just a minimum. I have not issue with using never
 instead. ;)
 
 12 months was just a rough estimate of two major releases which would be 
 good
 enough for bibisect at least.

those of us who fix regression bugs occasionally have reason to re-built
essentially arbitrarily old revisions, and i've personally been
frustrated this year by the repository zoo of pre-3.5 days making that
essentially impossible; please let's not add another way to make it more
difficult.

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


[Libreoffice-commits] core.git: vcl/source

2014-05-20 Thread Chris Laplante
 vcl/source/fontsubset/xlat.cxx |3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

New commits:
commit eff1d991de77700505ce18dfaaa3e01d76551ffd
Author: Chris Laplante mostthings...@gmail.com
Date:   Fri May 16 15:05:42 2014 -0400

Move #include from middle of file to top

Change-Id: I715f606ee0f94e9ffd3ef9ec889948a56a9cdc19
Reviewed-on: https://gerrit.libreoffice.org/9381
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/vcl/source/fontsubset/xlat.cxx b/vcl/source/fontsubset/xlat.cxx
index 56b4e24..3509ada 100644
--- a/vcl/source/fontsubset/xlat.cxx
+++ b/vcl/source/fontsubset/xlat.cxx
@@ -18,6 +18,7 @@
  */
 
 #include rtl/textcvt.h
+#include xlat.hxx
 #include tools/debug.hxx
 
 namespace {
@@ -138,8 +139,6 @@ void ConverterCache::convertStr( int nSelect, const 
sal_Unicode* pSrc, sal_uInt1
 
 } // anonymous namespace
 
-#include xlat.hxx
-
 namespace vcl
 {
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: 4 commits - bin/lo-all-static-libs configure.ac external/coinmp

2014-05-20 Thread Tor Lillqvist
 bin/lo-all-static-libs|7 +
 configure.ac  |   42 +++---
 external/coinmp/ExternalPackage_coinmp.mk |2 +
 external/coinmp/ExternalProject_coinmp.mk |2 +
 4 files changed, 33 insertions(+), 20 deletions(-)

New commits:
commit c1fd24e18fe82e761cfa2a07dc8acf13365dfd81
Author: Tor Lillqvist t...@collabora.com
Date:   Tue May 20 14:32:17 2014 +0300

Add the CoinMP libs

Change-Id: I37b4c1081055f98c9254d64480efeb2c205e711d

diff --git a/bin/lo-all-static-libs b/bin/lo-all-static-libs
index 5a71eae..77c0507 100755
--- a/bin/lo-all-static-libs
+++ b/bin/lo-all-static-libs
@@ -52,6 +52,13 @@ echo $INSTDIR/$LIBO_LIB_FOLDER/lib*.a \
  $foolibs \
  $WORKDIR/LinkTarget/StaticLibrary/lib*.a \
  $oslibs \
+ $WORKDIR/UnpackedTarball/coinmp/Cbc/src/.libs/*.a \
+ $WORKDIR/UnpackedTarball/coinmp/Cgl/src/.libs/*.a \
+ $WORKDIR/UnpackedTarball/coinmp/Clp/src/.libs/*.a \
+ $WORKDIR/UnpackedTarball/coinmp/Clp/src/OsiClp/.libs/*.a \
+ $WORKDIR/UnpackedTarball/coinmp/CoinMP/src/.libs/*.a \
+ $WORKDIR/UnpackedTarball/coinmp/CoinUtils/src/.libs/*.a \
+ $WORKDIR/UnpackedTarball/coinmp/Osi/src/Osi/.libs/*.a \
  $WORKDIR/UnpackedTarball/glew/lib/*.a \
  $WORKDIR/UnpackedTarball/icu/source/lib/*.a \
  $WORKDIR/UnpackedTarball/lcms2/src/.libs/*.a \
commit dd633de592fde183c06101cb082744c9ce270796
Author: Tor Lillqvist t...@collabora.com
Date:   Tue May 20 14:31:57 2014 +0300

Make CoinMP build for iOS

Change-Id: I8adff18896115d7dd0fce49916a18dc830506a36

diff --git a/external/coinmp/ExternalPackage_coinmp.mk 
b/external/coinmp/ExternalPackage_coinmp.mk
index bbea3dd..b0d29c2 100644
--- a/external/coinmp/ExternalPackage_coinmp.mk
+++ b/external/coinmp/ExternalPackage_coinmp.mk
@@ -13,6 +13,8 @@ $(eval $(call 
gb_ExternalPackage_use_external_project,coinmp,coinmp))
 
 ifeq ($(OS),WNT)
 $(eval $(call 
gb_ExternalPackage_add_file,coinmp,$(LIBO_LIB_FOLDER)/CoinMP.dll,CoinMP/MSVisualStudio/v9/$(if
 $(MSVC_USE_DEBUG_RUNTIME),Debug,Release)/CoinMP.dll))
+else ifneq ($(DISABLE_DYNLOADING),)
+# Just use the static archives from workdir. See bin/lo-all-static-libs
 else ifeq ($(OS),MACOSX)
 $(eval $(call 
gb_ExternalPackage_add_file,coinmp,$(LIBO_LIB_FOLDER)/libCbc.3.dylib,Cbc/src/.libs/libCbc.3.dylib))
 $(eval $(call 
gb_ExternalPackage_add_file,coinmp,$(LIBO_LIB_FOLDER)/libCbcSolver.3.dylib,Cbc/src/.libs/libCbcSolver.3.dylib))
diff --git a/external/coinmp/ExternalProject_coinmp.mk 
b/external/coinmp/ExternalProject_coinmp.mk
index 22769e3..71e1982 100644
--- a/external/coinmp/ExternalProject_coinmp.mk
+++ b/external/coinmp/ExternalProject_coinmp.mk
@@ -27,6 +27,8 @@ else
 $(call gb_ExternalProject_get_state_target,coinmp,build) :
+$(call gb_ExternalProject_run,build,\
./configure COIN_SKIP_PROJECTS=Data/Sample \
+   $(if $(CROSS_COMPILING),--build=$(BUILD_PLATFORM) 
--host=$(HOST_PLATFORM)) \
+   $(if $(DISABLE_DYNLOADING),--disable-shared) \
--enable-dependency-linking F77=unavailable \
 $(MAKE) \
)
commit 4474d705ae2366c056630646677c85f95e2a7c97
Author: Tor Lillqvist t...@collabora.com
Date:   Tue May 20 13:18:15 2014 +0300

No reason to avoid CoinMP for Android and iOS

Change-Id: I78eb3a706251ba64ad87e7985fc43c90bea2f767

diff --git a/configure.ac b/configure.ac
index 91f464d..96259be 100644
--- a/configure.ac
+++ b/configure.ac
@@ -677,7 +677,7 @@ linux-android*)
 enable_lotuswordpro=no
 enable_mpl_subset=yes
 enable_opengl=no
-enable_coinmp=no
+enable_coinmp=yes
 enable_lpsolve=no
 enable_report_builder=no
 with_theme=tango
@@ -7433,7 +7433,7 @@ if test $_os = iOS; then
 enable_mpl_subset=yes
 enable_opengl=no
 enable_lotuswordpro=no
-enable_coinmp=no
+enable_coinmp=yes
 enable_lpsolve=no
 enable_postgresql_sdbc=no
 enable_extension_integration=no
commit 8517dc95a6ad6eec34aa5295b5133c6c18fcd1e9
Author: Tor Lillqvist t...@collabora.com
Date:   Tue May 20 13:03:06 2014 +0300

If we don't want lpsolve we don't want lpsolve

Change-Id: I6038faf296831da0855eb7cd3628c5c6978d4cb3

diff --git a/configure.ac b/configure.ac
index b4052c6..91f464d 100644
--- a/configure.ac
+++ b/configure.ac
@@ -9560,24 +9560,26 @@ else
 fi
 AC_SUBST(ENABLE_LPSOLVE)
 
-AC_MSG_CHECKING([which lpsolve to use])
-if test $with_system_lpsolve = yes; then
-AC_MSG_RESULT([external])
-SYSTEM_LPSOLVE=TRUE
-AC_CHECK_HEADER(lpsolve/lp_lib.h, [],
-   [ AC_MSG_ERROR(lpsolve headers not found.)], [])
-save_LIBS=$LIBS
-# some systems need this. Like Ubuntu
-AC_CHECK_LIB(m, floor)
-AC_CHECK_LIB(dl, dlopen)
-AC_CHECK_LIB([lpsolve55], [make_lp], [:],
-[ AC_MSG_ERROR(lpsolve library not found or too old.)], [])
-LIBS=$save_LIBS
-libo_MINGW_CHECK_DLL([lpsolve55])
-else
-

Re: Newbie question

2014-05-20 Thread yusupmuk
Thank you!



--
View this message in context: 
http://nabble.documentfoundation.org/Newbie-question-tp4109464p4109483.html
Sent from the Dev mailing list archive at Nabble.com.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: getting started from scratch w/ C++

2014-05-20 Thread Miklos Vajna
On Mon, May 19, 2014 at 09:29:45AM -0700, Tom Meinen tom_o...@yahoo.com wrote:
 If you have used either or both of these books and have insight on them, I'm 
 all ears.

See
https://wiki.openoffice.org/wiki/Recommended_Reading#C.2B.2B_Programming

That applies to LibreOffice as well.


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


[Libreoffice-commits] core.git: Branch 'libreoffice-4-2' - sw/source

2014-05-20 Thread Stephan Bergmann
 sw/source/filter/ww8/writerwordglue.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 4a86cd75a8e83b6910c5285812e86bf6575fa92b
Author: Stephan Bergmann sberg...@redhat.com
Date:   Mon May 19 23:26:22 2014 +0200

-Werror,-Wunused-result

Change-Id: Iaa06237d3e554d3a3d9c87f6c5dba715343afa25
(cherry picked from commit 9cf5e4c2500fc5d579896ea08802aba5ca671c96)

diff --git a/sw/source/filter/ww8/writerwordglue.cxx 
b/sw/source/filter/ww8/writerwordglue.cxx
index 056268e..bafa0f0 100644
--- a/sw/source/filter/ww8/writerwordglue.cxx
+++ b/sw/source/filter/ww8/writerwordglue.cxx
@@ -812,7 +812,7 @@ namespace sw
 {
 const sal_Int32 nFindLen = rFind.getLength();
 const sal_Int32 nDiff = rReplace.getLength() - 
nFindLen;
-rParams.replaceAt( nI, nFindLen, rReplace);
+rParams = rParams.replaceAt( nI, nFindLen, rReplace);
 nI += nFindLen + nDiff - 1;
 nLen += nDiff;
 bReplaced = true;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'refs/notes/commits' - c9/e4fd337abdb6460441450a0ea48eef4afa05d9

2014-05-20 Thread Caolán McNamara
 c9/e4fd337abdb6460441450a0ea48eef4afa05d9 |1 +
 1 file changed, 1 insertion(+)

New commits:
commit b0c0c835b42fd6e02bf7c723e7cdfc170d1696a7
Author: Caolán McNamara caol...@redhat.com
Date:   Tue May 20 13:32:02 2014 +0100

Notes added by 'git notes add'

diff --git a/c9/e4fd337abdb6460441450a0ea48eef4afa05d9 
b/c9/e4fd337abdb6460441450a0ea48eef4afa05d9
new file mode 100644
index 000..6ff9947
--- /dev/null
+++ b/c9/e4fd337abdb6460441450a0ea48eef4afa05d9
@@ -0,0 +1 @@
+reject: erack - theres probably always be some who want it or not, *dither*, 
ok, leave it out
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: Bug 58187

2014-05-20 Thread Michael Stahl
On 20/05/14 10:55, Ulrich Kitzinger wrote:
 Hello,
 
 I'd like to try to work on bug
 https://bugs.freedesktop.org/show_bug.cgi?id=58187 (In the Writer
 Navigator an expanded tree is collapsed when you edit an entry).
 Does anybody have any hint for me, which code fragments to look at? For
 example, what's the type of the tree view or the tree items in the
 navigator? Up to now all tries to find the corresponding collapse method
 have been fruitless.
 

i would expect the navigator code to be somewhere around here:
sw/source/core/uibase/utlui/navipi.cxx
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Am I contributor

2014-05-20 Thread yusupmuk
Hello,

This is also a question of newbie developer who have just started to be
familiar with the project and submitted simple bug fix (that needs to be
reviewed).

I wanted to know, currently what is my status? Am I contributor? I can
review commits/patches in Gerrit, submit my own ones. I created account in
www.ohloh.net where one can list all his contributions to some projects. 
When I specified my name (same as in gerrit) and project (LibreOffice) it
said that there is not such contributor in the project. I followed all steps
written in wiki page of libreoffice-development and sent my statement for
adding me to the contributors list to the email
libreoffice@lists.freedesktop.org.

Am I supposed to do something else or I just need to wait when people will
add me to the contributors list ?

Thanks for the replies :)



--
View this message in context: 
http://nabble.documentfoundation.org/Am-I-contributor-tp4109488.html
Sent from the Dev mailing list archive at Nabble.com.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] core.git: sal/cppunittester

2014-05-20 Thread Michael Meeks
 sal/cppunittester/cppunittester.cxx |8 
 1 file changed, 8 insertions(+)

New commits:
commit ef02ced3ec5e66d5c25de52fc37355ae0f44235e
Author: Michael Meeks michael.me...@collabora.com
Date:   Tue May 20 13:38:49 2014 +0100

Win32 - warn of extraordinary behaviour with LoadLibrary and long PATHs

Change-Id: I0a8ee157aa548a5b04df67a2355e3f4fd15e84cd

diff --git a/sal/cppunittester/cppunittester.cxx 
b/sal/cppunittester/cppunittester.cxx
index 6e5258b..7ac8ab5 100644
--- a/sal/cppunittester/cppunittester.cxx
+++ b/sal/cppunittester/cppunittester.cxx
@@ -206,6 +206,14 @@ public:
 manager.load(testlib, args);
 } catch (const CppUnit::DynamicLibraryManagerException e) {
 std::cerr  DynamicLibraryManagerException: \  e.what()  
\\n;
+#ifdef WIN32
+const char *pPath = getenv (PATH);
+if (pPath  strlen (pPath)  256)
+{
+std::cerr  Windows has significant build problems with long 
PATH variables ;
+std::cerr  please check your PATH variable and 
re-autogen.\n;
+}
+#endif
 return false;
 }
 #endif
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Bug 75025] LibreOffice 4.3 most annoying bugs

2014-05-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=75025

Bug 75025 depends on bug 77891, which changed state.

Bug 77891 Summary: LibO crashes as soon as one enters text in a module
https://bugs.freedesktop.org/show_bug.cgi?id=77891

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

-- 
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] core.git: sal/osl

2014-05-20 Thread Mukhiddin Yusupov
 sal/osl/w32/thread.c |   16 
 sal/osl/w32/time.c   |2 +-
 2 files changed, 9 insertions(+), 9 deletions(-)

New commits:
commit 0562a17c64f4016f2d3ba01e1c19c27feadb2543
Author: Mukhiddin Yusupov ymukhid...@gmail.com
Date:   Mon May 19 00:17:33 2014 +0200

fdo#43157 - Clean up OSL_ASSERT, DBG_ASSERT

Changed OSL_ASSERT with SAL_WARN_IF

Change-Id: Ie81eeb51f6978162abd372f7e9910344a2baaccd
Reviewed-on: https://gerrit.libreoffice.org/9401
Reviewed-by: Christian Lohmaier lohmaier+libreoff...@googlemail.com
Tested-by: Christian Lohmaier lohmaier+libreoff...@googlemail.com

diff --git a/sal/osl/w32/thread.c b/sal/osl/w32/thread.c
index 2eeea06..ac3ddc6 100644
--- a/sal/osl/w32/thread.c
+++ b/sal/osl/w32/thread.c
@@ -73,7 +73,7 @@ static oslThread oslCreateThread(oslWorkerFunction pWorker,
 /* alloc mem. for our internal data structure */
 pThreadImpl= malloc(sizeof(osl_TThreadImpl));
 
-OSL_ASSERT(pThreadImpl);
+SAL_WARN_IF(!pThreadImpl, sal.osl, !pThreadImpl - malloc error);
 
 if ( pThreadImpl == 0 )
 {
@@ -160,7 +160,7 @@ void SAL_CALL osl_resumeThread(oslThread Thread)
 {
 osl_TThreadImpl* pThreadImpl= (osl_TThreadImpl*)Thread;
 
-OSL_ASSERT(pThreadImpl);/* valid ptr? */
+SAL_WARN_IF(!pThreadImpl, sal.osl, !pThreadImpl - invalid ptr);
/* valid ptr? */
 
 ResumeThread(pThreadImpl-m_hThread);
 }
@@ -172,7 +172,7 @@ void SAL_CALL osl_suspendThread(oslThread Thread)
 {
 osl_TThreadImpl* pThreadImpl= (osl_TThreadImpl*)Thread;
 
-OSL_ASSERT(pThreadImpl);/* valid ptr? */
+SAL_WARN_IF(!pThreadImpl, sal.osl, !pThreadImpl - invalid ptr);
/* valid ptr? */
 
 SuspendThread(pThreadImpl-m_hThread);
 }
@@ -186,7 +186,7 @@ void SAL_CALL osl_setThreadPriority(oslThread Thread,
 int winPriority;
 osl_TThreadImpl* pThreadImpl= (osl_TThreadImpl*)Thread;
 
-OSL_ASSERT(pThreadImpl);/* valid ptr? */
+SAL_WARN_IF(!pThreadImpl, sal.osl, !pThreadImpl - invalid ptr);
/* valid ptr? */
 
 /*  map enum to WIN32 levels
 it would be faster and more elegant to preset
@@ -216,13 +216,13 @@ void SAL_CALL osl_setThreadPriority(oslThread Thread,
 break;
 
 case osl_Thread_PriorityUnknown:
-OSL_ASSERT(FALSE);  /* only fools try this...*/
+SAL_WARN_IF(TRUE, sal.osl, Unknown thread priority);  /* only 
fools try this...*/
 
 /* let release-version behave friendly */
 return;
 
 default:
-OSL_ASSERT(FALSE);  /* enum expanded, but forgotten here...*/
+SAL_WARN_IF(TRUE, sal.osl, Expanded, but forgotten priority);  
/* enum expanded, but forgotten here...*/
 
 /* let release-version behave friendly */
 return;
@@ -281,7 +281,7 @@ oslThreadPriority SAL_CALL osl_getThreadPriority(const 
oslThread Thread)
 break;
 
 default:
-OSL_ASSERT(FALSE);  /* WIN32 API changed, incorporate new 
prio-level! */
+SAL_WARN_IF(TRUE, sal.osl, New priority-level needed);  /* 
WIN32 API changed, incorporate new prio-level! */
 
 /* release-version behaves friendly */
 Priority= osl_Thread_PriorityUnknown;
@@ -442,7 +442,7 @@ static void RemoveKeyFromList( PTLS pTls )
 pTls-pPrev-pNext = pTls-pNext;
 else
 {
-OSL_ASSERT( pTls == g_pThreadKeyList );
+SAL_WARN_IF( pTls != g_pThreadKeyList, sal.osl, pTls != 
g_pThreadKeyList );
 g_pThreadKeyList = pTls-pNext;
 }
 
diff --git a/sal/osl/w32/time.c b/sal/osl/w32/time.c
index dd30c44..95f2ec4 100644
--- a/sal/osl/w32/time.c
+++ b/sal/osl/w32/time.c
@@ -35,7 +35,7 @@ sal_Bool SAL_CALL osl_getSystemTime(TimeValue* pTimeVal)
 FILETIME   CurTime, OffTime;
 __int64Value;
 
-OSL_ASSERT(pTimeVal != 0);
+SAL_WARN_IF(pTimeVal == 0, sal.osl, pTimeVal == 0);
 
 GetSystemTime(SystemTime);
 SystemTimeToFileTime(SystemTime, CurTime);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Bug 43157] Clean up OSL_ASSERT, DBG_ASSERT, etc.

2014-05-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=43157

--- Comment #26 from Commit Notification 
libreoffice-comm...@lists.freedesktop.org ---
Mukhiddin Yusupov committed a patch related to this issue.
It has been pushed to master:

http://cgit.freedesktop.org/libreoffice/core/commit/?id=0562a17c64f4016f2d3ba01e1c19c27feadb2543

fdo#43157 - Clean up OSL_ASSERT, DBG_ASSERT



The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.

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


Re: Am I contributor

2014-05-20 Thread Bjoern Michaelsen
On Tue, May 20, 2014 at 05:54:51AM -0700, yusupmuk wrote:
 I created account in www.ohloh.net where one can list all his contributions
 to some projects. 
 [...]
 Am I supposed to do something else or I just need to wait when people will
 add me to the contributors list ?

Ohloh watches commits on the master branch, so once the change you uploaded to
gerrit has been accepted/merged, you will show up there (possible with some ~48
hour delay, as ohloh doesnt run every hour).

Best,

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


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

2014-05-20 Thread Markus Mohrhard
 sc/inc/chartarr.hxx  |   24 
 sc/source/core/tool/chartarr.cxx |   36 ++--
 sc/source/ui/unoobj/cellsuno.cxx |6 +++---
 3 files changed, 29 insertions(+), 37 deletions(-)

New commits:
commit ee212bf4db7e38279d8c236c944ef3e09e3c0b72
Author: Markus Mohrhard markus.mohrh...@collabora.co.uk
Date:   Mon May 19 13:42:54 2014 +0200

kill some old 16 bit column indices

Change-Id: Ie1c3f3b11ed216b53f3fd385614f6fcb2c38220c

diff --git a/sc/inc/chartarr.hxx b/sc/inc/chartarr.hxx
index 4e48d27..0dfcede 100644
--- a/sc/inc/chartarr.hxx
+++ b/sc/inc/chartarr.hxx
@@ -33,26 +33,26 @@ class ScDocument;
 
 class ScMemChart
 {
-short   nRowCnt;
-short   nColCnt;
-double* pData;
+SCROW nRowCnt;
+SCCOL nColCnt;
+double* pData;
 OUString* pColText;
 OUString* pRowText;
 
 ScMemChart(const ScMemChart rMemChart);  // not implemented
 
 public:
-ScMemChart(short nCols, short nRows);
+ScMemChart(SCCOL nCols, SCROW nRows);
 ~ScMemChart();
 
-short GetColCount() const { return nColCnt; }
-short GetRowCount() const { return nRowCnt; }
-const OUString GetColText(short nCol) const { return pColText[nCol]; }
-const OUString GetRowText(short nRow) const { return pRowText[nRow]; }
-double GetData(short nCol, short nRow) const { return pData[nCol * nRowCnt 
+ nRow]; }
-void SetData(short nCol, short nRow, const double rVal) { pData[nCol * 
nRowCnt + nRow] = rVal; }
-void SetColText(short nCol, const OUString rText) { pColText[nCol] = 
rText; }
-void SetRowText(short nRow, const OUString rText) { pRowText[nRow] = 
rText; }
+SCCOL GetColCount() const { return nColCnt; }
+SCROW GetRowCount() const { return nRowCnt; }
+const OUString GetColText(SCCOL nCol) const { return pColText[nCol]; }
+const OUString GetRowText(SCROW nRow) const { return pRowText[nRow]; }
+double GetData(SCCOL nCol, SCROW nRow) const { return pData[nCol * nRowCnt 
+ nRow]; }
+void SetData(SCCOL nCol, SCROW nRow, const double rVal) { pData[nCol * 
nRowCnt + nRow] = rVal; }
+void SetColText(SCCOL nCol, const OUString rText) { pColText[nCol] = 
rText; }
+void SetRowText(SCROW nRow, const OUString rText) { pRowText[nRow] = 
rText; }
 };
 
 class SC_DLLPUBLIC ScChartArray // only parameter-struct
diff --git a/sc/source/core/tool/chartarr.cxx b/sc/source/core/tool/chartarr.cxx
index cc6311c..e3d3a19 100644
--- a/sc/source/core/tool/chartarr.cxx
+++ b/sc/source/core/tool/chartarr.cxx
@@ -33,7 +33,7 @@
 
 using ::std::vector;
 
-ScMemChart::ScMemChart(short nCols, short nRows)
+ScMemChart::ScMemChart(SCCOL nCols, SCROW nRows)
 {
 nRowCnt = nRows;
 nColCnt = nCols;
@@ -43,8 +43,8 @@ ScMemChart::ScMemChart(short nCols, short nRows)
 {
 double *pFill = pData;
 
-for (short i = 0; i  nColCnt; i++)
-for (short j = 0; j  nRowCnt; j++)
+for (SCCOL i = 0; i  nColCnt; i++)
+for (SCROW j = 0; j  nRowCnt; j++)
 *(pFill ++) = 0.0;
 }
 
@@ -238,12 +238,8 @@ ScMemChart* ScChartArray::CreateMemChartSingle()
 aRows.push_back(nStrRow);
 }
 
-
-//  Data
-
-
-ScMemChart* pMemChart = new ScMemChart(
-static_castshort(nColCount), static_castshort(nRowCount) );
+//  Data
+ScMemChart* pMemChart = new ScMemChart( nColCount, nRowCount );
 if (pMemChart)
 {
 if ( bValidData )
@@ -256,7 +252,7 @@ ScMemChart* ScChartArray::CreateMemChartSingle()
 // DBL_MIN is a Hack for Chart to recognize empty cells.
 ScAddress aPos(aCols[nCol], aRows[nRow], nTab1);
 double nVal = getCellValue(*pDocument, aPos, DBL_MIN, 
bCalcAsShown);
-pMemChart-SetData(static_castshort(nCol), 
static_castshort(nRow), nVal);
+pMemChart-SetData(nCol, nRow, nVal);
 }
 }
 }
@@ -265,7 +261,7 @@ ScMemChart* ScChartArray::CreateMemChartSingle()
 // Flag marking data as invalid?
 for (nCol=0; nColnColCount; nCol++)
 for (nRow=0; nRownRowCount; nRow++)
-pMemChart-SetData( static_castshort(nCol), 
static_castshort(nRow), DBL_MIN );
+pMemChart-SetData( nCol, nRow, DBL_MIN );
 }
 
 
@@ -287,7 +283,7 @@ ScMemChart* ScChartArray::CreateMemChartSingle()
 
 aString = aBuf.makeStringAndClear();
 }
-pMemChart-SetColText( static_castshort(nCol), aString);
+pMemChart-SetColText( nCol, aString);
 }
 
 
@@ -309,7 +305,7 @@ ScMemChart* ScChartArray::CreateMemChartSingle()
 aBuf.append(static_castsal_Int32(aRows[nRow]+1));
 aString = aBuf.makeStringAndClear();
 }
-pMemChart-SetRowText( static_castshort(nRow), aString);
+pMemChart-SetRowText( 

Re: Am I contributor

2014-05-20 Thread yusupmuk
Ok, I did not know that. Hopefully I will see myself there. Thank you



--
View this message in context: 
http://nabble.documentfoundation.org/Am-I-contributor-tp4109488p4109494.html
Sent from the Dev mailing list archive at Nabble.com.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] core.git: sal/osl

2014-05-20 Thread Stephan Bergmann
 sal/osl/w32/thread.c |   16 
 sal/osl/w32/time.c   |2 +-
 2 files changed, 9 insertions(+), 9 deletions(-)

New commits:
commit 92621df9e72b9d4e44e448f92df14c4ba731d5e4
Author: Stephan Bergmann sberg...@redhat.com
Date:   Tue May 20 15:46:31 2014 +0200

Revert fdo#43157 - Clean up OSL_ASSERT, DBG_ASSERT

This reverts commit 0562a17c64f4016f2d3ba01e1c19c27feadb2543,
sal/log.hxx functionality is only available in C++.

diff --git a/sal/osl/w32/thread.c b/sal/osl/w32/thread.c
index ac3ddc6..2eeea06 100644
--- a/sal/osl/w32/thread.c
+++ b/sal/osl/w32/thread.c
@@ -73,7 +73,7 @@ static oslThread oslCreateThread(oslWorkerFunction pWorker,
 /* alloc mem. for our internal data structure */
 pThreadImpl= malloc(sizeof(osl_TThreadImpl));
 
-SAL_WARN_IF(!pThreadImpl, sal.osl, !pThreadImpl - malloc error);
+OSL_ASSERT(pThreadImpl);
 
 if ( pThreadImpl == 0 )
 {
@@ -160,7 +160,7 @@ void SAL_CALL osl_resumeThread(oslThread Thread)
 {
 osl_TThreadImpl* pThreadImpl= (osl_TThreadImpl*)Thread;
 
-SAL_WARN_IF(!pThreadImpl, sal.osl, !pThreadImpl - invalid ptr);
/* valid ptr? */
+OSL_ASSERT(pThreadImpl);/* valid ptr? */
 
 ResumeThread(pThreadImpl-m_hThread);
 }
@@ -172,7 +172,7 @@ void SAL_CALL osl_suspendThread(oslThread Thread)
 {
 osl_TThreadImpl* pThreadImpl= (osl_TThreadImpl*)Thread;
 
-SAL_WARN_IF(!pThreadImpl, sal.osl, !pThreadImpl - invalid ptr);
/* valid ptr? */
+OSL_ASSERT(pThreadImpl);/* valid ptr? */
 
 SuspendThread(pThreadImpl-m_hThread);
 }
@@ -186,7 +186,7 @@ void SAL_CALL osl_setThreadPriority(oslThread Thread,
 int winPriority;
 osl_TThreadImpl* pThreadImpl= (osl_TThreadImpl*)Thread;
 
-SAL_WARN_IF(!pThreadImpl, sal.osl, !pThreadImpl - invalid ptr);
/* valid ptr? */
+OSL_ASSERT(pThreadImpl);/* valid ptr? */
 
 /*  map enum to WIN32 levels
 it would be faster and more elegant to preset
@@ -216,13 +216,13 @@ void SAL_CALL osl_setThreadPriority(oslThread Thread,
 break;
 
 case osl_Thread_PriorityUnknown:
-SAL_WARN_IF(TRUE, sal.osl, Unknown thread priority);  /* only 
fools try this...*/
+OSL_ASSERT(FALSE);  /* only fools try this...*/
 
 /* let release-version behave friendly */
 return;
 
 default:
-SAL_WARN_IF(TRUE, sal.osl, Expanded, but forgotten priority);  
/* enum expanded, but forgotten here...*/
+OSL_ASSERT(FALSE);  /* enum expanded, but forgotten here...*/
 
 /* let release-version behave friendly */
 return;
@@ -281,7 +281,7 @@ oslThreadPriority SAL_CALL osl_getThreadPriority(const 
oslThread Thread)
 break;
 
 default:
-SAL_WARN_IF(TRUE, sal.osl, New priority-level needed);  /* 
WIN32 API changed, incorporate new prio-level! */
+OSL_ASSERT(FALSE);  /* WIN32 API changed, incorporate new 
prio-level! */
 
 /* release-version behaves friendly */
 Priority= osl_Thread_PriorityUnknown;
@@ -442,7 +442,7 @@ static void RemoveKeyFromList( PTLS pTls )
 pTls-pPrev-pNext = pTls-pNext;
 else
 {
-SAL_WARN_IF( pTls != g_pThreadKeyList, sal.osl, pTls != 
g_pThreadKeyList );
+OSL_ASSERT( pTls == g_pThreadKeyList );
 g_pThreadKeyList = pTls-pNext;
 }
 
diff --git a/sal/osl/w32/time.c b/sal/osl/w32/time.c
index 95f2ec4..dd30c44 100644
--- a/sal/osl/w32/time.c
+++ b/sal/osl/w32/time.c
@@ -35,7 +35,7 @@ sal_Bool SAL_CALL osl_getSystemTime(TimeValue* pTimeVal)
 FILETIME   CurTime, OffTime;
 __int64Value;
 
-SAL_WARN_IF(pTimeVal == 0, sal.osl, pTimeVal == 0);
+OSL_ASSERT(pTimeVal != 0);
 
 GetSystemTime(SystemTime);
 SystemTimeToFileTime(SystemTime, CurTime);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Bug 43157] Clean up OSL_ASSERT, DBG_ASSERT, etc.

2014-05-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=43157

--- Comment #27 from Commit Notification 
libreoffice-comm...@lists.freedesktop.org ---
Stephan Bergmann committed a patch related to this issue.
It has been pushed to master:

http://cgit.freedesktop.org/libreoffice/core/commit/?id=92621df9e72b9d4e44e448f92df14c4ba731d5e4

Revert fdo#43157 - Clean up OSL_ASSERT, DBG_ASSERT



The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.

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


[Bug 30732] Character formatting not retained in entries of TOC, table lists, etc.

2014-05-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=30732

--- Comment #26 from eneko.in...@gmail.com ---
(In reply to comment #23)
 I am not 100% clear on which formats should be retained. I am currently
 retaining super and sub-script, because this is the most important thing for
 anyone working in sciences and a major problem for anyone in this field
 trying to write a text with libre office.
 
 If more complex logic is required, I would suggest to track this in a
 separate bug.

Yes, as Daniel suggests, italics are important too if your headings contain
words in foreign languages and latin phrases (e.g. Indonesia's policy of
/Konfrontasi/), document titles (e.g. Sports in /Pravda/ (1960-1965)), or
you just need some kind of emphasis (e.g. On why we /need/ italics). And in
lists of tables, illustrations, etc., this is surely much more common. I'm sure
that must be the case also in the sciences.

-- 
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] core.git: 2 commits - include/vcl slideshow/Library_OGLTrans.mk slideshow/source vcl/source

2014-05-20 Thread David Tardon
 include/vcl/opengl/OpenGLContext.hxx   |3 
 slideshow/Library_OGLTrans.mk  |2 
 slideshow/source/engine/OGLTrans/generic/OGLTrans_TransitionImpl.hxx   |   12 
 slideshow/source/engine/OGLTrans/generic/OGLTrans_TransitionerImpl.cxx |  747 
++
 vcl/source/opengl/OpenGLContext.cxx|   31 
 5 files changed, 203 insertions(+), 592 deletions(-)

New commits:
commit 9a076c316ddb105430325121713ede6270cc1aee
Author: David Tardon dtar...@redhat.com
Date:   Tue May 20 11:47:29 2014 +0200

normalize member var. names

Change-Id: Ib6215520d51ada6f956f8400de36f61a5ff025ba

diff --git 
a/slideshow/source/engine/OGLTrans/generic/OGLTrans_TransitionerImpl.cxx 
b/slideshow/source/engine/OGLTrans/generic/OGLTrans_TransitionerImpl.cxx
index 4df199f..a5e9dcc 100644
--- a/slideshow/source/engine/OGLTrans/generic/OGLTrans_TransitionerImpl.cxx
+++ b/slideshow/source/engine/OGLTrans/generic/OGLTrans_TransitionerImpl.cxx
@@ -231,10 +231,10 @@ private:
 
 /** OpenGL handle to the leaving slide's texture
 */
-unsigned int GLleavingSlide;
+unsigned int maLeavingSlideGL;
 /** OpenGL handle to the entering slide's texture
 */
-unsigned int GLenteringSlide;
+unsigned int maEnteringSlideGL;
 
 Reference presentation::XSlideShowView  mxView;
 Reference rendering::XIntegerBitmap  mxLeavingBitmap;
@@ -242,15 +242,15 @@ private:
 
 /** raw bytes of the entering bitmap
 */
-uno::Sequencesal_Int8 EnteringBytes;
+uno::Sequencesal_Int8 maEnteringBytes;
 
 /** raw bytes of the leaving bitmap
 */
-uno::Sequencesal_Int8 LeavingBytes;
+uno::Sequencesal_Int8 maLeavingBytes;
 
 #if defined( GLX_EXT_texture_from_pixmap )
-GLXPixmap LeavingPixmap;
-GLXPixmap EnteringPixmap;
+GLXPixmap maLeavingPixmapGL;
+GLXPixmap maEnteringPixmapGL;
 #endif
 bool mbRestoreSync;
 bool mbUseLeavingPixmap;
@@ -264,11 +264,11 @@ private:
 
 /** the form the raw bytes are in for the bitmaps
 */
-rendering::IntegerBitmapLayout SlideBitmapLayout;
+rendering::IntegerBitmapLayout maSlideBitmapLayout;
 
 /** the size of the slides
 */
-geometry::IntegerSize2D SlideSize;
+geometry::IntegerSize2D maSlideSize;
 
 /** Our Transition to be used.
 */
@@ -277,11 +277,11 @@ private:
 public:
 /** whether we are running on ATI fglrx with bug related to textures
  */
-bool cbBrokenTexturesATI;
+bool mbBrokenTexturesATI;
 
 /** GL version
  */
-float cnGLVersion;
+float mnGLVersion;
 
 #ifdef UNX
 float mnGLXVersion;
@@ -290,11 +290,11 @@ public:
 /**
Whether the display has GLX extension on X11, always true otherwise (?)
  */
-bool cbGLXPresent;
+bool mbGLXPresent;
 
 /** Whether Mesa is the OpenGL vendor
  */
-bool cbMesa;
+bool mbMesa;
 
 /**
whether texture from pixmap extension is available
@@ -330,28 +330,28 @@ bool OGLTransitionerImpl::initialize( const Reference 
presentation::XSlideShowV
 
 setSlides( xLeavingSlide, xEnteringSlide );
 
-return cbGLXPresent;
+return mbGLXPresent;
 }
 
 void OGLTransitionerImpl::impl_initializeFlags( bool const bGLXPresent )
 {
-cbGLXPresent = bGLXPresent;
+mbGLXPresent = bGLXPresent;
 if ( bGLXPresent ) {
 const GLubyte* version = glGetString( GL_VERSION );
 if( version  version[0] ) {
-cnGLVersion = version[0] - '0';
+mnGLVersion = version[0] - '0';
 if( version[1] == '.'  version[2] )
-cnGLVersion += (version[2] - '0')/10.0;
+mnGLVersion += (version[2] - '0')/10.0;
 } else
-cnGLVersion = 1.0;
-SAL_INFO(slideshow.opengl, GL version:   version   parsed:  
 cnGLVersion   );
+mnGLVersion = 1.0;
+SAL_INFO(slideshow.opengl, GL version:   version   parsed:  
 mnGLVersion   );
 
 const GLubyte* vendor = glGetString( GL_VENDOR );
-cbMesa = ( vendor  strstr( (const char *) vendor, Mesa ) );
-SAL_INFO(slideshow.opengl, GL vendor:   vendor   identified 
as Mesa:   cbMesa   );
+mbMesa = ( vendor  strstr( (const char *) vendor, Mesa ) );
+SAL_INFO(slideshow.opengl, GL vendor:   vendor   identified 
as Mesa:   mbMesa   );
 
 /* TODO: check for version once the bug in fglrx driver is fixed */
-cbBrokenTexturesATI = (vendor  strcmp( (const char *) vendor, ATI 
Technologies Inc. ) == 0 );
+mbBrokenTexturesATI = (vendor  strcmp( (const char *) vendor, ATI 
Technologies Inc. ) == 0 );
 }
 }
 
@@ -390,10 +390,10 @@ bool OGLTransitionerImpl::initWindowFromSlideShowView( 
const Reference presenta
 mpContext-setWinPosAndSize(Point(aCanvasArea.X, aCanvasArea.Y), 
Size(aCanvasArea.Width, aCanvasArea.Height));
 SAL_INFO(slideshow.opengl, canvas area:   aCanvasArea.X  ,  
aCanvasArea.Y   -   

[Libreoffice-commits] core.git: external/coinmp

2014-05-20 Thread Tor Lillqvist
 external/coinmp/UnpackedTarball_coinmp.mk |2 
 external/coinmp/android.build.patch.1 |  143 ++
 external/coinmp/no-binaries.patch.1   |   25 +
 3 files changed, 170 insertions(+)

New commits:
commit 63bc29b2a80e746a439f248c1bb8daa3d516370c
Author: Tor Lillqvist t...@collabora.com
Date:   Tue May 20 17:04:59 2014 +0300

Make CoinMP build for Android

(No idea whether it works, of course.)

Patch the config.sub files to recognize arm-linux-androideabi.

Don't build any binary programs as that fails for Android becuase we don't
pass in the right C++ library to use anyway. (And those programs aren't 
really
useful to us anyway, on any platform, I guess?)

Change-Id: I70c7a527db41081a51548ce6983b6a9ae8a08bc7

diff --git a/external/coinmp/UnpackedTarball_coinmp.mk 
b/external/coinmp/UnpackedTarball_coinmp.mk
index 5daf123..d27d46c 100644
--- a/external/coinmp/UnpackedTarball_coinmp.mk
+++ b/external/coinmp/UnpackedTarball_coinmp.mk
@@ -16,6 +16,8 @@ $(eval $(call gb_UnpackedTarball_fix_end_of_line,coinmp,\
 ))
 
 $(eval $(call gb_UnpackedTarball_add_patches,coinmp,\
+   external/coinmp/android.build.patch.1 \
+   external/coinmp/no-binaries.patch.1 \
external/coinmp/windows.build.patch.1 \
 ))
 
diff --git a/external/coinmp/android.build.patch.1 
b/external/coinmp/android.build.patch.1
new file mode 100644
index 000..3d8897b
--- /dev/null
+++ b/external/coinmp/android.build.patch.1
@@ -0,0 +1,143 @@
+# -*- Mode: Diff -*-
+# Patch to make the CoinMP configury recognize Android
+#
+--- coinmp/config.sub
 coinmp/config.sub
+@@ -120,7 +120,7 @@
+ # Here we must recognize all the valid KERNEL-OS combinations.
+ maybe_os=`echo $1 | sed 's/^\(.*\)-\([^-]*-[^-]*\)$/\2/'`
+ case $maybe_os in
+-  nto-qnx* | linux-gnu* | linux-dietlibc | linux-newlib* | linux-uclibc* | \
++  nto-qnx* | linux-gnu* | linux-android* | linux-dietlibc | linux-newlib* | 
linux-uclibc* | \
+   uclinux-uclibc* | uclinux-gnu* | kfreebsd*-gnu* | knetbsd*-gnu* | 
netbsd*-gnu* | \
+   storm-chaos* | os2-emx* | rtmk-nova*)
+ os=-$maybe_os
+@@ -1218,7 +1218,7 @@
+ | -udi* | -eabi* | -lites* | -ieee* | -go32* | -aux* \
+ | -chorusos* | -chorusrdb* \
+ | -cygwin* | -pe* | -psos* | -moss* | -proelf* | -rtems* \
+-| -mingw32* | -linux-gnu* | -linux-newlib* | -linux-uclibc* \
++| -mingw32* | -linux-gnu* | -linux-androideabi* | -linux-newlib* 
| -linux-uclibc* \
+ | -uxpv* | -beos* | -mpeix* | -udk* \
+ | -interix* | -uwin* | -mks* | -rhapsody* | -darwin* | -opened* \
+ | -openstep* | -oskit* | -conix* | -pw32* | -nonstopux* \
+--- coinmp/CoinUtils/config.sub
 coinmp/CoinUtils/config.sub
+@@ -120,7 +120,7 @@
+ # Here we must recognize all the valid KERNEL-OS combinations.
+ maybe_os=`echo $1 | sed 's/^\(.*\)-\([^-]*-[^-]*\)$/\2/'`
+ case $maybe_os in
+-  nto-qnx* | linux-gnu* | linux-dietlibc | linux-newlib* | linux-uclibc* | \
++  nto-qnx* | linux-gnu* | linux-android* | linux-dietlibc | linux-newlib* | 
linux-uclibc* | \
+   uclinux-uclibc* | uclinux-gnu* | kfreebsd*-gnu* | knetbsd*-gnu* | 
netbsd*-gnu* | \
+   storm-chaos* | os2-emx* | rtmk-nova*)
+ os=-$maybe_os
+@@ -1218,7 +1218,7 @@
+ | -udi* | -eabi* | -lites* | -ieee* | -go32* | -aux* \
+ | -chorusos* | -chorusrdb* \
+ | -cygwin* | -pe* | -psos* | -moss* | -proelf* | -rtems* \
+-| -mingw32* | -linux-gnu* | -linux-newlib* | -linux-uclibc* \
++| -mingw32* | -linux-gnu* | -linux-androideabi* | -linux-newlib* 
| -linux-uclibc* \
+ | -uxpv* | -beos* | -mpeix* | -udk* \
+ | -interix* | -uwin* | -mks* | -rhapsody* | -darwin* | -opened* \
+ | -openstep* | -oskit* | -conix* | -pw32* | -nonstopux* \
+--- coinmp/Osi/config.sub
 coinmp/Osi/config.sub
+@@ -120,7 +120,7 @@
+ # Here we must recognize all the valid KERNEL-OS combinations.
+ maybe_os=`echo $1 | sed 's/^\(.*\)-\([^-]*-[^-]*\)$/\2/'`
+ case $maybe_os in
+-  nto-qnx* | linux-gnu* | linux-dietlibc | linux-newlib* | linux-uclibc* | \
++  nto-qnx* | linux-gnu* | linux-android* | linux-dietlibc | linux-newlib* | 
linux-uclibc* | \
+   uclinux-uclibc* | uclinux-gnu* | kfreebsd*-gnu* | knetbsd*-gnu* | 
netbsd*-gnu* | \
+   storm-chaos* | os2-emx* | rtmk-nova*)
+ os=-$maybe_os
+@@ -1218,7 +1218,7 @@
+ | -udi* | -eabi* | -lites* | -ieee* | -go32* | -aux* \
+ | -chorusos* | -chorusrdb* \
+ | -cygwin* | -pe* | -psos* | -moss* | -proelf* | -rtems* \
+-| -mingw32* | -linux-gnu* | -linux-newlib* | -linux-uclibc* \
++| -mingw32* | -linux-gnu* | -linux-androideabi* | -linux-newlib* 
| -linux-uclibc* \
+ | -uxpv* | -beos* | -mpeix* | -udk* \
+ | -interix* | -uwin* | -mks* | -rhapsody* | -darwin* | -opened* \
+ | -openstep* | -oskit* | -conix* | -pw32* | 

[Bug 75025] LibreOffice 4.3 most annoying bugs

2014-05-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=75025

Bug 75025 depends on bug 77891, which changed state.

Bug 77891 Summary: LibO crashes as soon as one enters text in a module
https://bugs.freedesktop.org/show_bug.cgi?id=77891

   What|Removed |Added

 Status|RESOLVED|REOPENED
 Resolution|FIXED   |---

-- 
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] core.git: Branch 'private/kohei/external-ref-refresh' - sc/source

2014-05-20 Thread Kohei Yoshida
 sc/source/core/data/documen8.cxx |2 +-
 sc/source/ui/docshell/externalrefmgr.cxx |   13 +++--
 2 files changed, 8 insertions(+), 7 deletions(-)

New commits:
commit e274a615f99c96359fed477a8532c7693917a8f4
Author: Kohei Yoshida kohei.yosh...@collabora.com
Date:   Tue May 20 10:19:25 2014 -0400

cp#172: Purge one document shell at a time, to avoid freeze.

Import especially when we have a whole bunch of large-ish documents open
in the background.

Change-Id: I614e6daab3481c09dae47c8407497d77aec40480

diff --git a/sc/source/core/data/documen8.cxx b/sc/source/core/data/documen8.cxx
index 4563b02..6b2f028 100644
--- a/sc/source/core/data/documen8.cxx
+++ b/sc/source/core/data/documen8.cxx
@@ -806,7 +806,7 @@ void ScDocument::UpdateExternalRefLinks(Window* pWin)
 ScProgress aProgress(GetDocumentShell(), Updating external links, 
aRefLinks.size());
 for (size_t i = 0, n = aRefLinks.size(); i  n; ++i)
 {
-aProgress.SetState(i);
+aProgress.SetState(i+1);
 
 ScExternalRefLink* pRefLink = aRefLinks[i];
 if (pRefLink-Update())
diff --git a/sc/source/ui/docshell/externalrefmgr.cxx 
b/sc/source/ui/docshell/externalrefmgr.cxx
index ded749f..525bbce 100644
--- a/sc/source/ui/docshell/externalrefmgr.cxx
+++ b/sc/source/ui/docshell/externalrefmgr.cxx
@@ -2796,19 +2796,20 @@ void 
ScExternalRefManager::notifyAllLinkListeners(sal_uInt16 nFileId, LinkUpdate
 
 void ScExternalRefManager::purgeStaleSrcDocument(sal_Int32 nTimeOut)
 {
-DocShellMap aNewDocShells;
+// To avoid potentially freezing Calc, we close one stale document at a 
time.
 DocShellMap::iterator itr = maDocShells.begin(), itrEnd = 
maDocShells.end();
 for (; itr != itrEnd; ++itr)
 {
 // in 100th of a second.
 sal_Int32 nSinceLastAccess = (Time( Time::SYSTEM ) - 
itr-second.maLastAccess).GetTime();
-if (nSinceLastAccess  nTimeOut)
-aNewDocShells.insert(*itr);
-else
-// Timed out.  Let's close this.
+if (nSinceLastAccess = nTimeOut)
+{
+// Timed out.  Let's close this, and exit the loop.
 itr-second.maShell-DoClose();
+maDocShells.erase(itr);
+break;
+}
 }
-maDocShells.swap(aNewDocShells);
 
 if (maDocShells.empty())
 maSrcDocTimer.Stop();
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: officecfg/registry sw/source sw/uiconfig

2014-05-20 Thread Michal Siedlaczek
 officecfg/registry/schema/org/openoffice/Office/Writer.xcs |   26 
 sw/source/core/uibase/inc/optload.hxx  |   37 
 sw/source/core/uibase/inc/wordcountdialog.hxx  |4 
 sw/source/ui/config/optload.cxx|   40 
 sw/source/ui/dialog/wordcountdialog.cxx|   27 
 sw/uiconfig/swriter/ui/optgeneralpage.ui   |  600 +++--
 sw/uiconfig/swriter/ui/wordcount.ui|   52 +
 7 files changed, 500 insertions(+), 286 deletions(-)

New commits:
commit 43cd2f79da7a639d09fe43e6194f7568cbc78970
Author: Michal Siedlaczek michal.siedlac...@gmail.com
Date:   Sat Jan 25 11:19:16 2014 +0100

Displaying the number of standardized pages in the Word count window

The number of standardized pages (unit defined in the options)
for the entire document and the selected area is displayed
(if enabled in options) in the Word count window.
The number of pages of size P for an area containing N characters equals 
P/N.
This value is a float with one decimal place precision.
The default size of the standardized page is 1800 characters
(including whitespace characters).

The configuration can be altered in Options-Writer-General.

Conflicts:
sw/source/ui/config/optload.cxx

Change-Id: If13e87c73cb7706ff6618fa0352ba29c7c670aec
Reviewed-on: https://gerrit.libreoffice.org/7642
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/officecfg/registry/schema/org/openoffice/Office/Writer.xcs 
b/officecfg/registry/schema/org/openoffice/Office/Writer.xcs
index b59764c..409e7d1 100644
--- a/officecfg/registry/schema/org/openoffice/Office/Writer.xcs
+++ b/officecfg/registry/schema/org/openoffice/Office/Writer.xcs
@@ -5463,6 +5463,32 @@
 /info
 value—–/value
   /prop
+  prop oor:name=ShowStandardizedPageCount oor:type=xs:boolean 
oor:nillable=false
+info
+  descconfigures if standardized page count should be displayed on 
the Word Count window/desc
+  labelShow Standardized Page Count/label
+/info
+valuefalse/value
+  /prop
+  prop oor:name=StandardizedPageSize oor:type=xs:int 
oor:nillable=false
+info
+  descconfigures how many characters is considered one standardized 
page/desc
+  labelCharacters Per Standardized Page/label
+/info
+constraints
+  minInclusive oor:value=1
+info
+  descSpecifies the minimum value allowed./desc
+/info
+  /minInclusive
+  maxInclusive oor:value=10
+info
+  descSpecifies the maximum value allowed./desc
+/info
+  /maxInclusive
+/constraints
+value1800/value
+  /prop
 /group
 group oor:name=Navigator
   info
diff --git a/sw/source/core/uibase/inc/optload.hxx 
b/sw/source/core/uibase/inc/optload.hxx
index 015..2c1f739 100644
--- a/sw/source/core/uibase/inc/optload.hxx
+++ b/sw/source/core/uibase/inc/optload.hxx
@@ -39,25 +39,28 @@ class SwWrtShell;
 class SwLoadOptPage : public SfxTabPage
 {
 private:
-RadioButton* m_pAlwaysRB;
-RadioButton* m_pRequestRB;
-RadioButton* m_pNeverRB;
-
-CheckBox*m_pAutoUpdateFields;
-CheckBox*m_pAutoUpdateCharts;
-
-ListBox* m_pMetricLB;
-FixedText*   m_pTabFT;
-MetricField* m_pTabMF;
-CheckBox*m_pUseSquaredPageMode;
-CheckBox*m_pUseCharUnit;
-Edit*m_pWordCountED;
-
-SwWrtShell*  m_pWrtShell;
-sal_uInt16   m_nLastTab;
-sal_Int32m_nOldLinkMode;
+RadioButton* m_pAlwaysRB;
+RadioButton* m_pRequestRB;
+RadioButton* m_pNeverRB;
+
+CheckBox*m_pAutoUpdateFields;
+CheckBox*m_pAutoUpdateCharts;
+
+ListBox* m_pMetricLB;
+FixedText*   m_pTabFT;
+MetricField* m_pTabMF;
+CheckBox*m_pUseSquaredPageMode;
+CheckBox*m_pUseCharUnit;
+Edit*m_pWordCountED;
+CheckBox*m_pShowStandardizedPageCount;
+NumericField*m_pStandardizedPageSizeNF;
+
+SwWrtShell*  m_pWrtShell;
+sal_uInt16   m_nLastTab;
+sal_Int32m_nOldLinkMode;
 
 DECL_LINK(MetricHdl, void *);
+DECL_LINK(StandardizedPageCountCheckHdl, void *);
 
 public:
 SwLoadOptPage(Window* pParent, const SfxItemSet rSet);
diff --git a/sw/source/core/uibase/inc/wordcountdialog.hxx 
b/sw/source/core/uibase/inc/wordcountdialog.hxx
index 9656015..fef9d92 100644
--- a/sw/source/core/uibase/inc/wordcountdialog.hxx
+++ b/sw/source/core/uibase/inc/wordcountdialog.hxx
@@ -31,18 +31,22 @@ class SwWordCountFloatDlg : public SfxModelessDialog
 virtual voidActivate() SAL_OVERRIDE;
 void SetValues(const SwDocStat rCurrent, const SwDocStat rDoc);
 void showCJK(bool bShowCJK);
+void 

[Bug 40469] Modifed Frame Style Options Not Preserved, Keep Ratio Option Unavailable

2014-05-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=40469

--- Comment #11 from Andrei Furtuna furtunas...@gmail.com ---
Cannot identify described bug in version 4.3 dev, keep ration option is
available and working properly. 

Gilvan Vilarim, can you describe more detailed the bug situation you got?

-- 
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] core.git: 2 commits - cui/source include/svx include/vcl svx/source vcl/source

2014-05-20 Thread Michael Stahl
 cui/source/tabpages/paragrph.cxx |6 --
 include/svx/relfld.hxx   |3 ++-
 include/vcl/builder.hxx  |2 ++
 svx/source/dialog/relfld.cxx |   14 ++
 vcl/source/window/builder.cxx|6 ++
 5 files changed, 24 insertions(+), 7 deletions(-)

New commits:
commit 68bc8af6b6c4d0399bd10ed450d81aa2f17931a1
Author: Michael Stahl mst...@redhat.com
Date:   Tue May 20 16:20:42 2014 +0200

SvxRelativeField: set the unit from the passed stringmap

In the Paragraph Indents  Spacing tab there are fields with unit:
object class=svxlo-SvxRelativeField id=spinED_LEFTINDENT:0cm

But the unit is effectively ignored, and the resulting SvxRelativeField
with FUNIT_NONE and  max somehow results in 0.18cm max when the
dialog sets cm as unit.

Change-Id: I6001c93079fd7d899f1e119913b29ea4898b1c34

diff --git a/include/svx/relfld.hxx b/include/svx/relfld.hxx
index 9b9d726..91fb36d 100644
--- a/include/svx/relfld.hxx
+++ b/include/svx/relfld.hxx
@@ -19,6 +19,7 @@
 #ifndef INCLUDED_SVX_RELFLD_HXX
 #define INCLUDED_SVX_RELFLD_HXX
 
+#include tools/fldunit.hxx
 #include vcl/field.hxx
 #include svx/svxdllapi.h
 
@@ -39,7 +40,7 @@ protected:
 voidModify() SAL_OVERRIDE;
 
 public:
-SvxRelativeField( Window* pParent, WinBits nBits);
+SvxRelativeField(Window* pParent, WinBits nBits, FieldUnit eUnit);
 
 voidEnableRelativeMode( sal_uInt16 nMin = 50, sal_uInt16 nMax 
= 150,
 sal_uInt16 nStep = 5 );
diff --git a/include/vcl/builder.hxx b/include/vcl/builder.hxx
index f8208f5..3f4841e 100644
--- a/include/vcl/builder.hxx
+++ b/include/vcl/builder.hxx
@@ -13,6 +13,7 @@
 #include typeinfo
 #include osl/module.hxx
 #include tools/resmgr.hxx
+#include tools/fldunit.hxx
 #include vcl/dllapi.h
 #include vcl/window.hxx
 #include xmlreader/xmlreader.hxx
@@ -303,6 +304,7 @@ public:
 static OString convertMnemonicMarkup(const OString rIn);
 
 static OString extractCustomProperty(stringmap rMap);
+static FieldUnit detectUnit(OString const);
 
 static bool extractDropdown(stringmap rMap);
 
diff --git a/svx/source/dialog/relfld.cxx b/svx/source/dialog/relfld.cxx
index 1031c89..b4db001 100644
--- a/svx/source/dialog/relfld.cxx
+++ b/svx/source/dialog/relfld.cxx
@@ -22,7 +22,8 @@
 
 
 
-SvxRelativeField::SvxRelativeField( Window* pParent, WinBits nBits)
+SvxRelativeField::SvxRelativeField(
+Window *const pParent, WinBits const nBits, FieldUnit const eUnit)
 : MetricField( pParent, nBits)
 , nRelMin(0)
 , nRelMax(0)
@@ -32,15 +33,20 @@ SvxRelativeField::SvxRelativeField( Window* pParent, 
WinBits nBits)
 , bNegativeEnabled(false)
 
 {
-
+SetUnit(eUnit);
 SetDecimalDigits( 2 );
 SetMin( 0 );
 SetMax(  );
 }
 
-extern C SAL_DLLPUBLIC_EXPORT Window* SAL_CALL makeSvxRelativeField(Window 
*pParent, VclBuilder::stringmap )
+extern C SAL_DLLPUBLIC_EXPORT Window* SAL_CALL
+makeSvxRelativeField(Window *const pParent, VclBuilder::stringmap  rMap)
 {
-return new SvxRelativeField(pParent, WB_BORDER | WB_SPIN | WB_REPEAT | 
WB_LEFT | WB_GROUP);
+OString const custom(VclBuilder::extractCustomProperty(rMap));
+FieldUnit const eUnit(VclBuilder::detectUnit(custom));
+SvxRelativeField *const pRet = new SvxRelativeField(pParent,
+WB_BORDER | WB_SPIN | WB_REPEAT | WB_LEFT | WB_GROUP, eUnit);
+return pRet;
 }
 
 
diff --git a/vcl/source/window/builder.cxx b/vcl/source/window/builder.cxx
index f3858d1..fb0356a 100644
--- a/vcl/source/window/builder.cxx
+++ b/vcl/source/window/builder.cxx
@@ -904,6 +904,12 @@ namespace
 }
 }
 
+FieldUnit VclBuilder::detectUnit(OString const rString)
+{
+OString const unit(extractUnit(rString));
+return detectMetricUnit(unit);
+}
+
 void VclBuilder::ensureDefaultWidthChars(VclBuilder::stringmap rMap)
 {
 OString sWidthChars(width-chars);
commit 35199df7b7af9d9dd3e98eb5f1b24ac1d407345c
Author: Michael Stahl mst...@redhat.com
Date:   Tue May 20 11:18:55 2014 +0200

(related: fdo#62176) Revert reset min/max values in paragraph ...

... dialog before setting the actual value

This prevents handling negative first-line indent in Impress, for which
it is necessary that the values are set before the limits.

This reverts commit 3d7ff1e907713d8ed53a1e193641a2f16de3c38b.

Change-Id: I47807d7d49716acab3657de1da64c6e9a99fc44d

diff --git a/cui/source/tabpages/paragrph.cxx b/cui/source/tabpages/paragrph.cxx
index 6dba191..0e71f68 100644
--- a/cui/source/tabpages/paragrph.cxx
+++ b/cui/source/tabpages/paragrph.cxx
@@ -431,8 +431,6 @@ void SvxStdParagraphTabPage::Reset( const SfxItemSet rSet )
 SetFieldUnit( *m_pLineDistAtMetricBox, eFUnit );
 }
 
-ELRLoseFocusHdl( NULL ); // reset min/max values
-
 sal_uInt16 _nWhich = GetWhich( SID_ATTR_LRSPACE );
 SfxItemState eItemState = rSet.GetItemState( _nWhich );
 
@@ -590,6 

[Libreoffice-commits] core.git: dbaccess/source

2014-05-20 Thread Stephan Bergmann
 dbaccess/source/core/api/FilteredContainer.cxx |   18 --
 1 file changed, 18 deletions(-)

New commits:
commit dc24203e5f20dcbf2818651a8802e67338293b3c
Author: Stephan Bergmann sberg...@redhat.com
Date:   Tue May 20 17:01:38 2014 +0200

fdo#78971 There's even a git hook to prevent commits containing SAL_DEBUG

Change-Id: I5a46e1eed129edf3648c68c36951d6fdf9bd370d

diff --git a/dbaccess/source/core/api/FilteredContainer.cxx 
b/dbaccess/source/core/api/FilteredContainer.cxx
index 21c0b2a..e65092f 100644
--- a/dbaccess/source/core/api/FilteredContainer.cxx
+++ b/dbaccess/source/core/api/FilteredContainer.cxx
@@ -302,15 +302,9 @@ sal_Int32 createWildCardVector(Sequence OUString  
_rTableFilter, ::std::vecto
 
 try
 {
-SAL_DEBUG(OFilteredContainer::construct(). 
-  Getting metadata ...);
-
 Reference XConnection  xCon( m_xConnection, UNO_SET_THROW );
 m_xMetaData.set( xCon-getMetaData(), UNO_SET_THROW );
 
-SAL_DEBUG(OFilteredContainer::construct(). 
-  Metadata got.);
-
 // create a table table filter suitable for the 
XDatabaseMetaData::getTables call,
 // taking into account both the externally-provided table type 
filter, and any
 // table type restriction which is inherent to the container
@@ -318,9 +312,6 @@ sal_Int32 createWildCardVector(Sequence OUString  
_rTableFilter, ::std::vecto
 OUString sInherentTableTypeRestriction( getTableTypeRestriction() 
);
 if ( !sInherentTableTypeRestriction.isEmpty() )
 {
-SAL_DEBUG(OFilteredContainer::construct(). 
-  NOT InherentTableTypeRestriction.);
-
 if ( _rTableTypeFilter.getLength() != 0 )
 {
 const OUString* tableType= 
_rTableTypeFilter.getConstArray();
@@ -342,9 +333,6 @@ sal_Int32 createWildCardVector(Sequence OUString  
_rTableFilter, ::std::vecto
 }
 else
 {
-SAL_DEBUG(OFilteredContainer::construct(). 
-  InherentTableTypeRestriction.);
-
 // no container-inherent restriction for the table types
 if ( _rTableTypeFilter.getLength() == 0 )
 {   // no externally-provided table type filter = use the 
default filter
@@ -356,16 +344,10 @@ sal_Int32 createWildCardVector(Sequence OUString  
_rTableFilter, ::std::vecto
 }
 }
 
-SAL_DEBUG(OFilteredContainer::construct(). 
-  Getting tables ...);
-
 static const OUString sAll(%);
 Reference XResultSet  xTables = m_xMetaData-getTables( Any(), 
sAll, sAll, aTableTypeFilter );
 Reference XRow  xCurrentRow( xTables, UNO_QUERY_THROW );
 
-SAL_DEBUG(OFilteredContainer::construct(). 
-  Tables got.);
-
 TableInfos aUnfilteredTables;
 
 OUString sCatalog, sSchema, sName, sType;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-4-2' - dbaccess/source

2014-05-20 Thread Stephan Bergmann
 dbaccess/source/core/api/FilteredContainer.cxx |   18 --
 1 file changed, 18 deletions(-)

New commits:
commit 862abf22751c02a495ded3971b9a6e5c3553b514
Author: Stephan Bergmann sberg...@redhat.com
Date:   Tue May 20 17:01:38 2014 +0200

fdo#78971 There's even a git hook to prevent commits containing SAL_DEBUG

Change-Id: I5a46e1eed129edf3648c68c36951d6fdf9bd370d
(cherry picked from commit dc24203e5f20dcbf2818651a8802e67338293b3c)

diff --git a/dbaccess/source/core/api/FilteredContainer.cxx 
b/dbaccess/source/core/api/FilteredContainer.cxx
index 7e5de81..edcf256 100644
--- a/dbaccess/source/core/api/FilteredContainer.cxx
+++ b/dbaccess/source/core/api/FilteredContainer.cxx
@@ -303,15 +303,9 @@ sal_Int32 createWildCardVector(Sequence OUString  
_rTableFilter, ::std::vecto
 
 try
 {
-SAL_DEBUG(OFilteredContainer::construct(). 
-  Getting metadata ...);
-
 Reference XConnection  xCon( m_xConnection, UNO_SET_THROW );
 m_xMetaData.set( xCon-getMetaData(), UNO_SET_THROW );
 
-SAL_DEBUG(OFilteredContainer::construct(). 
-  Metadata got.);
-
 // create a table table filter suitable for the 
XDatabaseMetaData::getTables call,
 // taking into account both the externally-provided table type 
filter, and any
 // table type restriction which is inherent to the container
@@ -319,9 +313,6 @@ sal_Int32 createWildCardVector(Sequence OUString  
_rTableFilter, ::std::vecto
 OUString sInherentTableTypeRestriction( getTableTypeRestriction() 
);
 if ( !sInherentTableTypeRestriction.isEmpty() )
 {
-SAL_DEBUG(OFilteredContainer::construct(). 
-  NOT InherentTableTypeRestriction.);
-
 if ( _rTableTypeFilter.getLength() != 0 )
 {
 const OUString* tableType= 
_rTableTypeFilter.getConstArray();
@@ -343,9 +334,6 @@ sal_Int32 createWildCardVector(Sequence OUString  
_rTableFilter, ::std::vecto
 }
 else
 {
-SAL_DEBUG(OFilteredContainer::construct(). 
-  InherentTableTypeRestriction.);
-
 // no container-inherent restriction for the table types
 if ( _rTableTypeFilter.getLength() == 0 )
 {   // no externally-provided table type filter = use the 
default filter
@@ -357,16 +345,10 @@ sal_Int32 createWildCardVector(Sequence OUString  
_rTableFilter, ::std::vecto
 }
 }
 
-SAL_DEBUG(OFilteredContainer::construct(). 
-  Getting tables ...);
-
 static const OUString sAll(%);
 Reference XResultSet  xTables = m_xMetaData-getTables( Any(), 
sAll, sAll, aTableTypeFilter );
 Reference XRow  xCurrentRow( xTables, UNO_QUERY_THROW );
 
-SAL_DEBUG(OFilteredContainer::construct(). 
-  Tables got.);
-
 TableInfos aUnfilteredTables;
 
 OUString sCatalog, sSchema, sName, sType;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'feature/chart-3d-chart2' - avmedia/source

2014-05-20 Thread Zolnai Tamás
 avmedia/source/framework/modeltools.cxx |   20 ++--
 1 file changed, 14 insertions(+), 6 deletions(-)

New commits:
commit f6a3c83bd499bc8df21f937bfa5d4db14b4a5352
Author: Zolnai Tamás tamas.zol...@collabora.com
Date:   Tue May 20 16:46:43 2014 +0200

glTF embedding: handle missing image on a better way

Using empty tree leads to empty image url which
cause problem.

Change-Id: If815ce588fb303905035185404ede93fa424fcba

diff --git a/avmedia/source/framework/modeltools.cxx 
b/avmedia/source/framework/modeltools.cxx
index 1adef63..c3fa11e 100644
--- a/avmedia/source/framework/modeltools.cxx
+++ b/avmedia/source/framework/modeltools.cxx
@@ -25,6 +25,7 @@
 #include boost/property_tree/ptree.hpp
 #include boost/property_tree/json_parser.hpp
 #include boost/foreach.hpp
+#include boost/optional.hpp
 
 #ifdef ENABLE_COLLADA2GLTF
 #include COLLADA2GLTFWriter.h
@@ -71,7 +72,7 @@ static void lcl_EmbedExternals(const OUString rSourceURL, 
uno::Referenceembed:
 
 // Parse json, read externals' URI and modify this relative URI's so they 
remain valid in the new context.
 std::vectorstd::string vExternals;
-ptree aTree, aEmptyTree;
+ptree aTree;
 try
 {
 json_parser::read_json( sUrl, aTree );
@@ -85,12 +86,19 @@ static void lcl_EmbedExternals(const OUString rSourceURL, 
uno::Referenceembed:
 aTree.put(buffers. + rVal.first + 
.path.,sBufferUri.substr(sBufferUri.find_last_of('/')+1));
 }
 // Images for textures
-BOOST_FOREACH(ptree::value_type rVal,aTree.get_child(images, 
aEmptyTree))
+boost::optional ptree  aImages = aTree.get_child_optional(images);
+if( aImages )
 {
-const std::string sImageUri(rVal.second.getstd::string(path));
-vExternals.push_back(sImageUri);
-// Change path: make it contain only a file name
-aTree.put(images. + rVal.first + 
.path.,sImageUri.substr(sImageUri.find_last_of('/')+1));
+BOOST_FOREACH(ptree::value_type rVal,aImages.get())
+{
+const std::string 
sImageUri(rVal.second.getstd::string(path));
+if( !sImageUri.empty() )
+{
+vExternals.push_back(sImageUri);
+// Change path: make it contain only a file name
+aTree.put(images. + rVal.first + 
.path.,sImageUri.substr(sImageUri.find_last_of('/')+1));
+}
+}
 }
 // Shaders (contains names only)
 BOOST_FOREACH(ptree::value_type rVal,aTree.get_child(programs))
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: sd/source

2014-05-20 Thread Takeshi Abe
 sd/source/ui/func/fuinsfil.cxx |   21 +++--
 sd/source/ui/func/fuline.cxx   |8 +++-
 sd/source/ui/func/fulinend.cxx |4 ++--
 sd/source/ui/func/fulink.cxx   |4 ++--
 sd/source/ui/func/fumorph.cxx  |4 ++--
 sd/source/ui/func/fuoaprms.cxx |5 ++---
 sd/source/ui/func/fuolbull.cxx |7 +--
 7 files changed, 19 insertions(+), 34 deletions(-)

New commits:
commit 1c9ba8a81ca3b92e8fd684b4c16a06b5180ceb64
Author: Takeshi Abe t...@fixedpoint.jp
Date:   Wed May 21 00:21:12 2014 +0900

Avoid possible memory leaks in case of exceptions

Change-Id: Idb8f0df3848416d96299ebc47ac5bd53d766e52c

diff --git a/sd/source/ui/func/fuinsfil.cxx b/sd/source/ui/func/fuinsfil.cxx
index 5110cdb..3959348 100644
--- a/sd/source/ui/func/fuinsfil.cxx
+++ b/sd/source/ui/func/fuinsfil.cxx
@@ -63,6 +63,7 @@
 #include unmovss.hxx
 #include Outliner.hxx
 #include sdabstdlg.hxx
+#include boost/scoped_ptr.hpp
 
 using namespace ::com::sun::star::lang;
 using namespace ::com::sun::star::uno;
@@ -321,7 +322,7 @@ bool FuInsertFile::InsSDDinDrMode(SfxMedium* pMedium)
 
 mpDocSh-SetWaitCursor( false );
 SdAbstractDialogFactory* pFact = SdAbstractDialogFactory::Create();
-AbstractSdInsertPagesObjsDlg* pDlg = pFact ? 
pFact-CreateSdInsertPagesObjsDlg( NULL, mpDoc, pMedium, aFile ) : 0;
+boost::scoped_ptrAbstractSdInsertPagesObjsDlg pDlg(pFact ? 
pFact-CreateSdInsertPagesObjsDlg( NULL, mpDoc, pMedium, aFile ) : 0);
 
 if( !pDlg )
 return false;
@@ -404,8 +405,6 @@ bool FuInsertFile::InsSDDinDrMode(SfxMedium* pMedium)
 mpDoc-RemoveUnnecessaryMasterPages();
 }
 
-delete pDlg;
-
 return (bOK);
 }
 
@@ -414,7 +413,7 @@ bool FuInsertFile::InsSDDinDrMode(SfxMedium* pMedium)
 void FuInsertFile::InsTextOrRTFinDrMode(SfxMedium* pMedium)
 {
 SdAbstractDialogFactory* pFact = SdAbstractDialogFactory::Create();
-AbstractSdInsertPagesObjsDlg* pDlg = pFact ? 
pFact-CreateSdInsertPagesObjsDlg(NULL, mpDoc, NULL, aFile ) : 0;
+boost::scoped_ptrAbstractSdInsertPagesObjsDlg pDlg(pFact ? 
pFact-CreateSdInsertPagesObjsDlg(NULL, mpDoc, NULL, aFile ) : 0);
 if( !pDlg )
 return;
 
@@ -439,7 +438,7 @@ void FuInsertFile::InsTextOrRTFinDrMode(SfxMedium* pMedium)
- the draw outliner of the drawing engine has to draw something in
  between
- the global outliner could be used in SdPage::CreatePresObj */
-SdrOutliner* pOutliner = new ::sd::Outliner( mpDoc, 
OUTLINERMODE_TEXTOBJECT );
+boost::scoped_ptrSdrOutliner pOutliner(new ::sd::Outliner( mpDoc, 
OUTLINERMODE_TEXTOBJECT ));
 
 // set reference device
 pOutliner-SetRefDevice( SD_MOD()-GetRefDevice( *mpDocSh ) );
@@ -542,10 +541,7 @@ void FuInsertFile::InsTextOrRTFinDrMode(SfxMedium* pMedium)
 }
 }
 }
-delete pOutliner;
 }
-
-delete pDlg;
 }
 
 
@@ -595,7 +591,7 @@ void FuInsertFile::InsTextOrRTFinOlMode(SfxMedium* pMedium)
- the draw outliner of the drawing engine has to draw something in
  between
- the global outliner could be used in SdPage::CreatePresObj */
-::Outliner* pOutliner = new ::Outliner( mpDoc-GetItemPool(), 
OUTLINERMODE_OUTLINEOBJECT );
+boost::scoped_ptr ::Outliner pOutliner(new ::Outliner( 
mpDoc-GetItemPool(), OUTLINERMODE_OUTLINEOBJECT ));
 
pOutliner-SetStyleSheetPool((SfxStyleSheetPool*)mpDoc-GetStyleSheetPool());
 
 // set reference device
@@ -631,7 +627,7 @@ void FuInsertFile::InsTextOrRTFinOlMode(SfxMedium* pMedium)
 
 mpDocSh-SetWaitCursor( false );
 
-SfxProgress* pProgress = new SfxProgress( mpDocSh, 
SD_RESSTR(STR_CREATE_PAGES), nNewPages);
+boost::scoped_ptrSfxProgress pProgress(new SfxProgress( mpDocSh, 
SD_RESSTR(STR_CREATE_PAGES), nNewPages));
 if( pProgress )
 pProgress-SetState( 0, 100 );
 
@@ -675,13 +671,10 @@ void FuInsertFile::InsTextOrRTFinOlMode(SfxMedium* 
pMedium)
 
 pDocliner-GetUndoManager().LeaveListAction();
 
-if( pProgress )
-delete pProgress;
+pProgress.reset();
 
 mpDocSh-SetWaitCursor( true );
 }
-
-delete pOutliner;
 }
 
 
diff --git a/sd/source/ui/func/fuline.cxx b/sd/source/ui/func/fuline.cxx
index ee03d31..46eafb8 100644
--- a/sd/source/ui/func/fuline.cxx
+++ b/sd/source/ui/func/fuline.cxx
@@ -36,6 +36,7 @@
 #include app.hrc
 #include svx/svxdlg.hxx
 #include svx/dialogs.hrc
+#include boost/scoped_ptr.hpp
 
 namespace sd {
 
@@ -72,11 +73,11 @@ void FuLine::DoExecute( SfxRequest rReq )
 if( rMarkList.GetMarkCount() == 1 )
 pObj = rMarkList.GetMark(0)-GetMarkedSdrObj();
 
-SfxItemSet* pNewAttr = new SfxItemSet( mpDoc-GetPool() );
+boost::scoped_ptrSfxItemSet pNewAttr(new SfxItemSet( 
mpDoc-GetPool() ));
 mpView-GetAttributes( *pNewAttr );
 
 SvxAbstractDialogFactory* pFact = SvxAbstractDialogFactory::Create();
-SfxAbstractTabDialog * 

[Libreoffice-commits] core.git: sw/source

2014-05-20 Thread Stephan Bergmann
 sw/source/ui/config/optload.cxx |6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

New commits:
commit 1122d513efbb3066b4e8aefbc8dc29ce7a7e9bcf
Author: Stephan Bergmann sberg...@redhat.com
Date:   Tue May 20 17:59:56 2014 +0200

loplugin:implicitboolconversion,literaltoboolconversion

Change-Id: Ib783fd0557361286ce7b913a429c9ab8fc101c75

diff --git a/sw/source/ui/config/optload.cxx b/sw/source/ui/config/optload.cxx
index d4db9ff..af52de9 100644
--- a/sw/source/ui/config/optload.cxx
+++ b/sw/source/ui/config/optload.cxx
@@ -217,7 +217,7 @@ bool SwLoadOptPage::FillItemSet( SfxItemSet rSet )
   m_pShowStandardizedPageCount-IsChecked(),
   batch);
 batch-commit();
-bRet = sal_True;
+bRet = true;
 }
 
 if (m_pStandardizedPageSizeNF-GetText() != 
m_pStandardizedPageSizeNF-GetSavedValue())
@@ -228,11 +228,11 @@ bool SwLoadOptPage::FillItemSet( SfxItemSet rSet )
   m_pStandardizedPageSizeNF-GetValue(),
   batch);
 batch-commit();
-bRet = sal_True;
+bRet = true;
 }
 
 bool bIsSquaredPageModeFlag = m_pUseSquaredPageMode-IsChecked();
-if ( bIsSquaredPageModeFlag != m_pUseSquaredPageMode-GetSavedValue() )
+if ( bIsSquaredPageModeFlag != 
static_castbool(m_pUseSquaredPageMode-GetSavedValue()) )
 {
 pMod-ApplyDefaultPageMode( bIsSquaredPageModeFlag );
 if ( m_pWrtShell )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Bug 64945] inconvenient localized symbol code

2014-05-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=64945

Thomas Arnhold thomas-l...@arnhold.org changed:

   What|Removed |Added

 CC||thomas-l...@arnhold.org
   See Also||https://bugs.freedesktop.or
   ||g/show_bug.cgi?id=32415

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


[Bug 64945] inconvenient localized symbol code

2014-05-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=64945

--- Comment #9 from Thomas Arnhold thomas-l...@arnhold.org ---
As Christian stated in the gerrit patchset, this is a localization problem.

At the moment starmath behaves like the following:

1. Inside the odf file the symbol names are in English.
2. If you have a localized version of LibreOffice (e.g. German), all symbol
names are mapped to German for the UI. You see them in German, and have to
write them in German.
3. If you store your formulas inside a odf file the localized symbol names are
mapped back to the English names.

Ergo: Changing the tranlated %OMÉGA to a better representation in french should
be no problem.

For the feature of supporting English symbol names in general, see Bug 32415.

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


[Bug 64945] inconvenient localized symbol code

2014-05-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=64945

--- Comment #10 from Thomas Arnhold thomas-l...@arnhold.org ---
Copy from gerrit:

%OMEGA is the capital omega %omega is the lower case character
Allowing a mixed version %Omega will make the formulas harder to read,
especially with short ones like %Mu, %MU, %mu...

So I'm against mixed capitalization.

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


[Bug 64945] inconvenient localized symbol code

2014-05-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=64945

--- Comment #11 from Thomas Arnhold thomas-l...@arnhold.org ---
A cleaner solution for capital symbols would be %CaptialMu for example.

-- 
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] core.git: include/osl sal/osl sal/util unotools/source

2014-05-20 Thread Stephan Bergmann
 include/osl/file.h |   16 
 include/osl/file.hxx   |   11 +--
 sal/osl/unx/file_misc.cxx  |   26 ++
 sal/osl/w32/file_dirvol.cxx|6 ++
 sal/util/sal.map   |1 +
 unotools/source/ucbhelper/tempfile.cxx |5 -
 6 files changed, 58 insertions(+), 7 deletions(-)

New commits:
commit f65de4feee506b80be0986ce28f7a2311e4fbe2c
Author: Stephan Bergmann sberg...@redhat.com
Date:   Tue May 20 18:07:54 2014 +0200

fdo#60338: Introduce osl_createDirectoryWithFlags

...so that utl::TempFile can pass osl_File_OpenFlag_Private and doesn't 
have to
resort to umask (the calls to umask around Directory::create had somewhat
erroneously been removed recently with 
1d72a0262c4570631d0aa8f98e34e21fb9d6ae42
Related fdo#60338: Create missing temp file dir with user's original 
umask,
mistaking this for creation of intermediate directories in the hierarchy).

On Windows, the flags argument to osl_createDirectoryWithFlags is ignored
completely for now.

Change-Id: Iac56a5049d579be729a3f338aa62105123edb6cb

diff --git a/include/osl/file.h b/include/osl/file.h
index c00a302..1f1ae1b 100644
--- a/include/osl/file.h
+++ b/include/osl/file.h
@@ -1130,6 +1130,22 @@ SAL_DLLPUBLIC oslFileError SAL_CALL osl_closeFile( 
oslFileHandle Handle );
 
 SAL_DLLPUBLIC oslFileError SAL_CALL osl_createDirectory( rtl_uString* 
pustrDirectoryURL );
 
+/** Create a directory, passing flags.
+
+@param url
+File URL of the directory to create.
+
+@param flags
+A combination of the same osl_File_OpenFlag_*s used by osl_openFile,
+except that osl_File_OpenFlag_Create is implied and ignored.  Support for
+the various flags can differ across operating systems.
+
+@see osl_createDirectory()
+
+@since LibreOffice 4.3
+*/
+SAL_DLLPUBLIC oslFileError SAL_CALL osl_createDirectoryWithFlags(
+rtl_uString * url, sal_uInt32 flags);
 
 /** Remove an empty directory.
 
diff --git a/include/osl/file.hxx b/include/osl/file.hxx
index 1e600e7..8f7f3b4 100644
--- a/include/osl/file.hxx
+++ b/include/osl/file.hxx
@@ -1849,6 +1849,10 @@ public:
 @param ustrDirectoryURL [in]
 Full qualified URL of the directory to create.
 
+@param flags [in]
+Optional flags, see osl_createDirectoryWithFlags for details.  This
+defaulted parameter is new since LibreOffice 4.3.
+
 @return
 E_None on success
 E_INVAL the format of the parameters was not valid
@@ -1871,9 +1875,12 @@ public:
 @see remove()
 */
 
-inline static RC create( const ::rtl::OUString ustrDirectoryURL )
+inline static RC create(
+const ::rtl::OUString ustrDirectoryURL,
+sal_Int32 flags = osl_File_OpenFlag_Read | osl_File_OpenFlag_Write )
 {
-return static_cast RC ( osl_createDirectory( ustrDirectoryURL.pData 
) );
+return static_cast RC (
+osl_createDirectoryWithFlags( ustrDirectoryURL.pData, flags ) );
 }
 
 /** Remove an empty directory.
diff --git a/sal/osl/unx/file_misc.cxx b/sal/osl/unx/file_misc.cxx
index 787866e..eb1e151 100644
--- a/sal/osl/unx/file_misc.cxx
+++ b/sal/osl/unx/file_misc.cxx
@@ -131,7 +131,8 @@ oslFileType DirectoryItem_Impl::getFileType() const
 return osl_File_Type_Unknown;
 }
 
-static oslFileError osl_psz_createDirectory(const sal_Char* pszPath);
+static oslFileError osl_psz_createDirectory(
+char const * pszPath, sal_uInt32 flags);
 static oslFileError osl_psz_removeDirectory(const sal_Char* pszPath);
 
 oslFileError SAL_CALL osl_openDirectory(rtl_uString* ustrDirectoryURL, 
oslDirectory* pDirectory)
@@ -395,6 +396,13 @@ oslFileError SAL_CALL osl_releaseDirectoryItem( 
oslDirectoryItem Item )
 
 oslFileError SAL_CALL osl_createDirectory( rtl_uString* ustrDirectoryURL )
 {
+return osl_createDirectoryWithFlags(
+ustrDirectoryURL, osl_File_OpenFlag_Read | osl_File_OpenFlag_Write);
+}
+
+oslFileError osl_createDirectoryWithFlags(
+rtl_uString * ustrDirectoryURL, sal_uInt32 flags)
+{
 char path[PATH_MAX];
 oslFileError eRet;
 
@@ -410,7 +418,7 @@ oslFileError SAL_CALL osl_createDirectory( rtl_uString* 
ustrDirectoryURL )
   return oslTranslateFileError( OSL_FET_ERROR, errno );
 #endif/* MACOSX */
 
-return osl_psz_createDirectory( path );
+return osl_psz_createDirectory( path, flags );
 }
 
 oslFileError SAL_CALL osl_removeDirectory( rtl_uString* ustrDirectoryURL )
@@ -433,10 +441,20 @@ oslFileError SAL_CALL osl_removeDirectory( rtl_uString* 
ustrDirectoryURL )
 return osl_psz_removeDirectory( path );
 }
 
-static oslFileError osl_psz_createDirectory( const sal_Char* pszPath )
+oslFileError osl_psz_createDirectory(char const * pszPath, sal_uInt32 flags)
 {
 int nRet=0;
-int mode = S_IRWXU | S_IRWXG | S_IRWXO;
+int mode
+= (((flags  osl_File_OpenFlag_Read) == 0
+

[Libreoffice-commits] core.git: Branch 'feature/textbox' - sw/inc sw/source

2014-05-20 Thread Miklos Vajna
 sw/inc/textboxhelper.hxx |9 +
 sw/source/core/doc/textboxhelper.cxx |   30 ++
 sw/source/core/unocore/unodraw.cxx   |9 -
 3 files changed, 47 insertions(+), 1 deletion(-)

New commits:
commit f1889b1a4cdd9d175103bbf9074c75690547e70f
Author: Miklos Vajna vmik...@collabora.co.uk
Date:   Tue May 20 18:33:05 2014 +0200

SwXDrawPage::getCount(): ignore textboxes

Change-Id: Ie011f808b35d6f5176965889eeda5cc120974f88

diff --git a/sw/inc/textboxhelper.hxx b/sw/inc/textboxhelper.hxx
index f7750b3..bac2dcc 100644
--- a/sw/inc/textboxhelper.hxx
+++ b/sw/inc/textboxhelper.hxx
@@ -10,10 +10,14 @@
 #ifndef INCLUDED_SW_INC_TEXTBOXHELPER_HXX
 #define INCLUDED_SW_INC_TEXTBOXHELPER_HXX
 
+#include list
+
 #include com/sun/star/uno/Any.h
 #include com/sun/star/uno/Type.h
 
+class SdrPage;
 class SwFrmFmt;
+class SwDoc;
 
 /**
  * A TextBox is a TextFrame, that is tied to a drawinglayer shape.
@@ -34,6 +38,11 @@ public:
 static void syncProperty(SwFrmFmt* pShape, sal_uInt16 nWID, sal_uInt8 
nMemberID, const OUString rPropertyName, const css::uno::Any rValue);
 /// If we have an associated TextFrame, then return that.
 static SwFrmFmt* findTextBox(SwFrmFmt* pShape);
+
+/// Look up TextFrames in a document, which are in fact TextBoxes.
+static std::listSwFrmFmt* findTextBoxes(SwDoc* pDoc);
+/// Count number of shapes in the document, excluding TextBoxes.
+static sal_Int32 getCount(SdrPage* pPage, std::listSwFrmFmt* 
rTextBoxes);
 };
 
 #endif // INCLUDED_SW_INC_TEXTBOXHELPER_HXX
diff --git a/sw/source/core/doc/textboxhelper.cxx 
b/sw/source/core/doc/textboxhelper.cxx
index 85b18b9..4671580 100644
--- a/sw/source/core/doc/textboxhelper.cxx
+++ b/sw/source/core/doc/textboxhelper.cxx
@@ -18,9 +18,11 @@
 #include unotextbodyhf.hxx
 #include unotextrange.hxx
 #include unomid.h
+#include dflyobj.hxx
 
 #include svx/svdoashp.hxx
 #include svx/unopage.hxx
+#include svx/svdpage.hxx
 
 #include com/sun/star/table/BorderLine2.hpp
 #include com/sun/star/text/SizeType.hpp
@@ -81,6 +83,34 @@ void SwTextBoxHelper::destroy(SwFrmFmt* pShape)
 }
 }
 
+std::listSwFrmFmt* SwTextBoxHelper::findTextBoxes(SwDoc* pDoc)
+{
+std::listSwFrmFmt* aRet;
+
+SwFrmFmts rSpzFrmFmts = *pDoc-GetSpzFrmFmts();
+for (SwFrmFmts::iterator it = rSpzFrmFmts.begin(); it != 
rSpzFrmFmts.end(); ++it)
+{
+SwFrmFmt* pTextBox = findTextBox(*it);
+if (pTextBox)
+aRet.push_back(pTextBox);
+}
+
+return aRet;
+}
+
+sal_Int32 SwTextBoxHelper::getCount(SdrPage* pPage, std::listSwFrmFmt* 
rTextBoxes)
+{
+sal_Int32 nRet = 0;
+for (size_t i = 0; i  pPage-GetObjCount(); ++i)
+{
+SwVirtFlyDrawObj* pObject = PTR_CAST(SwVirtFlyDrawObj, 
pPage-GetObj(i));
+if (pObject  std::find(rTextBoxes.begin(), rTextBoxes.end(), 
pObject-GetFmt()) != rTextBoxes.end())
+continue;
+++nRet;
+}
+return nRet;
+}
+
 SwFrmFmt* SwTextBoxHelper::findTextBox(SwFrmFmt* pShape)
 {
 SwFrmFmt* pRet = 0;
diff --git a/sw/source/core/unocore/unodraw.cxx 
b/sw/source/core/unocore/unodraw.cxx
index 6cef384..0c639fa 100644
--- a/sw/source/core/unocore/unodraw.cxx
+++ b/sw/source/core/unocore/unodraw.cxx
@@ -30,6 +30,7 @@
 #include swunohelper.hxx
 #include textboxhelper.hxx
 #include doc.hxx
+#include docary.hxx
 #include IDocumentUndoRedo.hxx
 #include fmtcntnt.hxx
 #include fmtflcnt.hxx
@@ -516,7 +517,13 @@ sal_Int32 SwXDrawPage::getCount(void) throw( 
uno::RuntimeException, std::excepti
 else
 {
 ((SwXDrawPage*)this)-GetSvxPage();
-return pDrawPage-getCount();
+
+std::listSwFrmFmt* aTextBoxes = SwTextBoxHelper::findTextBoxes(pDoc);
+
+if (aTextBoxes.empty())
+return pDrawPage-getCount();
+else
+return SwTextBoxHelper::getCount(pDrawPage-GetSdrPage(), 
aTextBoxes);
 }
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Bug 64945] inconvenient localized symbol code

2014-05-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=64945

--- Comment #12 from Thomas Arnhold thomas-l...@arnhold.org ---
Here's the symbol code definition: starmath/source/symbol.src

StringArray RID_EXPORT_SYMBOL_NAMES holds the export names for the odf file (do
not change!)

StringArray RID_UI_SYMBOL_NAMES holds the localized symbol names, which are
language dependent.

-- 
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] core.git: Branch 'feature/textbox' - sw/inc sw/source

2014-05-20 Thread Miklos Vajna
 sw/inc/textboxhelper.hxx |3 +++
 sw/source/core/doc/textboxhelper.cxx |   32 ++--
 sw/source/core/unocore/unodraw.cxx   |6 +-
 3 files changed, 38 insertions(+), 3 deletions(-)

New commits:
commit 83857d750b9205db02d4b8bfc3c304364fae0619
Author: Miklos Vajna vmik...@collabora.co.uk
Date:   Tue May 20 19:15:29 2014 +0200

SwXDrawPage::getByIndex(): ignore textboxes

Change-Id: Ie91619e1f44694a547bfbbc5ff8110099c8002e6

diff --git a/sw/inc/textboxhelper.hxx b/sw/inc/textboxhelper.hxx
index bac2dcc..d3532c0 100644
--- a/sw/inc/textboxhelper.hxx
+++ b/sw/inc/textboxhelper.hxx
@@ -14,6 +14,7 @@
 
 #include com/sun/star/uno/Any.h
 #include com/sun/star/uno/Type.h
+#include com/sun/star/lang/IndexOutOfBoundsException.hpp
 
 class SdrPage;
 class SwFrmFmt;
@@ -43,6 +44,8 @@ public:
 static std::listSwFrmFmt* findTextBoxes(SwDoc* pDoc);
 /// Count number of shapes in the document, excluding TextBoxes.
 static sal_Int32 getCount(SdrPage* pPage, std::listSwFrmFmt* 
rTextBoxes);
+/// Get a shape by index, excluding TextBoxes.
+static css::uno::Any getByIndex(SdrPage* pPage, sal_Int32 nIndex, 
std::listSwFrmFmt* rTextBoxes) throw(css::lang::IndexOutOfBoundsException);
 };
 
 #endif // INCLUDED_SW_INC_TEXTBOXHELPER_HXX
diff --git a/sw/source/core/doc/textboxhelper.cxx 
b/sw/source/core/doc/textboxhelper.cxx
index 4671580..d2cd4d7 100644
--- a/sw/source/core/doc/textboxhelper.cxx
+++ b/sw/source/core/doc/textboxhelper.cxx
@@ -98,19 +98,47 @@ std::listSwFrmFmt* SwTextBoxHelper::findTextBoxes(SwDoc* 
pDoc)
 return aRet;
 }
 
+/// If the passed SdrObject is in fact a TextFrame, that is used as a TextBox.
+bool lcl_isTextBox(SdrObject* pSdrObject, std::listSwFrmFmt* rTextBoxes)
+{
+SwVirtFlyDrawObj* pObject = PTR_CAST(SwVirtFlyDrawObj, pSdrObject);
+return pObject  std::find(rTextBoxes.begin(), rTextBoxes.end(), 
pObject-GetFmt()) != rTextBoxes.end();
+}
+
 sal_Int32 SwTextBoxHelper::getCount(SdrPage* pPage, std::listSwFrmFmt* 
rTextBoxes)
 {
 sal_Int32 nRet = 0;
 for (size_t i = 0; i  pPage-GetObjCount(); ++i)
 {
-SwVirtFlyDrawObj* pObject = PTR_CAST(SwVirtFlyDrawObj, 
pPage-GetObj(i));
-if (pObject  std::find(rTextBoxes.begin(), rTextBoxes.end(), 
pObject-GetFmt()) != rTextBoxes.end())
+if (lcl_isTextBox(pPage-GetObj(i), rTextBoxes))
 continue;
 ++nRet;
 }
 return nRet;
 }
 
+uno::Any SwTextBoxHelper::getByIndex(SdrPage* pPage, sal_Int32 nIndex, 
std::listSwFrmFmt* rTextBoxes) throw(lang::IndexOutOfBoundsException)
+{
+if (nIndex  0 || nIndex = getCount(pPage, rTextBoxes))
+throw lang::IndexOutOfBoundsException();
+
+SdrObject* pRet = 0;
+sal_Int32 nCount = 0; // Current logical index.
+for (size_t i = 0; i  pPage-GetObjCount(); ++i)
+{
+if (lcl_isTextBox(pPage-GetObj(i), rTextBoxes))
+continue;
+if (nCount == nIndex)
+{
+pRet = pPage-GetObj(i);
+break;
+}
+++nCount;
+}
+assert(pRet);
+return uno::makeAny(uno::Referencedrawing::XShape(pRet-getUnoShape(), 
uno::UNO_QUERY));
+}
+
 SwFrmFmt* SwTextBoxHelper::findTextBox(SwFrmFmt* pShape)
 {
 SwFrmFmt* pRet = 0;
diff --git a/sw/source/core/unocore/unodraw.cxx 
b/sw/source/core/unocore/unodraw.cxx
index 0c639fa..f3851d9 100644
--- a/sw/source/core/unocore/unodraw.cxx
+++ b/sw/source/core/unocore/unodraw.cxx
@@ -538,7 +538,11 @@ uno::Any SwXDrawPage::getByIndex(sal_Int32 nIndex)
 throw lang::IndexOutOfBoundsException();
 
 ((SwXDrawPage*)this)-GetSvxPage();
-return pDrawPage-getByIndex( nIndex );
+std::listSwFrmFmt* aTextBoxes = SwTextBoxHelper::findTextBoxes(pDoc);
+if (aTextBoxes.empty())
+return pDrawPage-getByIndex( nIndex );
+else
+return SwTextBoxHelper::getByIndex(pDrawPage-GetSdrPage(), nIndex, 
aTextBoxes);
 }
 
 uno::Type  SwXDrawPage::getElementType(void) throw( uno::RuntimeException, 
std::exception )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: slideshow/source

2014-05-20 Thread David Tardon
 slideshow/source/engine/OGLTrans/generic/OGLTrans_Shaders.cxx |   87 +-
 1 file changed, 46 insertions(+), 41 deletions(-)

New commits:
commit 4ffc7955361f38140fbacc41a0f473de5173c2dc
Author: David Tardon dtar...@redhat.com
Date:   Tue May 20 17:19:13 2014 +0200

avoid duplication

Change-Id: I2de8da0ce3648657977825309d4dd9902c0f8900

diff --git a/slideshow/source/engine/OGLTrans/generic/OGLTrans_Shaders.cxx 
b/slideshow/source/engine/OGLTrans/generic/OGLTrans_Shaders.cxx
index 9e04452..191bca8 100644
--- a/slideshow/source/engine/OGLTrans/generic/OGLTrans_Shaders.cxx
+++ b/slideshow/source/engine/OGLTrans/generic/OGLTrans_Shaders.cxx
@@ -47,15 +47,42 @@
 
 #else // UNX == X11
 
-namespace unx
-{
 #define GLX_GLXEXT_PROTOTYPES 1
 #include GL/glx.h
 #include GL/glxext.h
+
+#endif
+
+namespace
+{
+
+typedef void (*GLFunction_t)();
+
+#ifdef _WIN32
+
+GLFunction_t lcl_glGetProcAddress(const char *const name)
+{
+return wglGetProcAddress(name);
+}
+
+#elif defined(MACOSX)
+
+GLFunction_t lcl_glGetProcAddress(const char *const name)
+{
+return 0;
+}
+
+#else // UNX == X11
+
+GLFunction_t lcl_glGetProcAddress(const char *const name)
+{
+return glXGetProcAddress(reinterpret_castconst GLubyte*(name));
 }
 
 #endif
 
+}
+
 bool OGLShaders::cbInitialized = false;
 
 #ifdef GL_VERSION_2_0
@@ -83,46 +110,24 @@ bool OGLShaders::Initialize()
 {
 #ifdef GL_VERSION_2_0
 if( !cbInitialized ) {
-#ifdef _WIN32
-glActiveTexture = (PFNGLACTIVETEXTUREPROC) wglGetProcAddress( 
glActiveTexture );
-glCreateShader = (PFNGLCREATESHADERPROC) wglGetProcAddress( 
glCreateShader );
-glShaderSource = (PFNGLSHADERSOURCEPROC) wglGetProcAddress( 
glShaderSource );
-glCompileShader = (PFNGLCOMPILESHADERPROC) wglGetProcAddress( 
glCompileShader );
-glGetShaderiv = (PFNGLGETSHADERIVPROC) wglGetProcAddress( 
glGetShaderiv );
-glGetShaderInfoLog = (PFNGLGETSHADERINFOLOGPROC) wglGetProcAddress( 
glGetShaderInfoLog );
-glDeleteShader = (PFNGLDELETESHADERPROC) wglGetProcAddress( 
glDeleteShader );
-glCreateProgram = (PFNGLCREATEPROGRAMPROC) wglGetProcAddress( 
glCreateProgram );
-glAttachShader = (PFNGLATTACHSHADERPROC) wglGetProcAddress( 
glAttachShader );
-glLinkProgram = (PFNGLLINKPROGRAMPROC) wglGetProcAddress( 
glLinkProgram );
-glGetProgramiv = (PFNGLGETPROGRAMIVPROC) wglGetProcAddress( 
glGetProgramiv );
-glGetProgramInfoLog = (PFNGLGETPROGRAMINFOLOGPROC) wglGetProcAddress( 
glGetProgramInfoLog );
-glUseProgram = (PFNGLUSEPROGRAMPROC) wglGetProcAddress( glUseProgram 
);
-glDeleteProgram = (PFNGLDELETEPROGRAMPROC) wglGetProcAddress( 
glDeleteProgram );
-glGetUniformLocation = (PFNGLGETUNIFORMLOCATIONPROC) 
wglGetProcAddress( glGetUniformLocation );
-glUniform1i = (PFNGLUNIFORM1IPROC) wglGetProcAddress( glUniform1i );
-glUniform1f = (PFNGLUNIFORM1FPROC) wglGetProcAddress( glUniform1f );
+glActiveTexture = (PFNGLACTIVETEXTUREPROC) lcl_glGetProcAddress( 
glActiveTexture );
+glCreateShader = (PFNGLCREATESHADERPROC) lcl_glGetProcAddress( 
glCreateShader );
+glShaderSource = (PFNGLSHADERSOURCEPROC) lcl_glGetProcAddress( 
glShaderSource );
+glCompileShader = (PFNGLCOMPILESHADERPROC) lcl_glGetProcAddress( 
glCompileShader );
+glGetShaderiv = (PFNGLGETSHADERIVPROC) lcl_glGetProcAddress( 
glGetShaderiv );
+glGetShaderInfoLog = (PFNGLGETSHADERINFOLOGPROC) lcl_glGetProcAddress( 
glGetShaderInfoLog );
+glDeleteShader = (PFNGLDELETESHADERPROC) lcl_glGetProcAddress( 
glDeleteShader );
+glCreateProgram = (PFNGLCREATEPROGRAMPROC) lcl_glGetProcAddress( 
glCreateProgram );
+glAttachShader = (PFNGLATTACHSHADERPROC) lcl_glGetProcAddress( 
glAttachShader );
+glLinkProgram = (PFNGLLINKPROGRAMPROC) lcl_glGetProcAddress( 
glLinkProgram );
+glGetProgramiv = (PFNGLGETPROGRAMIVPROC) lcl_glGetProcAddress( 
glGetProgramiv );
+glGetProgramInfoLog = (PFNGLGETPROGRAMINFOLOGPROC) 
lcl_glGetProcAddress( glGetProgramInfoLog );
+glUseProgram = (PFNGLUSEPROGRAMPROC) lcl_glGetProcAddress( 
glUseProgram );
+glDeleteProgram = (PFNGLDELETEPROGRAMPROC) lcl_glGetProcAddress( 
glDeleteProgram );
+glGetUniformLocation = (PFNGLGETUNIFORMLOCATIONPROC) 
lcl_glGetProcAddress( glGetUniformLocation );
+glUniform1i = (PFNGLUNIFORM1IPROC) lcl_glGetProcAddress( glUniform1i 
);
+glUniform1f = (PFNGLUNIFORM1FPROC) lcl_glGetProcAddress( glUniform1f 
);
 cbInitialized = true;
-#elif defined(MACOSX)
-#else
-glActiveTexture = (PFNGLACTIVETEXTUREPROC) unx::glXGetProcAddress( 
(unsigned char *) glActiveTexture );
-glCreateShader = (PFNGLCREATESHADERPROC) unx::glXGetProcAddress( 
(unsigned char *) glCreateShader );
-glShaderSource = (PFNGLSHADERSOURCEPROC) unx::glXGetProcAddress( 
(unsigned char *) glShaderSource );
-glCompileShader = (PFNGLCOMPILESHADERPROC) 

[Libreoffice-commits] core.git: sdext/Library_pdfimport.mk

2014-05-20 Thread Jan-Marek Glogowski
 sdext/Library_pdfimport.mk |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit a130817f7deb61666262075e3918630f8d340c06
Author: Jan-Marek Glogowski glo...@fbihome.de
Date:   Tue May 20 19:57:58 2014 +0200

Correct linker library sequence for pdfimport

Otherwise I get a lot of missing fontconfig symbols.

Change-Id: I9bc3e395d54380a36cd6dd73cf95b7f4e4cab54e

diff --git a/sdext/Library_pdfimport.mk b/sdext/Library_pdfimport.mk
index 2c98ed7..3e3d170 100644
--- a/sdext/Library_pdfimport.mk
+++ b/sdext/Library_pdfimport.mk
@@ -32,9 +32,9 @@ $(eval $(call gb_Library_use_static_libraries,pdfimport,\
 
 $(eval $(call gb_Library_use_externals,pdfimport,\
boost_headers \
-$(if $(filter-out WNT MACOSX,$(OS)),fontconfig) \
 zlib \
 poppler \
+$(if $(filter-out WNT MACOSX,$(OS)),fontconfig) \
 ))
 
 $(eval $(call gb_Library_add_exception_objects,pdfimport,\
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: ucb/source

2014-05-20 Thread Julien Nabet
 ucb/source/core/ucbcmds.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit f332958999cc38ad778ac381cc67d249e9bd918a
Author: Julien Nabet serval2...@yahoo.fr
Date:   Tue May 20 20:09:41 2014 +0200

Fix wrong copy paste, should be ucb::CheckinArgument

See 
http://nabble.documentfoundation.org/Wrong-copy-paste-in-ucbcmds-cxx-ucb-module-td4109103.html
for more details

Change-Id: I4f6754cf0350e969cece52008b310e1d8f182622

diff --git a/ucb/source/core/ucbcmds.cxx b/ucb/source/core/ucbcmds.cxx
index 22f5dc9..c4ed448 100644
--- a/ucb/source/core/ucbcmds.cxx
+++ b/ucb/source/core/ucbcmds.cxx
@@ -251,7 +251,7 @@ CommandProcessorInfo::CommandProcessorInfo()
 = ucb::CommandInfo(
 OUString( CHECKIN_NAME ), // Name
 CHECKIN_HANDLE, // Handle
-cppu::UnoTypeucb::GlobalTransferCommandArgument::get() ); // 
ArgType
+cppu::UnoTypeucb::CheckinArgument::get() ); // ArgType
 }
 
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'private/kohei/external-ref-refresh' - 2 commits - sc/inc sc/source

2014-05-20 Thread Kohei Yoshida
 sc/inc/externalrefmgr.hxx|   24 
 sc/inc/sc.hrc|3 
 sc/source/core/data/documen8.cxx |2 
 sc/source/ui/docshell/externalrefmgr.cxx |  170 ++-
 sc/source/ui/src/scstring.src|5 
 5 files changed, 197 insertions(+), 7 deletions(-)

New commits:
commit 36162e779da890e99c962563d060355c72a2b8e5
Author: Kohei Yoshida kohei.yosh...@collabora.com
Date:   Tue May 20 13:57:13 2014 -0400

cp#172: Populate cache tables when updating all external links.

This way, even after the loaded doc shells get purged due to timeout,
we won't reload those external documents from disk again.  One caveat
is that we currently don't pre-populate empty cells even if they are
referenced by the host document.

Change-Id: I1de2987836bf2fc5d9d7044b406fb99faa534164

diff --git a/sc/inc/externalrefmgr.hxx b/sc/inc/externalrefmgr.hxx
index bbb3094..9bcc64d 100644
--- a/sc/inc/externalrefmgr.hxx
+++ b/sc/inc/externalrefmgr.hxx
@@ -55,6 +55,12 @@ class SharedStringPool;
 
 }
 
+namespace sc {
+
+class ColumnSpanSet;
+
+}
+
 class ScExternalRefLink : public ::sfx2::SvBaseLink
 {
 public:
@@ -145,6 +151,8 @@ public:
 Table();
 ~Table();
 
+void clear();
+
 /**
  * Add cell value to the cache.
  *
@@ -268,6 +276,13 @@ public:
 void setAllCacheTableReferencedStati( bool bReferenced );
 bool areAllCacheTablesReferenced() const;
 
+/**
+ * Collect all cached non-empty cell positions, inferred directly from the
+ * cached data, not the cached range metadata stored separately in the
+ * Table.
+ */
+void getAllCachedDataSpans( sal_uInt16 nFileId, sc::ColumnSpanSet rSet ) 
const;
+
 private:
 struct ReferencedStatus
 {
@@ -295,8 +310,17 @@ public:
 ScExternalRefCache::TableTypeRef getCacheTable(sal_uInt16 nFileId, size_t 
nTabIndex) const;
 ScExternalRefCache::TableTypeRef getCacheTable(sal_uInt16 nFileId, const 
OUString rTabName, bool bCreateNew, size_t* pnIndex);
 
+/**
+ * Clear all caches including the cache tables.
+ */
 void clearCache(sal_uInt16 nFileId);
 
+/**
+ * Clear all caches but keep the tables.  All cache tables will be empty
+ * after the call, but the tables will not be removed.
+ */
+void clearCacheTables(sal_uInt16 nFileId);
+
 private:
 struct RangeHash
 {
diff --git a/sc/source/ui/docshell/externalrefmgr.cxx 
b/sc/source/ui/docshell/externalrefmgr.cxx
index 525bbce..9f6b657 100644
--- a/sc/source/ui/docshell/externalrefmgr.cxx
+++ b/sc/source/ui/docshell/externalrefmgr.cxx
@@ -53,6 +53,8 @@
 #include vcl/msgbox.hxx
 #include stringutil.hxx
 #include scmatrix.hxx
+#include columnspanset.hxx
+#include column.hxx
 
 #include memory
 #include algorithm
@@ -254,6 +256,13 @@ ScExternalRefCache::Table::~Table()
 {
 }
 
+void ScExternalRefCache::Table::clear()
+{
+maRows.clear();
+maCachedRanges.RemoveAll();
+meReferenced = REFERENCED_MARKED;
+}
+
 void ScExternalRefCache::Table::setReferencedFlag( 
ScExternalRefCache::Table::ReferencedFlag eFlag )
 {
 meReferenced = eFlag;
@@ -445,9 +454,6 @@ void ScExternalRefCache::Table::setCachedCellRange(SCCOL 
nCol1, SCROW nRow1, SCC
 maCachedRanges.Append(aRange);
 else
 maCachedRanges.Join(aRange);
-
-OUString aStr;
-maCachedRanges.Format(aStr, SCA_VALID);
 }
 
 void ScExternalRefCache::Table::setWholeTableCached()
@@ -1106,6 +1112,35 @@ bool ScExternalRefCache::areAllCacheTablesReferenced() 
const
 return maReferenced.mbAllReferenced;
 }
 
+void ScExternalRefCache::getAllCachedDataSpans( sal_uInt16 nFileId, 
sc::ColumnSpanSet rSet ) const
+{
+const DocItem* pDocItem = getDocItem(nFileId);
+if (!pDocItem)
+// This document is not cached.
+return;
+
+const std::vectorTableTypeRef rTables = pDocItem-maTables;
+for (size_t nTab = 0, nTabCount = rTables.size(); nTab  nTabCount; ++nTab)
+{
+const Table rTable = *rTables[nTab];
+std::vectorSCROW aRows;
+rTable.getAllRows(aRows);
+std::vectorSCROW::const_iterator itRow = aRows.begin(), itRowEnd = 
aRows.end();
+for (; itRow != itRowEnd; ++itRow)
+{
+SCROW nRow = *itRow;
+std::vectorSCCOL aCols;
+rTable.getAllCols(nRow, aCols);
+std::vectorSCCOL::const_iterator itCol = aCols.begin(), itColEnd 
= aCols.end();
+for (; itCol != itColEnd; ++itCol)
+{
+SCCOL nCol = *itCol;
+rSet.set(nTab, nCol, nRow, true);
+}
+}
+}
+}
+
 ScExternalRefCache::ReferencedStatus::ReferencedStatus() :
 mbAllReferenced(false)
 {
@@ -1199,6 +1234,28 @@ void ScExternalRefCache::clearCache(sal_uInt16 nFileId)
 maDocs.erase(nFileId);
 }
 
+void ScExternalRefCache::clearCacheTables(sal_uInt16 nFileId)
+{
+osl::MutexGuard aGuard(maMtxDocs);

[SOLVED] Re: Wrong copy paste in ucbcmds.cxx? (ucb module)

2014-05-20 Thread julien2412
sberg wrote
 ...
 I wonder if line 254 could be a wrong copy paste and should be:
 cppu::UnoType
 ucb::CheckinArgument
 ::get() );
 
 Yes, that's wrong.  (See UniversalContentBroker::execute in 
 ucb/source/core/ucb.cxx, where CHECKIN_HANDLE/CHECKIN_NAME expects to 
 get a CheckinArgument, not a GlobalTransferCommandArgument.

Thank you Stephan for your feedback.
I pushed the fix on master (see
http://cgit.freedesktop.org/libreoffice/core/commit/?id=f332958999cc38ad778ac381cc67d249e9bd918a)
I didn't gerrit review on 4.2 since it's not related to a known bug but if
you want to, I can.

Julien



--
View this message in context: 
http://nabble.documentfoundation.org/Wrong-copy-paste-in-ucbcmds-cxx-ucb-module-tp4109103p4109522.html
Sent from the Dev mailing list archive at Nabble.com.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Bug 64945] inconvenient localized symbol code

2014-05-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=64945

--- Comment #13 from Thomas Arnhold thomas-l...@arnhold.org ---
Maybe we should approve some (less) user confusion and remove the translation
of those Greek symbol names. None of the other function, operators,... are
translated! Absolutely none!

To reduce the user confusion we could add the Greek symbols to the Elements
Dock, so that they are easy accessible (see
f5e2d31331839fd8c6039bec057a9c28ed3c7396).

What's the opinion about that?

-- 
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] core.git: Branch 'private/kohei/external-ref-refresh' - sc/inc sc/source

2014-05-20 Thread Kohei Yoshida
 sc/inc/scopetools.hxx  |9 +
 sc/source/core/data/documen8.cxx   |3 +++
 sc/source/core/tool/scopetools.cxx |   14 ++
 3 files changed, 26 insertions(+)

New commits:
commit a1c7e073fe545e6a0bd8c5eecb152106b9ec35fa
Author: Kohei Yoshida kohei.yosh...@collabora.com
Date:   Tue May 20 14:24:44 2014 -0400

Turn the mouse cursor to the wait hand during the external link update.

Change-Id: I983127828b28c72dd12d4778b88051964e9bceaa

diff --git a/sc/inc/scopetools.hxx b/sc/inc/scopetools.hxx
index 443ceaf..785fd70 100644
--- a/sc/inc/scopetools.hxx
+++ b/sc/inc/scopetools.hxx
@@ -13,6 +13,7 @@
 #include scdllapi.h
 
 class ScDocument;
+class Window;
 
 namespace sc {
 
@@ -55,6 +56,14 @@ public:
 ~IdleSwitch();
 };
 
+class WaitPointerSwitch
+{
+Window* mpFrameWin;
+public:
+WaitPointerSwitch(Window* pWin);
+~WaitPointerSwitch();
+};
+
 }
 
 #endif
diff --git a/sc/source/core/data/documen8.cxx b/sc/source/core/data/documen8.cxx
index e8ce20c..1247563 100644
--- a/sc/source/core/data/documen8.cxx
+++ b/sc/source/core/data/documen8.cxx
@@ -87,6 +87,7 @@
 #include globalnames.hxx
 #include stringutil.hxx
 #include documentlinkmgr.hxx
+#include scopetools.hxx
 
 #include boost/scoped_ptr.hpp
 
@@ -802,6 +803,8 @@ void ScDocument::UpdateExternalRefLinks(Window* pWin)
 aRefLinks.push_back(pRefLink);
 }
 
+sc::WaitPointerSwitch aWaitSwitch(pWin);
+
 pExternalRefMgr-enableDocTimer(false);
 ScProgress aProgress(GetDocumentShell(), 
ScResId(SCSTR_UPDATE_EXTDOCS).toString(), aRefLinks.size());
 for (size_t i = 0, n = aRefLinks.size(); i  n; ++i)
diff --git a/sc/source/core/tool/scopetools.cxx 
b/sc/source/core/tool/scopetools.cxx
index 96f4458..0664219 100644
--- a/sc/source/core/tool/scopetools.cxx
+++ b/sc/source/core/tool/scopetools.cxx
@@ -9,6 +9,7 @@
 
 #include scopetools.hxx
 #include document.hxx
+#include vcl/window.hxx
 
 namespace sc {
 
@@ -56,6 +57,19 @@ IdleSwitch::~IdleSwitch()
 mrDoc.EnableIdle(mbOldValue);
 }
 
+WaitPointerSwitch::WaitPointerSwitch(Window* pWin) :
+mpFrameWin(pWin)
+{
+if (mpFrameWin)
+mpFrameWin-EnterWait();
+}
+
+WaitPointerSwitch::~WaitPointerSwitch()
+{
+if (mpFrameWin)
+mpFrameWin-LeaveWait();
+}
+
 }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: sc/source

2014-05-20 Thread Julien Nabet
 sc/source/ui/app/inputhdl.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 04831249713827fb175faa118205bb132387dad9
Author: Julien Nabet serval2...@yahoo.fr
Date:   Tue May 20 20:50:44 2014 +0200

Following fdo#78838, let's make this part safe too

(Thank you Eike)

Change-Id: Icc7061c7f7296302d6ed21f0e74a48cbaa82011a

diff --git a/sc/source/ui/app/inputhdl.cxx b/sc/source/ui/app/inputhdl.cxx
index b9e1b90..be0220d 100644
--- a/sc/source/ui/app/inputhdl.cxx
+++ b/sc/source/ui/app/inputhdl.cxx
@@ -718,7 +718,6 @@ void ScInputHandler::GetFormulaData()
 else
 {
 pFormulaData = new ScTypedCaseStrSet;
-miAutoPosFormula = pFormulaData-end();
 }
 
 if( pFormulaDataPara )
@@ -749,6 +748,7 @@ void ScInputHandler::GetFormulaData()
 pFormulaDataPara-insert(ScTypedStrData(aEntry, 0.0, 
ScTypedStrData::Standard));
 }
 }
+miAutoPosFormula = pFormulaData-end();
 pDoc-GetFormulaEntries( *pFormulaData );
 pDoc-GetFormulaEntries( *pFormulaDataPara );
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Bug 64945] inconvenient localized symbol code

2014-05-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=64945

Laurent BP jumbo4...@yahoo.fr changed:

   What|Removed |Added

 CC||jumbo4...@yahoo.fr

--- Comment #14 from Laurent BP jumbo4...@yahoo.fr ---
Hi,

I agree to remove translation of Greek letters: in French it is often confusing
about how to write letters, for instance %rhô instead of %rho. But as mentioned
in bug 32415, it would be better to use UNICODE letters (easy access through
panel Element Dock) than letters of special font.

I do not agree with mixed capitalization: it would be too confusing.

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


[Bug 64945] inconvenient localized symbol code

2014-05-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=64945

--- Comment #15 from Tomaz Vajngerl qui...@gmail.com ---
I think the best (forward and backwards compatible) solution to this is to
embrace unicode characters for Greek symbols and always write those instead of
symbol names. We already use unicode characters when we import from MathML and
doc / docx instead of symbol names (obviously because we don't know how to map
between them). 

To make this complete we need auto-replace so we can directly convert the
symbol name into the unicode character (along with other user-defined symbols
in the catalog).

I already did some experiments with unicode Greek symbols in Elements Dock
(write the unicode symbol which is locale independent instead of symbol name)
but they are useless if we don't embrace them everywhere. This is something I
wanted to work on in the future..

-- 
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] core.git: 2 commits - vcl/inc vcl/osx

2014-05-20 Thread Tor Lillqvist
 vcl/inc/osx/salinst.h |1 -
 vcl/osx/salinst.cxx   |4 +++-
 2 files changed, 3 insertions(+), 2 deletions(-)

New commits:
commit 0f836296ea3cdc10326ad54983b634168285b8e2
Author: Tor Lillqvist t...@collabora.com
Date:   Tue May 20 23:34:02 2014 +0300

Don't use two separate names for the same thing

Change-Id: Iad76d452d5dbaa03099c1a649d7f278b0151f340

diff --git a/vcl/inc/osx/salinst.h b/vcl/inc/osx/salinst.h
index 6d9a8d6..8a88a26 100644
--- a/vcl/inc/osx/salinst.h
+++ b/vcl/inc/osx/salinst.h
@@ -147,7 +147,6 @@ public:
 static const short AppExecuteSVMain   = 0x7fff;
 static const short AppEndLoopEvent= 1;
 static const short AppStartTimerEvent = 10;
-static const short AppleRemoteEvent   = 15;
 static const short YieldWakeupEvent   = 20;
 
 static NSMenu* GetDynamicDockMenu();
diff --git a/vcl/osx/salinst.cxx b/vcl/osx/salinst.cxx
index 2fce74c..b779109 100644
--- a/vcl/osx/salinst.cxx
+++ b/vcl/osx/salinst.cxx
@@ -475,7 +475,7 @@ void AquaSalInstance::handleAppDefinedEvent( NSEvent* 
pEvent )
 }
 break;
 #if !HAVE_FEATURE_MACOSX_SANDBOX
-case AppleRemoteEvent:
+case AppleRemoteControlEvent: // Defined in 
apple_remote/RemoteMainController.h
 {
 sal_Int16 nCommand = 0;
 SalData* pSalData = GetSalData();
commit c41ad388af65863b89788f874031cb129ce0f8b2
Author: Tor Lillqvist t...@collabora.com
Date:   Tue May 20 23:32:00 2014 +0300

We don't support the Apple Remote in sandboxed mode

Change-Id: Ie0a456647376b9d249850227d3481be5b4ee991d

diff --git a/vcl/osx/salinst.cxx b/vcl/osx/salinst.cxx
index 7b10141..2fce74c 100644
--- a/vcl/osx/salinst.cxx
+++ b/vcl/osx/salinst.cxx
@@ -474,6 +474,7 @@ void AquaSalInstance::handleAppDefinedEvent( NSEvent* 
pEvent )
 }
 }
 break;
+#if !HAVE_FEATURE_MACOSX_SANDBOX
 case AppleRemoteEvent:
 {
 sal_Int16 nCommand = 0;
@@ -536,6 +537,7 @@ void AquaSalInstance::handleAppDefinedEvent( NSEvent* 
pEvent )
 
 }
 break;
+#endif
 
 case YieldWakeupEvent:
 // do nothing, fall out of Yield
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'feature/refactor-god-objects'

2014-05-20 Thread Valentin Kettner
New branch 'feature/refactor-god-objects' available with the following commits:
commit 3056fa190eeb24bd6f264790497b763d20f42cf9
Author: Valentin Kettner vakevk+libreoff...@gmail.com
Date:   Tue May 20 22:45:09 2014 +0200

Finished cleaning up of IDocumentDeviceAccess methods in SwDoc.

Moved all SwDoc::getPrintData and SwDoc::setPrintData calls to their
appropriate interface methods (SwDoc::getIDocumentDeviceAccess::).

Change-Id: I5671b365238abae6d94f125fb7a99481d81392f9

commit 0a7bfbd5d5882f7835c959dddbccc1c9070491f2
Author: Valentin Kettner vakevk+libreoff...@gmail.com
Date:   Tue May 20 22:01:36 2014 +0200

Cleaning up of IDocumentDeviceAccess methods in SwDoc.

Moved all SwDoc::getJobSetup and SwDoc::setJobSetup calls to their
appropriate interface methods (SwDoc::getIDocumentDeviceAccess::).

Change-Id: I155a0341363ec42eaad3290d2e6feb8c7d73bbb1

commit 45f3f91834216d2e132128c90e56d46134865d38
Author: Valentin Kettner vakevk+libreoff...@gmail.com
Date:   Tue May 20 21:35:48 2014 +0200

Cleaning up of IDocumentDeviceAccess methods in SwDoc.

Moved all SwDoc::getReferenceDevice and SwDoc::setReferenceDevice calls to 
their
appropriate interface methods (SwDoc::getIDocumentDeviceAccess::).

Change-Id: I2a1b488500fa4f0bfbb7ebadccde2c94b70efd85

commit cf66bc9b5d438cdd5fd72ddc7f15b03eddf9291b
Author: Valentin Kettner vakevk+libreoff...@gmail.com
Date:   Tue May 20 19:31:06 2014 +0200

Removed getVirtualDevice and setVirtualDevice from SwDoc.

Nothing else had to be chaned because they werent call from anywhere
else.

Change-Id: I708b54b909c5a1814bcec858dd356e17e5c6c473

commit 5964e208481f4b05bf6d6da7bf23e83979b17c17
Author: Valentin Kettner vakevk+libreoff...@gmail.com
Date:   Tue May 20 17:29:37 2014 +0200

Started cleaning up of IDocumentDeviceAccess methods in SwDoc.

Moved all SwDoc::getPrinter and SwDoc::setPrinter calls to their
appropriate interface methods (SwDoc::getIDocumenterDeviceAccess::).

Change-Id: Icbe5369ba03a05d9e2afb1300a9c67b0a0a20c60

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Bug 65675] LibreOffice 4.2 most annoying bugs

2014-05-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=65675

Mihkel Tõnnov mihh...@gmail.com changed:

   What|Removed |Added

 Depends on||78883

--- Comment #145 from Mihkel Tõnnov mihh...@gmail.com ---
(In reply to comment #144)
 added bug 78883  - regression, bibisected

...except that you actually removed it, too :) Adding it back.

-- 
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] core.git: 6 commits - chart2/source include/vcl vcl/source

2014-05-20 Thread Markus Mohrhard
 chart2/source/view/charttypes/GL3DBarChart.cxx |6 ++-
 chart2/source/view/inc/GL3DRenderer.hxx|   17 ++---
 chart2/source/view/main/3DChartObjects.cxx |   10 ++---
 chart2/source/view/main/GL3DRenderer.cxx   |   47 ++---
 include/vcl/opengl/OpenGLHelper.hxx|7 +++
 vcl/source/opengl/OpenGLHelper.cxx |   36 +++
 6 files changed, 106 insertions(+), 17 deletions(-)

New commits:
commit 61b2ae31c6642e09b3052599ae22077e8d41eabd
Author: Markus Mohrhard markus.mohrh...@collabora.co.uk
Date:   Wed May 21 01:49:59 2014 +0200

use FBO for the picking

Change-Id: Ibab6daa1f76300d159ccd4cfbca061c8d8c1951f

diff --git a/chart2/source/view/charttypes/GL3DBarChart.cxx 
b/chart2/source/view/charttypes/GL3DBarChart.cxx
index 1653668..d36a008 100644
--- a/chart2/source/view/charttypes/GL3DBarChart.cxx
+++ b/chart2/source/view/charttypes/GL3DBarChart.cxx
@@ -217,14 +217,16 @@ public:
 
 }
 
-void GL3DBarChart::clickedAt(const Point )
+void GL3DBarChart::clickedAt(const Point rPos)
 {
+sal_uInt32 nId = 1;
 {
 PickingModeSetter(mpRenderer.get());
 render();
+nId = mpRenderer-GetPixelColorFromPoint(rPos.X(), rPos.Y());
 }
 if (mpCamera)
-mpCamera-zoom(1);
+mpCamera-zoom(nId);
 }
 
 }
diff --git a/chart2/source/view/inc/GL3DRenderer.hxx 
b/chart2/source/view/inc/GL3DRenderer.hxx
index 52d341a..b08e90e 100644
--- a/chart2/source/view/inc/GL3DRenderer.hxx
+++ b/chart2/source/view/inc/GL3DRenderer.hxx
@@ -179,6 +179,8 @@ public:
 void ProcessUnrenderedShape();
 
 void SetPickingMode(bool bPickingMode);
+
+sal_uInt32 GetPixelColorFromPoint(long nX, long nY);
 private:
 void MoveModelf(PosVecf3 trans,PosVecf3 angle,PosVecf3 scale);
 
@@ -362,6 +364,10 @@ private:
 
 bool mbPickingMode;
 SceneBox m_SenceBox;
+
+GLuint mnPickingFbo;
+GLuint mnPickingRbo;
+GLuint mnPickingTexture;
 };
 
 }
diff --git a/chart2/source/view/main/GL3DRenderer.cxx 
b/chart2/source/view/main/GL3DRenderer.cxx
index 92cf61d..cea7e0f 100644
--- a/chart2/source/view/main/GL3DRenderer.cxx
+++ b/chart2/source/view/main/GL3DRenderer.cxx
@@ -249,6 +249,8 @@ void OpenGL3DRenderer::init()
 glBufferData(GL_ARRAY_BUFFER, sizeof(squareVertices), squareVertices, 
GL_STATIC_DRAW);
 glBindBuffer(GL_ARRAY_BUFFER, 0);
 
+OpenGLHelper::createFramebuffer(m_iWidth, m_iHeight, mnPickingFbo, 
mnPickingRbo, mnPickingTexture);
+
 CHECK_GL_ERROR();
 Init3DUniformBlock();
 
@@ -1580,6 +1582,22 @@ void OpenGL3DRenderer::MoveModelf(PosVecf3 
trans,PosVecf3 angle,PosVecf3 scal
 void OpenGL3DRenderer::SetPickingMode(bool bPickingMode)
 {
 mbPickingMode = bPickingMode;
+if(mbPickingMode)
+{
+glBindFramebuffer(GL_FRAMEBUFFER, mnPickingFbo);
+}
+else
+{
+glBindFramebuffer(GL_FRAMEBUFFER, 0);
+}
+}
+
+sal_uInt32 OpenGL3DRenderer::GetPixelColorFromPoint(long nX, long nY)
+{
+boost::scoped_arraysal_uInt8 buf(new sal_uInt8[4]);
+glReadPixels(nX, nY, 1, 1, GL_BGRA, GL_UNSIGNED_BYTE, buf.get());
+Color aColor(buf[3], buf[2], buf[1], buf[0]);
+return aColor.GetColor();
 }
 
 }
commit 8c39739a38237df6e8f39f312cf08fadae896ae4
Author: Markus Mohrhard markus.mohrh...@collabora.co.uk
Date:   Wed May 21 00:41:37 2014 +0200

add method for creating framebuffer objects to OpenGLHelper

Change-Id: I08bd2e58ee98a68accae256fcbcc288a8c56ae0b

diff --git a/include/vcl/opengl/OpenGLHelper.hxx 
b/include/vcl/opengl/OpenGLHelper.hxx
index 3cb482b..70b1d2a 100644
--- a/include/vcl/opengl/OpenGLHelper.hxx
+++ b/include/vcl/opengl/OpenGLHelper.hxx
@@ -29,6 +29,13 @@ public:
 static void renderToFile(long nWidth, long nHeight, const OUString 
rFileName);
 
 static const char* GLErrorString(GLenum errorCode);
+
+/**
+ * The caller is responsible for deleting the buffer objects identified by
+ * nFramebufferId, nRenderbufferId and nTexturebufferId
+ */
+static void createFramebuffer(long nWidth, long nHeight,
+GLuint nFramebufferId, GLuint nRenderbufferId, GLuint 
nTexturebufferId);
 };
 
 VCLOPENGL_DLLPUBLIC std::ostream operator(std::ostream rStrm, const 
glm::mat4 rMatrix);
diff --git a/vcl/source/opengl/OpenGLHelper.cxx 
b/vcl/source/opengl/OpenGLHelper.cxx
index 0e9d710..ad2506d 100644
--- a/vcl/source/opengl/OpenGLHelper.cxx
+++ b/vcl/source/opengl/OpenGLHelper.cxx
@@ -268,5 +268,41 @@ std::ostream operator(std::ostream rStrm, const 
glm::mat4 rMatrix)
 return rStrm;
 }
 
+void OpenGLHelper::createFramebuffer(long nWidth, long nHeight,
+GLuint nFramebufferId, GLuint nRenderbufferId, GLuint 
nTexturebufferId)
+{
+// create a renderbuffer
+glGenRenderbuffers(1, nRenderbufferId);
+glBindRenderbuffer(GL_RENDERBUFFER, nRenderbufferId);
+glRenderbufferStorage(GL_RENDERBUFFER, GL_DEPTH_COMPONENT, nWidth, 
nHeight);
+glBindRenderbuffer(GL_RENDERBUFFER, 0);
+
+// create 

[Libreoffice-commits] core.git: chart2/source

2014-05-20 Thread Markus Mohrhard
 chart2/source/view/main/GL3DRenderer.cxx |4 
 1 file changed, 4 insertions(+)

New commits:
commit 19979ae27055cb910bfc368bfc2899d211f56be1
Author: Markus Mohrhard markus.mohrh...@collabora.co.uk
Date:   Wed May 21 02:09:11 2014 +0200

forgot to release the OpenGL resources

Change-Id: I2baefbe611f9ed28defc46ca3af332d13b32c561

diff --git a/chart2/source/view/main/GL3DRenderer.cxx 
b/chart2/source/view/main/GL3DRenderer.cxx
index cea7e0f..cf75ea6 100644
--- a/chart2/source/view/main/GL3DRenderer.cxx
+++ b/chart2/source/view/main/GL3DRenderer.cxx
@@ -110,6 +110,10 @@ OpenGL3DRenderer::~OpenGL3DRenderer()
 glDeleteBuffers(1, m_RenderVertexBuf);
 glDeleteBuffers(1, m_3DUBOBuffer);
 glDeleteBuffers(1, m_VertexBuffer);
+
+glDeleteFramebuffers(1, mnPickingFbo);
+glDeleteRenderbuffers(1, mnPickingRbo);
+glDeleteTextures(1, mnPickingTexture);
 }
 
 void OpenGL3DRenderer::ShaderResources::LoadShaders()
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'feature/chart-3d-chart2' - 8 commits - chart2/source include/vcl sc/inc sc/source vcl/source

2014-05-20 Thread Markus Mohrhard
 chart2/source/view/charttypes/GL3DBarChart.cxx |6 +-
 chart2/source/view/inc/GL3DRenderer.hxx|   17 +---
 chart2/source/view/main/3DChartObjects.cxx |   10 ++--
 chart2/source/view/main/GL3DRenderer.cxx   |   51 ++---
 include/vcl/opengl/OpenGLHelper.hxx|7 +++
 sc/inc/chartarr.hxx|   24 +--
 sc/source/core/tool/chartarr.cxx   |   36 ++---
 sc/source/ui/unoobj/cellsuno.cxx   |6 +-
 vcl/source/opengl/OpenGLHelper.cxx |   36 +
 9 files changed, 139 insertions(+), 54 deletions(-)

New commits:
commit ae1426a2667ba2a426753283eb6f501341d7a517
Author: Markus Mohrhard markus.mohrh...@collabora.co.uk
Date:   Wed May 21 02:09:11 2014 +0200

forgot to release the OpenGL resources

Change-Id: I2baefbe611f9ed28defc46ca3af332d13b32c561

diff --git a/chart2/source/view/main/GL3DRenderer.cxx 
b/chart2/source/view/main/GL3DRenderer.cxx
index cea7e0f..cf75ea6 100644
--- a/chart2/source/view/main/GL3DRenderer.cxx
+++ b/chart2/source/view/main/GL3DRenderer.cxx
@@ -110,6 +110,10 @@ OpenGL3DRenderer::~OpenGL3DRenderer()
 glDeleteBuffers(1, m_RenderVertexBuf);
 glDeleteBuffers(1, m_3DUBOBuffer);
 glDeleteBuffers(1, m_VertexBuffer);
+
+glDeleteFramebuffers(1, mnPickingFbo);
+glDeleteRenderbuffers(1, mnPickingRbo);
+glDeleteTextures(1, mnPickingTexture);
 }
 
 void OpenGL3DRenderer::ShaderResources::LoadShaders()
commit 9d9b631c60dc4b88d5cff0a4a8c9e5c94b019a47
Author: Markus Mohrhard markus.mohrh...@collabora.co.uk
Date:   Wed May 21 01:49:59 2014 +0200

use FBO for the picking

Change-Id: Ibab6daa1f76300d159ccd4cfbca061c8d8c1951f

diff --git a/chart2/source/view/charttypes/GL3DBarChart.cxx 
b/chart2/source/view/charttypes/GL3DBarChart.cxx
index 1653668..d36a008 100644
--- a/chart2/source/view/charttypes/GL3DBarChart.cxx
+++ b/chart2/source/view/charttypes/GL3DBarChart.cxx
@@ -217,14 +217,16 @@ public:
 
 }
 
-void GL3DBarChart::clickedAt(const Point )
+void GL3DBarChart::clickedAt(const Point rPos)
 {
+sal_uInt32 nId = 1;
 {
 PickingModeSetter(mpRenderer.get());
 render();
+nId = mpRenderer-GetPixelColorFromPoint(rPos.X(), rPos.Y());
 }
 if (mpCamera)
-mpCamera-zoom(1);
+mpCamera-zoom(nId);
 }
 
 }
diff --git a/chart2/source/view/inc/GL3DRenderer.hxx 
b/chart2/source/view/inc/GL3DRenderer.hxx
index 52d341a..b08e90e 100644
--- a/chart2/source/view/inc/GL3DRenderer.hxx
+++ b/chart2/source/view/inc/GL3DRenderer.hxx
@@ -179,6 +179,8 @@ public:
 void ProcessUnrenderedShape();
 
 void SetPickingMode(bool bPickingMode);
+
+sal_uInt32 GetPixelColorFromPoint(long nX, long nY);
 private:
 void MoveModelf(PosVecf3 trans,PosVecf3 angle,PosVecf3 scale);
 
@@ -362,6 +364,10 @@ private:
 
 bool mbPickingMode;
 SceneBox m_SenceBox;
+
+GLuint mnPickingFbo;
+GLuint mnPickingRbo;
+GLuint mnPickingTexture;
 };
 
 }
diff --git a/chart2/source/view/main/GL3DRenderer.cxx 
b/chart2/source/view/main/GL3DRenderer.cxx
index 92cf61d..cea7e0f 100644
--- a/chart2/source/view/main/GL3DRenderer.cxx
+++ b/chart2/source/view/main/GL3DRenderer.cxx
@@ -249,6 +249,8 @@ void OpenGL3DRenderer::init()
 glBufferData(GL_ARRAY_BUFFER, sizeof(squareVertices), squareVertices, 
GL_STATIC_DRAW);
 glBindBuffer(GL_ARRAY_BUFFER, 0);
 
+OpenGLHelper::createFramebuffer(m_iWidth, m_iHeight, mnPickingFbo, 
mnPickingRbo, mnPickingTexture);
+
 CHECK_GL_ERROR();
 Init3DUniformBlock();
 
@@ -1580,6 +1582,22 @@ void OpenGL3DRenderer::MoveModelf(PosVecf3 
trans,PosVecf3 angle,PosVecf3 scal
 void OpenGL3DRenderer::SetPickingMode(bool bPickingMode)
 {
 mbPickingMode = bPickingMode;
+if(mbPickingMode)
+{
+glBindFramebuffer(GL_FRAMEBUFFER, mnPickingFbo);
+}
+else
+{
+glBindFramebuffer(GL_FRAMEBUFFER, 0);
+}
+}
+
+sal_uInt32 OpenGL3DRenderer::GetPixelColorFromPoint(long nX, long nY)
+{
+boost::scoped_arraysal_uInt8 buf(new sal_uInt8[4]);
+glReadPixels(nX, nY, 1, 1, GL_BGRA, GL_UNSIGNED_BYTE, buf.get());
+Color aColor(buf[3], buf[2], buf[1], buf[0]);
+return aColor.GetColor();
 }
 
 }
commit 8f6411c08f8382ff486af1f10fff5cf9c6525265
Author: Markus Mohrhard markus.mohrh...@collabora.co.uk
Date:   Wed May 21 00:41:37 2014 +0200

add method for creating framebuffer objects to OpenGLHelper

Change-Id: I08bd2e58ee98a68accae256fcbcc288a8c56ae0b

diff --git a/include/vcl/opengl/OpenGLHelper.hxx 
b/include/vcl/opengl/OpenGLHelper.hxx
index 3cb482b..70b1d2a 100644
--- a/include/vcl/opengl/OpenGLHelper.hxx
+++ b/include/vcl/opengl/OpenGLHelper.hxx
@@ -29,6 +29,13 @@ public:
 static void renderToFile(long nWidth, long nHeight, const OUString 
rFileName);
 
 static const char* GLErrorString(GLenum errorCode);
+
+/**
+ * The caller is responsible for deleting the buffer objects identified by
+ * 

[Libreoffice-commits] core.git: Branch 'private/kohei/external-ref-refresh' - sc/inc sc/source

2014-05-20 Thread Kohei Yoshida
 sc/inc/document.hxx  |3 
 sc/inc/xmlwrap.hxx   |   14 
 sc/source/core/data/documen2.cxx |1 
 sc/source/core/data/documen9.cxx |5 
 sc/source/filter/xml/xmlimprt.cxx|   23 +
 sc/source/filter/xml/xmlimprt.hxx|6 
 sc/source/filter/xml/xmlwrap.cxx |  473 +++
 sc/source/ui/docshell/docsh.cxx  |   10 
 sc/source/ui/docshell/externalrefmgr.cxx |4 
 9 files changed, 268 insertions(+), 271 deletions(-)

New commits:
commit ce3152ab14937cc37102b0be4264c793d288235a
Author: Kohei Yoshida kohei.yosh...@collabora.com
Date:   Tue May 20 21:01:46 2014 -0400

Let's not use ScDocument as a convenient anything goes storage place.

Change-Id: I0ae2f44b89b0db915e78a9b07835000e843d016f

diff --git a/sc/inc/document.hxx b/sc/inc/document.hxx
index 08f1ca4..e1aabf4 100644
--- a/sc/inc/document.hxx
+++ b/sc/inc/document.hxx
@@ -400,7 +400,6 @@ private:
 boolbInsertingFromOtherDoc;
 boolbLoadingMedium;
 boolbImportingXML;  // special handling of formula text
-boolbXMLFromWrapper;// distinguish ScXMLImportWrapper 
from external component
 boolbCalcingAfterLoad;  // in CalcAfterLoad 
TRUE
 // don't construct/destruct listeners temporarily
 boolbNoListening;
@@ -1754,8 +1753,6 @@ public:
 voidSetLoadingMedium( bool bVal );
 voidSetImportingXML( bool bVal );
 boolIsImportingXML() const { return bImportingXML; }
-voidSetXMLFromWrapper( bool bVal );
-boolIsXMLFromWrapper() const { return bXMLFromWrapper; }
 voidSetCalcingAfterLoad( bool bVal ) { bCalcingAfterLoad = 
bVal; }
 boolIsCalcingAfterLoad() const { return bCalcingAfterLoad; }
 voidSetNoListening( bool bVal ) { bNoListening = bVal; }
diff --git a/sc/inc/xmlwrap.hxx b/sc/inc/xmlwrap.hxx
index 8bb78af..e74c01a 100644
--- a/sc/inc/xmlwrap.hxx
+++ b/sc/inc/xmlwrap.hxx
@@ -26,10 +26,6 @@
 #include importfilterdata.hxx
 #include sal/types.h
 
-class ScDocument;
-class SfxMedium;
-class ScMySharedData;
-
 #include tools/errcode.hxx
 
 namespace com { namespace sun { namespace star {
@@ -43,10 +39,16 @@ namespace com { namespace sun { namespace star {
 namespace sax { struct InputSource; class XParser; class XWriter; } }
 } } }
 
+class ScDocument;
+class SfxMedium;
+class ScMySharedData;
+class ScDocShell;
+
 class ScXMLImportWrapper
 {
 sc::ImportPostProcessData maPostProcessData;
 
+ScDocShell mrDocShell;
 ScDocument rDoc;
 SfxMedium*  pMedium;
 ::com::sun::star::uno::Reference ::com::sun::star::embed::XStorage  
xStorage;
@@ -70,7 +72,9 @@ class ScXMLImportWrapper
 ScMySharedData* pSharedData);
 
 public:
-ScXMLImportWrapper(ScDocument rD, SfxMedium* pM, const 
::com::sun::star::uno::Reference ::com::sun::star::embed::XStorage );
+ScXMLImportWrapper(
+ScDocShell rDocSh, SfxMedium* pM, const 
css::uno::Referencecss::embed::XStorage xStor );
+
 bool Import(bool bStylesOnly, ErrCode );
 bool Export(bool bStylesOnly);
 
diff --git a/sc/source/core/data/documen2.cxx b/sc/source/core/data/documen2.cxx
index e0ed474..d65c42c 100644
--- a/sc/source/core/data/documen2.cxx
+++ b/sc/source/core/data/documen2.cxx
@@ -194,7 +194,6 @@ ScDocument::ScDocument( ScDocumentMode eMode, 
SfxObjectShell* pDocShell ) :
 bInsertingFromOtherDoc( false ),
 bLoadingMedium( false ),
 bImportingXML( false ),
-bXMLFromWrapper( false ),
 bCalcingAfterLoad( false ),
 bNoListening( false ),
 mbIdleEnabled(true),
diff --git a/sc/source/core/data/documen9.cxx b/sc/source/core/data/documen9.cxx
index 4d7920e..c9fe85b 100644
--- a/sc/source/core/data/documen9.cxx
+++ b/sc/source/core/data/documen9.cxx
@@ -620,11 +620,6 @@ void ScDocument::SetImportingXML( bool bVal )
 SetLoadingMedium(bVal);
 }
 
-void ScDocument::SetXMLFromWrapper( bool bVal )
-{
-bXMLFromWrapper = bVal;
-}
-
 rtl::ReferenceSvxForbiddenCharactersTable 
ScDocument::GetForbiddenCharacters()
 {
 return xForbiddenCharacters;
diff --git a/sc/source/filter/xml/xmlimprt.cxx 
b/sc/source/filter/xml/xmlimprt.cxx
index f95b2aa..2aa6c14 100644
--- a/sc/source/filter/xml/xmlimprt.cxx
+++ b/sc/source/filter/xml/xmlimprt.cxx
@@ -2102,7 +2102,7 @@ ScXMLImport::ScXMLImport(
 bRemoveLastChar(false),
 bNullDateSetted(false),
 bSelfImportingXMLSet(false),
-bFromWrapper(false),
+mbLockSolarMutex(true),
 mbHasNewCondFormatData(false)
 {
 pStylesImportHelper = new ScMyStylesImportHelper(*this);
@@ -2240,6 +2240,23 @@ ScXMLImport::~ScXMLImport() throw()
 delete pDetectiveOpArray;
 }
 
+void ScXMLImport::initialize( const css::uno::Sequencecss::uno::Any 
aArguments )
+throw 

[Libreoffice-commits] core.git: Branch 'private/kohei/external-ref-refresh' - sc/inc sc/source

2014-05-20 Thread Kohei Yoshida
 sc/inc/unonames.hxx   |2 ++
 sc/source/filter/xml/xmlimprt.cxx |5 +++--
 sc/source/filter/xml/xmlwrap.cxx  |3 ++-
 3 files changed, 7 insertions(+), 3 deletions(-)

New commits:
commit 1d17a0fe5312f7107beb50b6bb779f4eff8c67c8
Author: Kohei Yoshida kohei.yosh...@collabora.com
Date:   Tue May 20 21:09:41 2014 -0400

Let's use constant uno name for these.

Change-Id: I5e34f4d7561ef7f4f7b8b3b4d7d06cca072831c7

diff --git a/sc/inc/unonames.hxx b/sc/inc/unonames.hxx
index 1cc71a9..689dcea 100644
--- a/sc/inc/unonames.hxx
+++ b/sc/inc/unonames.hxx
@@ -668,6 +668,8 @@
 
 #define SC_UNO_EMBED_FONTS EmbedFonts
 
+#define SC_UNO_ODS_LOCK_SOLAR_MUTEX LockSolarMutex
+
 #endif
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/sc/source/filter/xml/xmlimprt.cxx 
b/sc/source/filter/xml/xmlimprt.cxx
index 2aa6c14..813190d 100644
--- a/sc/source/filter/xml/xmlimprt.cxx
+++ b/sc/source/filter/xml/xmlimprt.cxx
@@ -69,6 +69,7 @@
 #include editattributemap.hxx
 #include documentimport.hxx
 #include pivotsource.hxx
+#include unonames.hxx
 
 #include comphelper/extract.hxx
 
@@ -2253,8 +2254,8 @@ void ScXMLImport::initialize( const 
css::uno::Sequencecss::uno::Any aArgument
 if (!xInfoSetInfo.is())
 return;
 
-if (xInfoSetInfo-hasPropertyByName(LockSolarMutex))
-xInfoSet-getPropertyValue(LockSolarMutex) = mbLockSolarMutex;
+if (xInfoSetInfo-hasPropertyByName(SC_UNO_ODS_LOCK_SOLAR_MUTEX))
+xInfoSet-getPropertyValue(SC_UNO_ODS_LOCK_SOLAR_MUTEX) = 
mbLockSolarMutex;
 }
 
 SvXMLImportContext *ScXMLImport::CreateFontDeclsContext(const sal_uInt16 
nPrefix, const OUString rLocalName,
diff --git a/sc/source/filter/xml/xmlwrap.cxx b/sc/source/filter/xml/xmlwrap.cxx
index 6282b9f..4402b2b 100644
--- a/sc/source/filter/xml/xmlwrap.cxx
+++ b/sc/source/filter/xml/xmlwrap.cxx
@@ -71,6 +71,7 @@
 #include sheetdata.hxx
 #include XMLCodeNameProvider.hxx
 #include docsh.hxx
+#include unonames.hxx
 
 using namespace com::sun::star;
 
@@ -348,7 +349,7 @@ bool ScXMLImportWrapper::Import(bool bStylesOnly, ErrCode 
nError)
 };
 uno::Reference beans::XPropertySet  xInfoSet( 
comphelper::GenericPropertySet_CreateInstance( new comphelper::PropertySetInfo( 
aImportInfoMap ) ) );
 
-xInfoSet-setPropertyValue(LockSolarMutex, uno::makeAny(false));
+xInfoSet-setPropertyValue(SC_UNO_ODS_LOCK_SOLAR_MUTEX, 
uno::makeAny(false));
 
 //  get BuildId from parent container if available
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'private/kohei/external-ref-refresh' - sc/inc sc/source

2014-05-20 Thread Kohei Yoshida
Rebased ref, commits from common ancestor:
commit 4ea57c9f946a0078429a8a87b9d34372eb5c5072
Author: Kohei Yoshida kohei.yosh...@collabora.com
Date:   Tue May 20 21:09:41 2014 -0400

Let's use constant uno name for these.

Change-Id: I5e34f4d7561ef7f4f7b8b3b4d7d06cca072831c7

diff --git a/sc/inc/unonames.hxx b/sc/inc/unonames.hxx
index 1cc71a9..689dcea 100644
--- a/sc/inc/unonames.hxx
+++ b/sc/inc/unonames.hxx
@@ -668,6 +668,8 @@
 
 #define SC_UNO_EMBED_FONTS EmbedFonts
 
+#define SC_UNO_ODS_LOCK_SOLAR_MUTEX LockSolarMutex
+
 #endif
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/sc/source/filter/xml/xmlimprt.cxx 
b/sc/source/filter/xml/xmlimprt.cxx
index 2aa6c14..813190d 100644
--- a/sc/source/filter/xml/xmlimprt.cxx
+++ b/sc/source/filter/xml/xmlimprt.cxx
@@ -69,6 +69,7 @@
 #include editattributemap.hxx
 #include documentimport.hxx
 #include pivotsource.hxx
+#include unonames.hxx
 
 #include comphelper/extract.hxx
 
@@ -2253,8 +2254,8 @@ void ScXMLImport::initialize( const 
css::uno::Sequencecss::uno::Any aArgument
 if (!xInfoSetInfo.is())
 return;
 
-if (xInfoSetInfo-hasPropertyByName(LockSolarMutex))
-xInfoSet-getPropertyValue(LockSolarMutex) = mbLockSolarMutex;
+if (xInfoSetInfo-hasPropertyByName(SC_UNO_ODS_LOCK_SOLAR_MUTEX))
+xInfoSet-getPropertyValue(SC_UNO_ODS_LOCK_SOLAR_MUTEX) = 
mbLockSolarMutex;
 }
 
 SvXMLImportContext *ScXMLImport::CreateFontDeclsContext(const sal_uInt16 
nPrefix, const OUString rLocalName,
diff --git a/sc/source/filter/xml/xmlwrap.cxx b/sc/source/filter/xml/xmlwrap.cxx
index 6282b9f..cc1cbff 100644
--- a/sc/source/filter/xml/xmlwrap.cxx
+++ b/sc/source/filter/xml/xmlwrap.cxx
@@ -71,6 +71,7 @@
 #include sheetdata.hxx
 #include XMLCodeNameProvider.hxx
 #include docsh.hxx
+#include unonames.hxx
 
 using namespace com::sun::star;
 
@@ -343,12 +344,12 @@ bool ScXMLImportWrapper::Import(bool bStylesOnly, 
ErrCode nError)
 { OUString(OrganizerMode), 0, ::getBooleanCppuType(),
 ::com::sun::star::beans::PropertyAttribute::MAYBEVOID, 0 },
 { OUString(SourceStorage), 0, cppu::UnoTypeembed::XStorage::get(), 
::com::sun::star::beans::PropertyAttribute::MAYBEVOID, 0 },
-{ OUString(LockSolarMutex), 0, getBooleanCppuType(), 
css::beans::PropertyAttribute::MAYBEVOID, 0 },
+{ OUString(SC_UNO_ODS_LOCK_SOLAR_MUTEX), 0, getBooleanCppuType(), 
css::beans::PropertyAttribute::MAYBEVOID, 0 },
 { OUString(), 0, css::uno::Type(), 0, 0 }
 };
 uno::Reference beans::XPropertySet  xInfoSet( 
comphelper::GenericPropertySet_CreateInstance( new comphelper::PropertySetInfo( 
aImportInfoMap ) ) );
 
-xInfoSet-setPropertyValue(LockSolarMutex, uno::makeAny(false));
+xInfoSet-setPropertyValue(SC_UNO_ODS_LOCK_SOLAR_MUTEX, 
uno::makeAny(false));
 
 //  get BuildId from parent container if available
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'private/kohei/external-ref-refresh' - sc/inc sc/source

2014-05-20 Thread Kohei Yoshida
 sc/inc/xmlwrap.hxx   |9 +++-
 sc/source/filter/xml/xmlwrap.cxx |   81 ++-
 sc/source/ui/docshell/docsh.cxx  |8 +--
 3 files changed, 50 insertions(+), 48 deletions(-)

New commits:
commit ec639ad118ecb02ab0efea3a60c793d9b6ea3ef6
Author: Kohei Yoshida kohei.yosh...@collabora.com
Date:   Tue May 20 22:32:43 2014 -0400

Finer grained ODS import mode selection.

Change-Id: I18520837f8d25103bf8482a0204b8a7b7740feb1

diff --git a/sc/inc/xmlwrap.hxx b/sc/inc/xmlwrap.hxx
index e74c01a..b7ec5cd 100644
--- a/sc/inc/xmlwrap.hxx
+++ b/sc/inc/xmlwrap.hxx
@@ -72,10 +72,17 @@ class ScXMLImportWrapper
 ScMySharedData* pSharedData);
 
 public:
+
+static const sal_uInt8 STYLES   = 0x01;
+static const sal_uInt8 CONTENT  = 0x02;
+static const sal_uInt8 METADATA = 0x04;
+static const sal_uInt8 SETTINGS = 0x08;
+static const sal_uInt8 ALL  = STYLES | CONTENT | METADATA | SETTINGS;
+
 ScXMLImportWrapper(
 ScDocShell rDocSh, SfxMedium* pM, const 
css::uno::Referencecss::embed::XStorage xStor );
 
-bool Import(bool bStylesOnly, ErrCode );
+bool Import( sal_uInt8 nMode, ErrCode rError );
 bool Export(bool bStylesOnly);
 
 const sc::ImportPostProcessData GetImportPostProcessData() const;
diff --git a/sc/source/filter/xml/xmlwrap.cxx b/sc/source/filter/xml/xmlwrap.cxx
index cc1cbff..159c3d8 100644
--- a/sc/source/filter/xml/xmlwrap.cxx
+++ b/sc/source/filter/xml/xmlwrap.cxx
@@ -309,7 +309,7 @@ sal_uInt32 ScXMLImportWrapper::ImportFromComponent(const 
uno::Referenceuno::XCo
 return nReturn;
 }
 
-bool ScXMLImportWrapper::Import(bool bStylesOnly, ErrCode nError)
+bool ScXMLImportWrapper::Import( sal_uInt8 nMode, ErrCode rError )
 {
 uno::Referenceuno::XComponentContext xContext = 
comphelper::getProcessComponentContext();
 
@@ -349,6 +349,7 @@ bool ScXMLImportWrapper::Import(bool bStylesOnly, ErrCode 
nError)
 };
 uno::Reference beans::XPropertySet  xInfoSet( 
comphelper::GenericPropertySet_CreateInstance( new comphelper::PropertySetInfo( 
aImportInfoMap ) ) );
 
+// No need to lock solar mutex when calling from the wrapper.
 xInfoSet-setPropertyValue(SC_UNO_ODS_LOCK_SOLAR_MUTEX, 
uno::makeAny(false));
 
 //  get BuildId from parent container if available
@@ -403,14 +404,14 @@ bool ScXMLImportWrapper::Import(bool bStylesOnly, 
ErrCode nError)
 }
 }
 
-if (bStylesOnly)
-xInfoSet-setPropertyValue(OrganizerMode, uno::makeAny(sal_True));
+if (mrDocShell.GetCreateMode() == SFX_CREATE_MODE_ORGANIZER)
+xInfoSet-setPropertyValue(OrganizerMode, uno::makeAny(true));
 
 xInfoSet-setPropertyValue( SourceStorage, uno::Any( xStorage ) );
 
 bool bOasis = ( SotStorage::GetVersion( xStorage )  SOFFICE_FILEFORMAT_60 
);
 
-if (!bStylesOnly  bOasis)
+if ((nMode  METADATA) == METADATA  bOasis)
 {
 // RDF metadata: ODF = 1.2
 try
@@ -428,34 +429,37 @@ bool ScXMLImportWrapper::Import(bool bStylesOnly, 
ErrCode nError)
 ucb::InteractiveAugmentedIOException iaioe;
 if ( e.TargetException = iaioe )
 {
-nError = SCERR_IMPORT_UNKNOWN;
+rError = SCERR_IMPORT_UNKNOWN;
 }
 else
 {
-nError = SCWARN_IMPORT_FEATURES_LOST;
+rError = SCWARN_IMPORT_FEATURES_LOST;
 }
 }
 catch ( const uno::Exception )
 {
-nError = SCWARN_IMPORT_FEATURES_LOST;
+rError = SCWARN_IMPORT_FEATURES_LOST;
 }
 }
 
 // #i103539#: always read meta.xml for generator
 sal_uInt32 nMetaRetval(0);
-uno::Sequenceuno::Any aMetaArgs(1);
-uno::Any* pMetaArgs = aMetaArgs.getArray();
-pMetaArgs[0] = xInfoSet;
+if ((nMode  METADATA) == METADATA)
+{
+uno::Sequenceuno::Any aMetaArgs(1);
+uno::Any* pMetaArgs = aMetaArgs.getArray();
+pMetaArgs[0] = xInfoSet;
 
-SAL_INFO( sc.filter, meta import start );
+SAL_INFO( sc.filter, meta import start );
 
-nMetaRetval = ImportFromComponent(
-xContext, xModel, xXMLParser, aParserInput,
-bOasis ? 
OUString(com.sun.star.comp.Calc.XMLOasisMetaImporter)
-: 
OUString(com.sun.star.comp.Calc.XMLMetaImporter),
-meta.xml, Meta.xml, aMetaArgs, false);
+nMetaRetval = ImportFromComponent(
+xContext, xModel, xXMLParser, aParserInput,
+bOasis ? 
OUString(com.sun.star.comp.Calc.XMLOasisMetaImporter)
+: 
OUString(com.sun.star.comp.Calc.XMLMetaImporter),
+meta.xml, Meta.xml, aMetaArgs, false);
 
-SAL_INFO( sc.filter, meta import end );
+SAL_INFO( sc.filter, meta import end );
+}
 
 SvXMLGraphicHelper* pGraphicHelper = NULL;

[Libreoffice-commits] core.git: Branch 'private/kohei/external-ref-refresh' - include/sfx2 sc/inc sc/source sfx2/source

2014-05-20 Thread Kohei Yoshida
 include/sfx2/sfxmodelfactory.hxx |   12 +---
 sc/inc/unonames.hxx  |3 ++-
 sc/source/filter/xml/xmlimprt.cxx|   16 
 sc/source/filter/xml/xmlimprt.hxx|1 +
 sc/source/filter/xml/xmlwrap.cxx |5 +
 sc/source/ui/docshell/externalrefmgr.cxx |2 +-
 sfx2/source/doc/objxtor.cxx  |6 +-
 7 files changed, 35 insertions(+), 10 deletions(-)

New commits:
commit 1bd673611ecd795a1caaac676c6c5cef0eba023b
Author: Kohei Yoshida kohei.yosh...@collabora.com
Date:   Wed May 21 00:00:57 2014 -0400

cp#172: Skip styles import for external link cache documents.

This reduces external link update time by 10%.

Change-Id: Ic14d9ea7530818f839330a2004f6aa67ef1e831e

diff --git a/include/sfx2/sfxmodelfactory.hxx b/include/sfx2/sfxmodelfactory.hxx
index 7f7b05f..2067fbf 100644
--- a/include/sfx2/sfxmodelfactory.hxx
+++ b/include/sfx2/sfxmodelfactory.hxx
@@ -25,16 +25,14 @@
 #include com/sun/star/lang/XMultiServiceFactory.hpp
 #include com/sun/star/lang/XSingleServiceFactory.hpp
 
+#define SFXMODEL_STANDARD   (sal_uInt64)(0x)
+#define SFXMODEL_EMBEDDED_OBJECT(sal_uInt64)(0x0001)
+#define SFXMODEL_EXTERNAL_LINK  (sal_uInt64)(0x0002)
+#define SFXMODEL_DISABLE_EMBEDDED_SCRIPTS   (sal_uInt64)(0x0004)
+#define SFXMODEL_DISABLE_DOCUMENT_RECOVERY  (sal_uInt64)(0x0008)
 
 namespace sfx2
 {
-
-
-#define SFXMODEL_STANDARD   (sal_uInt64)(0x)
-#define SFXMODEL_EMBEDDED_OBJECT(sal_uInt64)(0x0001)
-#define SFXMODEL_DISABLE_EMBEDDED_SCRIPTS   (sal_uInt64)(0x0002)
-#define SFXMODEL_DISABLE_DOCUMENT_RECOVERY  (sal_uInt64)(0x0004)
-
 typedef ::com::sun::star::uno::Reference 
::com::sun::star::uno::XInterface  ( SAL_CALL * SfxModelFactoryFunc ) (
 const ::com::sun::star::uno::Reference 
::com::sun::star::lang::XMultiServiceFactory  _rxFactory,
 const sal_uInt64 _nCreationFlags
diff --git a/sc/inc/unonames.hxx b/sc/inc/unonames.hxx
index 689dcea..ca7a28c 100644
--- a/sc/inc/unonames.hxx
+++ b/sc/inc/unonames.hxx
@@ -668,7 +668,8 @@
 
 #define SC_UNO_EMBED_FONTS EmbedFonts
 
-#define SC_UNO_ODS_LOCK_SOLAR_MUTEX LockSolarMutex
+#define SC_UNO_ODS_LOCK_SOLAR_MUTEX ODSLockSolarMutex
+#define SC_UNO_ODS_IMPORT_STYLESODSImportStyles
 
 #endif
 
diff --git a/sc/source/filter/xml/xmlimprt.cxx 
b/sc/source/filter/xml/xmlimprt.cxx
index 813190d..9128026 100644
--- a/sc/source/filter/xml/xmlimprt.cxx
+++ b/sc/source/filter/xml/xmlimprt.cxx
@@ -2104,6 +2104,7 @@ ScXMLImport::ScXMLImport(
 bNullDateSetted(false),
 bSelfImportingXMLSet(false),
 mbLockSolarMutex(true),
+mbImportStyles(true),
 mbHasNewCondFormatData(false)
 {
 pStylesImportHelper = new ScMyStylesImportHelper(*this);
@@ -2256,6 +2257,9 @@ void ScXMLImport::initialize( const 
css::uno::Sequencecss::uno::Any aArgument
 
 if (xInfoSetInfo-hasPropertyByName(SC_UNO_ODS_LOCK_SOLAR_MUTEX))
 xInfoSet-getPropertyValue(SC_UNO_ODS_LOCK_SOLAR_MUTEX) = 
mbLockSolarMutex;
+
+if (xInfoSetInfo-hasPropertyByName(SC_UNO_ODS_IMPORT_STYLES))
+xInfoSet-getPropertyValue(SC_UNO_ODS_IMPORT_STYLES) = 
mbImportStyles;
 }
 
 SvXMLImportContext *ScXMLImport::CreateFontDeclsContext(const sal_uInt16 
nPrefix, const OUString rLocalName,
@@ -2765,6 +2769,9 @@ void ScXMLImport::SetType(uno::Reference 
beans::XPropertySet rProperties,
   const sal_Int16 nCellType,
   const OUString rCurrency)
 {
+if (!mbImportStyles)
+return;
+
 if ((nCellType != util::NumberFormat::TEXT)  (nCellType != 
util::NumberFormat::UNDEFINED))
 {
 if (rNumberFormat == -1)
@@ -2832,6 +2839,9 @@ void ScXMLImport::SetType(uno::Reference 
beans::XPropertySet rProperties,
 
 void ScXMLImport::AddStyleRange(const table::CellRangeAddress rCellRange)
 {
+if (!mbImportStyles)
+return;
+
 if (!xSheetCellRanges.is()  GetModel().is())
 {
 uno::Reference lang::XMultiServiceFactory 
xMultiServiceFactory(GetModel(), uno::UNO_QUERY);
@@ -2845,6 +2855,9 @@ void ScXMLImport::AddStyleRange(const 
table::CellRangeAddress rCellRange)
 
 void ScXMLImport::SetStyleToRanges()
 {
+if (!mbImportStyles)
+return;
+
 if (!sPrevStyleName.isEmpty())
 {
 uno::Reference beans::XPropertySet xProperties (xSheetCellRanges, 
uno::UNO_QUERY);
@@ -2901,6 +2914,9 @@ void ScXMLImport::SetStyleToRanges()
 void ScXMLImport::SetStyleToRange(const ScRange rRange, const OUString* 
pStyleName,
   const sal_Int16 nCellType, const OUString* 
pCurrency)
 {
+if (!mbImportStyles)
+return;
+
 if (sPrevStyleName.isEmpty())
 {
 nPrevCellType = nCellType;
diff --git a/sc/source/filter/xml/xmlimprt.hxx 
b/sc/source/filter/xml/xmlimprt.hxx
index 660f3bd..1f44982 100644
--- a/sc/source/filter/xml/xmlimprt.hxx
+++ 

[Libreoffice-commits] core.git: download.lst external/python3

2014-05-20 Thread Thomas Arnhold
 download.lst   |3 
 external/python3/UnpackedTarball_python3.mk|3 
 external/python3/python-3.3.0-implicit-int.patch.1 |   30 --
 external/python3/python-3.3.0-msvc-x64.patch.1 |   22 -
 external/python3/python-3.3.3-quoted-printable.patch.1 |  201 -
 external/python3/python-3.3.5-pyexpat-symbols.patch.1  |   28 ++
 6 files changed, 42 insertions(+), 245 deletions(-)

New commits:
commit 005fae2bddf4e43cb361bbdb9fc2cfb961693ffd
Author: Thomas Arnhold tho...@arnhold.org
Date:   Mon May 12 16:13:30 2014 +0200

upgrade to python-3.3.5

- remove now obselete patches, which were applied upstream.
- Hack to get MacOS to build

Change-Id: Id68e78e411efc92a46ea9e180f09c390fe5acb4a
Reviewed-on: https://gerrit.libreoffice.org/9311
Tested-by: LibreOffice gerrit bot ger...@libreoffice.org
Reviewed-by: David Tardon dtar...@redhat.com
Tested-by: David Tardon dtar...@redhat.com

diff --git a/download.lst b/download.lst
index 69aa304..8a3b889 100644
--- a/download.lst
+++ b/download.lst
@@ -110,7 +110,8 @@ export PNG_MD5SUM := 5266905cef49d1224437465ad4d67fd9
 export PNG_TARBALL := libpng-1.5.18.tar.gz
 export POPPLER_TARBALL := 
1cd27460f7e3379d1eb109cfd7bcdb39-poppler-0.22.5.tar.gz
 export POSTGRESQL_TARBALL := 
c0b4799ea9850eae3ead14f0a60e9418-postgresql-9.2.1.tar.bz2
-export PYTHON_TARBALL := f3ebe34d4d8695bf889279b54673e10c-Python-3.3.3.tar.bz2
+export PYTHON_MD5SUM := 803a75927f8f241ca78633890c798021
+export PYTHON_TARBALL := Python-3.3.5.tgz
 export RAPTOR_TARBALL := 4ceb9316488b0ea01acf011023cf7fff-raptor2-2.0.9.tar.gz
 export RASQAL_TARBALL := b12c5f9cfdb6b04efce5a4a186b8416b-rasqal-0.9.30.tar.gz
 export REDLAND_TARBALL := 
32f8e1417a64d3c6f2c727f9053f55ea-redland-1.0.16.tar.gz
diff --git a/external/python3/UnpackedTarball_python3.mk 
b/external/python3/UnpackedTarball_python3.mk
index d08ba1f..d6a1d04 100644
--- a/external/python3/UnpackedTarball_python3.mk
+++ b/external/python3/UnpackedTarball_python3.mk
@@ -24,13 +24,12 @@ $(eval $(call gb_UnpackedTarball_add_patches,python3,\
external/python3/python-3.3.0-msvc-disable.patch.1 \
external/python3/python-3.3.0-msvc-x64.patch.1 \
external/python3/python-3.3.0-ssl.patch.1 \
-   external/python3/python-3.3.0-implicit-int.patch.1 \
external/python3/python-3.3.0-gcc-4.8.patch.1 \
external/python3/python-3.3.0-pythreadstate.patch.1 \
external/python3/python-3.3.0-clang.patch.1 \
-   external/python3/python-3.3.3-quoted-printable.patch.1 \
external/python3/python-3.3.3-py17797.patch.1 \
external/python3/python-3.3.3-msvc2012-winxp.patch.1 \
+   external/python3/python-3.3.5-pyexpat-symbols.patch.1 \
 ))
 
 ifneq ($(filter DRAGONFLY FREEBSD LINUX NETBSD OPENBSD SOLARIS,$(OS)),)
diff --git a/external/python3/python-3.3.0-implicit-int.patch.1 
b/external/python3/python-3.3.0-implicit-int.patch.1
deleted file mode 100644
index 6e4d2b0..000
--- a/external/python3/python-3.3.0-implicit-int.patch.1
+++ /dev/null
@@ -1,30 +0,0 @@
-fix function names in import.h
-
-MSVC complains about some declarations in Include/import.h.
-Apparently the problem is a missing space between PyAPI_FUNC(int) and the
-function name, leading to concatenated int_PyImport... names and no
-return type.
-
-diff -ru python3.old/Include/import.h python3/Include/import.h
 python3.old/Include/import.h   2012-09-29 10:00:26.0 +0200
-+++ python3/Include/import.h   2012-11-27 16:09:26.449390966 +0100
-@@ -86,15 +86,15 @@
- 
- PyAPI_FUNC(void) _PyImport_ReInitLock(void);
- 
--PyAPI_FUNC(PyObject *)_PyImport_FindBuiltin(
-+PyAPI_FUNC(PyObject *) _PyImport_FindBuiltin(
- const char *name/* UTF-8 encoded string */
- );
--PyAPI_FUNC(PyObject *)_PyImport_FindExtensionObject(PyObject *, PyObject *);
--PyAPI_FUNC(int)_PyImport_FixupBuiltin(
-+PyAPI_FUNC(PyObject *) _PyImport_FindExtensionObject(PyObject *, PyObject *);
-+PyAPI_FUNC(int) _PyImport_FixupBuiltin(
- PyObject *mod,
- char *name  /* UTF-8 encoded string */
- );
--PyAPI_FUNC(int)_PyImport_FixupExtensionObject(PyObject*, PyObject *, PyObject 
*);
-+PyAPI_FUNC(int) _PyImport_FixupExtensionObject(PyObject*, PyObject *, 
PyObject *);
- 
- struct _inittab {
- char *name; /* ASCII encoded string */
diff --git a/external/python3/python-3.3.0-msvc-x64.patch.1 
b/external/python3/python-3.3.0-msvc-x64.patch.1
index 03fcfa7..8882a1c 100644
--- a/external/python3/python-3.3.0-msvc-x64.patch.1
+++ b/external/python3/python-3.3.0-msvc-x64.patch.1
@@ -210,7 +210,7 @@ diff -ru python3/PCbuild/pcbuild.sln 
python3.new/PCbuild/pcbuild.sln
ItemDefinitionGroup 
Condition='$(Configuration)|$(Platform)'=='Debug|Win32'
 --- python3/PCbuild/pythoncore.vcxproj
 +++ python3/PCbuild/pythoncore.vcxproj
-@@ -185,35 +185,35 @@
+@@ -195,35 +195,35 @@
ItemDefinitionGroup 

[Libreoffice-bugs] [Bug 63834] PDFIMPORT bullets disappear, underlined text becomes crossed out

2014-05-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=63834

--- Comment #4 from vv...@yandex.ru ---
Underlined text problem will be fixed within Bug 78689

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