| commit | 01f0ad327543deb470ab84f2dd79ed39dd8fed48 | |
| author | Teemu Murtola <teemu.murtola@gmail.com> | |
| Fri, 14 Aug 2015 12:32:54 +0000 (14 15:32 +0300) | ||
| committer | Gerrit Code Review <gerrit@gerrit.gromacs.org> | |
| Fri, 28 Aug 2015 18:31:59 +0000 (28 20:31 +0200) | ||
| tree | 0ae0df78f98ba269950ee23845d3f47b6c187b08 | treesnapshot (tar.gz zip) |
| parent | 65d7a4b97f3c7bd88f35509e6464632a5124f828 | commitdiff |
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
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: