Commit Graph

400411 Commits

Author SHA1 Message Date
Michael Stahl
745fe08e18 svl: convert some legacy DBG_ASSERT
Change-Id: I8fc4d849e434e245d871dc2d2eef42713e1359bb
2016-12-01 23:29:48 +01:00
Michael Stahl
bac1c69c65 sfx2: convert legacy DBG_ASSERT in SfxDocumentMetaData
Change-Id: Ic340d7226ea3af4bbd6e52c415b32d9be7cd453f
2016-12-01 23:29:48 +01:00
Noel Grandin
036603d72f convert nowParsing_ constants to scoped enum
Change-Id: I8df9ca0e9317a4d969f492699be926044415f68c
Reviewed-on: https://gerrit.libreoffice.org/31483
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
2016-12-01 21:51:41 +00:00
Noel Grandin
69c3e6bf84 fix bug in FmFormShell::GetState
Found when attempting an enum conversion.

Not sure what effect this will have, it was introduced with

    commit a0c7b2bc45
    Author: Oliver Bolte <obo@openoffice.org>
    Date:   Tue Nov 16 10:24:21 2004 +0000
    INTEGRATION: CWS eforms2 (1.48.52); FILE MERGED

which pushed a whole bunch of stuff together.

Change-Id: I571fc228c3c0e164aba261e49233ef68ba87e77d
Reviewed-on: https://gerrit.libreoffice.org/31491
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Tested-by: Julien Nabet <serval2412@yahoo.fr>
2016-12-01 21:47:52 +00:00
Caolán McNamara
8cc096b2f4 ditch old SAL_DO_NOT_USE_INVERT50 env var
Change-Id: I1b23db2335827cab6f1b98b40103b0508928b66e
2016-12-01 21:17:30 +00:00
Caolán McNamara
f1c08d6503 crashtesting: a gadzillion failures on export to odp
e.g. fdo35235-1.odp

Change-Id: I259cdc9ed073be2ad7d5208cd943d4f193f09c16
2016-12-01 21:16:22 +00:00
Miklos Vajna
c4cb8b5d14 xmlsecurity PDF verify: avoid seeking before the start of the stream
Happened when the doc was smaller than 1024 bytes.

Change-Id: Ie5eea5905a09722e7958495d26e6c78ee234d3ba
Reviewed-on: https://gerrit.libreoffice.org/31500
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
2016-12-01 19:35:22 +00:00
Miklos Vajna
b1f91c0a04 xmlsecurity PDF verify: handle no EOL at EOF
From a comment's point of view, EOF is just a terminator, similar to \r
or \n.

Change-Id: I120bf1e75f1eb81a550af643051e6fc472873eff
Reviewed-on: https://gerrit.libreoffice.org/31499
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
2016-12-01 19:34:52 +00:00
Miklos Vajna
cdf2ae1b66 xmlsecurity PDF verify: fix reading names containing ']'
Also fix parsing '<< /Foo [ /Bar ] >>'.

Change-Id: I3375001730b4d2e447b0dd8a7809a7bfb013126c
Reviewed-on: https://gerrit.libreoffice.org/31498
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
2016-12-01 19:34:25 +00:00
Miklos Vajna
e84993486b xmlsecurity PDF verify: don't abort read on partial sign
Map it to the partially signed (not all streams) ODF concept instead.

Change-Id: I7fc931e622b9f10a1261cd475b01a2f038e37ece
Reviewed-on: https://gerrit.libreoffice.org/31497
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
2016-12-01 19:34:14 +00:00
Khaled Hosny
dacd0e9419 Unused code
Change-Id: Idfc964930c242d752a78cd109d75d809bce4de11
Reviewed-on: https://gerrit.libreoffice.org/31470
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Khaled Hosny <khaledhosny@eglug.org>
2016-12-01 19:13:12 +00:00
Michael Meeks
aa68c99d88 tdf#104126 - comphelper thread-pool, use reliable std::condition_variable.
The existing osl::Condition is an API and reliability disaster area.

