| commit | 7f3b42b29473072a5529743d08158106088386ef | |
| author | Mark Abraham <mark.j.abraham@gmail.com> | |
| Sun, 9 Jul 2017 06:24:17 +0000 (9 08:24 +0200) | ||
| committer | Mark Abraham <mark.j.abraham@gmail.com> | |
| Thu, 13 Jul 2017 19:27:36 +0000 (13 21:27 +0200) | ||
| tree | 58770a1f1a977ed94ddff45afdbbdbaf0e977b27 | treesnapshot (tar.gz zip) |
| parent | 4df5596f512e11ae3961952f376d3f7323711158 | commitdiff |
Cleaned up mdrunner_start_threads
Threw an exception rather than return a nullptr to trigger a
fatal error.
Removed unnecessary variables. Parameters passed by value are copied,
so there is no need to make that explicit just because the return
value will be stored in the same variable that was just the source of
a parameter.
Noted TODOs for removing some allocation and leaks if we progress to
the point where mdrun filename arguments can be proved to be used only
in a const-safe read-only manner.
Updated and converted comments to Doxygen
Change-Id: I638c757d2d0aa5175dfe3769b499993eb613ea26
Threw an exception rather than return a nullptr to trigger a
fatal error.
Removed unnecessary variables. Parameters passed by value are copied,
so there is no need to make that explicit just because the return
value will be stored in the same variable that was just the source of
a parameter.
Noted TODOs for removing some allocation and leaks if we progress to
the point where mdrun filename arguments can be proved to be used only
in a const-safe read-only manner.
Updated and converted comments to Doxygen
Change-Id: I638c757d2d0aa5175dfe3769b499993eb613ea26
| src/programs/mdrun/runner.cpp | diffblobblamehistory |