There is no reason for this separation to exist.

It looks like this one is also meant to be called manually or something,
and the original version didn't skip the first argument either.

Daniel Bankston is right, the call probably meant to check whether
the intersection is not empty, as an optimization.

Caller of Region::Intersect was not ignore the return value in a conditional, so I refactored the conditional to not depend on the return value of Intersect without affect the original functionality of the code.  This usage of Intersect may be a possible defect and should be checked out.

Since they always return sal_Bool values that are not used by any callers, I changed the Region class methods (both versions of each Union, Intersect, Exclude, XOr) from sal_Bool return type to void return type.

Some string to OUString replacements in starmath

(infinite loop regression from a4e2453b)