Change-Id: Iab0a53abd723f0309f40742636315079a4b2c532

Change-Id: I60ec3a95bf7d631fefe061335bf0955216df14b9

After the document is created, an event is dispatched on the main loop
that calls SfxPickList::Notify(), which modifies document properties.

It tries to prevent setting the document to modified by calling
SfxObjectShell::EnableSetModified(false), but Writer cunningly outwits
it by simply having its own independent(?) modified flag that is set
unconditionally in DocumentStatisticsManager::DocInfoChgd().

Let's assume that if the modified flag shouldn't be modified in
SfxObjectShell, it shouldn't be modified in DocumentStatisticsManager.

Somehow in 4.4 and 4.3 the same thing was going on, but it didn't result
in a visibly enabled Save icon in the UI, but with 5.0 it does - cannot
easily bisect why that changed due to tdf#91383.

Change-Id: Id30fd831eb29910c9fb44ed3031bf8da23586bea

There are even more and it is causing a number of performance issues.
This just fixes the most obvious place.

Change-Id: Id85c13b5b7f4bb0bc077e62c39258067e17c0094

The code comment mentions correctly that we want to show all significant
digits but a value of -1 means that the last digit before the decimal
point is rounded away.

Change-Id: Id91a6076e37629502281c5dda426018f93d1e465

Change-Id: I5f7f993f2cf0604ec62cea3460b651c07a7b4383

... when deleting the temporarily inserted styles in
SwDocStyleSheet::FillStyleSheet().

Change-Id: Id4abc067ce10b41486f659350267c7e3933db472

...the code was like this ever since 8ab086b6
"initial import"

Change-Id: I808296a280c01f02adeba1018b9f9bc6977971d3

"return bDone;" was commented out ever since
8ab086b6 "initial import"

Change-Id: I604c7b49623e730cbbe486563f1a91f713ff6810

Change-Id: Id325d05060e427ad46acd45d0ec3f0e9159cea9f

Change-Id: I4a61595294c24a609a5952ce72f9f88524969784

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

...ever since its inception with fccfef48
"INTEGRATION: CWS impresshydra"

Change-Id: I098b4434bfb1ed2c10f1476c21f1427fe2292b60

Change-Id: I91ba41cb052c38aa3b047cf079090b01bbe93b39

...the odd setting of iconSize (to the same value inside the if as well as
unconditionally afterwards) was introduced in
17a5c821 "INTEGRATION: CWS vcl10: #109868# don't
use ridiculously large icon sizes," together with the olwm comment, and olwm-
specific code further down below that was removed again with
b118d56c "Remove Olwm support and Windowmaker
gravity handling;" so effectively revert the code to the state before
17a5c821

Change-Id: Ic68f6b5b44f48292a7a6983935427e529d8a5002

... which means the internal column names should already had it. An
unsuccessful cell lookup fallback doesn't necessarily fulfill a warning
condition.

Change-Id: I973b8edff83fcb47a34cc35fd9bf5163882f7742

Change-Id: I843b2cb12783549033fc81f82751704859139960

Change-Id: I9c5d9c53785828b3535c960f826ccd3720b2e4c3

...ever since c3607b3b "INTEGRATION: CWS
presfixes01"

Change-Id: I29107b0e55d16a8a88e25c7b929eeb40d56686f0

If the style does not actually have any font items, as is the case for
frame, page and number styles in Writer, the maFont will be
default-initialized and the maPixelSize incorrect.
Avoid using these variables.

Change-Id: I084cd8faa90a3d53310ceec55e8dae3af3dd586c

SwDocStyleSheet::FillStyleSheet() already takes care to remove all
temporarily created styles, so assume that works and suppress the
creation of user-visible Undo objects.

Change-Id: I748f0e8304c42e767b331ebd22be0290b9c0d89d

The dialog/sidebar should not actually create styles that don't exist yet,
because it messes up Undo and the (unused) styles are then unnecessarily
exported to documents.

Due to Writer's ... unusual SwDocStyleSheet class this is a bit tricky.

Add a new function GetItemSetForPreview() and use it from the style preview
code.

The implementation does not use FillPhysical so will temporarily create and
then delete any non-existing styles.

Skip page and numbering styles for now since they don't have a useful preview.

(regression from ca953076)

Change-Id: Id6ee30ea467fc24c991547a4c23a9ce14fdd86c7

This causes the temporary creation of frame styles to fail.

Change-Id: I5d148ae008660d9c0f457a4c0e9dc256e0dfc49a

Change-Id: I341fb7c0712dff77ebfed41560a487f85c27c25a

Change-Id: If8d9770717c21875b4472b0d94a1fa5a9d136085

+    rLineBoxes = pFndBox->GetLines()[nEnd]->GetBoxes();

This actually assigns the boxes vector of the last line to the first
line, which the previous code didn't do.

Notably after converting the types from boost::ptr_vector to
std::vector<std::unique_ptr>, the assignment produces a several page
error message from both GCC and clang, and infuriatingly neither
compiler was able to tell on which line in tblrwcl.cxx the template
was instantiated, so i had to bisect that.

(regression from be5e4247)

Change-Id: I646e3ca678895480d38ec48f38d720458860a985

...the computation of nTop, nBottom was done in this curious way ever since its
inception with 108632dc "CWS-TOOLING: integrate
CWS impressaccessibility3"

Change-Id: Ib516e944d96dc10ff12d0a82c0d8c921b57b0466

Change-Id: Id40a6802536120501133968d003267f21aaf5c27

Change-Id: Ic21d88b55b22c650de4fd69479b51d0f640fec6f

Change-Id: Ia829b0f723c8ded1237e52d48f034da3c1ac249e

translateFromChart2AxisIndexToOox toggles between just two
states, so convert consumers/suppliers of that to bool

Then in exportAxesId toggle between primary/seconday pairs based on that, vs
always exporting AXIS_PRIMARY_X + primary/secondary y

Change-Id: I2649f5fc07323a73a3a215fdc52d5f1a5c31c349

Change-Id: I7c19ce1bde1eb405fe64aba7504fc511c0ab646e

...ever since its inception with 485d404d
"vcl118: #i115788# fix file specs vs. URI"

Change-Id: I990c3610b29b060b516a21ec6cba615fa101ccb5

Instead of saying what to do, do it, so the CPPUNITTRACE="... /debugexe"
part can be copy&pasted.

Change-Id: I2ae54ad52ffa7dfbbf8b4e61f1fa8a0079c7b570

...needlessly added with 6d96720f "#98991# fix
CJKV vertical glyph rotation, avoid obsolete FT helper functions"

Change-Id: Ic0311ca8ae2b4c7d33c42f7fecaae4c95c24500a

...ever since its inception with a949f50c
"implemented GetGlyphOutline()"

Change-Id: I4591ac5c3d42053b3caa9d68bb12ad02dc319b66

...ever since its inception with 2d1f08d6
"CWS-TOOLING: integrate CWS otf01"

Change-Id: I92ec60f5704e632126f3ba98140c520ae5c9e784

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

...ever since its inception with 5624be5a
"CWS-TOOLING: integrate CWS graphite01"

Change-Id: Ib530362efe1b52f43a2a25e531cce9326001044d

...ever since its inception with d9bf8cb9 "add
serverside font feature"

Change-Id: I430b457dbf278be3299883ba0fac09a6b68c5d67