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.

Presumably a bitwise '&' is intended, i.e. a normal flag bit test.

If-zero out a leftover isolated expression without effect.
(Introduced in 2007 in CWS impress122.)

Convert DoubleStringArray to std::vector
Convert DoubleStringTable to std::map
Convert StringsTable to std::map

...once e.g. UCB is already initialized, as the invoked migration services use it.

Surely a bit test is what is intended here.

A sal_Bool variable is logical-ored with sal_False, wtf? The only way
this can have any meaning is if the sal_Bool variable actually has
some other value than sal_False (0) or sal_True (1); in that case the
logical or will turn it into a sal_True. But I doubt that is the case
here. The code has been like this since initial import in 2000.