Interesting use of for loops with the body of the loop returning
unconditionally.

Just use sal_Unicode as type for cLeader as it is being assigned to a
sal_Unicode field anyway.

static_cast is what we should need here, say the C++ experts.

A whole bunch of them.

At least in a MSVC --enable-dbgutil build.

When the user selects "Pages" radio button in the Range section, it is very
reasonable to expect that she would now want to specify the range. Thus moving
the focus automatically to the page range edit box would save the user a mouse
click.a

Code is contributed under the LGPLv3+ / MPL.

If I am completely wrong, feel free to revert
and fix it the right way