Commit graph

444054 commits

Author SHA1 Message Date
Noel Grandin
41b752756e tdf#128845 writer, cannot modify a bibliography entry
regression from
    commit 3b16e997f6
    Date:   Fri Jan 18 15:38:07 2019 +0200
    cleanup SwAuthEntry manual ref-counting

Change-Id: I30bd7af3cea51cedf5ca9bc43171665fb71830aa
Reviewed-on: https://gerrit.libreoffice.org/84388
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2019-12-04 09:43:16 +01:00
Miklos Vajna
ac5970258e CppunitTest_svx_unit: simplify testSemiTransparentText
Reading SvxXShadowPreview::Paint() shows how to use
sdr::contact::ObjectContactOfObjListPainter, which makes it possible to
avoid the whole chain of dummy SdrView, SdrPageView and SdrPageWindow,
just to turn an SdrObject into primitives.

Change-Id: I10c85476af20a2addbd070198b9c75341275ff8d
Reviewed-on: https://gerrit.libreoffice.org/84375
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
2019-12-04 09:01:40 +01:00
Noel Grandin
30c707666d tdf#129107 objects in basic disappear
Reverts part of "loplugin:useuniqueptr in SbModule"
   This reverts commit 263d732569.

because SbClassModuleObject is playing silly buggers with
ownership by messing with fields in its SbModule superclass.

Change-Id: I725332d080663e94b57f4bd4e1fb05aeeddf9038
Reviewed-on: https://gerrit.libreoffice.org/84352
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2019-12-04 08:59:40 +01:00
Stephan Bergmann
b7597b4525 use scoped enum
Change-Id: I8eadedbdf5f4bee218ae2f39c76300fc0624912b
Reviewed-on: https://gerrit.libreoffice.org/84374
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2019-12-04 08:49:55 +01:00
Chris Sherlock
3cde4fc908 drawinglayer: improve logging of EmfPlusStringFormat objects
Change-Id: I9bc28525c143e49dcbea017b73d40e690e1e7460
Reviewed-on: https://gerrit.libreoffice.org/84271
Tested-by: Jenkins
Reviewed-by: Bartosz Kosiorek <gang65@poczta.onet.pl>
2019-12-04 08:12:01 +01:00
Chris Sherlock
834a03f918 vcl: fix number of polygons shown in SAL_INFO message
Change-Id: I9a2c28f465b881583caae8d865d63b701213f8ee
Reviewed-on: https://gerrit.libreoffice.org/84139
Tested-by: Jenkins
Reviewed-by: Bartosz Kosiorek <gang65@poczta.onet.pl>
2019-12-04 08:10:16 +01:00
andreas kainz
c1b084de6f tdf#124258 Draw NB groupedbar_full not available for draw
Change-Id: Iff6768a377f9bcc6fb3da997ded90d0e78e8b9f3
Reviewed-on: https://gerrit.libreoffice.org/84382
Tested-by: Jenkins
Reviewed-by: andreas_kainz <kainz.a@gmail.com>
2019-12-04 07:59:09 +01:00
andreas kainz
2f0eed452c tdf#101513 arrangement of customizeable items in NB tabbed
Change-Id: I1cef677fcc91403dbaa92fd8dcf8a5fcc5bc6b03
Reviewed-on: https://gerrit.libreoffice.org/84383
Tested-by: Jenkins
Reviewed-by: andreas_kainz <kainz.a@gmail.com>
2019-12-04 07:58:12 +01:00
andreas kainz
68d9869e15 tdf#128927 groupedbar compact replace color with filter
Change-Id: Ie907e80c58d079262369fdfdce83e4b613a366bb
Reviewed-on: https://gerrit.libreoffice.org/84381
Tested-by: Jenkins
Reviewed-by: andreas_kainz <kainz.a@gmail.com>
2019-12-04 07:57:30 +01:00
Stephan Bergmann
ebdadcbe6a use scoped enum
Change-Id: I1b8604e8f06eaea39fbd505e632fc090c0a37d0e
Reviewed-on: https://gerrit.libreoffice.org/84379
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2019-12-04 07:45:57 +01:00
László Németh
5ac21d46f2 tdf#117982 Writer: fix invisible cell content due to spacing
If the cell content with its bottom paragraph spacing
is greater than the fix cell height, the content was
invisible.

