Change-Id: I1ee686ff96615f7b8aa69c0cd4caafaae07318b7
Reviewed-on: https://gerrit.libreoffice.org/31493Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>

Change-Id: I0c6610308f75a57e517b233db66c66b34eb25189
Reviewed-on: https://gerrit.libreoffice.org/31492Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>

Change-Id: Ibeb0aff07aec223f274932f2b3a492e58db09a02
Reviewed-on: https://gerrit.libreoffice.org/31488Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>

Change-Id: Ibcdc6a492abe5e26fb9b5244119be5dacc01b7b4
Reviewed-on: https://gerrit.libreoffice.org/31487Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>

Change-Id: I55ce4f6a9b033dc064beed8e70d35a5b5097846d
Reviewed-on: https://gerrit.libreoffice.org/31451Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>

.. which are not paragraphs.

Note: For some reason in character attributes sequence
we have a wrong Any value for "CharPosture" property.

Change-Id: Ia45526c01cc381d3d6a1b56dbf4f03fdd38a0989
Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/31591

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

and builds as root in that env, at least the default out of the box
experience is that for me, I suppose I could faff around getting it add a
user specifically for the build like I do for my coverity in docker
build, or just drop this.

This effectively reverts

commit 46625769
Author: Tor Lillqvist <tml@collabora.com>
Date:   Thu Apr 14 09:01:48 2016 +0300

    149

    Change-Id: I63f055756ff5f571d5ebe89aa4aee52d8ae2c510

I see your 149 and raise you a 1172

Change-Id: I3e81a725cdab1ff5ead0484dbfee11ac8fe303a9

make ctors with one parameter explicit

Change-Id: I08e753cea9f5ea17d654c889bad37ab7d0f4611e
Reviewed-on: https://gerrit.libreoffice.org/31588Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Jochen Nitschke <j.nitschke+logerrit@ok.de>

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

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

Especially in the oox code there are many...

Change-Id: I16915cc207c274e5bcdb5d1f4f8708db5a0479a1
Reviewed-on: https://gerrit.libreoffice.org/31578Reviewed-by: Kohei Yoshida <libreoffice@kohei.us>
Tested-by: Kohei Yoshida <libreoffice@kohei.us>

Change-Id: I6fa81689e14e605cd7449526e44245d4fdebb298
Reviewed-on: https://gerrit.libreoffice.org/31536Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Takeshi Abe <tabe@fixedpoint.jp>

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

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

Change-Id: Iaca5837e5502ef8eed4cddbca1d00e00c71c53c2
Reviewed-on: https://gerrit.libreoffice.org/31576Reviewed-by: Kohei Yoshida <libreoffice@kohei.us>
Tested-by: Kohei Yoshida <libreoffice@kohei.us>

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

This is the first version where hb_buffer_set_cluster_level() was
introduced. The layout engine depends on this feature in a few places
and though we can build without it, the result will be wrong in some
cases (i.e. any where we assume combining marks will have their own
cluster number).

Change-Id: I89187f317052e7933b83ad0350cace239333510b
Reviewed-on: https://gerrit.libreoffice.org/31521Reviewed-by: Rene Engelhard <rene@debian.org>
Reviewed-by: Khaled Hosny <khaledhosny@eglug.org>
Tested-by: Khaled Hosny <khaledhosny@eglug.org>

This is to fix a build after 67398a8d
No idea if packages should contain conditions too,
but at least gid_File_Bin_UnpackUpdate did not.

Change-Id: I56e1b23f6eeca9c62cdadd8d93de6ccda89b20c4

Which just lead to confusion as could be seen in two earlier commits..

Change-Id: I172127cf08bcd45c3c53e0cf00cf11a41e0dd3a6

Change-Id: Ifcfc4588d5bd6f713a40bd2300740f7a1c52337c

Assuming that the screenshots directory suffix
* should match the configured UI locale instead of the work locale
* should not be empty if configuration says "Default - ..."
  * actually ScreenshotTest::implSaveScreenshot() explicitly tests
    (maCurrentLanguage == "en-US") and then sets the suffix to empty, so no
    other locale should exhibit the same

