Remove read_stx_conf() and get_stx_coordnum() uses
authorTeemu Murtola <teemu.murtola@gmail.com>
Fri, 14 Aug 2015 12:32:54 +0000 (15:32 +0300)
committerGerrit Code Review <gerrit@gerrit.gromacs.org>
Fri, 28 Aug 2015 18:31:59 +0000 (20:31 +0200)
commit01f0ad327543deb470ab84f2dd79ed39dd8fed48
tree0ae0df78f98ba269950ee23845d3f47b6c187b08
parent65d7a4b97f3c7bd88f35509e6464632a5124f828
Remove read_stx_conf() and get_stx_coordnum() uses

Replace all uses with read_tps_conf(), which is superior in multiple
ways:
 - It removes the need for most boilerplate code needed to read stuff,
   and makes the ugliness of needing to read the same file twice
   internal to confio.cpp.
 - read_tps_conf() at least provides an API that makes it possible to
   not leak memory.  Nearly all callers that used to call
   read_stx_conf() still leak it, though, including read_tps_conf(), but
   at least it is now possible to fix incrementally.

Change-Id: Ic489087f5947bbfd764a1801cbf4e85cb046a812
17 files changed:
src/gromacs/fileio/confio.cpp
src/gromacs/fileio/confio.h
src/gromacs/fileio/tests/confio.cpp
src/gromacs/gmxana/gmx_anadock.cpp
src/gromacs/gmxana/gmx_chi.cpp
src/gromacs/gmxana/gmx_dyndom.cpp
src/gromacs/gmxana/gmx_editconf.cpp
src/gromacs/gmxana/gmx_genpr.cpp
src/gromacs/gmxana/gmx_make_edi.cpp
src/gromacs/gmxana/gmx_make_ndx.cpp
src/gromacs/gmxana/gmx_morph.cpp
src/gromacs/gmxana/gmx_rmsf.cpp
src/gromacs/gmxpreprocess/genconf.cpp
src/gromacs/gmxpreprocess/grompp.cpp
src/gromacs/gmxpreprocess/pdb2gmx.cpp
src/gromacs/gmxpreprocess/read-conformation.cpp
src/gromacs/gmxpreprocess/x2top.cpp