Change-Id: I5922941dc0941950912a7ea2cec0aedf1079544c
Reviewed-on: https://gerrit.libreoffice.org/84010
Tested-by: Jenkins
Reviewed-by: László Németh <nemeth@numbertext.org>
2019-12-04 07:12:51 +01:00
Andrea Gelmini
90417749fa Removed duplicated includes
Change-Id: Ica64e5b201689930ff400070fb773832d4a97d78
Reviewed-on: https://gerrit.libreoffice.org/84191
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
2019-12-04 07:06:49 +01:00
Noel Grandin
8d84a0dc8d some std::set->o3tl::sorted_vector
which is much better cache-wise

Change-Id: Iacec0df48f043bcdd761ed30c1b1513850a00152
Reviewed-on: https://gerrit.libreoffice.org/84311
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2019-12-04 07:01:53 +01:00
andreas kainz
60e8941fd5 tdf#101513 add customizeable option to writer NB grouped compact
Change-Id: I3e8af6cee6b541f3208c2ad86dfe2bdc5ef83683
Reviewed-on: https://gerrit.libreoffice.org/84282
Reviewed-by: andreas_kainz <kainz.a@gmail.com>
Tested-by: andreas_kainz <kainz.a@gmail.com>
2019-12-03 23:01:15 +01:00
andreas kainz
ebf3de3f17 Glade remove GtkMenuItem widget visible False item
Change-Id: Ibe88210114c8b0dc483d8fe6a36d13dc0fcd599c
Reviewed-on: https://gerrit.libreoffice.org/84333
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-12-03 22:21:04 +01:00
Stephan Bergmann
b8bb7fd853 Adapt SAL_WARN to C++20 deleted ostream << for sal_Unicode (aka char16_t)
<http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2019/p1423r3.html> "char8_t
backward compatibility remediation", as implemented now by <https://gcc.gnu.org/
git/?p=gcc.git;a=commit;h=0c5b35933e5b150df0ab487efb2f11ef5685f713> "libstdc++:
P1423R3 char8_t remediation (2/4)" for -std=c++2a, deletes operator << overloads
that would print an integer rather than a (presumably expected) character.

Change-Id: Ic70d3e90e4b990d297e35f07379fe4952e138820
Reviewed-on: https://gerrit.libreoffice.org/84321
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2019-12-03 22:17:21 +01:00
Caolán McNamara
41b8d8eb67 cid#1456145 Missing move assignment operator
Change-Id: I55ed7d87def93a66668f5cf4f90fed1e1e5bcbee
Reviewed-on: https://gerrit.libreoffice.org/84256
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-12-03 22:14:56 +01:00
Caolán McNamara
c10182573e toolbar item visibilty defaults to false in GtkBuilder but true in vcl
Change-Id: I91152db48fee77b95fa542790839a574b87161db
Reviewed-on: https://gerrit.libreoffice.org/84361
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-12-03 22:14:34 +01:00
Stephan Bergmann
1bfd91e1b3 Use o3tl::doAccess, prevent -Werror=maybe-uninitialized
Change-Id: Id40ebafeb227c432af8689b7db246a3615bc482b
Reviewed-on: https://gerrit.libreoffice.org/84344
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2019-12-03 21:30:07 +01:00
Stephan Bergmann
5d8f0fad50 Adapt CPPUNIT_ASSERT to C++20 deleted ostream << for sal_Unicode (aka char16_t)
<http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2019/p1423r3.html> "char8_t
backward compatibility remediation", as implemented now by <https://gcc.gnu.org/
git/?p=gcc.git;a=commit;h=0c5b35933e5b150df0ab487efb2f11ef5685f713> "libstdc++:
P1423R3 char8_t remediation (2/4)" for -std=c++2a, deletes operator << overloads
that would print an integer rather than a (presumably expected) character.

