Simplified handling of simulation runners
commit6f1f895fcbdf44d4dfe01ac237ebe45fc4130b98
authorMark Abraham <mark.j.abraham@gmail.com>
Wed, 28 Feb 2018 15:20:03 +0000 (28 16:20 +0100)
committerMark Abraham <mark.j.abraham@gmail.com>
Thu, 5 Apr 2018 17:41:51 +0000 (5 19:41 +0200)
treeab9c1ac8b1a868a690ef8572d5fa9a3e2a530f65
parentad79f7789085a1ba0dd400460450d081bda3331b
Simplified handling of simulation runners

Argments to runners are now passed via struct members. That struct is
expected to change over the medium term, as most of its contents
should either be in a container of modules, or data with such a
module. As such, it does not make sense to do a lot of coding to
manage its contents in a sound way. The practical effects of using
this struct are almost the same as the old approach of passing a
forest of arguments - some variables are declared in a scope,
initialized with values, then used once.

The form is chosen so that
* the do_md() etc. functions do not need a large number of useless
  textual changes now
* future changes to names and types of the former parameters do not
  need to be edited in matching way in multiple places of code and
  their doxygen
* we don't need to mark things gmx_unused anywhere

Useless unchecked return values have been removed.

Some excessive includes in integrator.h were removed, which
generates a few minor fixes elsewhere.

Refs #1793

Change-Id: I678598175192c9c68113fdd79fcee17f8e5c504e
src/gromacs/mdrun/CMakeLists.txt
src/gromacs/mdrun/integrator.cpp [copied from src/gromacs/mdrun/minimize.h with 55% similarity]
src/gromacs/mdrun/integrator.h
src/gromacs/mdrun/md.cpp
src/gromacs/mdrun/minimize.cpp
src/gromacs/mdrun/minimize.h
src/gromacs/mdrun/runner.cpp
src/gromacs/mdrun/tpi.cpp