Rework handling of units in the validator and analyser

[agarny]

The analyser (in PR #818) has various methods to handle units, some how which are also present in units.cpp (e.g. updateUnitsMap()). Those methods should therefore reworked, so that only one copy exists. Most likely the copy in the analyser since it does more things.