But for simplicity (and to avoid issues with non-printing characters), keep
printing an integer here.

Change-Id: I751b99ee32d418eb488131ffa130d6f7d6d38dc7
Reviewed-on: https://gerrit.libreoffice.org/84348
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2019-12-03 21:18:57 +01:00
Caolán McNamara
f3a63a7a20 drop some unnecessary includes
Change-Id: Ic88ed0d36f1aae9f32a0097354042526fc18d649
Reviewed-on: https://gerrit.libreoffice.org/84244
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-12-03 21:05:02 +01:00
Caolán McNamara
eef7c4d30b weld OAddFieldWindow
Change-Id: I1e0ca144943fd37769c7b6cfc7e77c4f001b31a0
Reviewed-on: https://gerrit.libreoffice.org/84233
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-12-03 21:04:45 +01:00
Stephan Bergmann
1f0c54c250 Adapt to C++20 deleted ostream << for sal_Unicode* (aka char16_t*)
<http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2019/p1423r3.html> "char8_t
backward compatibility remediation", as implemented now by <https://gcc.gnu.org/
git/?p=gcc.git;a=commit;h=0c5b35933e5b150df0ab487efb2f11ef5685f713> "libstdc++:
P1423R3 char8_t remediation (2/4)" for -std=c++2a, deletes operator << overloads
that would print a pointer rather than a (presumably expected) string.

But here it should be fine to print pointers, esp. as there are null pointers
involved.

Change-Id: I62fad4cb9eaaa612989f035f686086ef29093d70
Reviewed-on: https://gerrit.libreoffice.org/84351
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2019-12-03 20:56:53 +01:00
Stephan Bergmann
4e62db5876 Use o3tl::doAccess, prevent -Werror=maybe-uninitialized
Change-Id: I6a86db428dcf92083ee13298417b3d3027e45822
Reviewed-on: https://gerrit.libreoffice.org/84338
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2019-12-03 20:56:29 +01:00
Stephan Bergmann
941d53eee9 Use o3tl::doAccess, prevent -Werror=maybe-uninitialized
(and do not unbox Any where it is not necessary anyway)

Change-Id: Ic5461fb8b62157e9116cd2d557ee38fface28fb7
Reviewed-on: https://gerrit.libreoffice.org/84337
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2019-12-03 20:56:15 +01:00
Stephan Bergmann
ebd70d476c Also throw IllegalArgumentException for arguments of wrong type
Change-Id: I1b52accc3f0eec3e6232b8211bf7bcbf65ed18f8
Reviewed-on: https://gerrit.libreoffice.org/84350
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2019-12-03 20:22:03 +01:00
Stephan Bergmann
8eb8d64fa1 Don't count terminating newline when determining line length
Change-Id: I3a5d306f32697e160f008e73de38fc53f2f1dbda
Reviewed-on: https://gerrit.libreoffice.org/84349
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2019-12-03 20:21:33 +01:00
Stephan Bergmann
4b8ed95e33 Use o3tl::doAccess, prevent -Werror=maybe-uninitialized
Change-Id: Ibc7bffe3bb2bf67f85102c03f48e3b44d89c60cc
Reviewed-on: https://gerrit.libreoffice.org/84334
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2019-12-03 20:21:12 +01:00
Noel Grandin
f552271514 loplugin:mergeclasses ListBoxAccessibleBase with AccessibleListBoxEntry
Change-Id: I9173e64b7ddef3184cacbce55b060d677e7ec1d0
Reviewed-on: https://gerrit.libreoffice.org/84347
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2019-12-03 20:01:46 +01:00
Stephan Bergmann
4eb7a154e8 Silence false -Werror=maybe-uninitialized
...where success of >>= has already been checked with CPPUNIT_ASSERT

