GitLab

Keeping both functions to ensure that the octant ordering specified in
treeBoundBox is obeyed at the octree level, irrespective of the implementation
of the basic boundBox.

- a holdover from the old <root> <case> days

- mapFields and splitMeshRegions need more clarification

- use .obj files for debugging purposes instead.

- deprecate argList::additionalArgs() method and remove uses of it

- ensure that the standard options (eg, from timeSelector) also have
  some usage information

- space between keyword and bracket in 'for(..)', 'if(..)', 'while(..)'

- also drop various unused time options from src/OpenFOAM/include

- can use 'XX.empty()' instead of 'XX.size() == 0', 'XX.size() < 1' or
  'XX.size() <= 0' or for simpler coding.
  It also has the same number of characters as '!XX.size()' and /might/ be
  more readable

- many size checking had 'XX.size() > 0', 'XX.size() != 0', or 'XX.size() >= 1'
  when a simple 'XX.size()' suffices