Change-Id: I3be84e1c6a83e58c43c40c9c8720790d923a6694
Reviewed-on: https://gerrit.libreoffice.org/31163
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
Tested-by: Michael Meeks <michael.meeks@collabora.com>
2016-12-01 18:44:08 +00:00
Mike Kaganski
0afbe8d5ca tdf#104310: Accept also x14-style dataValidations
See https://msdn.microsoft.com/en-us/library/dd921584

Change-Id: I66c9474cbf83cea10ab0e7c2b44592673c8b683f
Reviewed-on: https://gerrit.libreoffice.org/31456
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-12-01 17:37:43 +00:00
Muhammet Kara
83b7e1f424 Add toolbar button for page orientation
Change-Id: Ia9458f8ac32bb8c6da6fc08e5fee527ca6fb8bd5
Reviewed-on: https://gerrit.libreoffice.org/31473
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Katarina Behrens <Katarina.Behrens@cib.de>
2016-12-01 17:36:16 +00:00
Jean-Tiare Le Bigot
18b3138a7a tdf#102621: import empty chart cells as NaN instead of 0
Change-Id: I574c3f719e52bc2244597532783130564621a891
Reviewed-on: https://gerrit.libreoffice.org/31303
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-12-01 17:32:03 +00:00
Michael Stahl
6982598524 gbuild: replace -O0 with -Og for GCC --enable-debug/dbgutil builds
With Fedora 25 GCC 6.2.1 and my usual config (except without
--with-package-format since that is slowest now) on commit
086631af59 i get with CCACHE_DISABLE=1:

With -O0:

"make check"
44490.70user 3127.01system 1:15:23elapsed 1052%CPU (0avgtext+0avgdata 2541312maxresident)k

"make check", incremental
7368.19user 153.98system 15:30.06elapsed 808%CPU (0avgtext+0avgdata 510200maxresident)k

"make CppunitTest_sc_functions_test"
272.13user 0.65system 4:33.47elapsed 99%CPU (0avgtext+0avgdata 258884maxresident)k

With -Og:

"make check"
46076.01user 3170.91system 1:16:19elapsed 1075%CPU (0avgtext+0avgdata 2613716maxresident)k

"make check", incremental
5478.33user 157.07system 11:50.42elapsed 793%CPU (0avgtext+0avgdata 454980maxresident)k

"make CppunitTest_sc_functions_test"
188.48user 0.62system 3:09.59elapsed 99%CPU (0avgtext+0avgdata 259096maxresident)k

So we now have so many tests that the time taken by GCC doing optimization
is brought back via the tests finishing faster, particularly if we
assume a non-negligible ccache hit rate for developers in practice.

Add this to gb_DEBUG_CFLAGS instead of gb_COMPILERNOOPTFLAGS because
presumably the bridges code that uses gb_COMPILERNOOPTFLAGS really wants
-O0; the gb_DEBUG_CFLAGS is added later so -Og overrides -O0.

It is an open question how well debugging in gdb actually works with
-Og, some experimentation is needed; "man gcc" claims:

       If you are not using some other optimization option, consider using -Og
       with -g.  With no -O option at all, some compiler passes that collect
       information useful for debugging do not run at all, so that -Og may
       result in a better debugging experience.

Change-Id: I103499f398b69397cf5aa9993a242680966ce999
Reviewed-on: https://gerrit.libreoffice.org/31334
Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com>
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-12-01 16:50:11 +00:00
Luke Deller
d6a1ba29a5 enable optimization (-Og) with --enable-debug
gcc-4.8 introduced a new optimization level -Og which enables
optimizations that do not interfere with debugging.

When configured with --enable-debug, use -Og rather than -O0
if available.

