Improve option initialization in traj. analysis.
authorTeemu Murtola <teemu.murtola@gmail.com>
Wed, 11 Jul 2012 08:25:03 +0000 (11:25 +0300)
committerTeemu Murtola <teemu.murtola@gmail.com>
Wed, 11 Jul 2012 08:31:58 +0000 (11:31 +0300)
commit6ac472288271ac8afcda1d934083271db0c3dc08
treed7ae87f328df046d2b0618ee63d625f0b27ca6fe
parentc304588522416755689c84d0bf71ff616aed6f7e
Improve option initialization in traj. analysis.

All methods that add options to an options object now take an externally
provided options object.  This both simplifies the calling code,
allowing them to control the lifetime of the objects, as well as the
implementing code, since they no longer need to have an internal options
object just for the purpose of returning it.  With the old interface, it
was also impossible to get a reference to the options object more than
once; with the new, the caller has much more flexibility.

Change-Id: I8a2e539fa5abefe696a26919b8591691a1cccc61
18 files changed:
share/template/template.cpp
share/template/template_doc.cpp
src/gromacs/selection/selectioncollection-impl.h
src/gromacs/selection/selectioncollection.cpp
src/gromacs/selection/selectioncollection.h
src/gromacs/trajectoryanalysis/analysismodule.cpp
src/gromacs/trajectoryanalysis/analysismodule.h
src/gromacs/trajectoryanalysis/analysissettings.h
src/gromacs/trajectoryanalysis/cmdlinerunner.cpp
src/gromacs/trajectoryanalysis/modules/angle.cpp
src/gromacs/trajectoryanalysis/modules/angle.h
src/gromacs/trajectoryanalysis/modules/distance.cpp
src/gromacs/trajectoryanalysis/modules/distance.h
src/gromacs/trajectoryanalysis/modules/select.cpp
src/gromacs/trajectoryanalysis/modules/select.h
src/gromacs/trajectoryanalysis/runnercommon.cpp
src/gromacs/trajectoryanalysis/runnercommon.h
src/gromacs/trajectoryanalysis/tests/test_selection.cpp