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>

Rectangles that are empty (i.e. IsEmpty() returns true)
were incorrectly considered to mean equivalent to "EMPTY".
The latter means full-area, while the former mean zero-area.

This fixes the issue by restrict full-area to rectangles
with 2 billion units on the each side (roughly INT_MAX) or
more, and using this new check rather than IsEmpty().

Change-Id: I12aca17267f5dd33b2932012d1d9db3545f9af6f
Reviewed-on: https://gerrit.libreoffice.org/31458Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>

as this is what vnd.libreoffice.image:// expects for localized images

Change-Id: I454ff86b86e11248f47ab65f682fb57e52d0f7e5
Reviewed-on: https://gerrit.libreoffice.org/31057Reviewed-by: Katarina Behrens <Katarina.Behrens@cib.de>
Tested-by: Katarina Behrens <Katarina.Behrens@cib.de>

snuck in with commit 4d2c210c
"use early return to simplify flow"

Change-Id: I2a5cbff01c5f40d879d320fd3934bc0856756d37

Exception safety, ensure the Access classes are always destroyed.

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

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

Change-Id: I37ee0cc58614dbf56a67a2bf0b08d3bf85d34ea7

adjust things to not need one

Change-Id: I1079f50d8813f86c4828be602687b4cbffe61415

...that no longer trigger any warnings with the current code base under MSVC
2013 and beyond

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

Change-Id: I7d987fdab23a708a75e7d2e8bd2aabdb96e53140

Change-Id: I60652dbac6a76eacdbb4e951a636e7a40e88a6a6