Change-Id: Iff3f98d736681ae34e49b96510228a14ce456b34
Reviewed-on: https://gerrit.libreoffice.org/31333
Reviewed-by: Michael Stahl <mstahl@redhat.com>
Tested-by: Michael Stahl <mstahl@redhat.com>
2016-12-01 16:48:52 +00:00
Miklos Vajna
c623a9f846 xmlsecurity PDF verify: handle boolean type as dictionary value
This caused not finding the length of a stream -> could not actually
verify signature.

Change-Id: I696b6da49525eb53f7575c27f619d2116be51f1d
Reviewed-on: https://gerrit.libreoffice.org/31490
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Tested-by: Jenkins <ci@libreoffice.org>
2016-12-01 16:26:13 +00:00
Matúš Kukan
0b66adec8f tdf#90753: AutoInstall gengal with filelist feature
Change-Id: Ic72fb18eccb54b24f4205d997585cee753965b10
2016-12-01 17:07:54 +01:00
Matúš Kukan
39a4ca4072 tdf#97803: gbuild: Check that resource targets are registered
And replace gb_AllLangResTarget_ALLTARGETS with
gb_AllLangResTarget_REGISTERED which should have
the same content and is already used.

This actually helped to find a problem, fixed in
52d409f0d6

Change-Id: Iae551d7be221c5655dee1bc9ad273c8822d45178
2016-12-01 17:07:54 +01:00
Stephan Bergmann
4f7280e4cc BitmapFilter, BitmapScaleSuper only used within vcl
Change-Id: Ia8a1207a584b599f01c47b658692d3eeae52cb3a
2016-12-01 16:02:12 +01:00
Zdeněk Crhonek
6c1e4d4629 add MODE test case
Change-Id: Ib6aecbcc993f2b8898a2c9855181d3b7db1f9840
Reviewed-on: https://gerrit.libreoffice.org/31444
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-12-01 13:57:11 +00:00
Miklos Vajna
9b526a43e1 xmlsecurity PDF verify: CR is also a terminator of a comment
If we skip to the first NL, then we start tokenizing some XML as PDF
data and soon error out due to an unexpected keyword.

Change-Id: I86b540a014e5a92ea4376ed765385a2ee568a3c1
Reviewed-on: https://gerrit.libreoffice.org/31472
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
2016-12-01 12:44:52 +00:00
Miklos Vajna
0233ff9523 xmlsecurity PDF verify: tolerate missing %%EOF in incremental updates
This is broken, but work it around to avoid an infinite loop.

Change-Id: I132a3c19cfe53e6166bfc1a881d1bfa5071f85d4
Reviewed-on: https://gerrit.libreoffice.org/31471
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
2016-12-01 12:29:32 +00:00
Stephan Bergmann
d36b3bcb7e Fix race in ThreadTaskTag
Assume T0 calls ThreadPool::pushTask twice, then ThreadPool::waitUntilDone:

T0 calls ThreadTaskTag::onTaskPushed:
  mnTasksWorking = 1, maTasksComplete.reset

T1 runs the 1st task to completion and calls ThreadTaskTag::onTaskWorkerDone:
  mnTasksWorking = 0; suspended...

T0 calls ThreadTaskTag::onTaskPushed:
  mnTaskWorking = 1, maTasksComplete.reset

T1 continues in the call to ThreadTaskTag::onTaskWorkerDone:
  ..., maTasksComplete.set

T0 calls ThreadTaskTag::waitUntilDone and immediately returns

T2 only now starts to run the 2nd task

Change-Id: Ic29101a4791fca2a1a4d54b559f10ff706e8a20d
2016-12-01 12:35:36 +01:00
Stephan Bergmann
659210773a loplugin:unnecessaryoverride (dtors) in sal
Change-Id: I498bb634f2a5800f5effab99227aa4699ed91aae
2016-12-01 12:19:19 +01:00
Stephan Bergmann
8bd3d06d97 Throw RuntimeException with message
...seen JunitTest_sc_unoapi_4 fail once in
sc.ScHeaderFooterTextCursor::com::sun::text::XTextRange with an empty
RuntimeException when calling some remote getString.

