Fix issues with using int for number of steps
commit7f2e942dd8620be03aff8f058a007c06ac301bac
authorMark Abraham <mark.j.abraham@gmail.com>
Thu, 14 Jul 2016 15:31:49 +0000 (14 17:31 +0200)
committerMark Abraham <mark.j.abraham@gmail.com>
Thu, 14 Jul 2016 15:34:48 +0000 (14 17:34 +0200)
tree61f24a80d80cbf5436fd92233389bc0c1c14b028
parentf7d4d019eecd733a56cbf4eeb9c936c6bd775eb1
Fix issues with using int for number of steps

Mostly we use a 64-bit integer, but we messed up a few
things.

During mdrun -rerun, edr writing complained about the negative step
number, implied it might be working around it, and threatened to
crash, which it can't do. Silenced the complaint during writing,
and reduced the scope of the message when reading.

Fixed TNG wrapper routines to pass a 64-bit integer like they should.

Made various infrastructure use gmx_int64_t for consistency, and noted
where in a few places the practical range of the value stored in such
a type is likely to be smaller. We can't extend the definition of XTC
or TRR, so we're stuck. TNG is already good, though.

Fixes #2006

Change-Id: If485d9d92cb4b99a3bbe25e8e9fa082fc3fccd5f
src/gromacs/fileio/enxio.cpp
src/gromacs/fileio/g96io.cpp
src/gromacs/fileio/tngio.cpp
src/gromacs/fileio/tngio.h
src/gromacs/fileio/trrio.cpp
src/gromacs/fileio/trrio.h
src/gromacs/fileio/xtcio.cpp
src/gromacs/fileio/xtcio.h
src/gromacs/tools/dump.cpp
src/gromacs/trajectory/trajectoryframe.h