Change-Id: Ie4561315b67c3ade20f3947772af45e471d0752f

Testing for GetLocaleConfigString() starting with a language code could never
have worked if configured was "Default - ..." in which case the config string
is an empty string.

Change-Id: I3a68a168a0f8e53c8c389fc243e40053f5d45e1f

And a little cleanup.

Change-Id: Id066b4d37e217b8ab21ea162c23a999566a9e190

Change-Id: I5bd3a5f6168b20c925f5713085fcb0918bcccc8e

Change-Id: Iecf059b974b996fceb57b04197c7dac9b7c8dd47

we're presented with...

CXX="/usr/local/bin/clang++"
CC="/usr/local/bin/clang"
CXXFLAGS="-g -stdlib=libc++"

and out of the box we fail to build, but if we reshuffle to...

CXX="/usr/local/bin/clang++ -stdlib=libc++"
CC="/usr/local/bin/clang -stdlib=libc++"
CXXFLAGS="-g"
LDFLAGS="-lpthread"

then with this distro-config we build to completion

Change-Id: Ib6bee4dd69e0609e3c1238d86837a1680483ffa4

There does not seem to be any need for that atom thing as we are
perfectly happy using plain OUStrings in the same struct, not to mention
that font names are supposed to be unique so I don’t see what we are
saving here.

As this was the only use for unotools/atom, it goes with it.

Change-Id: If9d58d84fff0403f9b2c41fe594b99028b30c2f4
Reviewed-on: https://gerrit.libreoffice.org/31520Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Khaled Hosny <khaledhosny@eglug.org>

tables that span multiple pages, and rows greater than one page
caused problems, so just removing the enhancement that tried
to emulate what the original odt looked like.

reverts commit 129f93e4
from bug tdf#91083

Change-Id: Iac31116d731e1e4aa60ee6b8de973f7f9291e47a
Reviewed-on: https://gerrit.libreoffice.org/31551Reviewed-by: Justin Luth <justin_luth@sil.org>
Tested-by: Justin Luth <justin_luth@sil.org>

gets vcl built anyway

Change-Id: I521a53019f54d5e42c931043c758143f2f8bdb1d

This was the last unit test that was disabled on Windows due to missing
implementation.

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

Change-Id: I5e6a6debf01886c283f5a2d6b9cacc00a44f7134

Change-Id: Ib553e085337972f74857db0aad9bc885712d5ea3
Reviewed-on: https://gerrit.libreoffice.org/29788Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com>
Tested-by: Norbert Thiebaud <nthiebaud@gmail.com>

- also move from deprecated OSL_ENSURE to SAL_ macros
- also flattened and refactored the deeply nested copypasta

Change-Id: Ib7333ae9589b93bca999c8b1d003b712c59920ee
Reviewed-on: https://gerrit.libreoffice.org/31391Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>

https://retrace.fedoraproject.org/faf/problems/bthash/?bth=17ba9a57f40d5ca29778866a9f96c1c0dfc7593c&bth=2f1801e44e9cb0be2b013624521fb0959dc2c73a&bth=5e8a1e07503f53b13b3a6779084f8b0637aaf5a7&bth=8033013a2ce586bccc23245583fa86ef4a761dce&bth=975271167132418534cb07336601efb5b0c711f0&bth=de75c2ad7917bc889d6307b152f79d34e79ce74d

Change-Id: Ie93afa476eca9fdbf7ea17ff0189fbaa493a5531

Change-Id: I0cab6b2f9fffc8cf9fc5059b8dc48fa9407e42b5

Change-Id: Ibcfae1b3ec118d74921312a80eb56c97c623f4a9

Change-Id: I8ffb08bdab452f6a8618eaf466b722e7c8d226d9

Use case: the bugdoc has 2 signatures, one normal one and one with
SubFilter=ETSI.RFC3161. By not hiding the second signature it's possible
to counter-sign the document, even if we don't handle the contents of
the second one.

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

just keep the current logic and silence coverity about it

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