Change-Id: Id631feffce810b40825fe0fa49d8f1846f045033
2016-12-01 11:02:39 +01:00
Miklos Vajna
055fd58711 xmlsecurity PDF verify: support non-detached signatures
And a couple of other changes to accept the bugdoc from
<https://github.com/esig/dss/
dss-pades/target/test-classes/plugtest/esig2014/ESIG-PAdES/RO/Signature-P-RO-4.pdf>.

Change-Id: I0fca9ba0bfe927ef91ae2592a5026b05d19879fd
Reviewed-on: https://gerrit.libreoffice.org/31462
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Tested-by: Jenkins <ci@libreoffice.org>
2016-12-01 09:49:29 +00:00
Miklos Vajna
7920ba294b xmlsecurity: instantiate SAXEventKeeperImpl directly in XSecController
Going via UNO for a class in the same module is an overkill.

Change-Id: I577660513022fde1576df19b412fcdb1ee2ad041
Reviewed-on: https://gerrit.libreoffice.org/31461
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Tested-by: Jenkins <ci@libreoffice.org>
2016-12-01 09:42:07 +00:00
Khaled Hosny
6c436ba09c Kill old Windows layout engines
Change-Id: I33f8322a6371150698bf926165fb6dddb9d4092c
Reviewed-on: https://gerrit.libreoffice.org/31452
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Khaled Hosny <khaledhosny@eglug.org>
2016-12-01 08:50:01 +00:00
Muthu Subramanian
6e5e55c29e tdf#103477 Add unit test
Change-Id: Ic960da6a479523a9255357d5f4cede212ff9c6a2
Reviewed-on: https://gerrit.libreoffice.org/30404
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: jan iversen <jani@documentfoundation.org>
2016-12-01 08:46:01 +00:00
V Stuart Foote
9841942508 tdf#80196 - another attempt at renaming to tonal.soc and adjusting names
now for 5.4.0 master and dropping abreviations

Change-Id: I72dbbab2688709e8266be93b40040c4c6c970d55
Reviewed-on: https://gerrit.libreoffice.org/31217
Reviewed-by: Heiko Tietze <tietze.heiko@googlemail.com>
Reviewed-by: jan iversen <jani@documentfoundation.org>
Tested-by: jan iversen <jani@documentfoundation.org>
2016-12-01 08:38:51 +00:00
Stephan Bergmann
1222f58dcf loplugin:unnecessaryoverride (dtors) in testtools
Change-Id: If004c2c58cb92e211cea34b54c3c38c91ccb6e7d
2016-12-01 08:54:56 +01:00
Stephan Bergmann
33c34f4b87 loplugin:unnecessaryoverride (dtors) in toolkit
Change-Id: Id95c044e75eccb0f394fd31e4776eda4c023341d
2016-12-01 08:54:40 +01:00
Stephan Bergmann
b97f01c545 loplugin:unnecessaryoverride (dtors) in ucbhelper
Change-Id: I7d77af190087e5f7190849c1e316c9d4f178a6c2
2016-12-01 08:54:20 +01:00
Stephan Bergmann
06ce64bdc4 loplugin:unnecessaryoverride (dtors) in unotest
Change-Id: Ib9114cafb3732d11017b98405d1ca60052801675
2016-12-01 08:54:05 +01:00
Stephan Bergmann
182dfdd00e loplugin:unnecessaryoverride (dtors) in xmloff
Change-Id: I351ff5955d766118e3a6b2234181774af0d1f522
2016-12-01 08:53:50 +01:00
Muthu Subramanian
3770a6a3f3 tdf#103477 buClrTx => use text color instead
Change-Id: I82d9231d2d0ea95cadc803f9a8dab9c23b88a677
Reviewed-on: https://gerrit.libreoffice.org/30403
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: jan iversen <jani@documentfoundation.org>
2016-12-01 07:40:11 +00:00
Matúš Kukan
985e93ecb4 fix cross-toolset: gb_Helper_optional_for_host can't contain 'newline'
Otherwise, the 'newline' is passed as a second parameter to
gb_Module_add_moduledir which breaks build. :/

