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.

Ifdef out a couple of weird leftover lines that effectively do
nothing. The code was changed to be like this in 2004. Could some
Calc expert have a look please and clean up? Also, if the intent
really is that bEmbed is always false, a couple of lines below it is
pointless to check if it is true anyway.

Microsoft Installer thinks that it is a good idea to fall back to zh-CN
from zh-HK and zh-MO. It is wrong, because zh-CN uses Simplified Chinese
zh-HK and zh-MO use Traditional Chinese. So we need to fall back to zh-TW.

In GetBookmarks and Writer_Impl pFontRemoveLst field. Remove unused
field pBkmkArr in Writer_Impl class. Convert SwBookmarkNodeTable to
std::multimap, greatly simplifying code.