Refactor gmx_fatal()
authorTeemu Murtola <teemu.murtola@gmail.com>
Sun, 13 Apr 2014 05:29:48 +0000 (08:29 +0300)
committerGerrit Code Review <gerrit@gerrit.gromacs.org>
Wed, 16 Apr 2014 01:45:01 +0000 (03:45 +0200)
commit8a9449370c3148c3a82d107bece893d85b4a005a
tree765d06ad65aa21529d67de191a0341a75e0b97be
parent4a7ddb22c130689cfe9aedaff16053059c947153
Refactor gmx_fatal()

- Remove t_commrec and legacyheaders/ dependencies from
  utility/fatalerror.cpp by splitting gmx_fatal_collective() into a
  low-level helper (in fatalerror.cpp) and a high-level routine that
  knows t_commrec (in network.c).
- Move the declaration of gmx_fatal_collective to network.h to get rid
  of one more header in legacyheaders/.
- Refactor fatalerror.cpp such that the error handler is no longer
  responsible of terminating the program.  Split functionality into
  helper functions that can be called as appropriate, and remove a lot
  of duplication.
- Clean up gmx_abort() and code that calls it.
- Remove interactive prompt from the error handler exit path.

Change-Id: Ia0ab2e415a18bf6ed17bebd1fd8870946d16793e
src/gromacs/gmxlib/gmx_detect_hardware.c
src/gromacs/gmxlib/network.c
src/gromacs/legacyheaders/gmx_fatal_collective.h [deleted file]
src/gromacs/legacyheaders/network.h
src/gromacs/mdlib/domdec.c
src/gromacs/utility/basenetwork.cpp
src/gromacs/utility/basenetwork.h
src/gromacs/utility/exceptions.cpp
src/gromacs/utility/fatalerror.cpp
src/gromacs/utility/fatalerror.h
src/programs/mdrun/runner.c