* Fixes atom documentation
* Fixes#1461
This is a complicated one. Basically URANGE_CHECK when
used on unsigned integers has a problem when the size of
the range it’s checking is 0. The standard operations is
to check
URANGE(num, size-1)
Which (for unsigned integers) obviously rolls over.
This fixes all usage cases to be
URANGE(num+1, size)
And fixes the bugs found. (addBond and the mmff tests)
* Fixes#1461 - Updates URANGE_CHECK to be 0<=x<hi
Conformer.cpp: make sure we don't ask for space for zero atoms.
Kekulize.cpp: don't complain about marked bonds if we aren't removing markers.
molopstest.cpp: bad smiles corrected in one test.exe
ROMol.cpp, RWMol.cpp: extra preconditions added (should have been there all along)
Wrap/MolOps.cpp: docs for replacesubstructs.