Change-Id: I127d559ab0bf212e4a32db4f297954d3b94032fc
Reviewed-on: https://gerrit.libreoffice.org/84345
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2019-12-03 19:45:04 +01:00
Stephan Bergmann
e8cb5f2e11 Adapt to C++20 deleted ostream << for sal_Unicode (aka char16_t)
<http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2019/p1423r3.html> "char8_t
backward compatibility remediation", as implemented now by <https://gcc.gnu.org/
git/?p=gcc.git;a=commit;h=0c5b35933e5b150df0ab487efb2f11ef5685f713> "libstdc++:
P1423R3 char8_t remediation (2/4)" for -std=c++2a, deletes operator << overloads
that would print an integer rather than a (presumably expected) character.

But in these cases printing an integer is as expected, so add explicit casts.

Change-Id: I7c2f1afaa2982b284aef8af183b71466c37142c2
Reviewed-on: https://gerrit.libreoffice.org/84339
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2019-12-03 19:44:37 +01:00
Stephan Bergmann
815fad1e55 Silence false -Werror=maybe-uninitialized
...where success of >>= has already been checked with CPPUNIT_ASSERT

Change-Id: I9aa553749988b6b2e26d9a5ac5b376cc5997aba7
Reviewed-on: https://gerrit.libreoffice.org/84335
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2019-12-03 19:42:51 +01:00
Noel Grandin
0cf64cf29c SAL_INFO_IF -> SAL_WARN_IF
this is a very very useful warning when libraries fail to load

Change-Id: I09bf64f6c65f285d6ab41f988b255a4842233428
Reviewed-on: https://gerrit.libreoffice.org/84314
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2019-12-03 19:39:22 +01:00
Stephan Bergmann
00262b0898 Drop unnecessary o3tl::optional::reset
...where xRangeList goes out of scope anyway

Change-Id: Ibb107ac7eee5504a161749f141997bd96fd07e83
Reviewed-on: https://gerrit.libreoffice.org/84336
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2019-12-03 18:46:55 +01:00
Stephan Bergmann
caf2e75b4c use scoped enum
Change-Id: Ib40fad986f99d02e58f0564fb68608e86989173c
Reviewed-on: https://gerrit.libreoffice.org/84325
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2019-12-03 16:28:34 +01:00
Stephan Bergmann
5c5a56797d use scoped enum
Change-Id: If2dce427fd8d52224996aaef93081d5a2e239893
Reviewed-on: https://gerrit.libreoffice.org/84322
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2019-12-03 16:28:19 +01:00
Stephan Bergmann
48f845dace external/clucene: Adapt to C++20 deleted ostream << for non-plain char types
<http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2019/p1423r3.html> "char8_t
backward compatibility remediation", as implemented now by <https://gcc.gnu.org/
git/?p=gcc.git;a=commit;h=0c5b35933e5b150df0ab487efb2f11ef5685f713> "libstdc++:
P1423R3 char8_t remediation (2/4)" for -std=c++2a, deletes operator << overloads
that would print a pointer rather than a (presumably expected) string.

So this infoStream output appears to have always been broken (the strings use
TCHAR, which appears to unconditionally be a typedef for wchar_t, see
workdir/UnpackedTarball/clucene/src/shared/CLucene/clucene-config.h), and
appears to be just of informative nature, so just simplify it to not try to
print any problematic parts.

