Change-Id: I28d0260531132216c5f0bd187d400ed47cc43326
Reviewed-on: https://gerrit.libreoffice.org/54498Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>

To make sure they are loaded correctly for most common graphic
formats.

Change-Id: Id121df0ddc763a299c5714cadd0d61740876e3dd
Reviewed-on: https://gerrit.libreoffice.org/54467Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>

For table shape we used OWN_ATTR_BITMAP as the identifier for the
"RepalcementGraphic" property, but that is not really correct.
More correct would be to use OWN_ATTR_REPLACEMENT_GRAPHIC, which
name reflects exactly what the property is for and doesn't conflict
with OWN_ATTR_BITMAP which is used for the "Bitmap" property.

Change-Id: Ic76318addf8f53391f41251fe9d99dba57308165
Reviewed-on: https://gerrit.libreoffice.org/54466Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>

For main document window we use logic coordinates, but
for dialog / floating windows we use pixels. Use a different
method name to make it clear which unit we use in the code.

Change-Id: I7aee7a03013d24e1a2e37072e224e7d4e7830f02
Reviewed-on: https://gerrit.libreoffice.org/54500Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>

Change-Id: I6f96abdd4d85c8dfed8bf83e1b2a30192a54bd2d
Reviewed-on: https://gerrit.libreoffice.org/54494Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>

Trying to chase the culprit.

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

Don't know yet how that could had happened, but the sample
document has a bad ActiveSplitRange value that doesn't match
HorizontalSplitMode and VerticalSplitMode.

* sanitize the value when reading view settings
* prevent writing such value to help versions that don't sanitize

Change-Id: I1b450c7b8102148a24d545ff7568f725d7492a6a

after 64931362. Remove stray (void)dummy;

Change-Id: I73cdcfc04116c4c070e3dda80c25cc9ca52eab95

It will be needed for a later change in online code.

Change-Id: Ic8f898b78ccaaaec849a1894b5e70fb730d70d6d
Reviewed-on: https://gerrit.libreoffice.org/54443Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>

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

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

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

document, with meta:generator of "opxml2odf - Version KO 0.1" has
a styles.xml xmlns of...

xmlns:ofo="urn:oasis:names:tc:opendocument:xmlns:xsl-fo-compatible:1.0"

instead of the usual xmlns:fo=...

use the importer namespace map to get the importers namespace prefix
for the fastparser namespace uri for the token

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

The ocStyle token is only in the RPN tokens, the raw tokens array contains
only ocName, so it's necessary to check also RPN tokens.

Prevents a crash with tdf#91220/1 because of ScInterpreter::ScStyle() causing
a SfxBroadcaster::Broadcast() call.

Change-Id: I7fa04114b698918569014322c721751ab3d8c62f
Reviewed-on: https://gerrit.libreoffice.org/54326Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
Tested-by: Jenkins <ci@libreoffice.org>

...that ignore the out-parameter integral part

Change-Id: I05f07c1a8909023232f8aecf75ea5541d4eb81ca
Reviewed-on: https://gerrit.libreoffice.org/54474Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Tested-by: Stephan Bergmann <sbergman@redhat.com>

Someone removed maPageObjectSize in PageObjectLayouter
and replaced it with the local var aPageObjectSize. This
workms in principle, but was forgotten in one line where
maFocusIndicatorBoundingBox gets set.
Since the incoming rPageObjectWindowSize only has one
value set (width or height) and the other one gets set
in the call to CalculatePreviewBoundingBox, the whole
layouting/visualization of SlideSorter and SlideView in
Draw/Impress was broken.
Fixed that - please be more careful with those changes
in the future - take a look at the app where you do changes.

Change-Id: I6bd9f8b11e7ebc7b880e465b80b813f5e2173c8c

Change-Id: I219226060ebf4b8b86c8f0487fc679f22f73812a
Reviewed-on: https://gerrit.libreoffice.org/54468Reviewed-by: Tor Lillqvist <tml@collabora.com>
Tested-by: Tor Lillqvist <tml@collabora.com>

Change-Id: I68dfd0832747c7c319e40ee28a4461623e95102f
Reviewed-on: https://gerrit.libreoffice.org/54482Reviewed-by: Tor Lillqvist <tml@collabora.com>
Tested-by: Tor Lillqvist <tml@collabora.com>

Found with bin/find-unneeded-includes
Removal proposals are dealt with here,
and a bit of fallout management.

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

Change-Id: I391ea52f5137975073a674ac1ad380277fe99846
Reviewed-on: https://gerrit.libreoffice.org/54381Reviewed-by: Michael Stahl <Michael.Stahl@cib.de>
Reviewed-by: Rene Engelhard <rene@debian.org>
Tested-by: Rene Engelhard <rene@debian.org>
Tested-by: Jenkins <ci@libreoffice.org>

Change-Id: If7ea2fd921c2e673f76ecb79188dc5dcb4cba442
Reviewed-on: https://gerrit.libreoffice.org/54447Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Zdenek Crhonek <zcrhonek@gmail.com>

For example: Character dialog -> Highlighting or
Character dialog -> Font.

Change-Id: I0d1269cdb9d4f482caea27b23ae204351f7dac23
Reviewed-on: https://gerrit.libreoffice.org/54446Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>

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

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

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

Created mailmerge output document was not using current printer settings,
which can be taken not only from settings, but overriden by currently
opened template.

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

If TimeAnimationValueListEntry contains time with -1000,
key times have to be distributed evenly between 0 and 1.
([MS-PPT] 2.8.31)

Change-Id: I67a3b83f1f1832fade5df7908c58032bcb9b73ce
Reviewed-on: https://gerrit.libreoffice.org/53284Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Aron Budea <aron.budea@collabora.com>

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

Change-Id: Ie212f26b88ed5cf08bbc72a278adce3cdf04c400

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

As I said in 2ff121f2, passing a
POSIX-style locale name to the std::locale constructor on Windows is
decidedly odd, and apparently boost adds insult to injury by even
really attempting to use a UTF-8 one if that is what our code thinks
it wants. Or something.

This commit avoids the ton of
"warn:unotools.i18n:7996:3544:unotools/source/i18n/resmgr.cxx:123: CRT
Report Hook: ASSERT: f:\dd\vctools\crt\crtw32\stdcpp\xmbtowc.c(89) :
Assertion failed: ploc->_Mbcurmax == 1 || ploc->_Mbcurmax == 2" we
otherwise get when built with --enable-dbgutil and a current VS2015
(or 2017?).

Change-Id: I3a774dff7b12d9960f553c9f242e0463c6d13d5d
Reviewed-on: https://gerrit.libreoffice.org/54444Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Tor Lillqvist <tml@collabora.com>

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

regression from...

commit 2f6b3fb0
Date:   Wed Apr 25 18:21:46 2018 +0200

    Avoid comphelper::string::getTokenCount()

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

Change-Id: I3938a05d96cb1fc171611e54fa34f780ba569268