Christian Glombek license statement

2017-08-24 Thread Glombek, Christian Johannes

All of my past & future contributions to LibreOffice may be
licensed under the MPLv2/LGPLv3+ dual license.

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


[Libreoffice-commits] help.git: source/text

2017-08-24 Thread Olivier Hallot
 source/text/shared/00/0406.xhp  |8 
 source/text/swriter/guide/spellcheck_dialog.xhp |2 +-
 2 files changed, 5 insertions(+), 5 deletions(-)

New commits:
commit b5d455e09862a98eba8209445f859de29c1e449e
Author: Olivier Hallot 
Date:   Thu Aug 24 23:46:54 2017 -0300

tdf#112015 Spelling and Grammar -> Spelling

Valid for strings that refers to the menu, but not on
the descriptive pages, where the funcionality of spell check come
together with the grammar check.

Change-Id: If8a8e876782eed2da8f42d05726aa16e621fa062
Reviewed-on: https://gerrit.libreoffice.org/41548
Reviewed-by: Julien Nabet 
Tested-by: Julien Nabet 

diff --git a/source/text/shared/00/0406.xhp 
b/source/text/shared/00/0406.xhp
index 720d1851c..f7d45ac91 100644
--- a/source/text/shared/00/0406.xhp
+++ b/source/text/shared/00/0406.xhp
@@ -49,7 +49,7 @@
   Choose Tools - 
Gallery or click the Gallery icon on the 
Standard Bar - New Theme button - Files 
tab
 
 
-Choose 
Tools - Spelling and Grammar
+Choose 
Tools - Spelling
 
 F7 
key
 On 
Standard bar, click
@@ -62,7 +62,7 @@

   
   
- Spelling and Grammar
+ Spelling
   

 
@@ -75,9 +75,9 @@
 
   Choose Tools - Language - Chinese Conversion 
(Asian language support must be enabled) - Edit terms button
 
-  Choose Tools - 
Spelling and Grammar
+  Choose Tools - 
Spelling
 
-  Choose Tools - Spelling and 
Grammar, then click Options
+  Choose Tools - Spelling, 
then click Options
 
   
  Choose Tools - Language - Thesaurus
diff --git a/source/text/swriter/guide/spellcheck_dialog.xhp 
b/source/text/swriter/guide/spellcheck_dialog.xhp
index 176a0a55a..8f0fbe41e 100644
--- a/source/text/swriter/guide/spellcheck_dialog.xhp
+++ b/source/text/swriter/guide/spellcheck_dialog.xhp
@@ -44,7 +44,7 @@
 Click in 
the document, or select the text that you want to check.
   
   
-Choose 
Tools - Spelling and Grammar.
+Choose 
Tools - Spelling.
   
   
 When a 
possible spelling error is encountered, the Spellcheck dialog opens and $[officename] offers some 
suggested corrections.
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: helpcontent2

2017-08-24 Thread Olivier Hallot
 helpcontent2 |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 5bd31bf44301fdb1938d38eae7317745f35d9db1
Author: Olivier Hallot 
Date:   Thu Aug 24 23:46:54 2017 -0300

Updated core
Project: help  b5d455e09862a98eba8209445f859de29c1e449e

tdf#112015 Spelling and Grammar -> Spelling

Valid for strings that refers to the menu, but not on
the descriptive pages, where the funcionality of spell check come
together with the grammar check.

Change-Id: If8a8e876782eed2da8f42d05726aa16e621fa062
Reviewed-on: https://gerrit.libreoffice.org/41548
Reviewed-by: Julien Nabet 
Tested-by: Julien Nabet 

diff --git a/helpcontent2 b/helpcontent2
index 8ecf172a2713..b5d455e09862 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit 8ecf172a2713f691ac63908676446fd86eefb2c1
+Subproject commit b5d455e09862a98eba8209445f859de29c1e449e
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: solenv/bin

2017-08-24 Thread Markus Mohrhard
 solenv/bin/modules/installer/scriptitems.pm |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 2b78b9feab9521e614b9edae17709cb6e2001292
Author: Markus Mohrhard 
Date:   Fri Aug 25 07:42:14 2017 +0200

updater: fix the handling of ~ in the update config

Change-Id: I5c07039e336de9362fdd92be29de516da0846998

