Change-Id: Ia6f2cc3a85f81c4853beea7a9f7b267a7fe89285

This is not needed; the problem was in the Fedora 25 epoxy headers:
They got badly broken in an update, and should not be used from C++
code. If you use Fedora 25, make sure to use --without-system-epoxy
for now. See https://bugzilla.redhat.com/show_bug.cgi?id=1431994

This reverts commit cbb2e6b7.

Change-Id: I55c77ff279f1716004ac77cd3a08c4bb3fdf644b

found with the attached bin/find-unused-rid.py script

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

For some reason this was not detected earlier, but now when the epoxy
headers in f25 were updated, it is.

Change-Id: I7be830153792414b0f913aaaecb2a1336ebf6fce

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

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

This test wasn't round-trippable because the page breaks were
all lost, and all of the frames ended up on page 1.

fixed in 5.3 daily debug bibisect 12a5c2be4d64199c6540969c645118112084a639
best guess: author Tamás Zolnai 2016-10-28 14:10:41 (GMT)
best guess: commit b927c1f4
tdf#103544: DOCX exp.: Image loss when have a frame anchored to the same para.

This doc still needs some work - the frames are shifted backwards one page,
so frame 1 is on page 2 and the last page contains two frames.

Change-Id: I512c1d66b742b29f1a9fe8644355f3d9df5ed1f1
Reviewed-on: https://gerrit.libreoffice.org/35127Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Justin Luth <justin_luth@sil.org>

Change-Id: Icf8a52db84536ef540c5dcffceabfbeb5c1deb02
Reviewed-on: https://gerrit.libreoffice.org/35153Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>

Found by the "Random Condig" tb run #293.

Change-Id: I8fa7b66444b9b57f289c963cb1da5e772341dc30
Reviewed-on: https://gerrit.libreoffice.org/35156Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>

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

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

found while changing base of SvxFrameDirectionItem to SfxEnumItem

> warning: comparison between ‘enum SvxFrameDirection’
> and ‘enum com::sun::star::text::WritingMode’ [-Wenum-compare]

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

Could have sworn I did this originally, somehow it got lost.
Also fix a thinko in tools::ResId I introduced in commit
2b70fb58
"Use o3tl::strong_int on RESOURCE_TYPE"

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

Change-Id: I5c7d1d47f347b3ce8de6e2e4995da8724a464941
Reviewed-on: https://gerrit.libreoffice.org/35045Reviewed-by: pranavk <pranavk@collabora.co.uk>
Tested-by: pranavk <pranavk@collabora.co.uk>

to skip a push to the stack immediately followed by a pop.

Change-Id: I97eb48db2521d640acec4d8080cceb56561eb901
Reviewed-on: https://gerrit.libreoffice.org/35129Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>
Tested-by: Takeshi Abe <tabe@fixedpoint.jp>

Change-Id: I38e622df87dd4b5e37dd248c5181978e13890fda
Reviewed-on: https://gerrit.libreoffice.org/35132Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>

Change-Id: I6ec50033a452493852729dc8efe78da2708cbd96

Change-Id: Ibe41e1edb717b4e740971f4b972abdea769460eb

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

adds an extra SVSTREAM_WRITE_ERROR check to some of them, otherwise
they're all the same

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

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

Change-Id: Ia89f068d93b810b0c32df5df62a582322f3e5588
Reviewed-on: https://gerrit.libreoffice.org/35142Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>

No point in outputting the enum class name when we know it is a value
from that enum that we are outputting.

Fix some random whitespace craziness.

No point in putting that symbolic representation of an enumeration in
double-quotes. It is not a string in any sense.

We can pass an OUString directly in SAL_INFO.

Be consistent, put a double-quote on both sides of the string (font
family name), if at all.

This is a way too verbose commit message for such a trivial change.

Change-Id: I9e3f96856a634dc16f0e24f789c1898d7ec75fd9

Fix C4806: '&': unsafe operation: no value of type 'bool' promoted to
type 'int' can equal the given constant

Change-Id: Iedd54379a67373fb3499a0a780f2cbfc83f08e84

Change-Id: I376933a033a2dc2b5b7b59006b93d14a65463aed

Change-Id: I6b9423db247583ff1dcbd469db5dfab7f89e7457

Change-Id: Ifd2fd8088590493377749866f37b1e663f97fa76
Reviewed-on: https://gerrit.libreoffice.org/34983Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>

...and get member function const'ness right overall

Change-Id: Idb422039108d1dde9f6a8b881f6dff742030f6f0

Change-Id: Ide32700aab3427a8bf18c806968822cdf3f91bf2

Change-Id: Ia315f446c374ac08d8a4b7247c571b70cbbab5ae

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

Change-Id: I81a59c7847e2109198a35cbdb49f5bcf3505c4ba

Change-Id: I3afb31b642a47e767dda0614d223b6b7f22e5d54

Change-Id: If728a1da99d9146d60d15b5273a7a89764b82d51

I guess you can only see the problem if you not only do a from-scratch
build (like Jenkins does) but also empty the tarball cache.

Change-Id: I1238e35abe1a6009ce0e0c3406e20aa1879c65bc

if we are a hairline along the very right/bottom edge
of the canvas then distory the polygon inwards one pixel right/bottom so that
the hairline falls inside the paintable area and becomes visible

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

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

... so only SvNumberFormatter can use it, as intended.

Change-Id: I91875dfd00d1f83b3ae988bda922782336489f73

(user-provided virtual dtor in class with dependent base)

Change-Id: I37b3cc3d32e76a0286ecc91f999920e3dfe706cc

(vs. uses of sal_uInt16 representing such items' IDs).  APPEND and ITEM_NOTFOUND
are members of ToolBox now (and have been bumped to the corresponding type's
max).  The payload of VclEventIds

  ToolboxButtonStateChanged
  ToolboxClick
  ToolboxHighlightOff
  ToolboxItemAdded
  ToolboxItemDisabled
  ToolboxItemEnabled
  ToolboxItemRemoved
  ToolboxItemTextChanged
  ToolboxItemUpdated
  ToolboxItemWindowChanged
  ToolboxSelect

has been changed too (note that reading/writing of those payloads isn't very
consistent, though: for some of these, values are written that are never read,
while for some others no values are ever written but the reading code would like
to receive some value if the payload wasn't a nullptr).

Change-Id: I4856ffb3637404b8d866621a3503d7922b1b8141