We didn't check wether the window was big enough to draw all the widgets. Sometimes, when the window was displaying, it was too small and drawing outside of the window caused a crash, we didn't take into account that the root point for drawing gtk3 widgets wasn't always (0,0).

The uno::Sequence copy c'tor creates a flat copy and increments the ref
count. So if you use new and later delete together with the copy
constructor you get a double delete.

there are some lifetime problems related to SchXMLCell and therefore
this helps to limit the scope of possible problems

A right parenthesis was apparently misplaced, leading to the code
actually containing a bogus comma expression. The GetPropertyValue()
call's result was ignored and the subexpression after the comma, zero,
was used instead.

Surely it's the bitwise operator & that is wanted here. The code makes
no sense otherwise. We apparently have been generating UUIDs where
every second hex digit is always a '1', like
{x1x1x1x1-x1x1-x1x1-x1x1-x1x1x1x1x1x1} (where each x is a random hex
digit).

Remopve apparently accidentally leftover initializer parameter list
that now was just a comma expression with no effect.

It looks like the second call to PutValue() was accidentally inserted
in f462a51f.
Also, pDoc is valid when pField is.

SwXTextField::setPropertyValue: call Expand() for all text fields
Fixes getPresentation returning stale values.
(regression from CWS sw34bf01, 8485708f)
Also, without this the text formatting does not actually update and the new
content is not displayed until some other unrelated editing operation.

While only the text formatting is able to expand all fields correctly,
this is still good enough for many field types and fixes a race
condition when the field value is requested before layout is finished.
(regression from CWS sw34bf01, 8485708f)

another 20% compile time win, and 350K saved from the shared library

Gives a 50% shrink in compile time, and a ~1Mb shrink in binary size

Saves 4% compile time and 200K of space

Use static const char * arrays and a helper function instead
of inlining umpteen OUString intern calls.

 - before we only generated custom shapes used as presets definition
 - preset-*.pptx documents might be used for testing the presets

The UI change was too intrusive.

This reverts commit dfc9c72d.

each custom shape now has a virtual constructor we can call to
construct it, this shrinks the function size, and helps the
optimiser much accelerating compile time. It may even help
improve import time too.