diff --git a/solenv/bin/modules/installer/scriptitems.pm 
b/solenv/bin/modules/installer/scriptitems.pm
index 98b41f7b484a..db2b67c5c9bb 100644
--- a/solenv/bin/modules/installer/scriptitems.pm
+++ b/solenv/bin/modules/installer/scriptitems.pm
@@ -659,7 +659,7 @@ sub replace_setup_variables
 my $updatechannel = "";
 if ( $ENV{'UPDATE_CONFIG'} && $ENV{'UPDATE_CONFIG'} ne "")
 {
-open(CONFIG, $ENV{'UPDATE_CONFIG'});
+open(CONFIG, glob($ENV{'UPDATE_CONFIG'}));
 while ()
 {
 chomp;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Rebuilding LO freezes on macOS

2017-08-24 Thread Thorsten Wagner
Julien, Shinnok, Alex,

thanks for your feedback. I'm trying to raise number of openable file 
descriptors by means of ulimit first. Default seems to be 256 for OS X 10.11. 
Indeed issue probably takes place during population of "instdir". 
"Library/libscfiltlo.dylib" is referenced in make's last console output only.

I'll give feedback upon testing.

brgds

Thorsten

> On 24.08.2017, at 11:19, Alexander Thurgood  wrote:
> 
> Le 23/08/2017 à 21:28, Thorsten Wagner a écrit :
> 
> Hi THorsten,
> 
> 
>> (2) During building again build freezes while linking 
>> "Library/libscfiltlo.dylib". "make" loops consuming 100% cpu on a single 
>> core. Build has to be interrupted by killing "make" processes.
>> 
> 
> I see this occasionally, or though it tends to be during population of
> instdir (so towards the end of the build) and not specifically in
> libscfiltLO.dylib. Possibly the parallel build gets into a race
> condition somewhere (shrugs) and I have to kill it, clean it and restart.
> 
> I remember reading somewhere a long time ago about changing the ulimit
> number of openable file descriptors, something like this here:
> 
> https://unix.stackexchange.com/questions/108174/how-to-persist-ulimit-settings-in-macos
> 
> That might help, but if you need to push past the hard limit set by
> Apple as default, then you need to buy AppleServer for OSX from the
> AppStore.
> 
> 
> Alex
> 
> ___
> LibreOffice mailing list
> LibreOffice@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/libreoffice

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


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

2017-08-24 Thread Akshay Deep
 cui/source/dialogs/cuicharmap.cxx |5 +
 1 file changed, 5 insertions(+)

New commits:
commit 8d74698f407de0b02826777d5cb3f5aca2fd50f1
Author: Akshay Deep 
Date:   Thu Aug 24 20:47:43 2017 +

tdf#111795 - Special character preview control background color

Change-Id: Id5c3448b0cf82e5ed3aa9afabc5bfb093eddf2e9
Reviewed-on: https://gerrit.libreoffice.org/41524
Tested-by: Jenkins 
Reviewed-by: Heiko Tietze 

diff --git a/cui/source/dialogs/cuicharmap.cxx 
b/cui/source/dialogs/cuicharmap.cxx
index 0e7f66c93f4b..72d81f410f3f 100644
--- a/cui/source/dialogs/cuicharmap.cxx
+++ b/cui/source/dialogs/cuicharmap.cxx
@@ -1139,10 +1139,13 @@ void SvxShowText::Paint(vcl::RenderContext& 
rRenderContext, const ::tools::Recta
 rRenderContext.SetFont(maFont);
 
 Color aTextCol = rRenderContext.GetTextColor();
+Color aFillCol = rRenderContext.GetFillColor();
 
 const StyleSettings& rStyleSettings = 
Application::GetSettings().GetStyleSettings();
 const Color aWindowTextColor(rStyleSettings.GetDialogTextColor());
+const Color aWindowColor(rStyleSettings.GetWindowColor());
 rRenderContext.SetTextColor(aWindowTextColor);
+rRenderContext.SetFillColor(aWindowColor);
 
 const OUString aText = GetText();
 const Size aSize(GetOutputSizePixel());
@@ -1211,8 +1214,10 @@ void SvxShowText::Paint(vcl::RenderContext& 
rRenderContext, const ::tools::Recta
 }
 }
 
+rRenderContext.DrawRect(tools::Rectangle(Point(0, 0), 
Size(GetOutputSizePixel().Width(), GetOutputSizePixel().Height(;
 rRenderContext.DrawText(aPoint, aText);
 rRenderContext.SetTextColor(aTextCol);
+rRenderContext.SetFillColor(aFillCol);
 if (bShrankFont)
 rRenderContext.SetFont(aOrigFont);
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-5-4-1' - configure.ac

2017-08-24 Thread Christian Lohmaier
 configure.ac |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 0e082c465ed0ac73ceb5f95c01bb1b09a1b12d09
Author: Christian Lohmaier 
Date:   Thu Aug 24 23:39:40 2017 +0200

bump product version to 5.4.1.2.0+

Change-Id: I56daad178b971a4364250e20c7ada322ee9667de

diff --git a/configure.ac b/configure.ac
index 4cd0a4dc6d25..c38dd959f81b 100644
--- a/configure.ac
+++ b/configure.ac
@@ -9,7 +9,7 @@ dnl in order to create a configure script.
 # several non-alphanumeric characters, those are split off and used only for 
the
 # ABOUTBOXPRODUCTVERSIONSUFFIX in openoffice.lst. Why that is necessary, no 
idea.
 
-AC_INIT([LibreOffice],[5.4.1.1.0+],[],[],[http://documentfoundation.org/])
+AC_INIT([LibreOffice],[5.4.1.2.0+],[],[],[http://documentfoundation.org/])
 
 AC_PREREQ([2.59])
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] translations.git: Changes to 'refs/tags/libreoffice-5.4.1.2'

2017-08-24 Thread Christian Lohmaier
Tag 'libreoffice-5.4.1.2' created by Christian Lohmaier 
 at 2017-08-24 21:39 +

Tag libreoffice-5.4.1.2
-BEGIN PGP SIGNATURE-

iQIcBAABAgAGBQJZn0d/AAoJEPQ0oe+v7q6jisIP/R+1rVjXmoagi/efs0xaY9Bc
IqVNGV+zjVDwv9uP7SP0bERO01Gc5CXGPsJ6tTOnzYzPTxT76sh627CfggssOwgF
/vpfBhBwn+ogf6OZsvkcczEoMnfVaqaoseWzTpfx7oP8rWFvs+/M6ntaD3r99KZG
YZzykIpp16kQ0Zjvc7scXZriZ4/2gl/8xibYXHIUQAmLfwndoMcUnEErx+jmpKot
bGnXUDRBthibGPTcD8BJWC8ps9XpZMMf+ySW67oCPLkU+0dRk0YqjsojkCJ+lomE
s80xLFOnvJSc42ln2cIECpMEvUBfQjP5WWMWures6qUNoI7I/rWtnUURes7ZASIM
udM8rlxt5+CD5uW2tUIg6bwl9/bJHFIQgRCkwAdfT18syZGtsf+C2UQVpEtkG0KX
cIIdQ/syNBkrnF4L0gYplwqkuYaMT/EbbPsKZgxgT0wpgn3oRBPGEh2r4tdaM4+E
Myyo0jdAfNSntjByvQQAPEgL3eCz5FXgCt50BT45druogADwU/0BPxdGJwKo7KRK
H1xWG2vIHmWc8n++ij/PzG9xlYyAv+Wq562PKQj5xxDoRh6BRwWXoP/Cf58VROG+
JBiVEH8NDN9Pxm27rMHBNyYQ4w+1q/RhHtA+4wWVGNwn37+G8Lh1HgMBnl5s1r3u
ykQHR2CQO/d8po+/zfNk
=Iqdu
-END PGP SIGNATURE-

Changes since libreoffice-5-4-branch-point-20:
---
 0 files changed
---
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/tags/libreoffice-5.4.1.2'

2017-08-24 Thread Christian Lohmaier
Tag 'libreoffice-5.4.1.2' created by Christian Lohmaier 
 at 2017-08-24 21:39 +

Tag libreoffice-5.4.1.2
-BEGIN PGP SIGNATURE-

iQIcBAABAgAGBQJZn0eCAAoJEPQ0oe+v7q6joTQP/13NcESLKkH0rX8981k791jw
VnW2ck15QJhZysHmu95jqMqFUR9cFQItjFFzrYP4orC9JhFcdBebvBRZqSLL8V9M
SCSr9+pr4orA3zhI2yL0Le9zEPnYF/34iqoTEJV9mLKiLYsw32uiycI7U289WCc3
RUSkCG6sBXxgTSeOz1jx7fkkds1NdnB4c5WyFRONvRX2Zfx5EeBqn3Us2uT62FYg
vMwjxcISdao0cXQCxnyQYcI6xfXAcE1UrGMr6MXkFQjybSju+5Fb5lFOrPTLmL18
Wqw6b7HoTj7WztfjpH2MCvrsd675Q3SSkywz715MSw16EFAZeDaLhQBvvVnpIpcT
Dzxmxxr6S7Rngd2rVbGF1ZFlKT+ua12wdG2QWx0T/71LDqbBqw3nHxNtS5J10F6N
5TtK5CPFyDmIInRPhuwsbbTlzciJJhdzHUtSfaAqwA9b3m0zwben8OM218sumHRD
TrpY/dFdSvOtnZL72fXMfRDFOwXmUuo/QzXusk22NhL3B4Omd1HGzq1q45eG8zAJ
DESPFea6ALIofQ3n/3IldI4NJOxgnPBnKBJFsOhhLxWGrpWIeOXF7PNDHKifOsZy
DUW8L8mUwj0ZR8ZnASrpi1xoSjpdIqIoI4SQ2E6ENjHe5GIOUowZ/tdnO5p66RuJ
o11tebCTdu18trX5GznT
=RarF
-END PGP SIGNATURE-

Changes since libreoffice-5-4-branch-point-534:
---
 0 files changed
---
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] dictionaries.git: Changes to 'refs/tags/libreoffice-5.4.1.2'

2017-08-24 Thread Christian Lohmaier
Tag 'libreoffice-5.4.1.2' created by Christian Lohmaier 
 at 2017-08-24 21:38 +

Tag libreoffice-5.4.1.2
-BEGIN PGP SIGNATURE-

iQIcBAABAgAGBQJZn0dyAAoJEPQ0oe+v7q6j/4AQAIdXE+VDAXFDPF5qq3M7CPOC
PkOsbuXWWCQ9ocRO3emdJAHIo+fLLe0VLRhY7WL5y9pqNSCXoXZeCEKVJhW02Pje
goGv0P7dhqVweVKbiVzSBJ2Ma43sOh7cd8JV+TEjEPxbb3gMXM0XBapYMHMvAMS8
lio4pnf3GF7pRP9kucz71rKjXHf3HjRBCVMx5vjZFDPVu4tcDyzsW5DXtgRMLICH
4bWHdgJUDInWWpdDlvxpIL9Qq0q2QFM5M1R6pOwla0z/hJp2PEREHD9zpd/2pfie
KsKJWqcgvB+iFvuFfpy8E+1FcQtBTDdkV8YbMUc/hxxWXnt4SuB+eEBSBrDVifUV
iyBdyLq93ilEc+0Hq4w3PqjagbjLVtRJ3eOWk4CXWlWNQFsf2/rHSbph0kHbto31
Ordpm3EVz/nMeEdoSxLZbFPRBVnX7uIUp7Ti5gwcP44jvAjLRwWJDwQDI4QACwhh
hn+GYMkEAhl42+tCVhgwj0z9yi5RHske/9/ARJeb+I/AyAguOxjdYwB1aCJoBzep
opETd3VYzkzrqDN+juFdR8sPWDVXC4z72/CpnwUPHbv3G0NrDqQbTqAoQE42p2YT
T0UTShjaNCXOzxm40m68xQw2GtF8CBJHKkdMO9xxC5QxTRdpTbekyv9bOxq1fFGF
y5YY7MnDCWAGv/c4UpGX
=5I1q
-END PGP SIGNATURE-

Changes since libreoffice-5-4-branch-point-6:
---
 0 files changed
---
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] help.git: Changes to 'refs/tags/libreoffice-5.4.1.2'

2017-08-24 Thread Christian Lohmaier
Tag 'libreoffice-5.4.1.2' created by Christian Lohmaier 
 at 2017-08-24 21:39 +

Tag libreoffice-5.4.1.2
-BEGIN PGP SIGNATURE-

iQIcBAABAgAGBQJZn0d7AAoJEPQ0oe+v7q6j4icP/2FT4Hv0zoUfYejVLdgJONkz
nunvgT2we6b88K8ZnkUFh9zoeVQ/g59lSDDRvimS4CwDhEd2GTCEbvKW67isJah3
MDii3Zi2N8joD3TE+EGoVT3jYw7rXYJ4b9IEPujEcGg6N8sgMuurfVjqRC64b0Qz
HG7M8a+J03nJ29Y7Nhc5j2UCz9zlrLps2KNVmoJVVdguFdw1/St3gXRJW4fwV1rs
ugY5OGkBitR98lf9GMScmcSOg6spA6lsjrR86gXAL/ctCpdxyDf89Hytc60P0qTi
reBefWxvopSVPeneZmpQqZvFjIuz8xEXVS6sK1WTCyp85AnR8atbX3CUKIQ6ENlE
yL2TlpadFroiG/6Z4lT9bNZ34tZO0BRL0WnGC32phUuYi2FV/6XEsC1JTLpoMyYZ
XkqzipRMUSzo3I63547Yu2yVtV7fcUMqfcOOmH25apfFT+pFO+kFFVs95dX5FmX2
moDwNpq6zVp8AX9JGlkCgEyIC+LK8Od4S9kdhU2Q+qbZ2w6IjwyxWW5RJnmRnWhb
wbYipmS1WtoVNnM1t7414GZUXvGkxKV+MJFHKPg2Z31ybqrXL3xAW2vyKmGjiIz9
0SnRpvGNaSGugjWD+PRqwMHU4JnJfQjEWh+0qK0lbC/7eLjPUd+a4RFD7NKGgbuS
pKf99JHjFbynBABaIaqV
=SsTa
-END PGP SIGNATURE-

Changes since libreoffice-5-4-branch-point-54:
---
 0 files changed
---
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-08-24 Thread Michael Stahl
 sw/source/core/doc/DocumentContentOperationsManager.cxx |3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit b99f9c9559b9a3a2aa6337bb87be0ea1767ee7a7
Author: Michael Stahl 
Date:   Fri Aug 18 17:25:17 2017 +0200

tdf#99692 sw: fix the fix: don't set bNullContent if node was split

When showing the redlines in rhbz908615-13.odt, the following assertion
happens:

Assertion `IDocumentMarkAccess::IsLegalPaMForCrossRefHeadingBookmark(rPaM) 
&& "" "- creation of cross-reference 
bookmark with an illegal PaM that does not expand over exactly one whole 
paragraph."' failed.

This is because in DocumentContentOperationsManager::MoveRange() the
flag bNullContent is set after the node has been split; in this case the
nContent is of course always 0.

Later the function then restores aSavePam to the index 0 of the next
node, when it actually shouldn't do anything because the JoinNext()
already positioned it correctly at the merge-index of the re-joined node.

(regression from 850795942b3e168cab8ce88b4f2b421945ff29ca)

Change-Id: I64d50e70b19e2fd81537a9771fa8706898b17642
(cherry picked from commit 6a018363791945b6fd7f04f2aa311e4f4753f6aa)
Reviewed-on: https://gerrit.libreoffice.org/41304
Tested-by: Jenkins 
Reviewed-by: Miklos Vajna 
(cherry picked from commit cac440fd5a76f4a76565466cd9df363f5634e8bd)
Reviewed-on: https://gerrit.libreoffice.org/41464
Reviewed-by: Eike Rathke 
Reviewed-by: Christian Lohmaier 
Tested-by: Christian Lohmaier 

diff --git a/sw/source/core/doc/DocumentContentOperationsManager.cxx 
b/sw/source/core/doc/DocumentContentOperationsManager.cxx
index 32f7da7e99de..cfc4f4fe9605 100644
--- a/sw/source/core/doc/DocumentContentOperationsManager.cxx
+++ b/sw/source/core/doc/DocumentContentOperationsManager.cxx
@@ -2039,7 +2039,8 @@ bool DocumentContentOperationsManager::MoveRange( SwPaM& 
rPaM, SwPosition& rPos,
 // the manipulated range.
 // tdf#99692 don't Move() back if that would end up in another node
 // because moving backward is not necessarily the inverse of forward then.
-const bool bNullContent = aSavePam.GetPoint()->nContent == 0;
+// (but do Move() back if we have split the node)
+const bool bNullContent = !bSplit && aSavePam.GetPoint()->nContent == 0;
 if( bNullContent )
 {
 aSavePam.GetPoint()->nNode--;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-5-4-1' - translations

2017-08-24 Thread Christian Lohmaier
 translations |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit a2c8a955e5a0df44651e01be6d76a9ecdcf37a94
Author: Christian Lohmaier 
Date:   Thu Aug 24 22:49:15 2017 +0200

Updated core
Project: translations  e5a1db22b7e114706f4b7cbeb243c10c96cac3be

update translations for 5.4.1 rc2

and force-fix errors using pocheck

Change-Id: I757f8fd738860dc4b9acd543e4e00de668bb1e69
(cherry picked from commit 1f0ab2c64b5590f8d762b41b63f3ce1f8c6ac077)

diff --git a/translations b/translations
index b55ddde65dfb..e5a1db22b7e1 16
--- a/translations
+++ b/translations
@@ -1 +1 @@
-Subproject commit b55ddde65dfb9374b2c861fa5dcb38bf85087b49
+Subproject commit e5a1db22b7e114706f4b7cbeb243c10c96cac3be
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] translations.git: Branch 'libreoffice-5-4' - source/am source/ar source/be source/bg source/cs source/de source/es source/eu source/fr source/gl source/hr source/hsb source/id so

2017-08-24 Thread Christian Lohmaier
 source/am/dbaccess/source/ui/dlg.po|6 
 source/am/dbaccess/source/ui/querydesign.po|6 
 source/am/desktop/source/app.po|   10 
 source/am/dictionaries/en/dialog.po|6 
 source/am/extras/source/autocorr/emoji.po  |6 
 source/am/formula/source/core/resource.po  |   16 
 source/am/helpcontent2/source/text/scalc/01.po |   16 
 source/am/officecfg/registry/data/org/openoffice/Office/UI.po  |6 
 source/am/sc/source/ui/dbgui.po|   10 
 source/am/sc/source/ui/src.po  |  122 -
 source/am/sc/uiconfig/scalc/ui.po  |   30 
 source/am/scaddins/source/analysis.po  |   24 
 source/am/starmath/uiconfig/smath/ui.po|8 
 source/am/svx/source/dialog.po |8 
 source/am/svx/source/svdraw.po |   10 
 source/am/sw/source/uibase/utlui.po|   10 
 source/am/sw/uiconfig/swriter/ui.po|   14 
 source/am/wizards/source/formwizard.po |8 
 source/am/wizards/source/template.po   |   12 
 source/ar/officecfg/registry/data/org/openoffice/Office/UI.po  |   16 
 source/ar/sd/source/core.po|   10 
 source/ar/sd/source/ui/view.po |6 
 source/ar/sfx2/source/doc.po   |   12 
 source/ar/sfx2/source/view.po  |   18 
 source/ar/sfx2/uiconfig/ui.po  |   46 
 source/ar/svtools/source/misc.po   |   12 
 source/ar/svtools/uiconfig/ui.po   |   15 
 source/be/chart2/source/controller/dialogs.po  |   10 
 source/be/connectivity/source/resource.po  |6 
 source/be/cui/source/dialogs.po|8 
 source/be/cui/source/options.po|   16 
 source/be/cui/source/tabpages.po   |6 
 source/be/dbaccess/source/ui/tabledesign.po|6 
 source/be/filter/source/config/fragments/filters.po|   10 
 source/be/filter/source/config/fragments/types.po  |6 
 source/be/filter/uiconfig/ui.po|   24 
 source/be/formula/source/core/resource.po  |   12 
 source/be/librelogo/source/pythonpath.po   |   22 
 source/be/nlpsolver/help/en/com.sun.star.comp.Calc.NLPSolver.po|6 
 source/be/officecfg/registry/data/org/openoffice/Office.po |  169 -
 source/be/officecfg/registry/data/org/openoffice/Office/UI.po  |   28 
 source/be/readlicense_oo/docs.po   |7 
 source/be/reportdesign/uiconfig/dbreport/ui.po |8 
 source/be/sc/source/ui/StatisticsDialogs.po|   44 
 source/be/sc/source/ui/cctrl.po|   10 
 source/be/sc/source/ui/dbgui.po|   10 
 source/be/sc/source/ui/src.po  |  219 --
 source/be/sc/uiconfig/scalc/ui.po  |  249 +-
 source/be/scaddins/source/analysis.po  |   20 
 source/be/scp2/source/draw.po  |   14 
 source/be/sd/uiconfig/sdraw/ui.po  |   46 
 source/be/sd/uiconfig/simpress/ui.po   |   72 
 source/be/sfx2/source/dialog.po|   16 
 source/be/sfx2/uiconfig/ui.po  |   14 
 source/be/starmath/source.po   |6 
 source/be/starmath/uiconfig/smath/ui.po|   26 
 source/be/svtools/source/misc.po   |   22 
 source/be/svx/source/dialog.po |   70 
 source/be/svx/source/form.po   |   16 
 source/be/svx/source/gallery2.po   |   15 
 source/be/svx/source/items.po  |9 
 source/be/svx/source/src.po|9 
 source/be/svx/source/svdraw.po |   14 
 source/be/svx/uiconfig/ui.po   |   52 
 source/be/swext/mediawiki/help.po  |   18 
 source/bg/helpcontent2/source/text/scalc/guide.po   

[Libreoffice-commits] translations.git: Branch 'libreoffice-5-4-1' - source/am source/ar source/be source/bg source/cs source/de source/es source/eu source/fr source/gl source/hr source/hsb source/id

2017-08-24 Thread Christian Lohmaier
 source/am/dbaccess/source/ui/dlg.po|6 
 source/am/dbaccess/source/ui/querydesign.po|6 
 source/am/desktop/source/app.po|   10 
 source/am/dictionaries/en/dialog.po|6 
 source/am/extras/source/autocorr/emoji.po  |6 
 source/am/formula/source/core/resource.po  |   16 
 source/am/helpcontent2/source/text/scalc/01.po |   16 
 source/am/officecfg/registry/data/org/openoffice/Office/UI.po  |6 
 source/am/sc/source/ui/dbgui.po|   10 
 source/am/sc/source/ui/src.po  |  122 -
 source/am/sc/uiconfig/scalc/ui.po  |   30 
 source/am/scaddins/source/analysis.po  |   24 
 source/am/starmath/uiconfig/smath/ui.po|8 
 source/am/svx/source/dialog.po |8 
 source/am/svx/source/svdraw.po |   10 
 source/am/sw/source/uibase/utlui.po|   10 
 source/am/sw/uiconfig/swriter/ui.po|   14 
 source/am/wizards/source/formwizard.po |8 
 source/am/wizards/source/template.po   |   12 
 source/ar/officecfg/registry/data/org/openoffice/Office/UI.po  |   16 
 source/ar/sd/source/core.po|   10 
 source/ar/sd/source/ui/view.po |6 
 source/ar/sfx2/source/doc.po   |   12 
 source/ar/sfx2/source/view.po  |   18 
 source/ar/sfx2/uiconfig/ui.po  |   46 
 source/ar/svtools/source/misc.po   |   12 
 source/ar/svtools/uiconfig/ui.po   |   15 
 source/be/chart2/source/controller/dialogs.po  |   10 
 source/be/connectivity/source/resource.po  |6 
 source/be/cui/source/dialogs.po|8 
 source/be/cui/source/options.po|   16 
 source/be/cui/source/tabpages.po   |6 
 source/be/dbaccess/source/ui/tabledesign.po|6 
 source/be/filter/source/config/fragments/filters.po|   10 
 source/be/filter/source/config/fragments/types.po  |6 
 source/be/filter/uiconfig/ui.po|   24 
 source/be/formula/source/core/resource.po  |   12 
 source/be/librelogo/source/pythonpath.po   |   22 
 source/be/nlpsolver/help/en/com.sun.star.comp.Calc.NLPSolver.po|6 
 source/be/officecfg/registry/data/org/openoffice/Office.po |  169 -
 source/be/officecfg/registry/data/org/openoffice/Office/UI.po  |   28 
 source/be/readlicense_oo/docs.po   |7 
 source/be/reportdesign/uiconfig/dbreport/ui.po |8 
 source/be/sc/source/ui/StatisticsDialogs.po|   44 
 source/be/sc/source/ui/cctrl.po|   10 
 source/be/sc/source/ui/dbgui.po|   10 
 source/be/sc/source/ui/src.po  |  219 --
 source/be/sc/uiconfig/scalc/ui.po  |  249 +-
 source/be/scaddins/source/analysis.po  |   20 
 source/be/scp2/source/draw.po  |   14 
 source/be/sd/uiconfig/sdraw/ui.po  |   46 
 source/be/sd/uiconfig/simpress/ui.po   |   72 
 source/be/sfx2/source/dialog.po|   16 
 source/be/sfx2/uiconfig/ui.po  |   14 
 source/be/starmath/source.po   |6 
 source/be/starmath/uiconfig/smath/ui.po|   26 
 source/be/svtools/source/misc.po   |   22 
 source/be/svx/source/dialog.po |   70 
 source/be/svx/source/form.po   |   16 
 source/be/svx/source/gallery2.po   |   15 
 source/be/svx/source/items.po  |9 
 source/be/svx/source/src.po|9 
 source/be/svx/source/svdraw.po |   14 
 source/be/svx/uiconfig/ui.po   |   52 
 source/be/swext/mediawiki/help.po  |   18 
 source/bg/helpcontent2/source/text/scalc/guide.po   

[Libreoffice-commits] core.git: Branch 'libreoffice-5-4' - translations

2017-08-24 Thread Christian Lohmaier
 translations |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 1403f09257f502acaff7f0c1ea4dd15fb0eda9a4
Author: Christian Lohmaier 
Date:   Thu Aug 24 22:49:15 2017 +0200

Updated core
Project: translations  1f0ab2c64b5590f8d762b41b63f3ce1f8c6ac077

update translations for 5.4.1 rc2

and force-fix errors using pocheck

Change-Id: I757f8fd738860dc4b9acd543e4e00de668bb1e69

diff --git a/translations b/translations
index 264123910785..1f0ab2c64b55 16
--- a/translations
+++ b/translations
@@ -1 +1 @@
-Subproject commit 26412391078511f68986b8e3ad4a94229cf7c52d
+Subproject commit 1f0ab2c64b5590f8d762b41b63f3ce1f8c6ac077
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-5-4-1' - cli_ure/version unoil/climaker

2017-08-24 Thread Michael Stahl
 cli_ure/version/version.txt |   24 
 unoil/climaker/version.txt  |6 +++---
 2 files changed, 15 insertions(+), 15 deletions(-)

New commits:
commit 49eadf7e55a98ec284a76bc44373150c85fa7f1e
Author: Michael Stahl 
Date:   Wed Aug 16 15:32:04 2017 +0200

tdf#108709 cli_ure,unoil: bump CLI assembly versions for 5.4

perl cli_ure/source/scripts/increment_version.pl 
cli_ure/version/version.txt cli_ure/version/incversions.txt temp.txt && mv 
temp.txt cli_ure/version/version.txt

perl cli_ure/source/scripts/increment_version.pl unoil/climaker/version.txt 
unoil/climaker/incversions.txt temp.txt && mv temp.txt 
unoil/climaker/version.txt

Change-Id: Iaea028fc345d090317f7ebf128b683b4643a1093
(cherry picked from commit 17192ce5588f84192d1dd0d963622bda48566fdc)
Reviewed-on: https://gerrit.libreoffice.org/41215
Tested-by: Jenkins 
Reviewed-by: Michael Stahl 
(cherry picked from commit aac0c5a7d6e2738bc8f86327f3c2ba0a132b6354)
Reviewed-on: https://gerrit.libreoffice.org/41247
Reviewed-by: Christian Lohmaier 
Reviewed-by: Thorsten Behrens 
Reviewed-by: Eike Rathke 
Tested-by: Eike Rathke 

diff --git a/cli_ure/version/version.txt b/cli_ure/version/version.txt
index e24ddc511fe9..11cd9e737649 100644
--- a/cli_ure/version/version.txt
+++ b/cli_ure/version/version.txt
@@ -16,23 +16,23 @@
 #   the License at http://www.apache.org/licenses/LICENSE-2.0 .
 #
 
-CLI_URETYPES_NEW_VERSION=1.0.8.0
-CLI_URETYPES_OLD_VERSION=1.0.0.0-1.0.7.0
-CLI_URETYPES_POLICY_VERSION=8.0.0.0
+CLI_URETYPES_NEW_VERSION=1.0.9.0
+CLI_URETYPES_OLD_VERSION=1.0.0.0-1.0.8.0
+CLI_URETYPES_POLICY_VERSION=9.0.0.0
 CLI_URETYPES_POLICY_ASSEMBLY=policy.1.0.cli_uretypes
 
-CLI_BASETYPES_NEW_VERSION=1.0.19.0
-CLI_BASETYPES_OLD_VERSION=1.0.0.0-1.0.18.0
-CLI_BASETYPES_POLICY_VERSION=19.0.0.0
+CLI_BASETYPES_NEW_VERSION=1.0.20.0
+CLI_BASETYPES_OLD_VERSION=1.0.0.0-1.0.19.0
+CLI_BASETYPES_POLICY_VERSION=20.0.0.0
 CLI_BASETYPES_POLICY_ASSEMBLY=policy.1.0.cli_basetypes
 
-CLI_URE_NEW_VERSION=1.0.22.0
-CLI_URE_OLD_VERSION=1.0.0.0-1.0.21.0
-CLI_URE_POLICY_VERSION=22.0.0.0
+CLI_URE_NEW_VERSION=1.0.23.0
+CLI_URE_OLD_VERSION=1.0.0.0-1.0.22.0
+CLI_URE_POLICY_VERSION=23.0.0.0
 CLI_URE_POLICY_ASSEMBLY=policy.1.0.cli_ure
 
-CLI_CPPUHELPER_NEW_VERSION=1.0.22.0
-CLI_CPPUHELPER_OLD_VERSION=1.0.0.0-1.0.21.0
-CLI_CPPUHELPER_POLICY_VERSION=22.0.0.0
+CLI_CPPUHELPER_NEW_VERSION=1.0.23.0
+CLI_CPPUHELPER_OLD_VERSION=1.0.0.0-1.0.22.0
+CLI_CPPUHELPER_POLICY_VERSION=23.0.0.0
 CLI_CPPUHELPER_POLICY_ASSEMBLY=policy.1.0.cli_cppuhelper
  
diff --git a/unoil/climaker/version.txt b/unoil/climaker/version.txt
index abc28bc42473..ec48a082e47f 100644
--- a/unoil/climaker/version.txt
+++ b/unoil/climaker/version.txt
@@ -16,8 +16,8 @@
 #   the License at http://www.apache.org/licenses/LICENSE-2.0 .
 #
 
-CLI_OOOTYPES_NEW_VERSION=1.0.8.0
-CLI_OOOTYPES_OLD_VERSION=1.0.0.0-1.0.7.0
-CLI_OOOTYPES_POLICY_VERSION=8.0.0.0
+CLI_OOOTYPES_NEW_VERSION=1.0.9.0
+CLI_OOOTYPES_OLD_VERSION=1.0.0.0-1.0.8.0
+CLI_OOOTYPES_POLICY_VERSION=9.0.0.0
 CLI_OOOTYPES_POLICY_ASSEMBLY=policy.1.0.cli_oootypes
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-08-24 Thread Michael Stahl
 svtools/source/misc/ehdl.cxx  |2 +-
 sw/qa/core/data/odt/fail/82fff64a-0a21-4b09-bbdc-2914a5a150f0.odt |binary
 vcl/source/window/errinf.cxx  |2 +-
 3 files changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 6e090b4272b8fa12d2032ff0c0ea4bfb24c0d2d2
Author: Michael Stahl 
Date:   Thu Aug 24 21:20:25 2017 +0200

tdf#111934: add document for filters-tests

Thanks to infostu...@gmail.com for the document; unfortunately git
tells me that i can't set the commit author to just an email address.

Change-Id: I844e725d06cfa2a56adcd93aaa0a222fbb14028f

diff --git a/sw/qa/core/data/odt/fail/82fff64a-0a21-4b09-bbdc-2914a5a150f0.odt 
b/sw/qa/core/data/odt/fail/82fff64a-0a21-4b09-bbdc-2914a5a150f0.odt
new file mode 100644
index ..7d823df75213
Binary files /dev/null and 
b/sw/qa/core/data/odt/fail/82fff64a-0a21-4b09-bbdc-2914a5a150f0.odt differ
commit 4ebcb4009e2185961fe3d716bd07ef2013ecd070
Author: Michael Stahl 
Date:   Thu Aug 24 21:03:58 2017 +0200

svtools: fix checking of ErrCodeArea so error dialogs are shown again

Previously SfxErrorHandler::CreateString() compared the entire error
code so >=/<= worked but now only the ErrCodeArea is compared,
so only  works.

(regression from 158595543053c158a9bcb95f679e7b8a3c2e4e89)

Change-Id: I8ac46de154f93a7109ec29d899c71bfb470d008c

diff --git a/svtools/source/misc/ehdl.cxx b/svtools/source/misc/ehdl.cxx
index 2941c80397d7..effe4ecb5860 100644
--- a/svtools/source/misc/ehdl.cxx
+++ b/svtools/source/misc/ehdl.cxx
@@ -159,7 +159,7 @@ bool SfxErrorHandler::CreateString(const ErrorInfo *pErr, 
OUString &rStr) const
 
 {
 ErrCode nErrCode(sal_uInt32(pErr->GetErrorCode()) & ERRCODE_ERROR_MASK);
-if( pErr->GetErrorCode().GetArea() > lEnd || 
pErr->GetErrorCode().GetArea() <= lStart )
+if (pErr->GetErrorCode().GetArea() < lStart || lEnd < 
pErr->GetErrorCode().GetArea())
 return false;
 if(GetErrorString(nErrCode, rStr))
 {
commit 41fb6822cbc2f82333ea54db816f60616f827e49
Author: Michael Stahl 
Date:   Thu Aug 24 17:14:45 2017 +0200

vcl: fix double-free of ErrorInfo

The only value that ErrorHandler::GetErrorString() adds over
ErrorStringFactory::CreateString() is that it will delete the pInfo a
2nd time, because the ErrorInfo::GetErrorInfo() returns the same object
twice.

This may fix it, but it's generally hard to tell what this
brain damaged error handling code will do.

(regression from 1167df7df59e37fddef0c40c4d27cb2e82e10922)

Change-Id: I22f446bee669cae8c5288bcc2f1e1d0299f5b384

diff --git a/vcl/source/window/errinf.cxx b/vcl/source/window/errinf.cxx
index 0eb4c0160e8c..40849ca916e1 100644
--- a/vcl/source/window/errinf.cxx
+++ b/vcl/source/window/errinf.cxx
@@ -148,7 +148,7 @@ DialogMask ErrorHandler::HandleError(ErrCode nErrCodeId, 
DialogMask nFlags)
 }
 
 OUString aErr;
-if (ErrorHandler::GetErrorString(nErrCodeId, aErr))
+if (ErrorStringFactory::CreateString(pInfo, aErr))
 {
 if(!rData.pDsp)
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-08-24 Thread Julien Nabet
 svl/source/notify/lstner.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 80d8cbb8e1444ff89dd61b270b43995623e95662
Author: Julien Nabet 
Date:   Thu Aug 24 19:52:44 2017 +0200

SAL_INFO->SAL_WARN in lstner.cxx (svl module)

Since we're already in #ifdef DBG_UTIL, we can also directly use SAL_WARN

Change-Id: Ic7c564f6c08e05eb9bd5928ef031f212def094aa
Reviewed-on: https://gerrit.libreoffice.org/41531
Reviewed-by: Julien Nabet 
Tested-by: Julien Nabet 

diff --git a/svl/source/notify/lstner.cxx b/svl/source/notify/lstner.cxx
index 0b3d2e9a3af5..75928021351a 100644
--- a/svl/source/notify/lstner.cxx
+++ b/svl/source/notify/lstner.cxx
@@ -95,7 +95,7 @@ void SfxListener::StartListening( SfxBroadcaster& 
rBroadcaster, bool bPreventDup
 if (bListeningAlready && !bPreventDuplicates)
 {
 auto f = mpImpl->maCallStacks.find( &rBroadcaster );
-SAL_INFO("svl", "previous StartListening call came from: " << 
sal_backtrace_to_string(f->second.get()));
+SAL_WARN("svl", "previous StartListening call came from: " << 
sal_backtrace_to_string(f->second.get()));
 }
 #endif
 assert(!(bListeningAlready && !bPreventDuplicates) && "duplicate listener, 
try building with DBG_UTIL to find the other insert site.");
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-08-24 Thread Eike Rathke
 sc/source/core/tool/scmatrix.cxx |   17 +
 1 file changed, 9 insertions(+), 8 deletions(-)

New commits:
commit d5e600aeb89915f1fbc291462190573e555bf21d
Author: Eike Rathke 
Date:   Mon Aug 21 15:49:41 2017 +0200

Resolves: tdf#111943 really really limit the match, tdf#108292 follow-up

getRemainingCount() could deliver a wrapped around overflow value if mnIndex
was already greater than the end index, which could happen if when/for
non-matching larger block sizes were added, and if then a match was found
behind those blocks a non-requested/unexpected index was returned, which in
turn led to the assert() being hit in ScInterpreter::CalculateLookup(). In
non-debug could result in an invalid block position access.

This happened with the bug case document of tdf#111943 which in master can 
be
loaded.

Also, the start and end index are not dynamic and don't have to be 
recalculated
each time, so make them const; column argument values are unused after.

(cherry picked from commit 25b3806ac509006573e669acc33643af3bd77380)

Change-Id: Ic294cade4e8e7828bee394e5ade61d7127be6bbb
Reviewed-on: https://gerrit.libreoffice.org/41468
Reviewed-by: Christian Lohmaier 
Reviewed-by: Markus Mohrhard 
Reviewed-by: Michael Stahl 
Tested-by: Michael Stahl 

diff --git a/sc/source/core/tool/scmatrix.cxx b/sc/source/core/tool/scmatrix.cxx
index 659e22fa064a..0a3dca7cb12d 100644
--- a/sc/source/core/tool/scmatrix.cxx
+++ b/sc/source/core/tool/scmatrix.cxx
@@ -1267,24 +1267,25 @@ template
 class WalkAndMatchElements : public 
std::unary_function
 {
 Type maMatchValue;
-MatrixImplType::size_pair_type maSize;
-size_t mnCol1;
-size_t mnCol2;
+const size_t mnStartIndex;
+const size_t mnStopIndex;
 size_t mnResult;
 size_t mnIndex;
 
 public:
 WalkAndMatchElements(Type aMatchValue, const 
MatrixImplType::size_pair_type& aSize, size_t nCol1, size_t nCol2) :
 maMatchValue(aMatchValue),
-maSize(aSize),
-mnCol1(nCol1),
-mnCol2(nCol2),
+mnStartIndex( nCol1 * aSize.row ),
+mnStopIndex( (nCol2 + 1) * aSize.row ),
 mnResult(ResultNotSet),
 mnIndex(0) {}
 
 size_t getMatching() const { return mnResult; }
 
-size_t getRemainingCount() const { return ((mnCol2 + 1) * maSize.row) - 
mnIndex; }
+size_t getRemainingCount() const
+{
+return mnIndex < mnStopIndex ? mnStopIndex - mnIndex : 0;
+}
 
 size_t compare(const MatrixImplType::element_block_node_type& node) const;
 
@@ -1295,7 +1296,7 @@ public:
 return;
 
 // limit lookup to the requested columns
-if ((mnCol1 * maSize.row) <= mnIndex && getRemainingCount() > 0)
+if (mnStartIndex <= mnIndex && getRemainingCount() > 0)
 {
 mnResult = compare(node);
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-08-24 Thread Eike Rathke
 sc/source/core/data/documen7.cxx |3 ++-
 sc/source/core/data/table5.cxx   |   15 ---
 2 files changed, 14 insertions(+), 4 deletions(-)

New commits:
commit c84332184a5de3e4180514cf6e3282597ffe5dd6
Author: Eike Rathke 
Date:   Thu Aug 24 19:33:32 2017 +0200

Resolves: tdf#93171 follow-up, notify dependents of subtotal formula cells

Track ScHiddenRowsChanged collected cells and broadcast with ScDataChanged.

Change-Id: Id41d2731585f6749533319bf42ad655a845ecf30

diff --git a/sc/source/core/data/documen7.cxx b/sc/source/core/data/documen7.cxx
index 342fff0c5664..299f5e70d089 100644
--- a/sc/source/core/data/documen7.cxx
+++ b/sc/source/core/data/documen7.cxx
@@ -559,7 +559,8 @@ void ScDocument::TrackFormulas( SfxHintId nHintId )
 if (!pBASM)
 return;
 
-if (pBASM->IsInBulkBroadcast() && !IsFinalTrackFormulas() && nHintId == 
SfxHintId::ScDataChanged)
+if (pBASM->IsInBulkBroadcast() && !IsFinalTrackFormulas() &&
+(nHintId == SfxHintId::ScDataChanged || nHintId == 
SfxHintId::ScHiddenRowsChanged))
 {
 SetTrackFormulasPending();
 return;
diff --git a/sc/source/core/data/table5.cxx b/sc/source/core/data/table5.cxx
index f64e2dc865f1..d6c7a8527e4b 100644
--- a/sc/source/core/data/table5.cxx
+++ b/sc/source/core/data/table5.cxx
@@ -40,6 +40,7 @@
 #include "globalnames.hxx"
 #include "scmod.hxx"
 #include "printopt.hxx"
+#include "bcaslot.hxx"
 
 #include 
 
@@ -593,9 +594,17 @@ bool ScTable::SetRowHidden(SCROW nStartRow, SCROW nEndRow, 
bool bHidden)
 {
 if (IsStreamValid())
 SetStreamValid(false);
-for (SCCOL i = 0; i < aCol.size(); i++)
-{
-aCol[i].BroadcastRows(nStartRow, nEndRow, 
SfxHintId::ScHiddenRowsChanged);
+
+{   // Scoped bulk broadcast.
+// Only subtotal formula cells will accept the notification of
+// SfxHintId::ScHiddenRowsChanged, leaving the bulk will track
+// those and broadcast SfxHintId::ScDataChanged to notify all
+// dependents.
+ScBulkBroadcast aBulkBroadcast( pDocument->GetBASM(), 
SfxHintId::ScDataChanged);
+for (SCCOL i = 0; i < aCol.size(); i++)
+{
+aCol[i].BroadcastRows(nStartRow, nEndRow, 
SfxHintId::ScHiddenRowsChanged);
+}
 }
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-08-24 Thread Szymon Kłos
 sw/qa/extras/ooxmlexport/data/watermark-layer.docx |binary
 sw/qa/extras/ooxmlexport/ooxmlexport9.cxx  |   14 ++
 sw/source/core/edit/edfcol.cxx |   15 ++-
 writerfilter/source/dmapper/DomainMapper_Impl.cxx  |   10 +-
 4 files changed, 33 insertions(+), 6 deletions(-)

New commits:
commit 39c08074a286855dd014ce1c30b8f7ef95b10242
Author: Szymon Kłos 
Date:   Fri Aug 18 15:13:52 2017 +0200

Watermark: not visible if page background was set

Watermark was drawn under the page background.
It has to be placed on the upper layer to be visible.

Change-Id: I132a313eed6fb712aafdca14a38fe559aa4231c8
Reviewed-on: https://gerrit.libreoffice.org/41289
Tested-by: Jenkins 
Reviewed-by: Szymon Kłos 

diff --git a/sw/qa/extras/ooxmlexport/data/watermark-layer.docx 
b/sw/qa/extras/ooxmlexport/data/watermark-layer.docx
new file mode 100755
index ..9428686c1bad
Binary files /dev/null and b/sw/qa/extras/ooxmlexport/data/watermark-layer.docx 
differ
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport9.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlexport9.cxx
index 4e961079b829..515f40e4803b 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport9.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport9.cxx
@@ -30,8 +30,10 @@
 #include 
 
 #include 
+#include 
 #include 
 #include 
+#include 
 
 class Test : public SwModelTestBase
 {
@@ -993,6 +995,18 @@ DECLARE_OOXMLEXPORT_TEST(testActiveXControlAtRunEnd, 
"activex_control_at_run_end
 
CPPUNIT_ASSERT_EQUAL(text::TextContentAnchorType_AT_CHARACTER,getProperty(xPropertySet2,"AnchorType"));
 }
 
+DECLARE_OOXMLEXPORT_TEST(testWatermarkLayer, "watermark-layer.docx")
+{
+// Watermark was not visible if page background was set.
+
+SwXTextDocument* pTextDoc = 
dynamic_cast(mxComponent.get());
+SdrPage* pPage = 
pTextDoc->GetDocShell()->GetDoc()->getIDocumentDrawModelAccess().GetDrawModel()->GetPage(0);
+SdrObject* pObject = pPage->GetObj(0);
+
+CPPUNIT_ASSERT(pObject);
+CPPUNIT_ASSERT_EQUAL(static_cast(1), pObject->GetLayer().get());
+}
+
 CPPUNIT_PLUGIN_IMPLEMENT();
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/sw/source/core/edit/edfcol.cxx b/sw/source/core/edit/edfcol.cxx
index 4e309a33761f..456d06b65b96 100644
--- a/sw/source/core/edit/edfcol.cxx
+++ b/sw/source/core/edit/edfcol.cxx
@@ -62,6 +62,7 @@
 #include 
 #include 
 #include 
+#include 
 
 #include 
 
@@ -352,7 +353,8 @@ SfxWatermarkItem SwEditShell::GetWatermark()
 void lcl_placeWatermarkInHeader(const SfxWatermarkItem& rWatermark,
 const uno::Reference& xModel,
 const uno::Reference& 
xPageStyle,
-const uno::Reference& xHeaderText)
+const uno::Reference& xHeaderText,
+sal_Int16 nLayer)
 {
 uno::Reference xMultiServiceFactory(xModel, 
uno::UNO_QUERY);
 OUString aShapeServiceName = "com.sun.star.drawing.CustomShape";
@@ -474,6 +476,7 @@ void lcl_placeWatermarkInHeader(const SfxWatermarkItem& 
rWatermark,
 xPropertySet->setPropertyValue("Transformation", 
uno::makeAny(aMatrix));
 xPropertySet->setPropertyValue(UNO_NAME_HORI_ORIENT, 
uno::makeAny(static_cast(text::HoriOrientation::CENTER)));
 xPropertySet->setPropertyValue(UNO_NAME_VERT_ORIENT, 
uno::makeAny(static_cast(text::VertOrientation::CENTER)));
+xPropertySet->setPropertyValue(UNO_NAME_MISC_OBJ_LAYERID, 
uno::makeAny(nLayer));
 
 uno::Reference xTextRange(xShape, uno::UNO_QUERY);
 xTextRange->setString(rWatermark.GetText());
@@ -519,6 +522,8 @@ void SwEditShell::SetWatermark(const SfxWatermarkItem& 
rWatermark)
 uno::Reference 
xStyleFamilies(xStyleFamiliesSupplier->getStyleFamilies(), uno::UNO_QUERY);
 uno::Reference 
xStyleFamily(xStyleFamilies->getByName("PageStyles"), uno::UNO_QUERY);
 
+sal_Int16 nHeavenId = 
GetDoc()->getIDocumentDrawModelAccess().GetHeavenId().get();
+
 std::set aUsedPageStyles = lcl_getUsedPageStyles(this);
 for (const OUString& rPageStyleName : aUsedPageStyles)
 {
@@ -544,16 +549,16 @@ void SwEditShell::SetWatermark(const SfxWatermarkItem& 
rWatermark)
 uno::Reference xHeaderTextRight;
 
 xPageStyle->getPropertyValue(UNO_NAME_HEADER_TEXT) >>= xHeaderText;
-lcl_placeWatermarkInHeader(rWatermark, xModel, xPageStyle, 
xHeaderText);
+lcl_placeWatermarkInHeader(rWatermark, xModel, xPageStyle, 
xHeaderText, nHeavenId);
 
 xPageStyle->getPropertyValue(UNO_NAME_HEADER_TEXT_FIRST) >>= 
xHeaderTextFirst;
-lcl_placeWatermarkInHeader(rWatermark, xModel, xPageStyle, 
xHeaderTextFirst);
+lcl_placeWatermarkInHeader(rWatermark, xModel, xPageStyle, 
xHeaderTextFirst, nHeavenId);
 
 xPageStyle->getPropertyValue(UNO_NAME_HEADER_TEXT_LEFT) >>= 
xHeaderTextLeft;
-lcl_placeWatermarkInHeader(rWatermark, xModel, xPageStyle, 
xHeaderTextLeft);
+   

[Libreoffice-commits] core.git: writerperfect/CppunitTest_writerperfect_epubexport.mk writerperfect/Module_writerperfect.mk writerperfect/qa

2017-08-24 Thread Miklos Vajna
 writerperfect/CppunitTest_writerperfect_epubexport.mk   |   41 
 writerperfect/Module_writerperfect.mk   |1 
 writerperfect/qa/unit/EPUBExportTest.cxx|   88 
++
 writerperfect/qa/unit/data/writer/epubexport/outline-level.fodt |   15 +
 4 files changed, 145 insertions(+)

New commits:
commit ab8b8fb183067fd0bded089f513b614d35138e18
Author: Miklos Vajna 
Date:   Thu Aug 24 14:32:59 2017 +0200

Add initial CppunitTest_writerperfect_epubexport

Fails with the writerperfect/source/writer/exp/txtparai.cxx part of
commit b6f39c47fb477f16c65631523b0c18b4f262fadf (EPUB export: initial
index support, 2017-08-15) reverted.

Change-Id: I56457b3e3b312682e4c83ea08dbd459ca42152ef
Reviewed-on: https://gerrit.libreoffice.org/41518
Tested-by: Jenkins 
Reviewed-by: Miklos Vajna 

diff --git a/writerperfect/CppunitTest_writerperfect_epubexport.mk 
b/writerperfect/CppunitTest_writerperfect_epubexport.mk
new file mode 100644
index ..ea5285730b31
--- /dev/null
+++ b/writerperfect/CppunitTest_writerperfect_epubexport.mk
@@ -0,0 +1,41 @@
+# -*- Mode: makefile-gmake; tab-width: 4; indent-tabs-mode: t -*-
+#
+# This file is part of the LibreOffice project.
+#
+# This Source Code Form is subject to the terms of the Mozilla Public
+# License, v. 2.0. If a copy of the MPL was not distributed with this
+# file, You can obtain one at http://mozilla.org/MPL/2.0/.
+#
+
+$(eval $(call gb_CppunitTest_CppunitTest,writerperfect_epubexport))
+
+$(eval $(call gb_CppunitTest_add_exception_objects,writerperfect_epubexport, \
+writerperfect/qa/unit/EPUBExportTest \
+))
+
+$(eval $(call gb_CppunitTest_use_sdk_api,writerperfect_epubexport))
+
+$(eval $(call gb_CppunitTest_use_libraries,writerperfect_epubexport, \
+   comphelper \
+   cppu \
+   cppuhelper \
+   sal \
+   test \
+   unotest \
+   utl \
+   tl \
+   $(gb_UWINAPI) \
+))
+
+$(eval $(call 
gb_CppunitTest_use_external,writerperfect_epubexport,boost_headers))
+
+$(eval $(call gb_CppunitTest_use_sdk_api,writerperfect_epubexport))
+
+$(eval $(call gb_CppunitTest_use_ure,writerperfect_epubexport))
+$(eval $(call gb_CppunitTest_use_vcl,writerperfect_epubexport))
+
+$(eval $(call gb_CppunitTest_use_rdb,writerperfect_epubexport,services))
+
+$(eval $(call gb_CppunitTest_use_configuration,writerperfect_epubexport))
+
+# vim: set noet sw=4 ts=4:
diff --git a/writerperfect/Module_writerperfect.mk 
b/writerperfect/Module_writerperfect.mk
index b2102c54b987..859f0cbe2507 100644
--- a/writerperfect/Module_writerperfect.mk
+++ b/writerperfect/Module_writerperfect.mk
@@ -40,6 +40,7 @@ $(eval $(call gb_Module_add_check_targets,writerperfect,\
 $(eval $(call gb_Module_add_slowcheck_targets,writerperfect,\
CppunitTest_writerperfect_calc \
CppunitTest_writerperfect_draw \
+   CppunitTest_writerperfect_epubexport \
CppunitTest_writerperfect_import \
CppunitTest_writerperfect_impress \
CppunitTest_writerperfect_writer \
diff --git a/writerperfect/qa/unit/EPUBExportTest.cxx 
b/writerperfect/qa/unit/EPUBExportTest.cxx
new file mode 100644
index ..ef65012daca9
--- /dev/null
+++ b/writerperfect/qa/unit/EPUBExportTest.cxx
@@ -0,0 +1,88 @@
+/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
+/*
+ * This file is part of the LibreOffice project.
+ *
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/.
+ */
+
+#include 
+#include 
+#include 
+
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+
+using namespace ::com::sun::star;
+
+namespace
+{
+
+char const DATA_DIRECTORY[] = "/writerperfect/qa/unit/data/writer/epubexport/";
+
+/// Tests the EPUB export filter.
+class EPUBExportTest : public test::BootstrapFixture, public 
unotest::MacrosTest
+{
+uno::Reference mxComponentContext;
+uno::Reference mxComponent;
+
+public:
+virtual void setUp() override;
+virtual void tearDown() override;
+void testOutlineLevel();
+
+CPPUNIT_TEST_SUITE(EPUBExportTest);
+CPPUNIT_TEST(testOutlineLevel);
+CPPUNIT_TEST_SUITE_END();
+};
+
+void EPUBExportTest::setUp()
+{
+test::BootstrapFixture::setUp();
+
+
mxComponentContext.set(comphelper::getComponentContext(getMultiServiceFactory()));
+mxDesktop.set(frame::Desktop::create(mxComponentContext));
+}
+
+void EPUBExportTest::tearDown()
+{
+if (mxComponent.is())
+mxComponent->dispose();
+
+test::BootstrapFixture::tearDown();
+}
+
+void EPUBExportTest::testOutlineLevel()
+{
+// Import the bugdoc and export as EPUB.
+OUString aURL = m_directories.getURLFromSrc(DATA_DIRECTORY) + 
"outline-level.fodt";
+mxComponent = loadFromDesktop(aURL);
+CPPUNIT_ASSERT(mxComponent.is());
+
+uno::Reference xStorable(mxCompo

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

2017-08-24 Thread Patrick Jaap
 emfio/qa/cppunit/emf/EmfImportTest.cxx   |   42 +++
 emfio/qa/cppunit/emf/data/TestDrawLine.emf   |binary
 emfio/qa/cppunit/emf/data/TestDrawString.emf |binary
 test/source/primitive2dxmldump.cxx   |   18 +++
 4 files changed, 60 insertions(+)

New commits:
commit 8d1dc9a662d2e1bc833d5488f3f0013f98e7f3fd
Author: Patrick Jaap 
Date:   Tue Aug 22 12:31:15 2017 +0200

emfio: unit test for the EMF+ renderer

Two unit tests, which test the EMF+ records DrawString and DrawLine
independently.
To this end, a fixed view decomposition of metafiles is implemented.

The chosen primitives should be optimal, even for future extensions
of the EMF+ parser.

Change-Id: I12672d3b294ff0fdae63c7a5ce211517ef8f763c
Reviewed-on: https://gerrit.libreoffice.org/41425
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/emfio/qa/cppunit/emf/EmfImportTest.cxx 
b/emfio/qa/cppunit/emf/EmfImportTest.cxx
index ca766b8b67f3..bc14c4bd3062 100644
--- a/emfio/qa/cppunit/emf/EmfImportTest.cxx
+++ b/emfio/qa/cppunit/emf/EmfImportTest.cxx
@@ -39,12 +39,16 @@ class Test : public test::BootstrapFixture, public 
XmlTestTools
 void checkRectPrimitive(Primitive2DSequence const & rPrimitive);
 
 void testWorking();
+void TestDrawString();
+void TestDrawLine();
 
 Primitive2DSequence parseEmf(const OUString& aSource);
 
 public:
 CPPUNIT_TEST_SUITE(Test);
 CPPUNIT_TEST(testWorking);
+CPPUNIT_TEST(TestDrawString);
+CPPUNIT_TEST(TestDrawLine);
 CPPUNIT_TEST_SUITE_END();
 };
 
@@ -88,6 +92,44 @@ void Test::testWorking()
 checkRectPrimitive(aSequenceRect);
 }
 
+void Test::TestDrawString()
+{
+// This unit checks for a correct import of an EMF+ file with only one 
DrawString Record
+// Since the text is undecorated the optimal choice is a simpletextportion 
primitive
+
+// first, get the sequence of primitives and dump it
+Primitive2DSequence aSequence = 
parseEmf("/emfio/qa/cppunit/emf/data/TestDrawString.emf");
+CPPUNIT_ASSERT_EQUAL(1, (int) aSequence.getLength());
+Primitive2dXmlDump dumper;
+xmlDocPtr pDocument = 
dumper.dumpAndParse(comphelper::sequenceToContainer(aSequence));
+CPPUNIT_ASSERT (pDocument);
+
+// check correct import of the DrawString: height, position, text, color 
and font
+assertXPath(pDocument, 
"/primitive2D/metafile/transform/textsimpleportion", "height", "276");
+assertXPath(pDocument, 
"/primitive2D/metafile/transform/textsimpleportion", "x", "25");
+assertXPath(pDocument, 
"/primitive2D/metafile/transform/textsimpleportion", "y", "323");
+assertXPath(pDocument, 
"/primitive2D/metafile/transform/textsimpleportion", "text", "TEST");
+assertXPath(pDocument, 
"/primitive2D/metafile/transform/textsimpleportion", "fontcolor", "#00");
+assertXPath(pDocument, 
"/primitive2D/metafile/transform/textsimpleportion", "familyname", "CALIBRI");
+}
+
+void Test::TestDrawLine()
+{
+// This unit checks for a correct import of an EMF+ file with only one 
DrawLine Record
+// The line is colored and has a specified width, therefore a 
polypolygonstroke primitive is the optimal choice
+
+// first, get the sequence of primitives and dump it
+Primitive2DSequence aSequence = 
parseEmf("/emfio/qa/cppunit/emf/data/TestDrawLine.emf");
+CPPUNIT_ASSERT_EQUAL(1, (int) aSequence.getLength());
+Primitive2dXmlDump dumper;
+xmlDocPtr pDocument = 
dumper.dumpAndParse(comphelper::sequenceToContainer(aSequence));
+CPPUNIT_ASSERT (pDocument);
+
+// check correct import of the DrawLine: color and width of the line
+assertXPath(pDocument, 
"/primitive2D/metafile/transform/polypolygonstroke/line", "color", "#00");
+assertXPath(pDocument, 
"/primitive2D/metafile/transform/polypolygonstroke/line", "width", "33");
+}
+
 CPPUNIT_TEST_SUITE_REGISTRATION(Test);
 
 }
diff --git a/emfio/qa/cppunit/emf/data/TestDrawLine.emf 
b/emfio/qa/cppunit/emf/data/TestDrawLine.emf
new file mode 100644
index ..89946c523f98
Binary files /dev/null and b/emfio/qa/cppunit/emf/data/TestDrawLine.emf differ
diff --git a/emfio/qa/cppunit/emf/data/TestDrawString.emf 
b/emfio/qa/cppunit/emf/data/TestDrawString.emf
new file mode 100644
index ..c7976f53b3c0
Binary files /dev/null and b/emfio/qa/cppunit/emf/data/TestDrawString.emf differ
diff --git a/test/source/primitive2dxmldump.cxx 
b/test/source/primitive2dxmldump.cxx
index 6523c4c0c971..563fb75acc19 100644
--- a/test/source/primitive2dxmldump.cxx
+++ b/test/source/primitive2dxmldump.cxx
@@ -26,6 +26,9 @@
 #include 
 #include 
 #include 
+#include 
+#include 
+
 
 #include 
 #include 
@@ -190,6 +193,8 @@ void Primitive2dXmlDump::decomposeAndWrite(
 {
 rWriter.attribute("height", aScale.getY());
 }
+rWriter.attribute("x", aTranslate.getX());
+rWriter.attribute("y", aTransla

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

2017-08-24 Thread heiko tietze
 cui/uiconfig/ui/spellingdialog.ui|2 +-
 officecfg/registry/data/org/openoffice/Office/UI/GenericCommands.xcu |4 
++--
 2 files changed, 3 insertions(+), 3 deletions(-)

New commits:
commit 1098fba99acb8c8cc7e87c55ece12ef175593147
Author: heiko tietze 
Date:   Thu Aug 3 14:55:27 2017 +0200

tdf#87648 Spelling and Grammar

Renamed to just Spelling

Change-Id: I4433d75a8fd697e8d4dd25fa5d26411ed15b6c1b
Reviewed-on: https://gerrit.libreoffice.org/40717
Tested-by: Jenkins 
Reviewed-by: Heiko Tietze 

diff --git a/cui/uiconfig/ui/spellingdialog.ui 
b/cui/uiconfig/ui/spellingdialog.ui
index 0f576da899ac..07688929231b 100644
--- a/cui/uiconfig/ui/spellingdialog.ui
+++ b/cui/uiconfig/ui/spellingdialog.ui
@@ -6,7 +6,7 @@
   
 False
 6
-Spelling and Grammar: $LANGUAGE 
($LOCATION)
+Spelling: $LANGUAGE 
($LOCATION)
 False
 dialog
 
diff --git 
a/officecfg/registry/data/org/openoffice/Office/UI/GenericCommands.xcu 
b/officecfg/registry/data/org/openoffice/Office/UI/GenericCommands.xcu
index 62f495322389..ff7788129b3c 100644
--- a/officecfg/registry/data/org/openoffice/Office/UI/GenericCommands.xcu
+++ b/officecfg/registry/data/org/openoffice/Office/UI/GenericCommands.xcu
@@ -3577,10 +3577,10 @@
   Spelling
 
 
-  ~Spelling and Grammar...
+  ~Spelling...
 
 
-  Check Spelling and Grammar
+  Check Spelling
 
 
   1
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Minutes of ESC call 2017-08-24

2017-08-24 Thread Miklos Vajna
* Present:
+ Armin, Heiko, Olivier, Shinnok, Thorsten, Miklos, Sophie, Eike, Stephan,
  Michael S, Cloph, Jan-Marek

* Completed Action Items:
+ poke people if they are around for GSoC evals (Thorsten)

* Pending Action Items:
+ find out if we have projects & mentors for Capstone (Shinnok)
+ Update the roadmap wiki with new 5.3.6/7 dates (Christian)
+ https://wiki.documentfoundation.org/Development/ScreenshotsAtInstall
+ work out what a good idea is here (Kendy, Thorsten)
  + next week or remove...
+ create redirect / transparent proxy so that we can use
  help.libreoffice.org directly even for the new stuff (Christian)
  [ sitemap exists, but not functional yet ]
+ build a write-up of the differences cross-platform (Xisco)
  [ done, Heiko wants to discuss a possible multiplier ]

* Release Engineering update (Christian)
+ 5.3.6 RC1 is on the mirrors, to be announced early next week
+ 5.4.1 RC2 is due this week, a bit of delay due to additional fixes
  + 
https://gerrit.libreoffice.org/#/q/status:open+project:core+branch:libreoffice-5-4-1
+ Android & iOS Remote
+ Android viewer
+ 10% of users reported crashes, not yet rolling out
+ AI: connect to uznomis[m] / Ximeng to take a look if possible 
(Christian)
+ Online
+ Not enough tinderboxes? (Thorsten)
  + Christian’s box is the only one providing daily builds for macOS
+ Mac CI will be back in 1-2 weeks, that’s the hope
+ would be good to have 1-2 mac tinderboxes at any rate
  + Linux builds are always off for 2-3 days around releases
+ only affects RPM/DEB, Markus provides archives

* Documentation (Olivier)
 + More work on SEO data for BoD
 + Help pages for undocumented features (Impress Remote)
 + Updated help pages (SUBTOTAL,  CDATE)
 + More help fixes by G. Kelemen and A. Barrientos
 + Help online: Shall we keep Google search or find a internal solution?
   See “Search” tab of https://goo.gl/7zWZ8W
 + Is there any massive critics? (Thorsten)
 + The site is already crawled (large site, lots of pages * languages) 
 + Can we limit hits to a single language? (Christian)
   + Not at the moment (Olivier)
 + Perhaps something less invasive (i.e. not Google but DDG?)
   + Having this as someone else’ problem is great (Thorsten)
 + Doumentation team producing Guides updates weekly
   + Target is 6.0 release in sync with code
 + Staff activities.

* UX Update (Heiko)
+ Bugzilla (topicUI) statistics
257(257) (topicUI) bugs open, 419(419) (needsUXEval) needs to be 
evaluated by the UXteam
+ Updates:
BZ changes   1 week1 month   3 months   12 months  
 added  7(-1) 15(3) 31(2)  346(-1) 
 commented 44(-12)   170(7)484(-5)3311(13) 
   removed  0(-1)  2(-1) 5(-1)  44(-1) 
  resolved  6(-1) 31(-8)69(3)  287(6)  
+ top 10 contributors:
  Tietze, Heiko made 140 changes in 1 month, and 1603 changes in 1 year
  Philips, Yousuf made 49 changes in 1 month, and 1486 changes in 1 year
  Foote, V Stuart made 44 changes in 1 month, and 604 changes in 1 year
  Cor Nouws made 41 changes in 1 month, and 363 changes in 1 year
  Faulí Tarazona, Xisco made 23 changes in 1 month, and 466 changes in 
1 y
  Adolfo Jayme Barrientos made 22 changes in 1 month, and 51 changes in 
1 y
  Thomas Lendo made 21 changes in 1 month, and 407 changes in 1 year
  Timur made 13 changes in 1 month, and 70 changes in 1 year
  Telesto made 12 changes in 1 month, and 73 changes in 1 year
  Buovjaga made 11 changes in 1 month, and 352 changes in 1 year

+ Mascot >180 proposals, and counting
+ Open Source survey 1400 participants, evaluation comes next
+ Talked about taking direct formatting/character style from headings into 
ToC
  https://bugs.documentfoundation.org/show_bug.cgi?id=87604
  question is whether or not to take all properties
  + there are related tickets, resolved and open (Miklos)

* Crash Reporting (Caolan)
+ 24(+0) import failure, ~700(+0) export failures
+ 13 coverity issues
+ Google / ossfuzz: ?? fuzzers active now

* Crash Reporter (Xisco)
+ http://crashreport.libreoffice.org/stats/?days=128
+ http://crashreport.libreoffice.org/stats/version/5.3.4.2
+ 1982 (last 7 days)
+ http://crashreport.libreoffice.org/stats/version/5.3.5.2
+ 420(last 7 days)
+ http://crashreport.libreoffice.org/stats/version/5.4.0.3
+ 1982 (last 7 days)

* Hackfests & Events (Bjoern)
+ next venues / suggestions
+ LibreOffice Conference
+ Google Mentor Summit right after the conference
  + application deadline is next week, hurry up if you want to go (and are 
a mentor)

* mentoring/easyhack update (S

Minutes of the Design Hangout: 2017-Aug-24

2017-08-24 Thread Heiko Tietze
Present: Jay, Stuart, Andreas, Heiko, Tomaž 

 * Default list of favorite characters
   + https://bugs.documentfoundation.org/show_bug.cgi?id=111775
   + ₠ -> €
   + ₪ -> ©
   + Φ,Ψ -> ≤,≥
   + 😃 -> π (U+03c0)

 * Default set of list styles
   + https://bugs.documentfoundation.org/show_bug.cgi?id=106988
   + https://gerrit.libreoffice.org/#/c/37742/
   + Final decision needed to submit Jay's patch
   -> create a document without patch and open with patch (AI Heiko)

 * Removal of Format > Columns
   + https://gerrit.libreoffice.org/#/c/39017/
   + Final decision needed to submit Jay's patch
   -> accepted, +2'ed, merged

 * 'Spelling' or 'Spelling and Grammar'
   + https://bugs.documentfoundation.org/show_bug.cgi?id=87648
   + https://gerrit.libreoffice.org/#/c/40717/
   + "If we instead swap the .uno:SpellingAndGrammarDialog command for 
.uno:SpellDialog 
  in the menu and toolbar xml, we could make this change and backport it as 
well." (Jay)
   -> there might be code that removes "and Grammar" when the dialog is opened 
in other modules, eg. Calc
   -> merge but double-check

  * Insert field in Calc only in edit mode
   + https://bugs.documentfoundation.org/show_bug.cgi?id=89261
   + what if cell has content and in case of multi selection?
   + Better have dedicated functions/specifier for this, e.g. =INFO() (Heiko)
   + no, because it clutters the menu (Jay)
   + no, because the target is unclear in multiselection (heiko)
   -> WF

 * Use direct formatting and character style in ToC
   + https://bugs.documentfoundation.org/show_bug.cgi?id=87604
   + Limitation unclear, e.g. font size?
   + Scope unclear, alphabetical index, figures, tables...
   + Take all properties (Heiko)
   + implementation issues to pick single properties and not use font size, for 
example (Stuart, Jay)
   -> postponed for next week




signature.asc
Description: OpenPGP digital signature
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice


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

2017-08-24 Thread Eike Rathke
 sc/source/ui/app/inputhdl.cxx |8 
 sc/source/ui/app/inputwin.cxx |6 ++
 sc/source/ui/inc/inputhdl.hxx |1 +
 3 files changed, 15 insertions(+)

New commits:
commit d31a8263d564ebd7886f6debdf36cedd031915ea
Author: Eike Rathke 
Date:   Thu Aug 24 16:34:55 2017 +0200

Resolves: tdf#108795 merge table EditEngine language attributes to input 
line

... when editing there, so the SvxAutoCorrect behaviour is synchronized.
Depends on commit 06d14411a447cd798d1f0678a3c5e06f5278a1cb that disentangles
the stateful SvxAutoCorrect instance.

Change-Id: Ic6aac1f2acc36b500144be50a20a4784a3ba62dc
Reviewed-on: https://gerrit.libreoffice.org/41521
Reviewed-by: Eike Rathke 
Tested-by: Eike Rathke 

diff --git a/sc/source/ui/app/inputhdl.cxx b/sc/source/ui/app/inputhdl.cxx
index 9be7701dfb92..f9435aac55f1 100644
--- a/sc/source/ui/app/inputhdl.cxx
+++ b/sc/source/ui/app/inputhdl.cxx
@@ -2185,6 +2185,14 @@ bool ScInputHandler::StartTable( sal_Unicode cTyped, 
bool bFromCommand, bool bIn
 return bNewTable;
 }
 
+void ScInputHandler::MergeLanguageAttributes( ScEditEngineDefaulter& 
rDestEngine ) const
+{
+const SfxItemSet& rSrcSet = mpEditEngine->GetDefaults();
+rDestEngine.SetDefaultItem( rSrcSet.Get( EE_CHAR_LANGUAGE ));
+rDestEngine.SetDefaultItem( rSrcSet.Get( EE_CHAR_LANGUAGE_CJK ));
+rDestEngine.SetDefaultItem( rSrcSet.Get( EE_CHAR_LANGUAGE_CTL ));
+}
+
 static void lcl_SetTopSelection( EditView* pEditView, ESelection& rSel )
 {
 OSL_ENSURE( rSel.nStartPara==0 && rSel.nEndPara==0, "SetTopSelection: Para 
!= 0" );
diff --git a/sc/source/ui/app/inputwin.cxx b/sc/source/ui/app/inputwin.cxx
index c69dae4a98a0..dca31015375a 100644
--- a/sc/source/ui/app/inputwin.cxx
+++ b/sc/source/ui/app/inputwin.cxx
@@ -1108,7 +1108,13 @@ void ScTextWnd::StartEditEngine()
 
 ScInputHandler* pHdl = mpViewShell->GetInputHandler();
 if (pHdl)
+{
+bool bStarting = !pHdl->IsEditMode();
 pHdl->SetMode(SC_INPUT_TOP);
+if (bStarting)
+// necessary to sync SvxAutoCorrect behavior
+pHdl->MergeLanguageAttributes( *mpEditEngine);
+}
 
 SfxViewFrame* pViewFrm = SfxViewFrame::Current();
 if (pViewFrm)
diff --git a/sc/source/ui/inc/inputhdl.hxx b/sc/source/ui/inc/inputhdl.hxx
index 0301b0a72056..99fd54f299a2 100644
--- a/sc/source/ui/inc/inputhdl.hxx
+++ b/sc/source/ui/inc/inputhdl.hxx
@@ -180,6 +180,7 @@ public:
 const ScAddress& GetCursorPos() const   { return aCursorPos; }
 
 boolGetTextAndFields( ScEditEngineDefaulter& rDestEngine );
+voidMergeLanguageAttributes( ScEditEngineDefaulter& 
rDestEngine ) const;
 
 boolKeyInput( const KeyEvent& rKEvt, bool bStartEdit );
 voidEnterHandler( ScEnterMode nBlockMode = ScEnterMode::NORMAL 
);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-08-24 Thread Paul Trojahn
 include/oox/export/drawingml.hxx   |4 +-
 oox/source/export/drawingml.cxx|   15 +
 oox/source/export/shapes.cxx   |6 ++-
 sd/qa/unit/data/odp/tdf111798.odp  |binary
 sd/qa/unit/export-tests-ooxml2.cxx |   59 +
 5 files changed, 74 insertions(+), 10 deletions(-)

New commits:
commit 73ee631e58f392415f23e98460ff4b2f3a763d37
Author: Paul Trojahn 
Date:   Thu Aug 17 20:04:27 2017 +0200

tdf#111798 Fix deformed export of flipped custom shapes to pptx

Shapes were deformed because flipping wasn't considered when removing
the rotation. Also WriteShapeTransformation needs information about
flipping to convert the angle correctly.

Change-Id: I7d485e93c00e02b9ec6c73ad6ae2876e5bc6360a
Reviewed-on: https://gerrit.libreoffice.org/41462
Tested-by: Jenkins 
Reviewed-by: Tamás Zolnai 

diff --git a/include/oox/export/drawingml.hxx b/include/oox/export/drawingml.hxx
index 4ee841d33eb2..ecdad5b99b6a 100644
--- a/include/oox/export/drawingml.hxx
+++ b/include/oox/export/drawingml.hxx
@@ -189,8 +189,8 @@ public:
 const OUString& rURL, bool bRelPathToMedia = false , const Graphic 
*pGraphic=nullptr );
 void WriteBlipMode( const css::uno::Reference< css::beans::XPropertySet >& 
rXPropSet, const OUString& rURL );
 
-void WriteShapeTransformation( const css::uno::Reference< 
css::drawing::XShape >& rXShape,
-  sal_Int32 nXmlNamespace, bool bFlipH = false, bool bFlipV = 
false, bool bSuppressRotation = false );
+void WriteShapeTransformation(const css::uno::Reference< 
css::drawing::XShape >& rXShape,
+  sal_Int32 nXmlNamespace, bool bFlipH = false, bool bFlipV = 
false, bool bSuppressRotation = false, bool bSuppressFlipping = false );
 void WriteTransformation( const tools::Rectangle& rRectangle,
   sal_Int32 nXmlNamespace, bool bFlipH = false, bool bFlipV = 
false, sal_Int32 nRotation = 0 );
 
diff --git a/oox/source/export/drawingml.cxx b/oox/source/export/drawingml.cxx
index aec1aaecc6a5..3d5a7ace8262 100644
--- a/oox/source/export/drawingml.cxx
+++ b/oox/source/export/drawingml.cxx
@@ -1157,11 +1157,6 @@ void DrawingML::WriteStretch( const css::uno::Reference< 
css::beans::XPropertySe
 void DrawingML::WriteTransformation( const tools::Rectangle& rRect,
 sal_Int32 nXmlNamespace, bool bFlipH, bool bFlipV, sal_Int32 nRotation 
)
 {
-//OOXML flips shapes before rotating them.
-if(bFlipH)
-nRotation = nRotation * -1 + 6*360;
-if(bFlipV)
-nRotation = nRotation * -1 + 6*360;
 
 mpFS->startElementNS( nXmlNamespace, XML_xfrm,
   XML_flipH, bFlipH ? "1" : nullptr,
@@ -1183,7 +1178,7 @@ void DrawingML::WriteTransformation( const 
tools::Rectangle& rRect,
 mpFS->endElementNS( nXmlNamespace, XML_xfrm );
 }
 
-void DrawingML::WriteShapeTransformation( const Reference< XShape >& rXShape, 
sal_Int32 nXmlNamespace, bool bFlipH, bool bFlipV, bool bSuppressRotation  )
+void DrawingML::WriteShapeTransformation( const Reference< XShape >& rXShape, 
sal_Int32 nXmlNamespace, bool bFlipH, bool bFlipV, bool bSuppressRotation, bool 
bSuppressFlipping )
 {
 SAL_INFO("oox.shape",  "write shape transformation");
 
@@ -1220,6 +1215,14 @@ void DrawingML::WriteShapeTransformation( const 
Reference< XShape >& rXShape, sa
 if (xPropertySetInfo->hasPropertyByName("RotateAngle"))
 xPropertySet->getPropertyValue("RotateAngle") >>= nRotation;
 }
+
+// OOXML flips shapes before rotating them.
+if(bFlipH != bFlipV)
+nRotation = nRotation * -1 + 36000;
+
+if(bSuppressFlipping)
+bFlipH = bFlipV = false;
+
 WriteTransformation( tools::Rectangle( Point( aPos.X, aPos.Y ), Size( 
aSize.Width, aSize.Height ) ), nXmlNamespace, bFlipH, bFlipV, 
OOX_DRAWINGML_EXPORT_ROTATE_CLOCKWISIFY(nRotation) );
 }
 
diff --git a/oox/source/export/shapes.cxx b/oox/source/export/shapes.cxx
index b5d4a450b94b..df34231a225b 100644
--- a/oox/source/export/shapes.cxx
+++ b/oox/source/export/shapes.cxx
@@ -823,7 +823,7 @@ ShapeExport& ShapeExport::WriteCustomShape( const 
Reference< XShape >& xShape )
 
 if (bHasHandles && bCustGeom && pShape)
 {
-WriteShapeTransformation( xShape, XML_a ); // do not flip, polypolygon 
coordinates are flipped already
+WriteShapeTransformation( xShape, XML_a, bFlipH, bFlipV, false, true 
);// do not flip, polypolygon coordinates are flipped already
 tools::PolyPolygon aPolyPolygon( pShape->GetLineGeometry(true) );
 sal_Int32 nRotation = 0;
 // The RotateAngle property's value is independent from any flipping, 
and that's exactly what we need here.
@@ -831,8 +831,10 @@ ShapeExport& ShapeExport::WriteCustomShape( const 
Reference< XShape >& xShape )
 uno::Reference xPropertySetInfo = 
xPropertySet->getPropertySetInfo();
 if (xPropertySetInfo->hasPropertyByName("RotateAngle"))
 xPro

[Libreoffice-commits] core.git: Branch 'libreoffice-5-4-1' - external/libxml2

2017-08-24 Thread Michael Stahl
 
external/libxml2/0001-Fix-buffer-size-checks-in-xmlSnprintfElementContent.patch.1
  |  116 
 external/libxml2/0001-Fix-handling-of-parameter-entity-references.patch.1  
|  287 ++
 external/libxml2/0001-Fix-type-confusion-in-xmlValidateOneNamespace.patch.1
|   43 +
 
external/libxml2/0001-Increase-buffer-space-for-port-in-HTTP-redirect-supp.patch.1
 |   31 +
 external/libxml2/0001-Prevent-unwanted-external-entity-reference.patch.1   
|   35 +
 external/libxml2/UnpackedTarball_xml2.mk   
|5 
 6 files changed, 517 insertions(+)

New commits:
commit f86f98796c2a602c5ccd0ce046573978f9152f56
Author: Michael Stahl 
Date:   Wed Aug 23 12:10:38 2017 +0200

libxml2: bunch of CVE fixes

Change-Id: Ic786fef17cbdb574c342925a4c57875123ef3151
(cherry picked from commit 486d3b214c270fbd2651a0b0895d600e0f647750)
Reviewed-on: https://gerrit.libreoffice.org/41471
Reviewed-by: Markus Mohrhard 
Reviewed-by: Christian Lohmaier 
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git 
a/external/libxml2/0001-Fix-buffer-size-checks-in-xmlSnprintfElementContent.patch.1
 
b/external/libxml2/0001-Fix-buffer-size-checks-in-xmlSnprintfElementContent.patch.1
new file mode 100644
index ..047c88d943a7
--- /dev/null
+++ 
b/external/libxml2/0001-Fix-buffer-size-checks-in-xmlSnprintfElementContent.patch.1
@@ -0,0 +1,116 @@
+From 932cc9896ab41475d4aa429c27d9afd175959d74 Mon Sep 17 00:00:00 2001
+From: Nick Wellnhofer 
+Date: Sat, 3 Jun 2017 02:01:29 +0200
+Subject: [PATCH] Fix buffer size checks in xmlSnprintfElementContent
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+xmlSnprintfElementContent failed to correctly check the available
+buffer space in two locations.
+
+Fixes bug 781333 (CVE-2017-9047) and bug 781701 (CVE-2017-9048).
+
+Thanks to Marcel Böhme and Thuan Pham for the report.
+---
+ result/valid/781333.xml |  5 +
+ result/valid/781333.xml.err |  3 +++
+ result/valid/781333.xml.err.rdr |  6 ++
+ test/valid/781333.xml   |  4 
+ valid.c | 20 +++-
+ 5 files changed, 29 insertions(+), 9 deletions(-)
+ create mode 100644 result/valid/781333.xml
+ create mode 100644 result/valid/781333.xml.err
+ create mode 100644 result/valid/781333.xml.err.rdr
+ create mode 100644 test/valid/781333.xml
+
+diff --git a/result/valid/781333.xml b/result/valid/781333.xml
+new file mode 100644
+index ..45dc451d
+--- /dev/null
 b/result/valid/781333.xml
+@@ -0,0 +1,5 @@
++
++
++]>
++
+diff --git a/result/valid/781333.xml.err b/result/valid/781333.xml.err
+new file mode 100644
+index ..b401b49a
+--- /dev/null
 b/result/valid/781333.xml.err
+@@ -0,0 +1,3 @@
++./test/valid/781333.xml:4: element a: validity error : Element a content does 
not follow the DTD, expecting ( ..., got 
++
++^
+diff --git a/result/valid/781333.xml.err.rdr b/result/valid/781333.xml.err.rdr
+new file mode 100644
+index ..5ff56992
+--- /dev/null
 b/result/valid/781333.xml.err.rdr
+@@ -0,0 +1,6 @@
++./test/valid/781333.xml:4: element a: validity error : Element a content does 
not follow the DTD, expecting ( ..., got 
++
++^
++./test/valid/781333.xml:5: element a: validity error : Element a content does 
not follow the DTD, Expecting more child
++
++^
+diff --git a/test/valid/781333.xml b/test/valid/781333.xml
+new file mode 100644
+index ..b29e5a68
+--- /dev/null
 b/test/valid/781333.xml
+@@ -0,0 +1,4 @@
++
++]>
++
+diff --git a/valid.c b/valid.c
+index 19f84b82..9b2df56a 100644
+--- a/valid.c
 b/valid.c
+@@ -1262,22 +1262,23 @@ xmlSnprintfElementContent(char *buf, int size, 
xmlElementContentPtr content, int
+ case XML_ELEMENT_CONTENT_PCDATA:
+ strcat(buf, "#PCDATA");
+   break;
+-  case XML_ELEMENT_CONTENT_ELEMENT:
++  case XML_ELEMENT_CONTENT_ELEMENT: {
++int qnameLen = xmlStrlen(content->name);
++
++  if (content->prefix != NULL)
++qnameLen += xmlStrlen(content->prefix) + 1;
++  if (size - len < qnameLen + 10) {
++  strcat(buf, " ...");
++  return;
++  }
+   if (content->prefix != NULL) {
+-  if (size - len < xmlStrlen(content->prefix) + 10) {
+-  strcat(buf, " ...");
+-  return;
+-  }
+   strcat(buf, (char *) content->prefix);
+   strcat(buf, ":");
+   }
+-  if (size - len < xmlStrlen(content->name) + 10) {
+-  strcat(buf, " ...");
+-  return;
+-  }
+   if (content->name != NULL)
+   strcat(buf, (char *) content->name);
+   break;
++}
+   case XML_ELEMENT_CONTENT_SEQ:
+   if ((content->c1->type == XML_ELEMENT_CONTENT_OR) ||
+   (content->c1->type == XML_ELEM

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

2017-08-24 Thread Gabor Kelemen
 sw/source/ui/misc/glosbib.cxx |   10 --
 1 file changed, 8 insertions(+), 2 deletions(-)

New commits:
commit dffdc0b1995e2b24304ce0651ca886bbf9cf4f95
Author: Gabor Kelemen 
Date:   Thu Aug 17 01:02:26 2017 +0200

tdf#111870 Translate the "My AutoText" string in the Edit Categories window

Change-Id: I483fa998d140c8458948fe19c5338e7af0d32527
Reviewed-on: https://gerrit.libreoffice.org/41234
Tested-by: Jenkins 
Reviewed-by: Katarina Behrens 

diff --git a/sw/source/ui/misc/glosbib.cxx b/sw/source/ui/misc/glosbib.cxx
index 401d4901859b..a3db641950c8 100644
--- a/sw/source/ui/misc/glosbib.cxx
+++ b/sw/source/ui/misc/glosbib.cxx
@@ -94,6 +94,9 @@ SwGlossaryGroupDlg::SwGlossaryGroupDlg(vcl::Window * pParent,
 m_pPathLB->Enable();
 
 const size_t nCount = pHdl->GetGroupCnt();
+/* tdf#111870 "My AutoText" comes from mytexts.bau but should be translated
+   here as well, see also SwGlossaryDlg::Init */
+const OUString sMyAutoTextEnglish("My AutoText");
 for( size_t i = 0; i < nCount; ++i)
 {
 OUString sTitle;
@@ -102,9 +105,12 @@ SwGlossaryGroupDlg::SwGlossaryGroupDlg(vcl::Window * 
pParent,
 continue;
 GlosBibUserData* pData = new GlosBibUserData;
 pData->sGroupName = sGroup;
-pData->sGroupTitle = sTitle;
+if ( sTitle == sMyAutoTextEnglish )
+pData->sGroupTitle = SwResId(STR_MY_AUTOTEXT);
+else
+pData->sGroupTitle = sTitle;
 pData->sPath = m_pPathLB->GetEntry(sGroup.getToken(1, 
GLOS_DELIM).toInt32());
-SvTreeListEntry* pEntry = m_pGroupTLB->InsertEntry(sTitle + "\t" + 
pData->sPath);
+SvTreeListEntry* pEntry = m_pGroupTLB->InsertEntry(pData->sGroupTitle 
+ "\t" + pData->sPath);
 pEntry->SetUserData(pData);
 
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-08-24 Thread Tamás Zolnai
 sd/qa/unit/data/pptx/tdf108926.ppt |binary
 sd/qa/unit/import-tests.cxx|   19 ++
 sd/source/filter/ppt/pptin.cxx |  324 ++---
 3 files changed, 179 insertions(+), 164 deletions(-)

New commits:
commit b766cab621690a3aaf4e10fcf925ebab1c1925b8
Author: Tamás Zolnai 
Date:   Thu Aug 24 14:29:00 2017 +0200

tdf#108926: Notes placeholder text boxes not shown in PPTs

Regression from:
acb2943c8125f4ceed74f35f31776929dedeb8d8

Call this placeholder related code not only for
normal slide pages.

Change-Id: Iae185ac7e5d2505554692045516cc51dbdcd735b
Reviewed-on: https://gerrit.libreoffice.org/41517
Tested-by: Jenkins 
Reviewed-by: Tamás Zolnai 

diff --git a/sd/qa/unit/data/pptx/tdf108926.ppt 
b/sd/qa/unit/data/pptx/tdf108926.ppt
new file mode 100755
index ..c8455e226e1b
Binary files /dev/null and b/sd/qa/unit/data/pptx/tdf108926.ppt differ
diff --git a/sd/qa/unit/import-tests.cxx b/sd/qa/unit/import-tests.cxx
index 7546689852a5..f9177c6493d3 100644
--- a/sd/qa/unit/import-tests.cxx
+++ b/sd/qa/unit/import-tests.cxx
@@ -171,6 +171,7 @@ public:
 void testSmartArtRotation();
 void testTdf109223();
 void testTdf109187();
+void testTdf108926();
 
 bool checkPattern(sd::DrawDocShellRef const & rDocRef, int nShapeNumber, 
std::vector& rExpected);
 void testPatternImport();
@@ -250,6 +251,7 @@ public:
 CPPUNIT_TEST(testSmartArtRotation);
 CPPUNIT_TEST(testTdf109223);
 CPPUNIT_TEST(testTdf109187);
+CPPUNIT_TEST(testTdf108926);
 
 CPPUNIT_TEST_SUITE_END();
 };
@@ -2431,6 +2433,23 @@ void SdImportTest::testTdf109187()
 xDocShRef->DoClose();
 }
 
+void SdImportTest::testTdf108926()
+{
+sd::DrawDocShellRef xDocShRef = 
loadURL(m_directories.getURLFromSrc("sd/qa/unit/data/pptx/tdf108926.ppt"), PPT);
+uno::Reference< presentation::XPresentationPage > xPage (getPage(0, 
xDocShRef), uno::UNO_QUERY_THROW);
+uno::Reference< drawing::XDrawPage > xNotesPage (xPage->getNotesPage(), 
uno::UNO_QUERY_THROW);
+CPPUNIT_ASSERT_EQUAL(static_cast(2), xNotesPage->getCount());
+
+// Second object should be imported as an empty presentation shape
+uno::Reference< beans::XPropertySet > 
xPresentationShape(xNotesPage->getByIndex(1), uno::UNO_QUERY);
+CPPUNIT_ASSERT(xPresentationShape.is());
+bool bIsEmptyPresObject = false;
+xPresentationShape->getPropertyValue( "IsEmptyPresentationObject" )  >>= 
bIsEmptyPresObject;
+CPPUNIT_ASSERT(bIsEmptyPresObject);
+
+xDocShRef->DoClose();
+}
+
 CPPUNIT_TEST_SUITE_REGISTRATION(SdImportTest);
 
 CPPUNIT_PLUGIN_IMPLEMENT();
diff --git a/sd/source/filter/ppt/pptin.cxx b/sd/source/filter/ppt/pptin.cxx
index 7bec40a94c6c..ce06ae52a268 100644
--- a/sd/source/filter/ppt/pptin.cxx
+++ b/sd/source/filter/ppt/pptin.cxx
@@ -2355,215 +2355,211 @@ SdrObject* ImplSdPPTImport::ApplyTextObj( PPTTextObj* 
pTextObj, SdrTextObj* pObj
 {
 sal_uInt32 nPlacementId = pPlaceHolder->nPlacementId;
 PptPlaceholder nPlaceholderId = pPlaceHolder->nPlaceholderId;
-
-if ( eAktPageKind == PPT_SLIDEPAGE )
+PresObjKind ePresObjKind = PRESOBJ_NONE;
+boolbEmptyPresObj = true;
+boolbVertical = false;
+if ( ( pTextObj->GetShapeType() == mso_sptRectangle ) || ( 
pTextObj->GetShapeType() == mso_sptTextBox ) )
 {
-PresObjKind ePresObjKind = PRESOBJ_NONE;
-boolbEmptyPresObj = true;
-boolbVertical = false;
-if ( ( pTextObj->GetShapeType() == mso_sptRectangle ) || ( 
pTextObj->GetShapeType() == mso_sptTextBox ) )
+//if a placeholder with some custom attribute,the pTextObj 
will keep those attr,whose text size is zero,
+//so sdPage should renew a PresObj to process placeholder.
+bEmptyPresObj = ( pTextObj->Count() == 0 ) || ( 
pTextObj->Count() == 1 && pTextObj->First()->GetTextSize() == 0 );
+switch ( nPlaceholderId )
 {
-//if a placeholder with some custom attribute,the 
pTextObj will keep those attr,whose text size is zero,
-//so sdPage should renew a PresObj to process 
placeholder.
-bEmptyPresObj = ( pTextObj->Count() == 0 ) || ( 
pTextObj->Count() == 1 && pTextObj->First()->GetTextSize() == 0 );
-switch ( nPlaceholderId )
+case PptPlaceholder::NOTESBODY :
ePresObjKind = PRESOBJ_NOTES;   break;
+case PptPlaceholder::VERTICALTEXTTITLE :
+bVertical = true;
+SAL_FALLTHROUGH;
+case PptPlaceholder::TITLE :
ePresObjKind = PRESOBJ_TITLE;   break;
+case PptPlaceho

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

2017-08-24 Thread Yousuf Philips
 sw/uiconfig/swriter/menubar/menubar.xml |1 -
 1 file changed, 1 deletion(-)

New commits:
commit 04e719efbdb551d7c72fd0cc690f76c96bb66960
Author: Yousuf Philips 
Date:   Tue Jun 20 18:32:48 2017 +0400

tdf#91781 Removal of Format > Columns

Columns tab is available in Format > Page and in Format > Section >
Options which makes this menu entry redundant

Change-Id: Ia637b7fe5c95ba075bad921cacb9d2e41e159cb8
Reviewed-on: https://gerrit.libreoffice.org/39017
Tested-by: Jenkins 
Reviewed-by: Heiko Tietze 

diff --git a/sw/uiconfig/swriter/menubar/menubar.xml 
b/sw/uiconfig/swriter/menubar/menubar.xml
index 83a52c0ec1fb..ce25da98b5a2 100644
--- a/sw/uiconfig/swriter/menubar/menubar.xml
+++ b/sw/uiconfig/swriter/menubar/menubar.xml
@@ -453,7 +453,6 @@
   
   
   
-  
   
   
   
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Changes to 'refs/tags/libreoffice-5.4.1.2'

2017-08-24 Thread Andras Timar
Tag 'libreoffice-5.4.1.2' created by Andras Timar  
at 2017-08-24 13:28 +

libreoffice-5.4.1.2

Changes since libreoffice-5.4.0.2-13:
---
 0 files changed
---
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-08-24 Thread Grzegorz Araminowicz
 oox/source/drawingml/diagram/diagram.cxx|7 +
 oox/source/drawingml/diagram/diagram.hxx|5 +
 oox/source/drawingml/diagram/diagramlayoutatoms.cxx |   71 +++-
 oox/source/drawingml/diagram/diagramlayoutatoms.hxx |6 +
 sd/qa/unit/data/pptx/smartart-dir.pptx  |binary
 sd/qa/unit/data/pptx/smartart-maxdepth.pptx |binary
 sd/qa/unit/import-tests.cxx |   31 
 7 files changed, 102 insertions(+), 18 deletions(-)

New commits:
commit f6850953dfd0ff0847e5a24bf921456a12ae47e4
Author: Grzegorz Araminowicz 
Date:   Wed Aug 23 12:23:58 2017 +0200

SmartArt: implement dir and maxDepth if node functions

Change-Id: I4ef05b5bab1188cf349d1c7f5bbc9022bc79b21c
Reviewed-on: https://gerrit.libreoffice.org/41452
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/oox/source/drawingml/diagram/diagram.cxx 
b/oox/source/drawingml/diagram/diagram.cxx
index 2f33d9a79002..e0a8256e997d 100644
--- a/oox/source/drawingml/diagram/diagram.cxx
+++ b/oox/source/drawingml/diagram/diagram.cxx
@@ -65,8 +65,9 @@ void Point::dump() const
 
 } // dgm namespace
 
-DiagramData::DiagramData()
-: mpFillProperties( new FillProperties )
+DiagramData::DiagramData() :
+mpFillProperties( new FillProperties ),
+mnMaxDepth(0)
 {
 }
 
@@ -298,6 +299,8 @@ void Diagram::build(  )
 const sal_Int32 nDepth=calcDepth(aPresOfNodeIterCalcLevel->first,
  getData()->getConnections());
 aPresOfNodeIterCalcLevel->second = nDepth != 0 ? nDepth : -1;
+if (nDepth > getData()->getMaxDepth())
+getData()->setMaxDepth(nDepth);
 ++aPresOfNodeIterCalcLevel;
 }
 
diff --git a/oox/source/drawingml/diagram/diagram.hxx 
b/oox/source/drawingml/diagram/diagram.hxx
index f31757f6ad91..3604d2607344 100644
--- a/oox/source/drawingml/diagram/diagram.hxx
+++ b/oox/source/drawingml/diagram/diagram.hxx
@@ -182,6 +182,10 @@ public:
 ::std::vector &getExtDrawings()
 { return maExtDrawings; }
 const dgm::Point* getRootPoint() const;
+sal_Int32 getMaxDepth() const
+{ return mnMaxDepth; }
+void setMaxDepth(sal_Int32 nDepth)
+{ mnMaxDepth = nDepth; }
 void dump() const;
 private:
 FillPropertiesPtr mpFillProperties;
@@ -191,6 +195,7 @@ private:
 PointsNameMap maPointsPresNameMap;
 ConnectionNameMap maConnectionNameMap;
 StringMap maPresOfNameMap;
+sal_Int32 mnMaxDepth;
 };
 
 typedef std::shared_ptr< DiagramData > DiagramDataPtr;
diff --git a/oox/source/drawingml/diagram/diagramlayoutatoms.cxx 
b/oox/source/drawingml/diagram/diagramlayoutatoms.cxx
index 7bfe4996edef..791381ccda87 100644
--- a/oox/source/drawingml/diagram/diagramlayoutatoms.cxx
+++ b/oox/source/drawingml/diagram/diagramlayoutatoms.cxx
@@ -29,7 +29,6 @@
 #include "drawingml/textparagraph.hxx"
 #include "drawingml/textrun.hxx"
 #include "drawingml/customshapeproperties.hxx"
-#include "layoutnodecontext.hxx"
 
 using namespace ::com::sun::star;
 using namespace ::com::sun::star::uno;
@@ -63,17 +62,17 @@ ConditionAttr::ConditionAttr()
 : mnFunc( 0 )
 , mnArg( 0 )
 , mnOp( 0 )
+, mnVal( 0 )
 {
-
 }
 
 void ConditionAttr::loadFromXAttr( const Reference< XFastAttributeList >& 
xAttr )
 {
 mnFunc = xAttr->getOptionalValueToken( XML_func, 0 );
-// mnArg will be -1 for "none" or any other unknown value
-mnArg = LayoutNodeContext::tagToVarIdx( xAttr->getOptionalValueToken( 
XML_arg, XML_none ) );
+mnArg = xAttr->getOptionalValueToken( XML_arg, XML_none );
 mnOp = xAttr->getOptionalValueToken( XML_op, 0 );
 msVal = xAttr->getOptionalValue( XML_val );
+mnVal = xAttr->getOptionalValueToken( XML_val, 0 );
 }
 
 void LayoutAtom::dump(int level)
@@ -123,15 +122,36 @@ bool ConditionAtom::compareResult(sal_Int32 nOperator, 
sal_Int32 nFirst, sal_Int
 }
 }
 
-sal_Int32 ConditionAtom::getNodeCount() const
+bool ConditionAtom::compareResult(sal_Int32 nOperator, const OUString& sFirst, 
const OUString& sSecond)
+{
+switch (nOperator)
+{
+case XML_equ: return sFirst == sSecond;
+case XML_neq: return sFirst != sSecond;
+default:
+SAL_WARN("oox.drawingml", "unsupported operator: " << nOperator);
+return false;
+}
+}
+
+const dgm::Point* ConditionAtom::getPresNode() const
 {
-sal_Int32 nCount = 0;
 const DiagramData::PointsNameMap& rPoints = 
mrLayoutNode.getDiagram().getData()->getPointsPresNameMap();
 DiagramData::PointsNameMap::const_iterator aDataNode = 
rPoints.find(mrLayoutNode.getName());
 if (aDataNode != rPoints.end())
 {
-SAL_WARN_IF(aDataNode->second.size() > 1, "oox.drawingml", "multiple 
nodes found; calculating cnt for first one");
-const dgm::Point* pPoint = aDataNode->second.front();
+SAL_WARN_IF(aDataNode->second.size() > 1, "oox.drawingml", "multiple 
nodes 

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

2017-08-24 Thread Noel Grandin
 sc/inc/arealink.hxx  |2 +-
 sc/inc/datamapper.hxx|2 +-
 sc/inc/externalrefmgr.hxx|6 +++---
 sc/inc/tablink.hxx   |2 +-
 sc/source/ui/cctrl/checklistmenu.cxx |2 +-
 sc/source/ui/condformat/colorformat.cxx  |4 ++--
 sc/source/ui/condformat/condformatdlgentry.cxx   |4 ++--
 sc/source/ui/dataprovider/dataprovider.cxx   |2 +-
 sc/source/ui/dataprovider/datatransformation.cxx |2 +-
 sc/source/ui/dbgui/PivotLayoutDialog.cxx |   10 +-
 sc/source/ui/dbgui/PivotLayoutTreeListBase.cxx   |4 ++--
 sc/source/ui/dbgui/validate.cxx  |6 +++---
 sc/source/ui/docshell/arealink.cxx   |2 +-
 sc/source/ui/docshell/dbdocfun.cxx   |2 +-
 sc/source/ui/docshell/dbdocimp.cxx   |2 +-
 sc/source/ui/docshell/docfunc.cxx|4 ++--
 sc/source/ui/docshell/docsh.cxx  |2 +-
 sc/source/ui/docshell/docsh4.cxx |2 +-
 sc/source/ui/docshell/editable.cxx   |   16 
 sc/source/ui/docshell/externalrefmgr.cxx |   14 +++---
 sc/source/ui/docshell/impex.cxx  |4 ++--
 sc/source/ui/docshell/tablink.cxx|2 +-
 sc/source/ui/drawfunc/drawsh.cxx |2 +-
 sc/source/ui/drawfunc/drawsh5.cxx|6 +++---
 sc/source/ui/drawfunc/drtxtob.cxx|2 +-
 sc/source/ui/drawfunc/drtxtob2.cxx   |2 +-
 sc/source/ui/drawfunc/fuconrec.cxx   |4 ++--
 sc/source/ui/drawfunc/fuconstr.cxx   |2 +-
 sc/source/ui/drawfunc/fudraw.cxx |4 ++--
 sc/source/ui/drawfunc/fuins1.cxx |6 +++---
 sc/source/ui/drawfunc/fuins2.cxx |2 +-
 sc/source/ui/drawfunc/fupoor.cxx |2 +-
 sc/source/ui/drawfunc/fusel.cxx  |2 +-
 sc/source/ui/drawfunc/fusel2.cxx |2 +-
 sc/source/ui/drawfunc/futext.cxx |2 +-
 sc/source/ui/inc/PivotLayoutDialog.hxx   |8 
 sc/source/ui/inc/PivotLayoutTreeListBase.hxx |4 ++--
 sc/source/ui/inc/checklistmenu.hxx   |2 +-
 sc/source/ui/inc/datatransformation.hxx  |2 +-
 sc/source/ui/inc/dbdocfun.hxx|4 ++--
 sc/source/ui/inc/docsh.hxx   |2 +-
 sc/source/ui/inc/drawsh.hxx  |6 +++---
 sc/source/ui/inc/drtxtob.hxx |4 ++--
 sc/source/ui/inc/editable.hxx|   16 
 sc/source/ui/inc/fuconrec.hxx|2 +-
 sc/source/ui/inc/fuconstr.hxx|2 +-
 sc/source/ui/inc/fudraw.hxx  |2 +-
 sc/source/ui/inc/fuinsert.hxx|4 ++--
 sc/source/ui/inc/fupoor.hxx  |2 +-
 sc/source/ui/inc/fusel.hxx   |4 ++--
 sc/source/ui/inc/validate.hxx|   14 +++---
 51 files changed, 106 insertions(+), 106 deletions(-)

New commits:
commit c137c2a46cd2276938d572c97fb111b9aa31cdfd
Author: Noel Grandin 
Date:   Thu Aug 24 12:24:44 2017 +0200

loplugin:constparam in sc part5

Change-Id: Iba249d4d8f7cc686941106983031bae53c7528e1
Reviewed-on: https://gerrit.libreoffice.org/41510
Reviewed-by: Noel Grandin 
Tested-by: Noel Grandin 

diff --git a/sc/inc/arealink.hxx b/sc/inc/arealink.hxx
index 69b6b5a8d40f..5391d320ce09 100644
--- a/sc/inc/arealink.hxx
+++ b/sc/inc/arealink.hxx
@@ -42,7 +42,7 @@ private:
 boolbAddUndo;
 boolbInCreate;
 boolbDoInsert;  // is set to FALSE for first update
-static bool FindExtRange( ScRange& rRange, ScDocument* pSrcDoc, const 
OUString& rAreaName );
+static bool FindExtRange( ScRange& rRange, const ScDocument* pSrcDoc, 
const OUString& rAreaName );
 
 public:
 ScAreaLink( SfxObjectShell* pShell, const OUString& rFile,
diff --git a/sc/inc/datamapper.hxx b/sc/inc/datamapper.hxx
index 55390fc5a39b..f872f50143b8 100644
--- a/sc/inc/datamapper.hxx
+++ b/sc/inc/datamapper.hxx
@@ -85,7 +85,7 @@ public:
 const OUString& getID() const;
 double getUpdateFrequency() const;
 OUString getDBName() const;
-void setDBData(ScDBData* pDBData);
+void setDBData(const ScDBData* pDBData);
 
 void refresh(ScDocument* pDoc, bool bDeterministic = false);
 };
diff --git a/sc/inc/externalrefmgr.hxx b/sc/inc/externalrefmgr.hxx
index 5a63a0030797..30bc8136b034 100644
--- a/sc/inc/externalrefmgr.hxx
+++ b/sc/inc/externalrefmgr.hxx
@@ -398,7 +398,7 @@ public:
 class SC_DLLPUBLIC ApiGuard
 {
 public:
-ApiGuard(ScDocument* pDoc);
+ApiGuard(const ScDocument* pDoc);
 ~ApiGuard();
 private:
 ScExternalRefManager* m

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

2017-08-24 Thread Michael Stahl
 xmloff/source/style/xmlnumfi.cxx |5 +
 1 file changed, 5 insertions(+)

New commits:
commit 9fe857b7bd126ff4856fc4689d375881653b97a2
Author: Michael Stahl 
Date:   Thu Aug 24 13:56:35 2017 +0200

tdf#111934 xmloff: ODF import: self-referential conditional style crash

It's invalid input and also causes stack overflow.

Change-Id: Ie0b9dcaefcfcf254326151f345f4802ed66b994d

diff --git a/xmloff/source/style/xmlnumfi.cxx b/xmloff/source/style/xmlnumfi.cxx
index 25d54ecf4252..6a3d997f8fe5 100644
--- a/xmloff/source/style/xmlnumfi.cxx
+++ b/xmloff/source/style/xmlnumfi.cxx
@@ -1659,6 +1659,11 @@ sal_Int32 
SvXMLNumFormatContext::CreateAndInsert(SvNumberFormatter* pFormatter)
 {
 SvXMLNumFormatContext* pStyle = const_cast( 
static_cast(pStyles->FindStyleChildContext(
 XML_STYLE_FAMILY_DATA_STYLE, aMyConditions[i].sMapName)));
+if (this == pStyle)
+{
+SAL_INFO("xmloff.style", "invalid style:map references containing 
style");
+pStyle = nullptr;
+}
 if (pStyle)
 {
 if (pStyle->PrivateGetKey() > -1) // don't reset pStyle's 
bRemoveAfterUse flag
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-08-24 Thread Justin Luth
 sw/qa/extras/rtfexport/rtfexport3.cxx   |4 ++--
 sw/source/filter/ww8/rtfattributeoutput.cxx |2 ++
 2 files changed, 4 insertions(+), 2 deletions(-)

New commits:
commit 49877dc91b2f91baa656facd462ac1b1e832f182
Author: Justin Luth 
Date:   Tue Jul 25 19:55:14 2017 -0400

sw:rtf - timely export of footnote char properties

related to tdf#82173 and tdf#108949.
Write the footnote character properties now - otherwise they will
affect text incorrectly later on.

Change-Id: I33af1586a51f9028ef5b0d646e4ba3d746c77a2c
Reviewed-on: https://gerrit.libreoffice.org/40435
Tested-by: Jenkins 
Reviewed-by: Miklos Vajna 

diff --git a/sw/qa/extras/rtfexport/rtfexport3.cxx 
b/sw/qa/extras/rtfexport/rtfexport3.cxx
index 35d7fc9861ec..fbf63740f444 100644
--- a/sw/qa/extras/rtfexport/rtfexport3.cxx
+++ b/sw/qa/extras/rtfexport/rtfexport3.cxx
@@ -33,10 +33,10 @@ DECLARE_RTFEXPORT_TEST(testTdf108949, 
"tdf108949_footnoteCharFormat.odt")
 uno::Reference xFootnoteText;
 xFootnotes->getByIndex(0) >>= xFootnoteText;
 // This was green (0x00A800), the character property of the footnote 
character, not the footnote text
-//CPPUNIT_ASSERT_EQUAL(sal_Int32(0x00), 
getProperty(getRun(getParagraphOfText(1, xFootnoteText),1), 
"CharColor"));
+CPPUNIT_ASSERT_MESSAGE ("Footnote Text color",sal_Int32(0x00) >= 
getProperty(getRun(getParagraphOfText(1, xFootnoteText),1), 
"CharColor"));
 }
 
-DECLARE_RTFIMPORT_TEST(testTdf108949_footnote, "tdf108949_footnote.rtf")
+DECLARE_RTFEXPORT_TEST(testTdf108949_footnote, "tdf108949_footnote.rtf")
 {
 CPPUNIT_ASSERT_EQUAL_MESSAGE("Paragraph Numbering style", OUString(), 
getProperty(getParagraph(2), "NumberingStyleName"));
 
diff --git a/sw/source/filter/ww8/rtfattributeoutput.cxx 
b/sw/source/filter/ww8/rtfattributeoutput.cxx
index 5a1cd9252921..acb9385ebc6b 100644
--- a/sw/source/filter/ww8/rtfattributeoutput.cxx
+++ b/sw/source/filter/ww8/rtfattributeoutput.cxx
@@ -2657,6 +2657,8 @@ void RtfAttributeOutput::TextFootnote_Impl(const 
SwFormatFootnote& rFootnote)
 SAL_INFO("sw.rtf", OSL_THIS_FUNC << " start");
 
 m_aRun->append("{" OOO_STRING_SVTOOLS_RTF_SUPER " ");
+EndRunProperties(nullptr);
+m_aRun->append(' ');
 WriteTextFootnoteNumStr(rFootnote);
 m_aRun->append("{" OOO_STRING_SVTOOLS_RTF_IGNORE 
OOO_STRING_SVTOOLS_RTF_FOOTNOTE);
 if (rFootnote.IsEndNote() || m_rExport.m_pDoc->GetFootnoteInfo().ePos == 
FTNPOS_CHAPTER)
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-08-24 Thread Miklos Vajna
 vcl/Library_vcl.mk|8 
 vcl/source/gdi/pdfwriter_impl.cxx |   14 --
 2 files changed, 22 deletions(-)

New commits:
commit b23cd7ea0607ea1c0ec2022e80349d96368e3923
Author: Miklos Vajna 
Date:   Thu Aug 24 09:29:03 2017 +0200

vcl PDF export: remove no longer direct NSS dependency

The relevant code has been moved to svl.

Change-Id: Id2b77c797745af21d3d5f954357a02d54fb67908
Reviewed-on: https://gerrit.libreoffice.org/41498
Reviewed-by: Miklos Vajna 
Tested-by: Jenkins 

diff --git a/vcl/Library_vcl.mk b/vcl/Library_vcl.mk
index 00cbf3b6a23e..f6eaae2d4e13 100644
--- a/vcl/Library_vcl.mk
+++ b/vcl/Library_vcl.mk
@@ -63,14 +63,6 @@ $(eval $(call gb_Library_use_externals,vcl,\
$(if $(filter PDFIUM,$(BUILD_TYPE)),pdfium) \
 ))
 
-ifeq ($(TLS),NSS)
-$(eval $(call gb_Library_use_externals,vcl,\
-   $(if $(filter-out IOS WNT,$(OS)), \
-   nss3 \
-   plc4) \
-))
-endif
-
 $(eval $(call gb_Library_use_libraries,vcl,\
 $(call gb_Helper_optional,BREAKPAD, \
crashreport) \
diff --git a/vcl/source/gdi/pdfwriter_impl.cxx 
b/vcl/source/gdi/pdfwriter_impl.cxx
index 63d9848e2a22..9e3db7068a25 100644
--- a/vcl/source/gdi/pdfwriter_impl.cxx
+++ b/vcl/source/gdi/pdfwriter_impl.cxx
@@ -85,20 +85,6 @@
 
 #include "pdfwriter_impl.hxx"
 
-#if HAVE_FEATURE_NSS && !defined(_WIN32)
-// NSS headers for PDF signing
-#include "nss.h"
-#include "cert.h"
-#include "hasht.h"
-#include "secerr.h"
-#include "sechash.h"
-#include "cms.h"
-#include "cmst.h"
-
-// We use curl for RFC3161 time stamp requests
-#include 
-#endif
-
 #ifdef _WIN32
 // WinCrypt headers for PDF signing
 // Note: this uses Windows 7 APIs and requires the relevant data types
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: solenv/gbuild

2017-08-24 Thread Stephan Bergmann
 solenv/gbuild/platform/com_GCC_defs.mk |2 ++
 1 file changed, 2 insertions(+)

New commits:
commit 30935cf8adf8747b46ea485ba4a320b542a4f7a5
Author: Stephan Bergmann 
Date:   Thu Aug 24 14:34:39 2017 +0200

Explicitly specify -finput-charset=UTF-8 for GCC/Clang

...after 9a7d2d72cef7ff14a020c1024fbff8c00e4e4aff "tell msvc our source 
code is
written using utf-8" did the equivalent for MSVC.  That should cause the C++
string literal u8"ßa" (in vcl/qa/cppunit/mnemonic.cxx) introduced with that
commit to reliably be interpreted as intended by all our toolchains.

Change-Id: Ibbda8588a3ca66d1c2764f70aa999fad243f1bb1

diff --git a/solenv/gbuild/platform/com_GCC_defs.mk 
b/solenv/gbuild/platform/com_GCC_defs.mk
index 97a2538b1895..793e5d34f656 100644
--- a/solenv/gbuild/platform/com_GCC_defs.mk
+++ b/solenv/gbuild/platform/com_GCC_defs.mk
@@ -55,6 +55,7 @@ gb_CFLAGS_COMMON := \
-Wstrict-prototypes \
-Wundef \
-Wunused-macros \
+   -finput-charset=UTF-8 \
-fmessage-length=0 \
-fno-common \
-pipe \
@@ -67,6 +68,7 @@ gb_CXXFLAGS_COMMON := \
-Wextra \
-Wundef \
-Wunused-macros \
+   -finput-charset=UTF-8 \
-fmessage-length=0 \
-fno-common \
-pipe \
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-08-24 Thread Noel Grandin
 framework/source/uiconfiguration/CommandImageResolver.cxx |   15 
++---
 framework/source/uiconfiguration/CommandImageResolver.hxx |3 +
 framework/source/uiconfiguration/moduleuiconfigurationmanager.cxx |   16 
+-
 3 files changed, 9 insertions(+), 25 deletions(-)

New commits:
commit f9acc8649d4a31ddd84832124e0413d4f1fa1710
Author: Noel Grandin 
Date:   Wed Aug 23 15:15:20 2017 +0200

loplugin:useuniqueptr in framework

Change-Id: I89bcfaa605251912bd1f979b38f12e983dcaf76e
Reviewed-on: https://gerrit.libreoffice.org/41512
Tested-by: Jenkins 
Reviewed-by: Noel Grandin 

diff --git a/framework/source/uiconfiguration/CommandImageResolver.cxx 
b/framework/source/uiconfiguration/CommandImageResolver.cxx
index 98aec12e3b4c..12d902d4bfd5 100644
--- a/framework/source/uiconfiguration/CommandImageResolver.cxx
+++ b/framework/source/uiconfiguration/CommandImageResolver.cxx
@@ -64,14 +64,10 @@ OUString lclConvertToCanonicalName(const OUString& 
rFileName)
 
 CommandImageResolver::CommandImageResolver()
 {
-for (ImageList*& rp : m_pImageList)
-rp = nullptr;
 }
 
 CommandImageResolver::~CommandImageResolver()
 {
-for (ImageList* p : m_pImageList)
-delete p;
 }
 
 bool CommandImageResolver::registerCommands(Sequence& 
aCommandSequence)
@@ -129,20 +125,17 @@ ImageList* CommandImageResolver::getImageList(ImageType 
nImageType)
 if (sIconTheme != m_sIconTheme)
 {
 m_sIconTheme = sIconTheme;
-for (ImageList*& rp : m_pImageList)
-{
-delete rp;
-rp = nullptr;
-}
+for (auto& rp : m_pImageList)
+rp.reset();
 }
 
 if (!m_pImageList[nImageType])
 {
 OUString sIconPath = 
OUString::createFromAscii(ImageType_Prefixes[nImageType]);
-m_pImageList[nImageType] = new ImageList(m_aImageNameVector, 
sIconPath);
+m_pImageList[nImageType].reset( new ImageList(m_aImageNameVector, 
sIconPath) );
 }
 
-return m_pImageList[nImageType];
+return m_pImageList[nImageType].get();
 }
 
 Image CommandImageResolver::getImageFromCommandURL(ImageType nImageType, const 
OUString& rCommandURL)
diff --git a/framework/source/uiconfiguration/CommandImageResolver.hxx 
b/framework/source/uiconfiguration/CommandImageResolver.hxx
index d7788e4e49bf..8d5a3c9ef309 100644
--- a/framework/source/uiconfiguration/CommandImageResolver.hxx
+++ b/framework/source/uiconfiguration/CommandImageResolver.hxx
@@ -18,6 +18,7 @@
 
 #include "ImageList.hxx"
 
+#include 
 #include 
 #include 
 
@@ -33,7 +34,7 @@ private:
 std::vector m_aImageCommandNameVector;
 std::vector m_aImageNameVector;
 
-o3tl::enumarray m_pImageList;
+o3tl::enumarray> m_pImageList;
 OUString m_sIconTheme;
 
 ImageList* getImageList(ImageType nImageType);
diff --git a/framework/source/uiconfiguration/moduleuiconfigurationmanager.cxx 
b/framework/source/uiconfiguration/moduleuiconfigurationmanager.cxx
index aa94300e6154..96299c2bd83b 100644
--- a/framework/source/uiconfiguration/moduleuiconfigurationmanager.cxx
+++ b/framework/source/uiconfiguration/moduleuiconfigurationmanager.cxx
@@ -58,6 +58,7 @@
 #include 
 #include 
 #include 
+#include 
 
 using namespace css;
 using namespace com::sun::star::uno;
@@ -85,8 +86,6 @@ public:
 const css::uno::Reference< css::uno::XComponentContext >& 
xServiceManager,
 const css::uno::Sequence< css::uno::Any >& aArguments);
 
-virtual ~ModuleUIConfigurationManager() override;
-
 virtual OUString SAL_CALL getImplementationName() override
 {
 return 
OUString("com.sun.star.comp.framework.ModuleUIConfigurationManager");
@@ -201,7 +200,7 @@ private:
 voidimpl_reloadElementTypeData( UIElementType& 
rUserElementType, UIElementType const & rDefaultElementType, 
ConfigEventNotifyContainer& rRemoveNotifyContainer, ConfigEventNotifyContainer& 
rReplaceNotifyContainer );
 
 UIElementTypesVector  
m_aUIElements[LAYER_COUNT];
-PresetHandler*
m_pStorageHandler[css::ui::UIElementType::COUNT];
+std::unique_ptr
m_pStorageHandler[css::ui::UIElementType::COUNT];
 css::uno::Reference< css::embed::XStorage >   
m_xDefaultConfigStorage;
 css::uno::Reference< css::embed::XStorage >   
m_xUserConfigStorage;
 bool  m_bReadOnly;
@@ -854,9 +853,6 @@ ModuleUIConfigurationManager::ModuleUIConfigurationManager(
 , m_xContext( xContext )
 , m_aListenerContainer( m_mutex )
 {
-for (PresetHandler* & i : m_pStorageHandler)
-i = nullptr;
-
 // Make sure we have a default initialized entry for every layer and user 
interface element type!
 // The following code depends on this!
 m_aUIElements[LAYER_DEFAULT].resize( css::ui::UIElementType::COUNT );
@@ -888,7 +884,7 @@ ModuleUIConfig

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

2017-08-24 Thread Grzegorz Araminowicz
 oox/source/drawingml/diagram/diagramlayoutatoms.hxx |   17 --
 oox/source/drawingml/diagram/layoutnodecontext.cxx  |   49 
 oox/source/drawingml/diagram/layoutnodecontext.hxx  |2 
 3 files changed, 2 insertions(+), 66 deletions(-)

New commits:
commit 96b07db82b4a7834246ae8acc395cd2cb355630b
Author: Grzegorz Araminowicz 
Date:   Wed Aug 23 14:26:03 2017 +0200

SmartArt: get rid of tagToVarIdx() and change VarMap to std::map

it was unnecessarily complicated

Change-Id: I5ad4b4a6a401ffe6a67a471fa355ade9c1514ef1
Reviewed-on: https://gerrit.libreoffice.org/41458
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/oox/source/drawingml/diagram/diagramlayoutatoms.hxx 
b/oox/source/drawingml/diagram/diagramlayoutatoms.hxx
index a7ff21759174..cb7c84e9e532 100644
--- a/oox/source/drawingml/diagram/diagramlayoutatoms.hxx
+++ b/oox/source/drawingml/diagram/diagramlayoutatoms.hxx
@@ -22,9 +22,7 @@
 
 #include 
 #include 
-#include 
 
-#include 
 #include 
 
 #include "oox/drawingml/shape.hxx"
@@ -207,20 +205,7 @@ class LayoutNode
 : public LayoutAtom
 {
 public:
-enum {
-VAR_animLvl = 0,
-VAR_animOne,
-VAR_bulletEnabled,
-VAR_chMax,
-VAR_chPref,
-VAR_dir,
-VAR_hierBranch,
-VAR_orgChart,
-VAR_resizeHandles
-};
-// we know that the array is of fixed size
-// the use of Any allow having empty values
-typedef std::array VarMap;
+typedef std::map VarMap;
 
 LayoutNode(const Diagram& rDgm) : LayoutAtom(*this), mrDgm(rDgm), 
mnChildOrder(0) {}
 const Diagram& getDiagram() const
diff --git a/oox/source/drawingml/diagram/layoutnodecontext.cxx 
b/oox/source/drawingml/diagram/layoutnodecontext.cxx
index 77a3fa3b8ed8..6fc38a5608a0 100644
--- a/oox/source/drawingml/diagram/layoutnodecontext.cxx
+++ b/oox/source/drawingml/diagram/layoutnodecontext.cxx
@@ -160,12 +160,7 @@ public:
 
 virtual ContextHandlerRef onCreateContext( ::sal_Int32 aElement, const 
AttributeList& rAttribs ) override
 {
-sal_Int32 nIdx =  LayoutNodeContext::tagToVarIdx( getBaseToken( 
aElement ) );
-if( nIdx != -1 )
-{
-mVariables[ nIdx ] <<= rAttribs.getString( XML_val ).get();
-}
-
+mVariables[ getBaseToken(aElement) ] = rAttribs.getString( XML_val 
).get();
 return this;
 }
 private:
@@ -187,48 +182,6 @@ LayoutNodeContext::~LayoutNodeContext()
 {
 }
 
-/** convert the XML tag to a variable index in the array
- * @param aTag the tag, without namespace
- * @return the variable index. -1 is an error
- */
-sal_Int32 LayoutNodeContext::tagToVarIdx( sal_Int32 aTag )
-{
-sal_Int32 nIdx = -1;
-switch( aTag )
-{
-case DGM_TOKEN( animLvl ):
-nIdx = LayoutNode::VAR_animLvl;
-break;
-case DGM_TOKEN( animOne ):
-nIdx = LayoutNode::VAR_animOne;
-break;
-case DGM_TOKEN( bulletEnabled ):
-nIdx = LayoutNode::VAR_bulletEnabled;
-break;
-case DGM_TOKEN( chMax ):
-nIdx = LayoutNode::VAR_chMax;
-break;
-case DGM_TOKEN( chPref ):
-nIdx = LayoutNode::VAR_chPref;
-break;
-case DGM_TOKEN( dir ):
-nIdx = LayoutNode::VAR_dir;
-break;
-case DGM_TOKEN( hierBranch ):
-nIdx = LayoutNode::VAR_hierBranch;
-break;
-case DGM_TOKEN( orgChart ):
-nIdx = LayoutNode::VAR_orgChart;
-break;
-case DGM_TOKEN( resizeHandles ):
-nIdx = LayoutNode::VAR_resizeHandles;
-break;
-default:
-break;
-}
-return nIdx;
-}
-
 ContextHandlerRef
 LayoutNodeContext::onCreateContext( ::sal_Int32 aElement,
 const AttributeList& rAttribs )
diff --git a/oox/source/drawingml/diagram/layoutnodecontext.hxx 
b/oox/source/drawingml/diagram/layoutnodecontext.hxx
index 85f73f029f24..335c86fa13c9 100644
--- a/oox/source/drawingml/diagram/layoutnodecontext.hxx
+++ b/oox/source/drawingml/diagram/layoutnodecontext.hxx
@@ -33,8 +33,6 @@ public:
 virtual ~LayoutNodeContext() override;
 
 virtual ::oox::core::ContextHandlerRef onCreateContext( ::sal_Int32 
Element, const ::oox::AttributeList& rAttribs ) override;
-
-static ::sal_Int32 tagToVarIdx( ::sal_Int32 aTag );
 private:
 LayoutAtomPtr mpNode;
 };
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-08-24 Thread Noel Grandin
 forms/source/component/imgprod.cxx |   27 +--
 forms/source/component/imgprod.hxx |7 +--
 2 files changed, 14 insertions(+), 20 deletions(-)

New commits:
commit 0d46a29b2144eb840935e6bb040ee016d74b4d99
Author: Noel Grandin 
Date:   Wed Aug 23 15:05:45 2017 +0200

loplugin:useuniqueptr in forms

Change-Id: I656ef57a322a4d04d0a39ae8f9e1df6fff054a58
Reviewed-on: https://gerrit.libreoffice.org/41511
Tested-by: Jenkins 
Reviewed-by: Noel Grandin 

diff --git a/forms/source/component/imgprod.cxx 
b/forms/source/component/imgprod.cxx
index 1f7eb6045500..154b4114800c 100644
--- a/forms/source/component/imgprod.cxx
+++ b/forms/source/component/imgprod.cxx
@@ -161,16 +161,11 @@ ImageProducer::ImageProducer()
 , mnTransIndex(0)
 , mbConsInit(false)
 {
-mpGraphic = new Graphic;
+mpGraphic.reset( new Graphic );
 }
 
 ImageProducer::~ImageProducer()
 {
-delete mpGraphic;
-mpGraphic = nullptr;
-
-delete mpStm;
-mpStm = nullptr;
 }
 
 
@@ -206,19 +201,18 @@ void ImageProducer::SetImage( const OUString& rPath )
 maURL = rPath;
 mpGraphic->Clear();
 mbConsInit = false;
-delete mpStm;
+mpStm.reset();
 
 if ( ::svt::GraphicAccess::isSupportedURL( maURL ) )
 {
-mpStm = ::svt::GraphicAccess::getImageStream( 
::comphelper::getProcessComponentContext(), maURL );
+mpStm.reset( ::svt::GraphicAccess::getImageStream( 
::comphelper::getProcessComponentContext(), maURL ) );
 }
 else if( !maURL.isEmpty() )
 {
 SvStream* pIStm = ::utl::UcbStreamHelper::CreateStream( maURL, 
StreamMode::STD_READ );
-mpStm = pIStm ? new SvStream( new ImgProdLockBytes( pIStm, true ) ) : 
nullptr;
+if (pIStm)
+mpStm.reset( new SvStream( new ImgProdLockBytes( pIStm, true ) ) );
 }
-else
-mpStm = nullptr;
 }
 
 
@@ -228,8 +222,7 @@ void ImageProducer::SetImage( SvStream& rStm )
 mpGraphic->Clear();
 mbConsInit = false;
 
-delete mpStm;
-mpStm = new SvStream( new ImgProdLockBytes( &rStm, false ) );
+mpStm.reset( new SvStream( new ImgProdLockBytes( &rStm, false ) ) );
 }
 
 
@@ -238,12 +231,10 @@ void ImageProducer::setImage( css::uno::Reference< 
css::io::XInputStream > const
 maURL.clear();
 mpGraphic->Clear();
 mbConsInit = false;
-delete mpStm;
+mpStm.reset();
 
 if( rInputStmRef.is() )
-mpStm = new SvStream( new ImgProdLockBytes( rInputStmRef ) );
-else
-mpStm = nullptr;
+mpStm.reset( new SvStream( new ImgProdLockBytes( rInputStmRef ) ) );
 }
 
 
@@ -268,7 +259,7 @@ void ImageProducer::startProduction()
 if( ( mpGraphic->GetType() == GraphicType::NONE ) || 
mpGraphic->GetContext() )
 {
 if ( ImplImportGraphic( *mpGraphic ) )
-maDoneHdl.Call( mpGraphic );
+maDoneHdl.Call( mpGraphic.get() );
 }
 
 if( mpGraphic->GetType() != GraphicType::NONE )
diff --git a/forms/source/component/imgprod.hxx 
b/forms/source/component/imgprod.hxx
index a247b2890f92..2c0fed6e2c44 100644
--- a/forms/source/component/imgprod.hxx
+++ b/forms/source/component/imgprod.hxx
@@ -26,6 +26,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 
 
@@ -47,8 +48,10 @@ private:
 
 OUStringmaURL;
 ConsumerList_t  maConsList;
-Graphic*mpGraphic;
-SvStream*   mpStm;
+std::unique_ptr
+mpGraphic;
+std::unique_ptr
+mpStm;
 sal_uInt32  mnTransIndex;
 boolmbConsInit;
 Link maDoneHdl;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'distro/escriba/escriba-5.2.1' - 19 commits - offapi/com solenv/gbuild sw/inc sw/qa sw/source

2017-08-24 Thread Miklos Vajna
 offapi/com/sun/star/text/TextRangeContentProperties.idl  |6 
 solenv/gbuild/platform/filter-showIncludes.awk   |7 
 sw/inc/cmdid.h   |1 
 sw/inc/unoprnms.hxx  |1 
 sw/qa/extras/odfimport/data/i61225.sxw   |binary
 sw/qa/extras/odfimport/odfimport.cxx |8 
 sw/qa/extras/uiwriter/data/i95698.odt|binary
 sw/qa/extras/uiwriter/data/lines-in-section-in-table.odt |binary
 sw/qa/extras/uiwriter/data/paragraph-of-text-range.odt   |binary
 sw/qa/extras/uiwriter/data/rhbz739252-3.odt  |binary
 sw/qa/extras/uiwriter/data/table-in-sect.odt |binary
 sw/qa/extras/uiwriter/data/tdf108524.odt |binary
 sw/qa/extras/uiwriter/uiwriter.cxx   |  126 +++
 sw/source/core/inc/cellfrm.hxx   |1 
 sw/source/core/layout/findfrm.cxx|   20 +-
 sw/source/core/layout/flowfrm.cxx|5 
 sw/source/core/layout/sectfrm.cxx|   66 ++-
 sw/source/core/layout/tabfrm.cxx |   39 
 sw/source/core/unocore/unocrsrhelper.cxx |   13 +
 sw/source/core/unocore/unomapproperties.hxx  |1 
 sw/source/uibase/inc/unodispatch.hxx |7 
 sw/source/uibase/uno/unodispatch.cxx |   10 +
 22 files changed, 293 insertions(+), 18 deletions(-)

New commits:
commit 3ff777ef14f690946073aa7b48cd6953412e4db1
Author: Miklos Vajna 
Date:   Wed Aug 16 17:19:05 2017 +0200

i#95698 sw: fix crash on splitting in-table section containing a nested 
table

Found by crashtesting, ooo95698-1.odt crashed sw layout on load. The
intended use-case is splitting section frames inside a table frame, so
can just blacklist the non-interesting table-in-section-in-table case
that causes the problem here.

Reviewed-on: https://gerrit.libreoffice.org/41224
Reviewed-by: Miklos Vajna 
Tested-by: Jenkins 
(cherry picked from commit 802477ae75b39194442d9c01a1342d068c7b9300)

Conflicts:
sw/qa/extras/uiwriter/uiwriter.cxx

Change-Id: Ic47cd8c46cc71f7eaa36b03ec2c4a5df8ca8051c

diff --git a/sw/qa/extras/uiwriter/data/i95698.odt 
b/sw/qa/extras/uiwriter/data/i95698.odt
new file mode 100644
index ..9fe3ec207648
Binary files /dev/null and b/sw/qa/extras/uiwriter/data/i95698.odt differ
diff --git a/sw/qa/extras/uiwriter/uiwriter.cxx 
b/sw/qa/extras/uiwriter/uiwriter.cxx
index 3d566b5df278..c23c0e825d27 100644
--- a/sw/qa/extras/uiwriter/uiwriter.cxx
+++ b/sw/qa/extras/uiwriter/uiwriter.cxx
@@ -203,6 +203,7 @@ public:
 void testTdf108524();
 void testTableInSection();
 void testTableInNestedSection();
+void testTableInSectionInTable();
 void testLinesInSectionInTable();
 void testLinesMoveBackwardsInSectionInTable();
 
@@ -312,6 +313,7 @@ public:
 CPPUNIT_TEST(testTableInSection);
 CPPUNIT_TEST(testTableInNestedSection);
 CPPUNIT_TEST(testLinesInSectionInTable);
+CPPUNIT_TEST(testTableInSectionInTable);
 CPPUNIT_TEST(testLinesMoveBackwardsInSectionInTable);
 CPPUNIT_TEST_SUITE_END();
 
@@ -3820,6 +3822,14 @@ void SwUiWriterTest::testTableInNestedSection()
 assertXPath(pXmlDoc, "//page[2]//section/tab", 1);
 }
 
+void SwUiWriterTest::testTableInSectionInTable()
+{
+// The document has a table, containing a section, containing a nested
+// table.
+// This crashed the layout.
+createDoc("i95698.odt");
+}
+
 CPPUNIT_TEST_SUITE_REGISTRATION(SwUiWriterTest);
 CPPUNIT_PLUGIN_IMPLEMENT();
 
diff --git a/sw/source/core/layout/tabfrm.cxx b/sw/source/core/layout/tabfrm.cxx
index 45282d886929..0c6db983d50d 100644
--- a/sw/source/core/layout/tabfrm.cxx
+++ b/sw/source/core/layout/tabfrm.cxx
@@ -916,9 +916,21 @@ static bool lcl_FindSectionsInRow( const SwRowFrame& rRow )
 if (const SwFrame* pSectionLower = 
pTmpFrame->GetLower())
 {
 if (!pSectionLower->IsColumnFrame())
+{
 // Section has a single column only, try to
 // split that.
 bRet = false;
+
+for (const SwFrame* pFrame = pSectionLower; 
pFrame; pFrame = pFrame->GetNext())
+{
+if (pFrame->IsTabFrame())
+{
+// Section contains a table, no split 
in that case.
+bRet = true;
+break;
+}
+}
+}
 }
 }

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

2017-08-24 Thread Eike Rathke
 sc/source/core/tool/scmatrix.cxx |   17 +
 1 file changed, 9 insertions(+), 8 deletions(-)

New commits:
commit 9bedb5caae9cb4b535b9fc370480c1b4f5cd70f6
Author: Eike Rathke 
Date:   Mon Aug 21 15:49:41 2017 +0200

Resolves: tdf#111943 really really limit the match, tdf#108292 follow-up

getRemainingCount() could deliver a wrapped around overflow value if mnIndex
was already greater than the end index, which could happen if when/for
non-matching larger block sizes were added, and if then a match was found
behind those blocks a non-requested/unexpected index was returned, which in
turn led to the assert() being hit in ScInterpreter::CalculateLookup(). In
non-debug could result in an invalid block position access.

This happened with the bug case document of tdf#111943 which in master can 
be
loaded.

Also, the start and end index are not dynamic and don't have to be 
recalculated
each time, so make them const; column argument values are unused after.

(cherry picked from commit 25b3806ac509006573e669acc33643af3bd77380)

Change-Id: Ic294cade4e8e7828bee394e5ade61d7127be6bbb
Reviewed-on: https://gerrit.libreoffice.org/41396
Tested-by: Jenkins 
Reviewed-by: Markus Mohrhard 

diff --git a/sc/source/core/tool/scmatrix.cxx b/sc/source/core/tool/scmatrix.cxx
index 659e22fa064a..0a3dca7cb12d 100644
--- a/sc/source/core/tool/scmatrix.cxx
+++ b/sc/source/core/tool/scmatrix.cxx
@@ -1267,24 +1267,25 @@ template
 class WalkAndMatchElements : public 
std::unary_function
 {
 Type maMatchValue;
-MatrixImplType::size_pair_type maSize;
-size_t mnCol1;
-size_t mnCol2;
+const size_t mnStartIndex;
+const size_t mnStopIndex;
 size_t mnResult;
 size_t mnIndex;
 
 public:
 WalkAndMatchElements(Type aMatchValue, const 
MatrixImplType::size_pair_type& aSize, size_t nCol1, size_t nCol2) :
 maMatchValue(aMatchValue),
-maSize(aSize),
-mnCol1(nCol1),
-mnCol2(nCol2),
+mnStartIndex( nCol1 * aSize.row ),
+mnStopIndex( (nCol2 + 1) * aSize.row ),
 mnResult(ResultNotSet),
 mnIndex(0) {}
 
 size_t getMatching() const { return mnResult; }
 
-size_t getRemainingCount() const { return ((mnCol2 + 1) * maSize.row) - 
mnIndex; }
+size_t getRemainingCount() const
+{
+return mnIndex < mnStopIndex ? mnStopIndex - mnIndex : 0;
+}
 
 size_t compare(const MatrixImplType::element_block_node_type& node) const;
 
@@ -1295,7 +1296,7 @@ public:
 return;
 
 // limit lookup to the requested columns
-if ((mnCol1 * maSize.row) <= mnIndex && getRemainingCount() > 0)
+if (mnStartIndex <= mnIndex && getRemainingCount() > 0)
 {
 mnResult = compare(node);
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-08-24 Thread Dennis Francis
 sc/qa/unit/subsequent_export-test.cxx |   67 +-
 sc/source/filter/xml/xmlexprt.cxx |8 +++-
 2 files changed, 73 insertions(+), 2 deletions(-)

New commits:
commit ebe58344ba4bf6ad7de49564ebf10940d7636176
Author: Dennis Francis 
Date:   Tue Aug 15 11:21:04 2017 +0530

tdf#83562: Make the EditTextObject export of automatic colors...

... use the xml token "use-window-font-color" instead of "color"
to encode a boolean value of whether an auto color is used or not.
This is because the "auto" color is 0x and cannot be
written into "color" xml field which can store only 3 byte-hexcode.

This commit also adds a unit test in subsequent_export-test.cxx
to ensure the correct export of automatic color.

Change-Id: I42aab926f31669c1423bc09b96b45c410c9732de
Reviewed-on: https://gerrit.libreoffice.org/41252
Tested-by: Jenkins 
Reviewed-by: Dennis Francis 
(cherry picked from commit 06d104ecda085200fb6d22e0bd6d6cb89ca4f66c)
Reviewed-on: https://gerrit.libreoffice.org/41358
Reviewed-by: Michael Stahl 
(cherry picked from commit 1b4c5f66083a92f4bfc9bd20a2a5b1e863ec84c1)
Reviewed-on: https://gerrit.libreoffice.org/41467
Reviewed-by: Eike Rathke 
Reviewed-by: Christian Lohmaier 
Tested-by: Christian Lohmaier 

diff --git a/sc/qa/unit/subsequent_export-test.cxx 
b/sc/qa/unit/subsequent_export-test.cxx
index fb4d496243bf..26ac932a7765 100644
--- a/sc/qa/unit/subsequent_export-test.cxx
+++ b/sc/qa/unit/subsequent_export-test.cxx
@@ -26,6 +26,7 @@
 #include "userdat.hxx"
 #include "docsh.hxx"
 #include "patattr.hxx"
+#include "docpool.hxx"
 #include "scitems.hxx"
 #include "document.hxx"
 #include "cellform.hxx"
@@ -61,6 +62,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 #include 
@@ -1029,7 +1031,7 @@ void ScExportTest::testMiscRowHeightExport()
 
 namespace {
 
-void setAttribute( ScFieldEditEngine& rEE, sal_Int32 nPara, sal_Int32 nStart, 
sal_Int32 nEnd, sal_uInt16 nType )
+void setAttribute( ScFieldEditEngine& rEE, sal_Int32 nPara, sal_Int32 nStart, 
sal_Int32 nEnd, sal_uInt16 nType, sal_uInt32 nColor = COL_BLACK )
 {
 ESelection aSel;
 aSel.nStartPara = aSel.nEndPara = nPara;
@@ -1074,6 +1076,13 @@ void setAttribute( ScFieldEditEngine& rEE, sal_Int32 
nPara, sal_Int32 nStart, sa
 rEE.QuickSetAttribs(aItemSet, aSel);
 }
 break;
+case EE_CHAR_COLOR:
+{
+SvxColorItem aItem(nColor, nType);
+aItemSet.Put(aItem);
+rEE.QuickSetAttribs(aItemSet, aSel);
+}
+break;
 default:
 ;
 }
@@ -1256,6 +1265,23 @@ void ScExportTest::testRichTextExportODS()
 return false;
 }
 
+static bool isColor(const editeng::Section& rAttr, sal_uInt32 nColor)
+{
+if (rAttr.maAttributes.empty())
+return false;
+
+std::vector::const_iterator it = 
rAttr.maAttributes.begin(), itEnd = rAttr.maAttributes.end();
+for (; it != itEnd; ++it)
+{
+const SfxPoolItem* p = *it;
+if (p->Which() != EE_CHAR_COLOR)
+continue;
+
+return static_cast(p)->GetValue() == 
nColor;
+}
+return false;
+}
+
 bool checkB2(const EditTextObject* pText) const
 {
 if (!pText)
@@ -1482,6 +1508,32 @@ void ScExportTest::testRichTextExportODS()
 return true;
 }
 
+bool checkB10(const EditTextObject* pText) const
+{
+if (!pText)
+return false;
+
+if (pText->GetParagraphCount() != 1)
+return false;
+
+if (pText->GetText(0) != "BLUE AUTO")
+return false;
+
+std::vector aSecAttrs;
+pText->GetAllSections(aSecAttrs);
+if (aSecAttrs.size() != 2)
+return false;
+
+// auto color
+const editeng::Section* pAttr = &aSecAttrs[1];
+if (pAttr->mnParagraph != 0 ||pAttr->mnStart != 5 || pAttr->mnEnd 
!= 9)
+return false;
+
+if (pAttr->maAttributes.size() != 1 || !isColor(*pAttr, COL_AUTO))
+return false;
+
+return true;
+}
 
 } aCheckFunc;
 
@@ -1583,6 +1635,17 @@ void ScExportTest::testRichTextExportODS()
 pEditText = rDoc3.GetEditText(ScAddress(1,8,0));
 CPPUNIT_ASSERT_MESSAGE("Incorrect B9 value.", 
aCheckFunc.checkB9(pEditText));
 
+ScPatternAttr aCellFontColor(rDoc3.GetPool());
+aCellFontColor.GetItemSet().Put(SvxColorItem(COL_BLUE, 
ATTR_FONT_COLOR));
+// Set font color of B10 to blue.
+rDoc3.ApplyPattern(1, 9, 0, aCellFontColor);
+pEE->Clear();
+pEE->SetText("BLUE AUTO");
+// Set the color of the string "AUTO" to automatic color.
+setAttribute(*pEE

[Libreoffice-commits] online.git: loolwsd.xml.in wsd/DocumentBroker.cpp wsd/DocumentBroker.hpp wsd/LOOLWSD.cpp

2017-08-24 Thread Pranav Kant
 loolwsd.xml.in |3 +++
 wsd/DocumentBroker.cpp |6 --
 wsd/DocumentBroker.hpp |3 ---
 wsd/LOOLWSD.cpp|2 ++
 4 files changed, 9 insertions(+), 5 deletions(-)

New commits:
commit ad211ef51b1f87d8ab587d991bda253b54bb6be1
Author: Pranav Kant 
Date:   Thu Aug 24 16:40:39 2017 +0530

wsd: Configurable autosave and idlesave durations

But these save conditions are checked every 30 seconds only, so setting
them to less than 30 seconds wouldn't mean that save will be triggered
anytime sooner.

Change-Id: Id473a79af6a3170c72e372040460f2b7c15f150e

diff --git a/loolwsd.xml.in b/loolwsd.xml.in
index a8b67737..5097d9ac 100644
--- a/loolwsd.xml.in
+++ b/loolwsd.xml.in
@@ -16,6 +16,9 @@
 
 4
 3600
+
+30
+300
 0
 0
 8000
diff --git a/wsd/DocumentBroker.cpp b/wsd/DocumentBroker.cpp
index 77035837..6744b838 100644
--- a/wsd/DocumentBroker.cpp
+++ b/wsd/DocumentBroker.cpp
@@ -799,9 +799,11 @@ bool DocumentBroker::autoSave(const bool force)
 LOG_TRC("Time since last save of docKey [" << _docKey << "] is " << 
timeSinceLastSaveMs <<
 "ms and most recent activity was " << inactivityTimeMs << "ms 
ago.");
 
+static const auto idleSaveDurationMs = 
LOOLWSD::getConfigValue("per_document.idlesave_duration_secs", 30) * 1000;
+static const auto autoSaveDurationMs = 
LOOLWSD::getConfigValue("per_document.autosave_duration_secs", 300) * 1000;
 // Either we've been idle long enough, or it's auto-save time.
-if (inactivityTimeMs >= IdleSaveDurationMs ||
-timeSinceLastSaveMs >= AutoSaveDurationMs)
+if (inactivityTimeMs >= idleSaveDurationMs ||
+timeSinceLastSaveMs >= autoSaveDurationMs)
 {
 LOG_TRC("Sending timed save command for [" << _docKey << "].");
 sent = sendUnoSave(savingSessionId);
diff --git a/wsd/DocumentBroker.hpp b/wsd/DocumentBroker.hpp
index b08fd2e9..ecaaa30a 100644
--- a/wsd/DocumentBroker.hpp
+++ b/wsd/DocumentBroker.hpp
@@ -435,9 +435,6 @@ private:
 
 /// Unique DocBroker ID for tracing and debugging.
 static std::atomic DocBrokerId;
-
-static constexpr auto IdleSaveDurationMs = 30 * 1000;
-static constexpr auto AutoSaveDurationMs = 300 * 1000;
 };
 
 #endif
diff --git a/wsd/LOOLWSD.cpp b/wsd/LOOLWSD.cpp
index 37e56dd2..dc389dcf 100644
--- a/wsd/LOOLWSD.cpp
+++ b/wsd/LOOLWSD.cpp
@@ -633,6 +633,8 @@ void LOOLWSD::initialize(Application& self)
 { "num_prespawn_children", "1" },
 { "per_document.max_concurrency", "4" },
 { "per_document.idle_timeout_secs", "3600" },
+{ "per_document.idlesave_duration_secs", "30" },
+{ "per_document.autosave_duration_secs", "300" },
 { "per_document.limit_virt_mem_mb", "0" },
 { "per_document.limit_stack_mem_kb", "8000" },
 { "per_document.limit_file_size_mb", "50" },
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2017-08-24 Thread Michael Stahl
 dbaccess/source/core/dataaccess/databasecontext.cxx |   12 +++-
 1 file changed, 7 insertions(+), 5 deletions(-)

New commits:
commit a05e5adc62c74b42262fce9a309c1bc6c685893c
Author: Michael Stahl 
Date:   Thu Aug 24 12:41:24 2017 +0200

dbaccess: avoid deadlock in ODatabaseContext::registerObject()

Limit the scope of m_aMutex lock to what is necessary.

As observed by Jan-Marek Glogowski:

[Switching to thread 198 (Thread 0x2aab44e5b700 (LWP 46847))]
3  osl_acquireMutex(oslMutexImpl*) (pMutex=0x1e59d10) at 
sal/osl/unx/mutex.cxx:97
4  osl::Mutex::acquire() (this=0x1ebbe28) at include/osl/mutex.hxx:56
5  osl::Guard::Guard(osl::Mutex&, bool) (this=0x2aab44e56198, 
rT=..., bAquire=true) at include/osl/mutex.hxx:135
6  dbaccess::ODatabaseContext::getElementNames() (this=0x1ebbd90) at 
dbaccess/source/core/dataaccess/databasecontext.cxx:661
7  non-virtual thunk to dbaccess::ODatabaseContext::getElementNames() () at 
dbaccess/source/core/dataaccess/databasecontext.cxx:659
8  dbaui::SbaTableQueryBrowser::initializeTreeModel() (this=0xb174890) at 
dbaccess/source/ui/browser/unodatbr.cxx:2086
9  dbaui::SbaTableQueryBrowser::impl_initialize() (this=0xb174890) at 
dbaccess/source/ui/browser/unodatbr.cxx:3201
10 
dbaui::OGenericUnoController::initialize(com::sun::star::uno::Sequence
 const&) (this=0xb174890, aArguments=uno::Sequence of length 9 = {...}) at 
dbaccess/source/ui/browser/genericcontroller.cxx:299
11 
DBContentLoader::load(com::sun::star::uno::Reference
 const&, rtl::OUString const&, 
com::sun::star::uno::Sequence const&, 
com::sun::star::uno::Reference 
const&) (this=0xb1730a0, rFrame=uno::Reference to ((anonymous namespace)::Frame 
*) 0xb162690, rURL=".component:DB/DataSourceBrowser", rArgs=uno::Sequence of 
length 8 = {...}, rListener=uno::Reference to (framework::LoadEnvListener *) 
0xb174228) at dbaccess/source/ui/browser/dbloader.cxx:270
12 framework::LoadEnv::impl_loadContent() (this=0xb16f5b8) at 
framework/source/loadenv/loadenv.cxx:1077
13 framework::LoadEnv::startLoading() (this=0xb16f5b8) at 
framework/source/loadenv/loadenv.cxx:372

[Switching to thread 196 (Thread 0x2aab4565f700 (LWP 42841))]
3  osl_acquireMutex(oslMutexImpl*) (pMutex=0x773cd0) at 
sal/osl/unx/mutex.cxx:97
4  osl::Mutex::acquire() (this=0x77b4a8) at include/osl/mutex.hxx:56
5  comphelper::GenericSolarMutex::doAcquire(unsigned int) (this=0x77b4a0, 
nLockCount=1) at comphelper/source/misc/solarmutex.cxx:61
6  comphelper::SolarMutex::acquire(unsigned int) (this=0x77b4a0, 
nLockCount=1) at include/comphelper/solarmutex.hxx:74
7  
osl::ClearableGuard::ClearableGuard(comphelper::SolarMutex&)
 (this=0x2aab4565d478, t=...) at include/osl/mutex.hxx:170
8  
osl::ResettableGuard::ResettableGuard(comphelper::SolarMutex&)
 (this=0x2aab4565d478, rT=...) at include/osl/mutex.hxx:215
9  SolarMutexResettableGuard::SolarMutexResettableGuard() 
(this=0x2aab4565d478) at include/vcl/svapp.hxx:1416
10 
dbaccess::ModelMethodGuard::ModelMethodGuard(dbaccess::ModelDependentComponent 
const&) (this=0x2aab4565d478, _component=...) at 
dbaccess/source/core/dataaccess/ModelImpl.hxx:587
11 dbaccess::ODatabaseSource::getDatabaseDocument() (this=0x281d690) at 
dbaccess/source/core/dataaccess/datasource.cxx:1300
12 non-virtual thunk to dbaccess::ODatabaseSource::getDatabaseDocument() () 
at dbaccess/source/core/dataaccess/datasource.cxx:1298
13 dbaccess::ODatabaseContext::registerObject(rtl::OUString const&, 
com::sun::star::uno::Reference const&) 
(this=0x1ebbd90, _rName="MyFactory", _rxObject=uno::Reference to 
(dbaccess::ODatabaseSource *) 0x281d708) at 
dbaccess/source/core/dataaccess/databasecontext.cxx:433
14 gcc3::callVirtualMethod(void*, unsigned int, void*, 
_typelib_TypeDescriptionReference*, bool, unsigned long*, unsigned int, 
unsigned long*, double*) (pThis=0x1ebbdf8, nVtableIndex=4, pRegisterReturn=0x0, 
pReturnTypeRef=0x781090, bSimpleReturn=true, pStack=0x2aab4565d810, nStack=0, 
pGPR=0x2aab4565db30, pFPR=0x2aab4565daf0) at 
bridges/source/cpp_uno/gcc3_linux_x86-64/callvirtualmethod.cxx:77

Change-Id: I06e91d9c4e193c97f98f29f6b73eaf01b12922c0

diff --git a/dbaccess/source/core/dataaccess/databasecontext.cxx 
b/dbaccess/source/core/dataaccess/databasecontext.cxx
index e023806270b2..4d2305a6b9cf 100644
--- a/dbaccess/source/core/dataaccess/databasecontext.cxx
+++ b/dbaccess/source/core/dataaccess/databasecontext.cxx
@@ -423,9 +423,6 @@ void ODatabaseContext::setTransientProperties(const 
OUString& _sURL, ODatabaseMo
 
 void ODatabaseContext::registerObject(const OUString& _rName, const Reference< 
XInterface > & _rxObject)
 {
-MutexGuard aGuard(m_aMutex);
-
::connectivity::checkDisposed(DatabaseAccessContext_Base::rBHelper.bDisposed);
-
 if ( _rName.isEmpty() )
 throw IllegalArgumentException( OUString(), *this, 1 );
 
@@ -438,9 +435,14 @@ void ODatabaseContext::registerObject(con

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

2017-08-24 Thread Zdeněk Crhonek
 basic/qa/cppunit/test_vba.cxx |5 ++
 basic/qa/vba_tests/val.vb |   94 ++
 basic/qa/vba_tests/vartype.vb |   86 ++
 basic/qa/vba_tests/weekday.vb |   83 +
 basic/qa/vba_tests/weekdayname.vb |   86 ++
 basic/qa/vba_tests/year.vb|   66 ++
 6 files changed, 420 insertions(+)

New commits:
commit 74da6d5f9f875cc355776f8fa8fbff15f0316d0c
Author: Zdeněk Crhonek 
Date:   Thu Aug 24 11:02:01 2017 +0200

VBA tests-add VAL,VARTYPE,WEEKDAY,WEEKDAYNAME,YEAR tests

Change-Id: I3f1803617a2bdeaab9922ebb375ec4fd67318d20
Reviewed-on: https://gerrit.libreoffice.org/41499
Tested-by: Jenkins 
Reviewed-by: Zdenek Crhonek 

diff --git a/basic/qa/cppunit/test_vba.cxx b/basic/qa/cppunit/test_vba.cxx
index 9ba9521036a5..a8d5c7c9410c 100644
--- a/basic/qa/cppunit/test_vba.cxx
+++ b/basic/qa/cppunit/test_vba.cxx
@@ -137,6 +137,11 @@ void VBATest::testMiscVBAFunctions()
 "typename.vb",
 "ubound.vb",
 "ucase.vb",
+"val.vb",
+"vartype.vb",
+"weekday.vb",
+"weekdayname.vb",
+"year.vb",
 #ifndef WIN32 // missing 64bit Currency marshalling.
 "win32compat.vb", // windows compatibility hooks.
 #endif
diff --git a/basic/qa/vba_tests/val.vb b/basic/qa/vba_tests/val.vb
new file mode 100644
index ..08f2722c3c8d
--- /dev/null
+++ b/basic/qa/vba_tests/val.vb
@@ -0,0 +1,94 @@
+Option VBASupport 1
+Option Explicit
+Dim passCount As Integer
+Dim failCount As Integer
+Dim result As String
+
+Function doUnitTest() As String
+result = verify_testVal()
+If failCount <> 0 And passCount > 0 Then
+doUnitTest = result
+Else
+doUnitTest = "OK"
+End If
+End Function
+
+
+
+Function verify_testVal() As String
+
+passCount = 0
+failCount = 0
+
+result = "Test Results" & Chr$(10) & "" & Chr$(10)
+
+Dim testName As String
+Dim TestDateTime As Date
+Dim TestStr As String
+Dim date1, date2
+testName = "Test Val function"
+On Error GoTo errorHandler
+
+date2 = 2
+date1 = Val("02/04/2010")
+TestLog_ASSERT date1 = date2, "the return Val is: " & date1
+
+date2 = 1050
+date1 = Val("1050")
+TestLog_ASSERT date1 = date2, "the return Val is: " & date1
+
+date2 = 130.75
+date1 = Val("130.75")
+TestLog_ASSERT date1 = date2, "the return Val is: " & date1
+
+date2 = 50
+date1 = Val("50 Park Lane")
+TestLog_ASSERT date1 = date2, "the return Val is: " & date1
+
+date2 = 1320
+date1 = Val("1320 then some text")
+TestLog_ASSERT date1 = date2, "the return Val is: " & date1
+
+date2 = 0
+date1 = Val("L13.5")
+TestLog_ASSERT date1 = date2, "the return Val is: " & date1
+
+date2 = 0
+date1 = Val("sometext")
+TestLog_ASSERT date1 = date2, "the return Val is: " & date1
+
+REMdate2 = 1
+REMdate1 = Val("1, 2")
+REMTestLog_ASSERT date1 = date2, "the return Val is: " & date1
+REMtdf#111999
+
+date2 = -1
+date1 = Val("&H")
+TestLog_ASSERT date1 = date2, "the return Val is: " & date1
+result = result & Chr$(10) & "Tests passed: " & passCount & Chr$(10) & 
"Tests failed: " & failCount & Chr$(10)
+verify_testVal = result
+
+Exit Function
+errorHandler:
+TestLog_ASSERT (False), testName & ": hit error handler"
+End Function
+
+Sub TestLog_ASSERT(assertion As Boolean, Optional testId As String, Optional 
testComment As String)
+
+If assertion = True Then
+passCount = passCount + 1
+Else
+Dim testMsg As String
+If Not IsMissing(testId) Then
+testMsg = testMsg + " : " + testId
+End If
+If Not IsMissing(testComment) And Not (testComment = "") Then
+testMsg = testMsg + " (" + testComment + ")"
+End If
+
+result = result & Chr$(10) & " Failed: " & testMsg
+failCount = failCount + 1
+End If
+
+End Sub
+
diff --git a/basic/qa/vba_tests/vartype.vb b/basic/qa/vba_tests/vartype.vb
new file mode 100644
index ..05cfa308fe28
--- /dev/null
+++ b/basic/qa/vba_tests/vartype.vb
@@ -0,0 +1,86 @@
+Option VBASupport 1
+Option Explicit
+Dim passCount As Integer
+Dim failCount As Integer
+Dim result As String
+
+Function doUnitTest() As String
+result = verify_testVarType()
+If failCount <> 0 And passCount > 0 Then
+doUnitTest = result
+Else
+doUnitTest = "OK"
+End If
+End Function
+
+
+
+Function verify_testVarType() As String
+
+passCount = 0
+failCount = 0
+
+result = "Test Results" & Chr$(10) & "" & Chr$(10)
+
+Dim testName As String
+Dim TestDateTime As Date
+Dim TestStr As String
+Dim TestInt As Integer
+Dim TestLong As Long
+Dim TestDouble As Double
+Dim TestBoo As Boolean
+Dim date1, date2
+testName = "Test VarType function"
+On Error GoTo er

Re: tdf#109097 - mdds accelerating random lookups

2017-08-24 Thread Michael Meeks
Hi Dennis,

On 24/08/17 11:03, Dennis Francis wrote:
> As this bug report
> (https://bugs.documentfoundation.org/show_bug.cgi?id=109097) suggests
> it is highly desirable to improve the lookup time in multi_type_vector
> (currently in worst case it takes O(N)) without position hints.

Perhaps one further elaboration here is that we're working on exploring
threading the calc-ulation engine (with some success) - however this
requires avoiding highly granular locking - ie. that we can treat the
various calc core data-structures as immutable during that process. As
such the low-hanging fruit solutions of building caches /
last-position-looked-up etc. as we do lookups are rather problematic.

So - I'm well up for this change - assuming Kohei, and other calc guys
are happy.

HTH,

Michael.

-- 
michael.me...@collabora.com <><, Pseudo Engineer, itinerant idiot
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice


tdf#109097 - mdds accelerating random lookups

2017-08-24 Thread Dennis Francis
Hello,

As this bug report (
https://bugs.documentfoundation.org/show_bug.cgi?id=109097) suggests
it is highly desirable to improve the lookup time in multi_type_vector
(currently in worst case
it takes O(N)) without position hints. One way to improve the situation is
to store absolute
position of first element of each block in the block instead of the block
size member. This
can improve the lookup time up to log(log(N)) using linear interpolation
search or at least log(N)
if we fallback to binary search after a few initial passes of
linear-interpolation search.

Down side of this approach is that in the worst case, insert/deletes incur
O(N) cost because
absolute positions stored in each block subsequent to the delete/insert
block have to be changed,
but this still would be way better than old cell storage case when there
was no concept of blocks.

Before working on any change from my side, I would love to hear everyone's
thoughts on the topic first. Thanks !

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


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

2017-08-24 Thread Miklos Vajna
 sw/qa/extras/rtfimport/data/tdf108943.rtf   |   24 
 sw/qa/extras/rtfimport/rtfimport.cxx|   19 +++
 writerfilter/source/rtftok/rtfdispatchvalue.cxx |5 -
 3 files changed, 47 insertions(+), 1 deletion(-)

New commits:
commit e6426cc1bf174a67828f3e18ab1ea70eccc44b07
Author: Miklos Vajna 
Date:   Wed Aug 23 09:04:43 2017 +0200

tdf#108943 RTF import: handle \tx inside a list level definition

\tx in the list level definition has a different meaning (we should map
it to the ListtabStopPosition UNO property), compared to when it's used
as a plain text property.

(cherry picked from commit 57d535302596a62be914bee49d7b1df1fc622a65)

Conflicts:
sw/qa/extras/rtfimport/rtfimport.cxx

Change-Id: I7b98ae3d82bbd2c4ad41d0d7f4ad50525c4d0f9a
Reviewed-on: https://gerrit.libreoffice.org/41457
Tested-by: Jenkins 
Reviewed-by: Michael Stahl 

diff --git a/sw/qa/extras/rtfimport/data/tdf108943.rtf 
b/sw/qa/extras/rtfimport/data/tdf108943.rtf
new file mode 100644
index ..478d101f4167
--- /dev/null
+++ b/sw/qa/extras/rtfimport/data/tdf108943.rtf
@@ -0,0 +1,24 @@
+{\rtf1
+{\fonttbl
+{\f0\fbidi \froman\fcharset238\fprq2 Times New Roman;}
+}
+\noqfpromote
+{\stylesheet
+{\s25 \fi-720\li720\ri0\ls1 ParaLevel1;}
+}
+{\*\listtable
+{\list\listtemplateid-1762123600
+{\listlevel\levelnfc0\levelnfcn0\leveljc0\leveljcn0
+\levelfollow0\levelstartat1\levelspace0\levelindent0
+{\leveltext\'02\'00.;}
+{\levelnumbers\'01;}
+\rtlch\fcs1 \af0 \ltrch\fcs0 \b0\i0\ulnone\fbias0 
\s25\fi-720\li720\jclisttab\tx720\lin720 }
+\listid1977027487}
+}
+{\*\listoverridetable
+{\listoverride\listid1977027487\listoverridecount0\ls1}
+}
+\pard\plain \s25 \fi-720\li720\ri0\tqr\tx720\ls1
+hello
+\par
+}
diff --git a/sw/qa/extras/rtfimport/rtfimport.cxx 
b/sw/qa/extras/rtfimport/rtfimport.cxx
index bf8188db99d6..f7f2e89d70a3 100644
--- a/sw/qa/extras/rtfimport/rtfimport.cxx
+++ b/sw/qa/extras/rtfimport/rtfimport.cxx
@@ -235,6 +235,25 @@ DECLARE_RTFIMPORT_TEST(testFdo45187, "fdo45187.rtf")
 CPPUNIT_ASSERT_EQUAL(sal_Int16(1), 
xTextRangeCompare->compareRegionStarts(xAnchor0, xAnchor1));
 }
 
+DECLARE_RTFIMPORT_TEST(testTdf108943, "tdf108943.rtf")
+{
+uno::Reference 
xPropertySet(getStyles("NumberingStyles")->getByName("WWNum1"), uno::UNO_QUERY);
+uno::Reference 
xLevels(xPropertySet->getPropertyValue("NumberingRules"), uno::UNO_QUERY);
+uno::Sequence aProps;
+xLevels->getByIndex(0) >>= aProps; // 1st level
+
+sal_Int32 nListtabStopPosition = 0;
+for (int i = 0; i < aProps.getLength(); ++i)
+{
+const beans::PropertyValue& rProp = aProps[i];
+
+if (rProp.Name == "ListtabStopPosition")
+nListtabStopPosition = rProp.Value.get();
+}
+// This was 0, \tx was handled in paragraphs only (and not in list 
definitions).
+CPPUNIT_ASSERT_EQUAL(static_cast(1270), nListtabStopPosition);
+}
+
 DECLARE_RTFIMPORT_TEST(testFdo46662, "fdo46662.rtf")
 {
 uno::Reference 
xPropertySet(getStyles("NumberingStyles")->getByName("WWNum3"), uno::UNO_QUERY);
diff --git a/writerfilter/source/rtftok/rtfdispatchvalue.cxx 
b/writerfilter/source/rtftok/rtfdispatchvalue.cxx
index 527d39da68db..f6a243e2fbc0 100644
--- a/writerfilter/source/rtftok/rtfdispatchvalue.cxx
+++ b/writerfilter/source/rtftok/rtfdispatchvalue.cxx
@@ -605,7 +605,10 @@ RTFError RTFDocumentImpl::dispatchValue(RTFKeyword 
nKeyword, int nParam)
 {
 m_aStates.top().aTabAttributes.set(NS_ooxml::LN_CT_TabStop_pos, 
pIntValue);
 auto pValue = 
std::make_shared(m_aStates.top().aTabAttributes);
-putNestedSprm(m_aStates.top().aParagraphSprms, 
NS_ooxml::LN_CT_PPrBase_tabs, NS_ooxml::LN_CT_Tabs_tab, pValue);
+if (m_aStates.top().eDestination == Destination::LISTLEVEL)
+putNestedSprm(m_aStates.top().aTableSprms, 
NS_ooxml::LN_CT_PPrBase_tabs, NS_ooxml::LN_CT_Tabs_tab, pValue);
+else
+putNestedSprm(m_aStates.top().aParagraphSprms, 
NS_ooxml::LN_CT_PPrBase_tabs, NS_ooxml::LN_CT_Tabs_tab, pValue);
 m_aStates.top().aTabAttributes.clear();
 }
 break;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: Rebuilding LO freezes on macOS

2017-08-24 Thread Alexander Thurgood
Le 23/08/2017 à 21:28, Thorsten Wagner a écrit :

Hi THorsten,


> (2) During building again build freezes while linking 
> "Library/libscfiltlo.dylib". "make" loops consuming 100% cpu on a single 
> core. Build has to be interrupted by killing "make" processes.
> 

I see this occasionally, or though it tends to be during population of
instdir (so towards the end of the build) and not specifically in
libscfiltLO.dylib. Possibly the parallel build gets into a race
condition somewhere (shrugs) and I have to kill it, clean it and restart.

I remember reading somewhere a long time ago about changing the ulimit
number of openable file descriptors, something like this here:

https://unix.stackexchange.com/questions/108174/how-to-persist-ulimit-settings-in-macos

That might help, but if you need to push past the hard limit set by
Apple as default, then you need to buy AppleServer for OSX from the
AppStore.


Alex

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


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

2017-08-24 Thread Noel Grandin
 sc/inc/colorscale.hxx|2 -
 sc/inc/formulacell.hxx   |4 +-
 sc/inc/formularesult.hxx |2 -
 sc/inc/scmod.hxx |6 +--
 sc/inc/xmlwrap.hxx   |   14 +++
 sc/inc/zforauto.hxx  |2 -
 sc/source/core/data/colorscale.cxx   |2 -
 sc/source/core/data/formulacell.cxx  |4 +-
 sc/source/core/tool/formularesult.cxx|2 -
 sc/source/core/tool/zforauto.cxx |2 -
 sc/source/filter/xml/XMLChangeTrackingExportHelper.cxx   |4 +-
 sc/source/filter/xml/XMLChangeTrackingExportHelper.hxx   |4 +-
 sc/source/filter/xml/XMLExportDDELinks.cxx   |2 -
 sc/source/filter/xml/XMLExportDDELinks.hxx   |2 -
 sc/source/filter/xml/XMLExportDataPilot.cxx  |   20 +--
 sc/source/filter/xml/XMLExportDataPilot.hxx  |   20 +--
 sc/source/filter/xml/XMLExportIterator.cxx   |2 -
 sc/source/filter/xml/XMLExportIterator.hxx   |2 -
 sc/source/filter/xml/XMLStylesExportHelper.cxx   |2 -
 sc/source/filter/xml/XMLStylesExportHelper.hxx   |2 -
 sc/source/filter/xml/XMLTableShapeImportHelper.cxx   |2 -
 sc/source/filter/xml/XMLTableShapeImportHelper.hxx   |2 -
 sc/source/filter/xml/XMLTableShapeResizer.cxx|4 +-
 sc/source/filter/xml/XMLTableShapeResizer.hxx|4 +-
 sc/source/filter/xml/xmlcelli.cxx|2 -
 sc/source/filter/xml/xmlexprt.cxx|4 +-
 sc/source/filter/xml/xmlexprt.hxx|4 +-
 sc/source/filter/xml/xmlimprt.cxx|2 -
 sc/source/filter/xml/xmlimprt.hxx|2 -
 sc/source/filter/xml/xmlsubti.cxx|2 -
 sc/source/filter/xml/xmlsubti.hxx|4 +-
 sc/source/filter/xml/xmlwrap.cxx |   12 +++---
 sc/source/ui/Accessibility/AccessibleCell.cxx|4 +-
 sc/source/ui/Accessibility/AccessibleDocument.cxx|2 -
 sc/source/ui/Accessibility/AccessibleDocumentPagePreview.cxx |8 ++--
 sc/source/ui/Accessibility/AccessiblePreviewCell.cxx |2 -
 sc/source/ui/Accessibility/AccessibleSpreadsheet.cxx |4 +-
 sc/source/ui/app/client.cxx  |2 -
 sc/source/ui/app/drwtrans.cxx|2 -
 sc/source/ui/app/inputhdl.cxx|   14 +++
 sc/source/ui/app/inputwin.cxx|4 +-
 sc/source/ui/app/scmod.cxx   |8 ++--
 sc/source/ui/app/transobj.cxx|2 -
 sc/source/ui/cctrl/checklistmenu.cxx |6 +--
 sc/source/ui/condformat/condformatdlgentry.cxx   |8 ++--
 sc/source/ui/inc/AccessibleCell.hxx  |4 +-
 sc/source/ui/inc/AccessiblePreviewCell.hxx   |2 -
 sc/source/ui/inc/AccessibleSpreadsheet.hxx   |4 +-
 sc/source/ui/inc/checklistmenu.hxx   |6 +--
 sc/source/ui/inc/client.hxx  |2 -
 sc/source/ui/inc/drwtrans.hxx|2 -
 sc/source/ui/inc/inputhdl.hxx|   12 +++---
 sc/source/ui/inc/inputwin.hxx|2 -
 sc/source/ui/inc/transobj.hxx|2 -
 54 files changed, 123 insertions(+), 123 deletions(-)

New commits:
commit fd736c4f7910f5b876c3ab8b8e0e804aa83347a0
Author: Noel Grandin 
Date:   Thu Aug 24 09:26:25 2017 +0200

loplugin:constparam in sc part4

Change-Id: I745f50257948a53ca07766955c4f4e86dd2a366e
Reviewed-on: https://gerrit.libreoffice.org/41497
Tested-by: Jenkins 
Reviewed-by: Noel Grandin 

diff --git a/sc/inc/colorscale.hxx b/sc/inc/colorscale.hxx
index 4fe901e8387d..0d33f1bb7146 100644
--- a/sc/inc/colorscale.hxx
+++ b/sc/inc/colorscale.hxx
@@ -68,7 +68,7 @@ public:
 void UpdateReference( const sc::RefUpdateContext& rCxt );
 void UpdateInsertTab( sc::RefUpdateInsertTabContext& rCxt );
 void UpdateDeleteTab( sc::RefUpdateDeleteTabContext& rCxt );
-void UpdateMoveTab( sc::RefUpdateMoveTabContext& rCxt );
+void UpdateMoveTab( const sc::RefUpdateMoveTabContext& rCxt );
 
 const ScTokenArray* GetFormula() const;
 OUString GetFormula( formula::FormulaGrammar::Grammar eGrammar ) const;
diff --git a/sc/inc/formulacell.hxx b/sc/inc/formulacell.hxx
index a7843f875f

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

2017-08-24 Thread Noel Grandin
 sc/inc/address.hxx |6 ++---
 sc/inc/charthelper.hxx |4 +--
 sc/inc/compiler.hxx|2 -
 sc/inc/consoli.hxx |4 +--
 sc/inc/dbdata.hxx  |4 +--
 sc/inc/detdata.hxx |4 +--
 sc/inc/detfunc.hxx |4 +--
 sc/inc/document.hxx|2 -
 sc/inc/orcusxml.hxx|3 --
 sc/inc/rangelst.hxx|8 +++
 sc/inc/rangenam.hxx|4 +--
 sc/inc/rangeutl.hxx|   16 +++
 sc/inc/tabprotection.hxx   |2 -
 sc/inc/token.hxx   |4 +--
 sc/inc/tokenarray.hxx  |2 -
 sc/inc/zforauto.hxx|2 -
 sc/source/core/data/documen5.cxx   |2 -
 sc/source/core/data/tabprotection.cxx  |6 ++---
 sc/source/core/inc/interpre.hxx|2 -
 sc/source/core/inc/refupdat.hxx|8 +++
 sc/source/core/tool/address.cxx|6 ++---
 sc/source/core/tool/charthelper.cxx|4 +--
 sc/source/core/tool/compiler.cxx   |2 -
 sc/source/core/tool/consoli.cxx|2 -
 sc/source/core/tool/dbdata.cxx |4 +--
 sc/source/core/tool/detdata.cxx|2 -
 sc/source/core/tool/detfunc.cxx|6 ++---
 sc/source/core/tool/interpr2.cxx   |2 -
 sc/source/core/tool/interpr4.cxx   |6 ++---
 sc/source/core/tool/orcusxml.cxx   |7 --
 sc/source/core/tool/progress.cxx   |4 +--
 sc/source/core/tool/rangelst.cxx   |8 +++
 sc/source/core/tool/rangenam.cxx   |4 +--
 sc/source/core/tool/rangeutl.cxx   |   18 -
 sc/source/core/tool/refupdat.cxx   |8 +++
 sc/source/core/tool/token.cxx  |   10 -
 sc/source/core/tool/zforauto.cxx   |2 -
 sc/source/filter/xml/XMLChangeTrackingExportHelper.cxx |   12 +--
 sc/source/filter/xml/XMLChangeTrackingExportHelper.hxx |4 +--
 39 files changed, 97 insertions(+), 103 deletions(-)

New commits:
commit 3eb5ab6efe4229db0ec2bb177d892f2604f49411
Author: Noel Grandin 
Date:   Wed Aug 23 18:37:52 2017 +0200

loplugin:constparam in sc part3

Change-Id: I8abc44b9f0bcd5e39fd8a94abafaf80aadcd26ca
Reviewed-on: https://gerrit.libreoffice.org/41496
Reviewed-by: Noel Grandin 
Tested-by: Noel Grandin 

diff --git a/sc/inc/address.hxx b/sc/inc/address.hxx
index 607a9a824691..0adeaa705861 100644
--- a/sc/inc/address.hxx
+++ b/sc/inc/address.hxx
@@ -543,7 +543,7 @@ public:
const 
css::uno::Sequence* pExternalLinks = nullptr,
const OUString* pErrRef = nullptr );
 
-SC_DLLPUBLIC ScRefFlags ParseAny( const OUString&, ScDocument*,
+SC_DLLPUBLIC ScRefFlags ParseAny( const OUString&, const ScDocument*,
   const ScAddress::Details& rDetails = 
ScAddress::detailsOOOa1 );
 SC_DLLPUBLIC ScRefFlags ParseCols( const OUString&,
const ScAddress::Details& rDetails = 
ScAddress::detailsOOOa1 );
@@ -907,12 +907,12 @@ template< typename T > void PutInOrder( T& nStart, T& 
nEnd )
 }
 }
 
-bool ConvertSingleRef( ScDocument* pDocument, const OUString& rRefString,
+bool ConvertSingleRef( const ScDocument* pDocument, const OUString& rRefString,
SCTAB nDefTab, ScRefAddress& rRefAddress,
const ScAddress::Details& rDetails,
ScAddress::ExternalInfo* pExtInfo = nullptr );
 
-bool ConvertDoubleRef( ScDocument* pDocument, const OUString& rRefString,
+bool ConvertDoubleRef( const ScDocument* pDocument, const OUString& rRefString,
SCTAB nDefTab, ScRefAddress& rStartRefAddress,
ScRefAddress& rEndRefAddress,
const ScAddress::Details& rDetails,
diff --git a/sc/inc/charthelper.hxx b/sc/inc/charthelper.hxx
index 9e7689e87fcf..7aad9522045a 100644
--- a/sc/inc/charthelper.hxx
+++ b/sc/inc/charthelper.hxx
@@ -38,7 +38,7 @@ class ScChartHelper
 {
 public:
 static sal_uInt16 DoUpdateAllCharts( ScDocument* pDoc );
-static void AdjustRangesOfChartsOnDestinationPage( ScDocument* pSrcDoc, 
ScDocument* pDestDoc, const SCTAB nSrcTab, const SCTAB nDestTab )

[Libreoffice-commits] core.git: Branch 'feature/gsoc17-revamp-customize-dialog' - cui/inc cui/source cui/uiconfig

2017-08-24 Thread Muhammet Kara
 cui/inc/strings.hrc   |1 
 cui/source/customize/SvxMenuConfigPage.cxx|   31 ++
 cui/source/customize/SvxToolbarConfigPage.cxx |   23 +++
 cui/source/customize/cfg.cxx  |   22 --
 cui/source/inc/SvxMenuConfigPage.hxx  |1 
 cui/source/inc/SvxToolbarConfigPage.hxx   |1 
 cui/source/inc/cfg.hxx|4 +++
 cui/uiconfig/ui/menuassignpage.ui |2 -
 8 files changed, 82 insertions(+), 3 deletions(-)

New commits:
commit e0d03d929929dec937d3fbf97ebf7d9eef1806a0
Author: Muhammet Kara 
Date:   Sun Aug 20 20:43:56 2017 +0300

Implement "Reset" button in the Customize dialog

For now, it works for the Toolbar and the Context Menu tabs.
Normal top-level menus cannot be reset individually at the moment
because they are sharing a single xml file, and the removeSettings()
mathod doesn't provide per-item removal from within a single file.

Change-Id: I42d62dc26130e4c03cf75a1ce6dc9ff367a58d47
Reviewed-on: https://gerrit.libreoffice.org/41367
Tested-by: Jenkins 
Reviewed-by: Katarina Behrens 

diff --git a/cui/inc/strings.hrc b/cui/inc/strings.hrc
index 74b0bcb4f80e..266f93bdd5de 100644
--- a/cui/inc/strings.hrc
+++ b/cui/inc/strings.hrc
@@ -69,6 +69,7 @@
 #define RID_SVXSTR_CONFIRM_MENU_RESET   
NC_("RID_SVXSTR_CONFIRM_MENU_RESET", "The menu configuration for %SAVE IN 
SELECTION% will be reset to the default settings. Do you want to continue?")
 #define RID_SVXSTR_CONFIRM_TOOLBAR_RESET
NC_("RID_SVXSTR_CONFIRM_TOOLBAR_RESET", "The toolbar configuration for %SAVE IN 
SELECTION% will be reset to the default settings. Do you want to continue?")
 #define RID_SVXSTR_CONFIRM_RESTORE_DEFAULT  
NC_("RID_SVXSTR_CONFIRM_RESTORE_DEFAULT", "This will delete all changes 
previously made to this toolbar. Do you really want to reset the toolbar?")
+#define RID_SVXSTR_CONFIRM_RESTORE_DEFAULT_MENU 
NC_("RID_SVXSTR_CONFIRM_RESTORE_DEFAULT_MENU", "This will delete all changes 
previously made to this context menu. Do you really want to reset?")
 #define RID_SVXSTR_MNUCFG_ALREADY_INCLUDED  
NC_("RID_SVXSTR_MNUCFG_ALREADY_INCLUDED", "Function is already included in this 
popup.")
 #define RID_SVXSTR_LABEL_NEW_NAME   
NC_("RID_SVXSTR_LABEL_NEW_NAME", "~New name")
 #define RID_SVXSTR_RENAME_MENU  
NC_("RID_SVXSTR_RENAME_MENU", "Rename Menu")
diff --git a/cui/source/customize/SvxMenuConfigPage.cxx 
b/cui/source/customize/SvxMenuConfigPage.cxx
index 32eb77e83a6c..decf6a9d251b 100644
--- a/cui/source/customize/SvxMenuConfigPage.cxx
+++ b/cui/source/customize/SvxMenuConfigPage.cxx
@@ -130,6 +130,9 @@ SvxMenuConfigPage::SvxMenuConfigPage(vcl::Window *pParent, 
const SfxItemSet& rSe
 
 m_pInsertBtn->SetSelectHdl(
 LINK( this, SvxMenuConfigPage, InsertHdl ) );
+m_pResetBtn->SetClickHdl(
+LINK( this, SvxMenuConfigPage, ResetMenuHdl ) );
+
 }
 
 SvxMenuConfigPage::~SvxMenuConfigPage()
@@ -360,6 +363,34 @@ IMPL_LINK( SvxMenuConfigPage, InsertHdl, MenuButton *, 
pButton, void )
 }
 }
 
+IMPL_LINK_NOARG( SvxMenuConfigPage, ResetMenuHdl, Button *, void )
+{
+SvxConfigEntry* pMenuData = GetTopLevelSelection();
+
+ScopedVclPtrInstance qbox(this,
+CuiResId(RID_SVXSTR_CONFIRM_RESTORE_DEFAULT_MENU), 
VclMessageType::Question, VclButtonsType::YesNo);
+
+// Resetting individual top-level menus is not possible at the moment.
+// So we are resetting only if it is a context menu
+if (!m_bIsMenuBar && qbox->Execute() == RET_YES)
+{
+sal_Int32 nPos = m_pTopLevelListBox->GetSelectEntryPos();
+ContextMenuSaveInData* pSaveInData = static_cast< 
ContextMenuSaveInData* >(GetSaveInData());
+
+pSaveInData->ResetContextMenu(pMenuData);
+
+// ensure that the UI is cleared before populating it
+m_pTopLevelListBox->Clear();
+m_pContentsListBox->Clear();
+
+ReloadTopLevelListBox();
+
+// Reselect the resetted menu
+m_pTopLevelListBox->SelectEntryPos(nPos);
+m_pTopLevelListBox->GetSelectHdl().Call(*m_pTopLevelListBox);
+}
+}
+
 SaveInData* SvxMenuConfigPage::CreateSaveInData(
 const css::uno::Reference< css::ui::XUIConfigurationManager >& xCfgMgr,
 const css::uno::Reference< css::ui::XUIConfigurationManager >& 
xParentCfgMgr,
diff --git a/cui/source/customize/SvxToolbarConfigPage.cxx 
b/cui/source/customize/SvxToolbarConfigPage.cxx
index c9a87a4b0e23..fdf1837952d7 100644
--- a/cui/source/customize/SvxToolbarConfigPage.cxx
+++ b/cui/source/customize/SvxToolbarConfigPage.cxx
@@ -140,6 +140,9 @@ SvxToolbarConfigPage::SvxToolbarConfigPage(vcl::Window 
*pParent, const SfxItemSe
 
 m_pInsertBtn->SetSelectHdl(
 LINK( this, SvxToolbarConfigPage, InsertHdl ) );
+m_pResetBtn->SetClickHdl(
+LINK( this, SvxToolbarConfigPage, ResetToolbarHdl ) 

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

2017-08-24 Thread Noel Grandin
 filter/source/graphicfilter/idxf/dxf2mtf.cxx  |2 +-
 filter/source/graphicfilter/idxf/dxfentrd.cxx |   10 --
 filter/source/graphicfilter/idxf/dxfentrd.hxx |2 +-
 3 files changed, 6 insertions(+), 8 deletions(-)

New commits:
commit 4fb9d7b69c4fb10107210c2f851040f7c259e413
Author: Noel Grandin 
Date:   Wed Aug 23 14:57:41 2017 +0200

loplugin:useuniqueptr in filter

Change-Id: I127b570bb1563dc75b21633019b3640f93715d77
Reviewed-on: https://gerrit.libreoffice.org/41495
Tested-by: Jenkins 
Reviewed-by: Noel Grandin 

diff --git a/filter/source/graphicfilter/idxf/dxf2mtf.cxx 
b/filter/source/graphicfilter/idxf/dxf2mtf.cxx
index a30da0c25912..692151d4012b 100644
--- a/filter/source/graphicfilter/idxf/dxf2mtf.cxx
+++ b/filter/source/graphicfilter/idxf/dxf2mtf.cxx
@@ -574,7 +574,7 @@ void DXF2GDIMetaFile::DrawHatchEntity(const DXFHatchEntity 
& rE, const DXFTransf
 {
 for ( std::deque::size_type i = 0; i < 
rPathData.aEdges.size(); i++ )
 {
-const DXFEdgeType* pEdge = rPathData.aEdges[ i ];
+const DXFEdgeType* pEdge = rPathData.aEdges[ i ].get();
 switch( pEdge->nEdgeType )
 {
 case 1 :
diff --git a/filter/source/graphicfilter/idxf/dxfentrd.cxx 
b/filter/source/graphicfilter/idxf/dxfentrd.cxx
index 8c8eb309b4c0..9995ef8270d7 100644
--- a/filter/source/graphicfilter/idxf/dxfentrd.cxx
+++ b/filter/source/graphicfilter/idxf/dxfentrd.cxx
@@ -562,8 +562,6 @@ DXFBoundaryPathData::DXFBoundaryPathData() :
 
 DXFBoundaryPathData::~DXFBoundaryPathData()
 {
-for (auto i: aEdges)
-delete i;
 }
 
 bool DXFBoundaryPathData::EvaluateGroup( DXFGroupReader & rDGR )
@@ -619,10 +617,10 @@ bool DXFBoundaryPathData::EvaluateGroup( DXFGroupReader & 
rDGR )
 sal_Int32 nEdgeType = rDGR.GetI();
 switch( nEdgeType )
 {
-case 1 : aEdges.push_back( new DXFEdgeTypeLine() ); break;
-case 2 : aEdges.push_back( new DXFEdgeTypeCircularArc() ); 
break;
-case 3 : aEdges.push_back( new DXFEdgeTypeEllipticalArc() ); 
break;
-case 4 : aEdges.push_back( new DXFEdgeTypeSpline() ); break;
+case 1 : aEdges.emplace_back( new DXFEdgeTypeLine() ); break;
+case 2 : aEdges.emplace_back( new DXFEdgeTypeCircularArc() ); 
break;
+case 3 : aEdges.emplace_back( new DXFEdgeTypeEllipticalArc() 
); break;
+case 4 : aEdges.emplace_back( new DXFEdgeTypeSpline() ); break;
 }
 }
 else if ( aEdges.size() )
diff --git a/filter/source/graphicfilter/idxf/dxfentrd.hxx 
b/filter/source/graphicfilter/idxf/dxfentrd.hxx
index 85dbd32b2c5e..849ffeca6cd4 100644
--- a/filter/source/graphicfilter/idxf/dxfentrd.hxx
+++ b/filter/source/graphicfilter/idxf/dxfentrd.hxx
@@ -412,7 +412,7 @@ struct DXFBoundaryPathData
 sal_Int32   nPointIndex;
 
 std::unique_ptr pP;
-std::deque aEdges;
+std::deque> aEdges;
 
 DXFBoundaryPathData();
 ~DXFBoundaryPathData();
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'distro/escriba/escriba-5.2' - 16 commits - sw/qa sw/source

2017-08-24 Thread Miklos Vajna
 sw/qa/extras/odfimport/data/i61225.sxw   |binary
 sw/qa/extras/odfimport/odfimport.cxx |8 +
 sw/qa/extras/uiwriter/data/i95698.odt|binary
 sw/qa/extras/uiwriter/data/lines-in-section-in-table.odt |binary
 sw/qa/extras/uiwriter/data/rhbz739252-3.odt  |binary
 sw/qa/extras/uiwriter/data/table-in-sect.odt |binary
 sw/qa/extras/uiwriter/data/tdf108524.odt |binary
 sw/qa/extras/uiwriter/uiwriter.cxx   |  103 +++
 sw/source/core/inc/cellfrm.hxx   |1 
 sw/source/core/layout/findfrm.cxx|   20 +-
 sw/source/core/layout/flowfrm.cxx|5 
 sw/source/core/layout/sectfrm.cxx|   66 -
 sw/source/core/layout/tabfrm.cxx |   39 +
 13 files changed, 225 insertions(+), 17 deletions(-)

New commits:
commit 0d644aeba1580865173840f6b41e6f2f0ac5ccf0
Author: Miklos Vajna 
Date:   Wed Aug 16 17:19:05 2017 +0200

i#95698 sw: fix crash on splitting in-table section containing a nested 
table

Found by crashtesting, ooo95698-1.odt crashed sw layout on load. The
intended use-case is splitting section frames inside a table frame, so
can just blacklist the non-interesting table-in-section-in-table case
that causes the problem here.

Reviewed-on: https://gerrit.libreoffice.org/41224
Reviewed-by: Miklos Vajna 
Tested-by: Jenkins 
(cherry picked from commit 802477ae75b39194442d9c01a1342d068c7b9300)

Conflicts:
sw/qa/extras/uiwriter/uiwriter.cxx

Change-Id: Ic47cd8c46cc71f7eaa36b03ec2c4a5df8ca8051c

diff --git a/sw/qa/extras/uiwriter/data/i95698.odt 
b/sw/qa/extras/uiwriter/data/i95698.odt
new file mode 100644
index ..9fe3ec207648
Binary files /dev/null and b/sw/qa/extras/uiwriter/data/i95698.odt differ
diff --git a/sw/qa/extras/uiwriter/uiwriter.cxx 
b/sw/qa/extras/uiwriter/uiwriter.cxx
index fd9031de8516..7d5bb5982431 100644
--- a/sw/qa/extras/uiwriter/uiwriter.cxx
+++ b/sw/qa/extras/uiwriter/uiwriter.cxx
@@ -205,6 +205,7 @@ public:
 void testTdf108524();
 void testTableInSection();
 void testTableInNestedSection();
+void testTableInSectionInTable();
 void testLinesInSectionInTable();
 void testLinesMoveBackwardsInSectionInTable();
 
@@ -316,6 +317,7 @@ public:
 CPPUNIT_TEST(testTableInSection);
 CPPUNIT_TEST(testTableInNestedSection);
 CPPUNIT_TEST(testLinesInSectionInTable);
+CPPUNIT_TEST(testTableInSectionInTable);
 CPPUNIT_TEST(testLinesMoveBackwardsInSectionInTable);
 CPPUNIT_TEST_SUITE_END();
 
@@ -3846,6 +3848,14 @@ void SwUiWriterTest::testTableInNestedSection()
 assertXPath(pXmlDoc, "//page[2]//section/tab", 1);
 }
 
+void SwUiWriterTest::testTableInSectionInTable()
+{
+// The document has a table, containing a section, containing a nested
+// table.
+// This crashed the layout.
+createDoc("i95698.odt");
+}
+
 CPPUNIT_TEST_SUITE_REGISTRATION(SwUiWriterTest);
 CPPUNIT_PLUGIN_IMPLEMENT();
 
diff --git a/sw/source/core/layout/tabfrm.cxx b/sw/source/core/layout/tabfrm.cxx
index 45282d886929..0c6db983d50d 100644
--- a/sw/source/core/layout/tabfrm.cxx
+++ b/sw/source/core/layout/tabfrm.cxx
@@ -916,9 +916,21 @@ static bool lcl_FindSectionsInRow( const SwRowFrame& rRow )
 if (const SwFrame* pSectionLower = 
pTmpFrame->GetLower())
 {
 if (!pSectionLower->IsColumnFrame())
+{
 // Section has a single column only, try to
 // split that.
 bRet = false;
+
+for (const SwFrame* pFrame = pSectionLower; 
pFrame; pFrame = pFrame->GetNext())
+{
+if (pFrame->IsTabFrame())
+{
+// Section contains a table, no split 
in that case.
+bRet = true;
+break;
+}
+}
+}
 }
 }
 }
commit d67b100f44f7c320af713355dc6022451347447d
Author: Miklos Vajna 
Date:   Fri Jul 14 16:05:10 2017 +0200

CppunitTest_sw_uiwriter: disable testLinesMoveBackwards... on macOS

No idea off the top of my head what is the problem here, seeing Linux
and Windows is happy; clang on Linux as well.

Change-Id: I56c79b37a5648d9afd02d8e161ea4a279cc89744
(cherry picked from commit 39dd0121f5994dee56f95bc57fae3323bf849a20)

diff --git a/sw/qa/extras/uiwriter/uiwriter.cxx 
b/sw/qa/extras/uiwriter/uiwriter.cxx
index ef555d8b3ac4..fd9031de8516 100644
--- a/sw

[Libreoffice-commits] core.git: chart2/qa emfio/qa include/unotest sc/qa sd/qa svgio/qa sw/qa test/source unotest/source vcl/qa xmlsecurity/qa

2017-08-24 Thread Stephan Bergmann
 chart2/qa/extras/charttest.hxx |8 
 emfio/qa/cppunit/emf/EmfImportTest.cxx |4 ++--
 include/unotest/directories.hxx|   14 ++
 sc/qa/extras/htmlexporttest.cxx|2 +-
 sc/qa/unit/tiledrendering/tiledrendering.cxx   |2 +-
 sd/qa/unit/SVGExportTests.cxx  |2 +-
 sd/qa/unit/import-tests.cxx|2 +-
 sd/qa/unit/tiledrendering/tiledrendering.cxx   |2 +-
 svgio/qa/cppunit/SvgImportTest.cxx |4 ++--
 sw/qa/extras/globalfilter/globalfilter.cxx |4 ++--
 sw/qa/extras/inc/swmodeltestbase.hxx   |6 +++---
 sw/qa/extras/tiledrendering/tiledrendering.cxx |2 +-
 sw/qa/extras/uiwriter/uiwriter.cxx |2 +-
 test/source/screenshot_test.cxx|7 +++
 unotest/source/cpp/directories.cxx |   18 --
 vcl/qa/cppunit/pdfexport/pdfexport.cxx |2 +-
 xmlsecurity/qa/unit/pdfsigning/pdfsigning.cxx  |2 +-
 xmlsecurity/qa/unit/signing/signing.cxx|2 +-
 18 files changed, 36 insertions(+), 49 deletions(-)

New commits:
commit 66604baf08c47cc0c77ab3b9ee7e77f987c64722
Author: Stephan Bergmann 
Date:   Wed Aug 23 18:11:38 2017 +0200

Consistently use OUString in test::Directories

Change-Id: Ia5fd8af34ee9d2f37e4450ab241f6bb9a06445b2
Reviewed-on: https://gerrit.libreoffice.org/41466
Tested-by: Jenkins 
Reviewed-by: Stephan Bergmann 

diff --git a/chart2/qa/extras/charttest.hxx b/chart2/qa/extras/charttest.hxx
index b75dac6d25b5..438565222db4 100644
--- a/chart2/qa/extras/charttest.hxx
+++ b/chart2/qa/extras/charttest.hxx
@@ -76,10 +76,10 @@ public:
 void load( const OUString& rDir, const OUString& rFileName );
 std::shared_ptr save( const OUString& rFileName );
 std::shared_ptr reload( const OUString& rFileName );
-uno::Sequence < OUString > getImpressChartColumnDescriptions( const char* 
pDir, const char* pName );
+uno::Sequence < OUString > getImpressChartColumnDescriptions( const 
OUString& pDir, const char* pName );
 OUString getFileExtension( const OUString& rFileName );
 
-uno::Reference< chart::XChartDocument > getChartDocFromImpress( const 
char* pDir, const char* pName );
+uno::Reference< chart::XChartDocument > getChartDocFromImpress( const 
OUString& pDir, const char* pName );
 
 uno::Reference getChartDocFromDrawImpress( 
sal_Int32 nPage, sal_Int32 nShape );
 
@@ -497,7 +497,7 @@ std::vector > 
getDataSeriesLabelsFromChartType( const Re
 return aRet;
 }
 
-uno::Reference< chart::XChartDocument > ChartTest::getChartDocFromImpress( 
const char* pDir, const char* pName )
+uno::Reference< chart::XChartDocument > ChartTest::getChartDocFromImpress( 
const OUString& pDir, const char* pName )
 {
 mxComponent = loadFromDesktop(m_directories.getURLFromSrc(pDir) + 
OUString::createFromAscii(pName), 
"com.sun.star.comp.Draw.PresentationDocument");
 uno::Reference< drawing::XDrawPagesSupplier > xDoc(mxComponent, 
uno::UNO_QUERY_THROW );
@@ -562,7 +562,7 @@ uno::Reference 
ChartTest::getChartDocFromWriter( sal_Int3
 return xChartDoc;
 }
 
-uno::Sequence < OUString > ChartTest::getImpressChartColumnDescriptions( const 
char* pDir, const char* pName )
+uno::Sequence < OUString > ChartTest::getImpressChartColumnDescriptions( const 
OUString& pDir, const char* pName )
 {
 uno::Reference< chart::XChartDocument > xChartDoc = 
getChartDocFromImpress( pDir, pName );
 uno::Reference< chart::XChartDataArray > xChartData ( 
xChartDoc->getData(), uno::UNO_QUERY_THROW);
diff --git a/emfio/qa/cppunit/emf/EmfImportTest.cxx 
b/emfio/qa/cppunit/emf/EmfImportTest.cxx
index a78aaeabeba2..ca766b8b67f3 100644
--- a/emfio/qa/cppunit/emf/EmfImportTest.cxx
+++ b/emfio/qa/cppunit/emf/EmfImportTest.cxx
@@ -40,7 +40,7 @@ class Test : public test::BootstrapFixture, public 
XmlTestTools
 
 void testWorking();
 
-Primitive2DSequence parseEmf(const char* aSource);
+Primitive2DSequence parseEmf(const OUString& aSource);
 
 public:
 CPPUNIT_TEST_SUITE(Test);
@@ -48,7 +48,7 @@ public:
 CPPUNIT_TEST_SUITE_END();
 };
 
-Primitive2DSequence Test::parseEmf(const char* aSource)
+Primitive2DSequence Test::parseEmf(const OUString& aSource)
 {
 const Reference xEmfParser = EmfTools::create(m_xContext);
 
diff --git a/include/unotest/directories.hxx b/include/unotest/directories.hxx
index 04ee9287fe68..c1eaac83bafa 100644
--- a/include/unotest/directories.hxx
+++ b/include/unotest/directories.hxx
@@ -30,19 +30,17 @@ public:
 const OUString& getSrcRootURL()   { return m_aSrcRootURL; }
 const OUString& getSrcRootPath()  { return m_aSrcRootPath; }
 
-// return a URL to a given c-str path from the source directory
-OUString getURLFromSrc( const char *pPath );
+// return a URL to a given path from the source directory
 OUString getURLFromSrc( const OUString& rPath );
 
-// return a Path to a giv

[Libreoffice-commits] core.git: Changes to 'distro/escriba/escriba-5.2.1'

2017-08-24 Thread Christian Lohmaier
New branch 'distro/escriba/escriba-5.2.1' available with the following commits:
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: Rebuilding LO freezes on macOS

2017-08-24 Thread Shinnok
Thorsten,

I can't reproduce this with latest LO on macOS 10.12.6, so maybe this is 
specific to your build env and system.
Can you run make with the -d flag and when it hangs kill the build process and 
reply with the link command and following output?

PS: You did make sure you are not running out of disk space or other potential 
causes like that?

Shinnok

> On Aug 23, 2017, at 10:28 PM, Thorsten Wagner  
> wrote:
> 
> Dear all,
> 
> at least since LO 5.2 I have a strange problem concerning building on macOS:
> 
> (1) Build is ok after cleaning and building for the first time.
> 
> (2) During building again build freezes while linking 
> "Library/libscfiltlo.dylib". "make" loops consuming 100% cpu on a single 
> core. Build has to be interrupted by killing "make" processes.
> 
> (3) Issue is reproducible. Only workaround is cleaning and rebuilding 
> completely.
> 
> I'm building on OS X 10.11 using Xcode 8.2 command line tools.
> 
> Does everyone has seen this behaviour too?
> 
> Any heap would be appreciated.
> 
> brgds Thorsten
> ___
> LibreOffice mailing list
> LibreOffice@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/libreoffice

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


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

2017-08-24 Thread Noel Grandin
 include/vcl/graphicfilter.hxx   |1 -
 vcl/source/filter/graphicfilter.cxx |2 --
 2 files changed, 3 deletions(-)

New commits:
commit 844b3c067ccef7c8fcd38f4456689065dee5a447
Author: Noel Grandin 
Date:   Wed Aug 23 20:30:39 2017 +0200

remove unused error code in vcl

ever since

commit 920d4463f6e59b815852c173e2974ffc7b4bb284
Date:   Thu Jun 23 13:13:28 2016 +0200
loplugin:singlevalfields in vcl(part1)

Change-Id: I3691d03dadff10585a119ca8db8aa193b5a01702
Reviewed-on: https://gerrit.libreoffice.org/41492
Tested-by: Jenkins 
Reviewed-by: Noel Grandin 

diff --git a/include/vcl/graphicfilter.hxx b/include/vcl/graphicfilter.hxx
index 62ed0745a3d9..7f15760ee786 100644
--- a/include/vcl/graphicfilter.hxx
+++ b/include/vcl/graphicfilter.hxx
@@ -41,7 +41,6 @@ struct ConvertData;
 #define ERRCODE_GRFILTER_FORMATERROR  ErrCode(ErrCodeArea::Vcl, 
ERRCODE_CLASS_GENERAL | 3)
 #define ERRCODE_GRFILTER_VERSIONERROR ErrCode(ErrCodeArea::Vcl, 
ERRCODE_CLASS_GENERAL | 4)
 #define ERRCODE_GRFILTER_FILTERERROR  ErrCode(ErrCodeArea::Vcl, 
ERRCODE_CLASS_GENERAL | 5)
-#define ERRCODE_GRFILTER_ABORTErrCode(ErrCodeArea::Vcl, 
ERRCODE_CLASS_GENERAL | 6)
 #define ERRCODE_GRFILTER_TOOBIG   ErrCode(ErrCodeArea::Vcl, 
ERRCODE_CLASS_GENERAL | 7)
 
 #define GRFILTER_OUTHINT_GREY   1
diff --git a/vcl/source/filter/graphicfilter.cxx 
b/vcl/source/filter/graphicfilter.cxx
index a5a4c53ebc5d..47f755b368b3 100644
--- a/vcl/source/filter/graphicfilter.cxx
+++ b/vcl/source/filter/graphicfilter.cxx
@@ -2449,8 +2449,6 @@ ErrCode GraphicFilter::LoadGraphic( const OUString 
&rPath, const OUString &rFilt
 aReturnString="version error";
 else if (nRes == ERRCODE_GRFILTER_FILTERERROR)
 aReturnString="filter error";
-else if (nRes == ERRCODE_GRFILTER_ABORT)
-aReturnString="import aborted";
 else if (nRes == ERRCODE_GRFILTER_TOOBIG)
 aReturnString="graphic is too big";
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits