Change-Id: I77f49f09ce9fe5897f78044c0522210978c904d1

...that happen to be requested concurrently from multiple threads

Change-Id: Icecfb0b8d88c9123064689643eddf90a99d3b1b8

...instead of dereferencing null pointers

Change-Id: I9d1709468a5b601b9f3d86dadea7a8e817f449d8

...except when checking for infinity

Change-Id: If6e3b8965169cc615fa5d76422e1925866714936

Change-Id: I5ecd63d79e8be77879d3789c45079929696abe03

Change-Id: Ib45af8a56ac3adc428c530908cf1bbd95e9cc64c
Reviewed-on: https://gerrit.libreoffice.org/21065Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: jan iversen <jani@documentfoundation.org>

Avoid hundreds of presumably pointless warning messages in some unit
tests at least. The code explicitly checks and handles the asserted
condition not being true right after the DBG_ASSERTs.

Change-Id: I5482c56aeeadb8f85dfdd03e12960abd7e769c56

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

Change-Id: I37948ff77d060dc18a583389966b52c06a95bdaf

"yeah, makes sense to remove it", says Stephan.

Change-Id: Ife7baa5814a856dca7f0f02ac3cb2521e2980851

Project: dictionaries  ce7755c7d3179d682dfaccace9e64b6a008c77d2

tdf#96782 updated German dictionaries

Change-Id: Ifa7aed93eaa0448ba530cc029ee55480060e1be5

Change-Id: Id3b155abec7b7a2ff43d0a35e98709f5ee51c6b9
Reviewed-on: https://gerrit.libreoffice.org/21069Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
Tested-by: Chris Sherlock <chris.sherlock79@gmail.com>

The Test currently tries to test the OpenCL and the SW interpreter and
assumes that it will show different results. However if we succeed in
generating an OpenCL interpreter we will never generate a SW interpreter.

There are a number of problems with the approach taken in this test.

1.) Assuming that we are able to execute OpenCL is an invalid
assumption.

2.) Changing the configuration inside of a test needs to be exeption
save as we are otherwise left with a strange config for the remaining
test cases.

3.) Changing the configuration does not change the formula group
interpreter. This needs to be done explicitly.

Change-Id: I679985ee7551b21aaa52038e107324980e7e1bb6

Change-Id: Ie0b3efdbe23a163aab1e8d128217ee1f1c7fd287

Change-Id: I2ef26c34a424e653d85597c85baa736c19004313

Change-Id: Id8b2db3b824ea40345aba99a5f24f2579c0ad630

Change-Id: I4e2345cdf9fd8d088f3522d563263b65abedb7c9

Change-Id: I588a9c0205e03f701b617fd1775e7f9b9b88c6b0

Change-Id: I544eac439cb7277e47a23a90c6fbc45496161c3b

Change-Id: I60ee2ccc6090c305ade4ba16d3d298866cf89c4e

This may sound crazy, but literally vcl::FontInfo serves no purpose
that I can see. The inheritance chain is like this:

        ┌────────────────────────┐
        │                        │
        │        vcl::Font       │
        │                        │
        └────────────────────────┘
                     ^
                    ╱ ╲
                     │
                     │
        ┌────────────────────────┐
        │                        │
        │      vcl::FontInfo     │
        │                        │
        └────────────────────────┘
                     ^
                    ╱ ╲
                     │
                     │

        ┌────────────────────────┐
        │                        │
        │       FontMetric       │
        │                        │
        └────────────────────────┘

vcl::FontInfo (which, incidentally, needs to be put into the vcl
namespace due to collisions with poppler!) literally does nothing
and is acting as a bridge between FontMetric and vcl::Font. Unlike
a bridge though, this bridge doesn't actually *do* anything.

So I'm removing it, which means one less class to deal with in the
vcl fonts world.

Change-Id: I32725669c9bca7fbb0846b6a062135464046e4f6
Reviewed-on: https://gerrit.libreoffice.org/21058Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
Tested-by: Chris Sherlock <chris.sherlock79@gmail.com>

Change-Id: I0a49bab5f410901d40c5b3b62a5a3f9d9946b8a0
Reviewed-on: https://gerrit.libreoffice.org/21057Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
Tested-by: Chris Sherlock <chris.sherlock79@gmail.com>

Change-Id: I74cde57e328047ac149593c5bb065d7a6066941d
Reviewed-on: https://gerrit.libreoffice.org/21056Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
Tested-by: Chris Sherlock <chris.sherlock79@gmail.com>

Change-Id: I15f64797340ae366e02c2b503767cd3d22a66eeb

Change-Id: Ib0a3d82af858304bf63a20bcb30b8de741491935
Reviewed-on: https://gerrit.libreoffice.org/21054Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
Tested-by: Chris Sherlock <chris.sherlock79@gmail.com>

Change-Id: Ia2fe20559745e705667f23cdd73cbe50c6371aaa
Reviewed-on: https://gerrit.libreoffice.org/21053Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
Tested-by: Chris Sherlock <chris.sherlock79@gmail.com>

Changed private variables to use accessor functions in ImplFontMetric.
It turns out there are some variables that are redundant as they are
already in ImplFontAttributes. These are:

- meFamilyType
- mbDevice

I also needed to change ImplFontMetric::GetSlant() to GetSlantType()
as the names conflicted.

Change-Id: Ic06a374fb929e3b2ffb18a6c66577c526c3b31a7
Reviewed-on: https://gerrit.libreoffice.org/21050Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
Tested-by: Chris Sherlock <chris.sherlock79@gmail.com>

Project: help  864de3b4e95b4550d7cfdb99ff2188e63f140cd0

Fix typo in AGGREGATE function help page

Change-Id: I0701328d135968735bec3b30adeba64fc6138f04
Reviewed-on: https://gerrit.libreoffice.org/21051Reviewed-by: Olivier Hallot <ohallot@collabora.co.uk>
Tested-by: Olivier Hallot <ohallot@collabora.co.uk>

Change-Id: I253f71f851621d8a3d1244468b9da63ba2d10b09

Change-Id: Ic6407a57440fbd887f9ec9d5963e5748aa89a29e

Change-Id: Ia35f2ec1b27373243736172f1d91c03bb2622c64

Change-Id: Ib2520679c3d62991f4ae6c5a325a44ab3cce1d89

Also significantly simplify, and remove paranoid ramblings
from the flushing code.

Change-Id: Ie7cc742caac290d77ac931c54cc4fe777a7872de

Change-Id: I959c0874c670027e79163ded6a1d8c8b55d66376
Reviewed-on: https://gerrit.libreoffice.org/21045Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
Tested-by: Michael Meeks <michael.meeks@collabora.com>

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

Change-Id: I5b09d1777c994acaf25457e786222886b2435af8
Reviewed-on: https://gerrit.libreoffice.org/21042Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
Tested-by: Michael Meeks <michael.meeks@collabora.com>

Project: help  efa3b7d1e3eb003e6babb49c9b83c47979da0a89

Fix tdf#86428 Wrap text help page duplicated

Fix for tdf#86428  Help Page “Wrap” Exists Two Times in Local Help

Wrap text help page 05180000.xhp was replaced by 05060200.xhp but
left there because of a link from the main format help page.

The 05060200.xhp page now carries the recent changes in Writer UI
and context menus.

Wrap page from guide (main0105.xhp) also updated.

Change-Id: Icd0c015ceca89dc1e3f8f7200ffe491ddb03ff7b
Reviewed-on: https://gerrit.libreoffice.org/21023Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
Tested-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>

Project: help  dd8cc9d27b6891661185f02bf7478b5934deff08

COUNTIFS: add missing argument name

Change-Id: Ic099ea1272f1c8c0ea6804cd77b15c75159d084b
Reviewed-on: https://gerrit.libreoffice.org/21014Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
Tested-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>

Use Qt submodule notation for Qt includes and also "correct"
some KDE headers AKA use <> notation for external headers.

Change-Id: I9457f75baae56514989800b016a11348a72a242b

Generalize the UnoMark pretty printer to actually print many
objects inheriting from sw::mark::MarkBase.

Also adds the mark name to the output.

Change-Id: Idaec06ba448702ee3a47b474736df954646c0300