Otherwise their instances created in other modules may end up
as non-exported even when used by something exported.

Otherwise when using a template from another module, an instance
of it may end up being non-exported even though it's used by
something exported.

clang errors out on this

The real reason for this change is that clang aborts with a fatal
error in backend (http://llvm.org/bugs/show_bug.cgi?id=12226),
but I'm not sure whether this was a correct construct in the first
place anyway. The contents are above 0x7f, which may be outside
the range of plain char, if it is signed, so maybe the result is undefined.
I don't want to spend the time checking in the spec,
but this new variant is technically the same and it is safe.

Follow-up to fdo#43467

It's a small-size object, so using std::vector is slightly more
efficient.

For the "context menu" key VCL synthesizes a cursor position that is the
center of the window, hence likely to be in the document body.
When SwEditWin::Command calls lcl_CheckHeaderFooterClick, it then moves
the document view cursor; prevent that by only calling that for actual
mouse events.  (regression from a6176612)

hack to fix the scenario where sometimes we get "Object required" error when comparing an uno property containing a struct with another struct ( if that property is "MAYBEVOID" )

It so happens that both group info and date info can be enabled, in which
case the date info takes precedence.

This is to avoid populating them twice in case the same cache is
referenced by multiple pivot tables.

This is to avoid group data to be added multiple times in case of
multiple pivot tables referencing the same pivot cache.

Though I stil need to fix the sorting issue, which occurs because I
use string items to represent the range items.