Change-Id: I02eba5ddef9f7a6dccbb7d135a7e8c68629da9c4
2016-12-01 07:46:56 +01:00
Zdeněk Crhonek
c5bc782048 add MINA test case
Change-Id: I746f85503aef14d3e5730ffcf67689ae00dc8f10
Reviewed-on: https://gerrit.libreoffice.org/31442
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-11-30 22:51:29 +00:00
Zdeněk Crhonek
931facfec4 add MINIFS test case
Change-Id: I7b4403dda5e4f06ad51618f3aac50b17103ee265
Reviewed-on: https://gerrit.libreoffice.org/31443
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-11-30 22:50:37 +00:00
Zdeněk Crhonek
8b39a58d02 add MODE.SNGL test file
Change-Id: I1943547d22df7580467814300e4d372fd20b197f
Reviewed-on: https://gerrit.libreoffice.org/31448
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-11-30 22:50:04 +00:00
Markus Mohrhard
1b15f2f280 split the functions test in even more categories
Change-Id: I3ca640a24e95d1e6e0ad66be7bf09c45d63b52bb
Reviewed-on: https://gerrit.libreoffice.org/31447
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-11-30 22:47:56 +00:00
Markus Mohrhard
ff22a22286 better varyColors export for charts, related tdf#103943
Change-Id: I4280e708c854c687b6281c56d5bccdb514afd81e
Reviewed-on: https://gerrit.libreoffice.org/31252
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-11-30 22:33:51 +00:00
Markus Mohrhard
f45af7c809 start to split the functions test by category
Change-Id: Ib06eaee7cf5eeb3dbe95ad42afc2a17caa0359c2
Reviewed-on: https://gerrit.libreoffice.org/31440
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-11-30 22:33:25 +00:00
Mark Page
64e6e7c6e7 Use smart pointers for gdi pdf functions
Change-Id: Ia78adfbd0d07449e12a7e0d02acf8a1a1108437c
Reviewed-on: https://gerrit.libreoffice.org/31421
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-11-30 22:08:03 +00:00
Mark Page
0de5101df0 When overreading gif sub blocks, do not read skipped bytes
We do not require to read in gif extension data when it is ignored
This is used to skip the Adobe XMP gif extension

Change-Id: Id64f373a4eae764227c0b7ac6687697666991ec7
Reviewed-on: https://gerrit.libreoffice.org/31410
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-11-30 22:05:58 +00:00
Mark Page
5de4492eee Use smart pointers for allocated objects in the gif loader
Change-Id: Ib9d3bc05fbccc882df8913d3ffcb36be6a9f06dd
Reviewed-on: https://gerrit.libreoffice.org/31401
Reviewed-by: Michael Stahl <mstahl@redhat.com>
Tested-by: Michael Stahl <mstahl@redhat.com>
2016-11-30 22:01:22 +00:00
Matúš Kukan
52d409f0d6 gbuild: Prevent wrong use of gb_Helper_optional_for_host
It's empty, unless gb_Side=build.
This fixes commit 1193443b15

Change-Id: I83efad36d8f40c460d09ee1bd1addd7fb26c177b
Reviewed-on: https://gerrit.libreoffice.org/31419
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-11-30 21:56:09 +00:00
Jochen Nitschke
61f61e7723 don't hold vector as pointer here ...
and make impl types private

Change-Id: I7ead64bac38da3d813e0b47fa124970937c36617
Reviewed-on: https://gerrit.libreoffice.org/31438
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-11-30 21:46:11 +00:00