Change-Id: Ie9f8edb03aff461a15718a0c025af57004aba0a9
Reviewed-on: https://gerrit.libreoffice.org/84320
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2019-12-03 16:06:05 +01:00
Stephan Bergmann
ff7d323f2d use scoped enum
Change-Id: Id28d593d94fc176194871db32c1c5a287b98a26a
Reviewed-on: https://gerrit.libreoffice.org/84319
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2019-12-03 15:52:14 +01:00
Stephan Bergmann
141722374d use scoped enum
Change-Id: Ife99f3bea2972e080b4c3da6f5068543d4010173
Reviewed-on: https://gerrit.libreoffice.org/84307
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2019-12-03 15:46:23 +01:00
Mike Kaganski
283a29e495 PopFieldContext calls CloseFieldCommand itself
Change-Id: I4ac75d6e73f4d60ac532829a56c5d23501e6a531
Reviewed-on: https://gerrit.libreoffice.org/84318
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2019-12-03 15:42:37 +01:00
Julien Nabet
4bfd34145c Simplify a bit use of LabelPlacementParam (oox/chartexport)
Change-Id: I940be544577d8955959d4b8c89feaa08f99e544a
Reviewed-on: https://gerrit.libreoffice.org/84272
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
2019-12-03 15:30:11 +01:00
Julien Nabet
99f0f57d68 Remove useless PatternPair (sc/vbainterior)
Following 3471d83f49
Use initializer list instead of using several inserts (sc/vba)

+ 1a5850e7c4
Following 3471d83f49 (sc/vba)
"Use initializer list instead of using several inserts (sc/vba)"
I had forgotten to remove the initial commented code

Change-Id: Idd9855e520383c17cf7564ae4646fd613fc45895
Reviewed-on: https://gerrit.libreoffice.org/84278
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
2019-12-03 15:29:26 +01:00
Stephan Bergmann
8c6ca69487 enum WindowsStateMask type itself is unused
(found with upcoming loplugin:unusedmember)

Change-Id: I2a7de15da061c4c0d4dc163c59370db6da763c2c
Reviewed-on: https://gerrit.libreoffice.org/84316
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2019-12-03 15:04:20 +01:00
Stephan Bergmann
96608e0c24 use scoped enum
Change-Id: I41a810d14be0a7f18c91b62e5d982c12db593183
Reviewed-on: https://gerrit.libreoffice.org/84315
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2019-12-03 14:58:25 +01:00
Caolán McNamara
4359ee40b4 tdf#129068 try dialog content area help before dialog itself
we already check current notebook page before the dialog, now
check the content area before the dialog

Change-Id: Ieedb5727ddec6d24adc6e70144977b5c3063c0fd
Reviewed-on: https://gerrit.libreoffice.org/84245
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
2019-12-03 14:44:15 +01:00
Stephan Bergmann
b1541d16d5 Use namespaces instead of structs
Change-Id: I82e49d666376c31f3a8e407aedd05cfaee2f2bde
Reviewed-on: https://gerrit.libreoffice.org/84301
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2019-12-03 13:08:09 +01:00
andreas kainz
92d521e5c7 tdf#101513 add customizable option to writer NB contextual single
Change-Id: I0c62b26d0b47cef7f02ee581412aa3cd2d6ac7e4
Reviewed-on: https://gerrit.libreoffice.org/84285
Tested-by: Jenkins
Reviewed-by: andreas_kainz <kainz.a@gmail.com>
2019-12-03 12:48:45 +01:00
Noel Grandin
f656547d4d remove some useless comment lines
which merely announce that the next declaration is a class

Change-Id: Ifdb1398bcd99816b13e0b3769b46d0562bfbc1dc
Reviewed-on: https://gerrit.libreoffice.org/84229
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2019-12-03 12:46:27 +01:00
Julien Nabet
f51fddc2e7 Simplify a bit B2DPolyRange::appendElement (basegfx)
push_back is more optimized than an insert

Change-Id: I8932c4ee61569b5eee3d5dbc5fbfd8eff624d061
Reviewed-on: https://gerrit.libreoffice.org/84276
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2019-12-03 12:29:35 +01:00
Samuel Mehrbrodt
895ac4705f Disable slide copy/cut when LockContentExtraction is set
Change-Id: I7bf9ba907a152fab81a9472bc7baf00331753828
Reviewed-on: https://gerrit.libreoffice.org/84289
Reviewed-by: Serge Krot (CIB) <Serge.Krot@cib.de>
Tested-by: Serge Krot (CIB) <Serge.Krot@cib.de>
Tested-by: Jenkins
2019-12-03 12:03:08 +01:00