Remove use of where(), if DEBUG etc.
authorMark Abraham <mark.j.abraham@gmail.com>
Fri, 2 Mar 2018 21:08:23 +0000 (22:08 +0100)
committerMark Abraham <mark.j.abraham@gmail.com>
Thu, 12 Apr 2018 15:57:31 +0000 (17:57 +0200)
commitc8c0605d074b04d5ddf358bbe22e019e267977b4
treea33b3096781709ff5bcf004e968fde35de1f2b4d
parente9e20e64729161c6795fcb75b3dd63a81c0bb74a
Remove use of where(), if DEBUG etc.

The where() debugging function causes run-time branches in
release-mode builds, and replaces functionality for which one should
use a debugger.

Other such preprocessing clutter should also go away. Since we don't
check whether any of it compiles or produces useful data, we may as
well delete it.

This coincidentally simplifies the call sigatures of some
functionality, too.

Retained some useful essentialdynamics debug code, which is now always
compiled, and written from setup code only at debug level 2.

Refs #2423
Fixes #2122

Change-Id: I2c60c162734f4c2ec1d56153853a36d28e6a66ff
23 files changed:
src/gromacs/essentialdynamics/edsam.cpp
src/gromacs/ewald/pme-redistribute.cpp
src/gromacs/ewald/pme.cpp
src/gromacs/listed-forces/bonded.cpp
src/gromacs/mdlib/constr.cpp
src/gromacs/mdlib/coupling.cpp
src/gromacs/mdlib/ebin.cpp
src/gromacs/mdlib/force.cpp
src/gromacs/mdlib/forcerec.cpp
src/gromacs/mdlib/md_support.cpp
src/gromacs/mdlib/rbin.cpp
src/gromacs/mdlib/settle.cpp
src/gromacs/mdlib/shake.cpp
src/gromacs/mdlib/stat.cpp
src/gromacs/mdlib/tgroup.cpp
src/gromacs/mdlib/update.cpp
src/gromacs/mdlib/update.h
src/gromacs/mdrun/md.cpp
src/gromacs/mdrun/minimize.cpp
src/gromacs/utility/fatalerror.cpp
src/gromacs/utility/fatalerror.h
src/gromacs/utility/futil.cpp
src/gromacs/utility/smalloc.cpp