Preparation for de-duplicating mdrun setup
authorM. Eric Irrgang <ericirrgang@gmail.com>
Sun, 14 Oct 2018 12:33:48 +0000 (15:33 +0300)
committerPaul Bauer <paul.bauer.q@gmail.com>
Mon, 15 Oct 2018 06:31:59 +0000 (08:31 +0200)
commitdf3b598f6d15d1e2e70e1c6c3bf5debc49069c47
tree24350372ecc2b24c5713c794adfff4ff19c63fd3
parent6da853b07596eafd31dd5e763d29a2ff14921c13
Preparation for de-duplicating mdrun setup

Recent work introducing the API extracted MdFilenames and then
essentially duplicated everything else. This commit expands
MdFilenames into LegacyMdrunOptions, and clarified the plan that all
such code needs to become either module-based option handling, or
aspects of SimulationContext suitable for both CLI and API.

This commit introduces the Legacy MdrunOptions struct, so that a
future commit can make it available to the API setup for an mdrun
session. The layout of the changes is intended to minimize the textual
diff for the CLI. Code motion for clarity and also use by the API will
follow.

Removed MdFilenames, made its data a std::vector in
LegacyMdrunOptions, and used it via an ArrayRef. This clarifies that
it is read-only input data supplied by the client.

Change-Id: I3438cfe1ab52eeb2fffbd16f4bd5254b2a74f03f
src/api/cpp/context-impl.h
src/api/cpp/context.cpp
src/gromacs/mdrun/legacyoptions.h [new file with mode: 0644]
src/gromacs/mdrun/runner.cpp
src/gromacs/mdrun/runner.h
src/gromacs/mdrun/simulationcontext.h
src/programs/legacymodules.cpp
src/programs/mdrun/mdrun.cpp
src/programs/mdrun/mdrun_main.h