Change-Id: I33d065780ece2aab5f5a07d0dfd5493005a22b33
Reviewed-on: https://gerrit.libreoffice.org/43917Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

Change-Id: Ibc14583137aae0d46e501927e602a510339abe38
Reviewed-on: https://gerrit.libreoffice.org/43872Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

Change-Id: I1a278302b995137d3e73620c003534498a59ba14
Reviewed-on: https://gerrit.libreoffice.org/43870Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

Change-Id: Ia08ac1ae46d4af6df7b0a590752e17a6d9a6836e
Reviewed-on: https://gerrit.libreoffice.org/43869Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

regression from

    commit 0ad8447d
    dyncolcontainer: convert nRight to std::vector

Change-Id: I9cdf80115a2be73ec2b30fd875a904de3333fddf
Reviewed-on: https://gerrit.libreoffice.org/43905Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

Change-Id: I1bdb332e40a2cf612b249dafa7620f700af19833
Reviewed-on: https://gerrit.libreoffice.org/43927Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Tested-by: Jenkins <ci@libreoffice.org>

Change-Id: I83438eaf2c1045a9212a63ed9fd8b3371e9b06aa
Reviewed-on: https://gerrit.libreoffice.org/43896Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

This reverts commit cdd4b528.

The manual acquire()/release() is necessary to prevent a double-delete
if the exception is thrown

Change-Id: I5acc0840befba3981e13b084b8d347f8a2766e70
Reviewed-on: https://gerrit.libreoffice.org/43889Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

Before this, -D_WIN32_WINNT=0x0601 used to land to VS project simply
as _WIN32_WINNT (without value).

Change-Id: I68ebdc24c3181c5d9e046280756e4cefee3de46c
Reviewed-on: https://gerrit.libreoffice.org/43928Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Tested-by: Mike Kaganski <mike.kaganski@collabora.com>

Change-Id: I5b4359727ebf58bc121325d490e63f2c919dd7ab
Reviewed-on: https://gerrit.libreoffice.org/43921Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>

Change-Id: I96f64b7c259f04a67684bb699ea3cc43b68e40fa
Reviewed-on: https://gerrit.libreoffice.org/43913Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Tested-by: Julien Nabet <serval2412@yahoo.fr>

Change-Id: I1ff394ecd1fe805d671d666d8bc2522c33c40b50
Reviewed-on: https://gerrit.libreoffice.org/43922Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>

Change-Id: Idba5a30bc44c7b2bd951f954ea3bfd048b022660
Reviewed-on: https://gerrit.libreoffice.org/43920Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Eike Rathke <erack@redhat.com>

Change-Id: Id9e306877125b66f908d24048d236c55483330d0
Reviewed-on: https://gerrit.libreoffice.org/43916Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
Tested-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>

Previously, defines string like this:

  -DDBG_UTIL -DNOMINMAX -D_DLL -UNOMINMAX

would produce this defines list:

  DBG_UTIL;NOMINMAX;_DLL -UNOMINMAX

where last "define" is incorrect; proper list should be

  DBG_UTIL;_DLL

so that the undef'ed element would be properly eliminated from the
result. This patch takes care of this.

Change-Id: Ia66a1d6d0a6e0bbfd0022b22285b005609871336
Reviewed-on: https://gerrit.libreoffice.org/43923Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Tested-by: Mike Kaganski <mike.kaganski@collabora.com>

(regression from 16598d52)

Change-Id: Ic621979926a9629aa8d8fb95a80957850ae0661b

...after 022b1b2a "tdf#96505 Get rid of cargo
cult long integer literals"

Change-Id: I9e5cc9d63c2eddd1ad766c2f6b01a9ff49a09bfd

Insert constructor everywhere, except a couple places that apparently
want to compare GetMapUnit().

Change-Id: I1910deb60562e5e949203435e827057f70a3f988

Change-Id: I3d4f15cd4929b553673f1a1d8950b6c9082a0cdc

Change-Id: Idc4668cba7412d0b4a3e486a8629c7f95a701091

Change-Id: Ibd26816cb00b3b8b2514aca536906b689c8cbd5e

Change-Id: Ic2eadf1942927efb6a6b209db591e869b61362f4

Change-Id: I16726fbc8bf4f67fe1018d54437d59b715738bfe

Change-Id: I4dc2d5a5fbbded5f4c7a1417c150fe6b1a73a69d

... and LocaleData2 service, which is necessary with new-style
services because optional interface at published service is not
possible.

Change-Id: I77703243a961b5385eaea146e4c9c42d5f30e1ad

Change-Id: I19bb1d566bd80f859810b4023d965c8fd80b98f2

...see a2d814ac
"loplugin:implicitboolconversion" and 24eeb4d2
"loplugin:implicitboolconversion" for things previously only found in C++17.  As
expected, no further occurrences were found.

Change-Id: Id0ab621b82dc3c40c8b5801413fceb73ade1408a

Change-Id: I8f984c3b3833437f6b4d65ab99da608a6868ff74

Change-Id: I899df8d0609d6a3c55d130cffc5b1b23eebece13

Change-Id: I8848784cd867714019f90bb7fc57ca7289e51f81
Reviewed-on: https://gerrit.libreoffice.org/43873Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

There can be a dual mode flag set in the EMF+ data. If set,
we should process either EMF or EMF+ data, because the contained
information should be equal (says the documentation).
The more advanced EMF+ renderer is chosen here.

The flag is read in the EMFPlusReader and blocks the reading of other
EMF records than HEADER, EOF and COMMENT.

Thanks to Chris, for pointing out the dual mode!

Change-Id: I8522930cebbb9e9ecc732397cbb3deea9e8c9127
Reviewed-on: https://gerrit.libreoffice.org/43122Reviewed-by: Patrick Jaap <patrick.jaap@tu-dresden.de>
Reviewed-by: Bartosz Kosiorek <gang65@poczta.onet.pl>
Tested-by: Bartosz Kosiorek <gang65@poczta.onet.pl>

Change-Id: I620b0278273e285100fe44d252389e6393a2022c
Reviewed-on: https://gerrit.libreoffice.org/43877Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

Change-Id: Ica00209bc6679edaa0c83fc7d31bd5d4653b5e96
Reviewed-on: https://gerrit.libreoffice.org/43879Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

(or does the comment need to go exactly at the line before the one containing
the "return" token?)

Change-Id: I5f92e59ee05d0b5ba3d6bda775e6ca02f185dbe8

Added Copy/Move/Delete since this is the only way to access
the local storage.

Change-Id: I9b054a4f23d32d5220ae500a37b52cb7e7a8587c

Added button in Document window, to signal if working local or in iCloud

Change-Id: Ib33b67c95dfad466664e2289390ea577ba5e3391

(i.e., using constructor="..." in the .component file) that implement multiple
services, but are not implemented as single-instance objects (i.e., always
return different objects from that constructor function).  In that case, the
"factory" would be the object itself, and would thus be different for different
instantiations of that implementation.  erAck ran into such a case with new code
he's writing.

Change-Id: Ib134a4c6d1400b43a7df3cf6a115f8206fb80c93

Change-Id: I8404ebed8ba84ee83528310ee63f796356674553
Reviewed-on: https://gerrit.libreoffice.org/43830Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Mark Hung <marklh9@gmail.com>

Change-Id: I1657b84d613b28d5762057954069d2a2fe2e3ab2
Reviewed-on: https://gerrit.libreoffice.org/43103Reviewed-by: Heiko Tietze <tietze.heiko@googlemail.com>
Tested-by: Heiko Tietze <tietze.heiko@googlemail.com>

Change-Id: I8ffdf86e98e261b337cfa81b1be0cb64539c9a9c
Reviewed-on: https://gerrit.libreoffice.org/43871Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>