C++-ify analysis nbsearch more.
authorTeemu Murtola <teemu.murtola@gmail.com>
Sun, 12 May 2013 11:38:35 +0000 (14:38 +0300)
committerGerrit Code Review <gerrit@gerrit.gromacs.org>
Thu, 27 Jun 2013 20:18:57 +0000 (22:18 +0200)
commit902c99cbbcf00c29ebd6e8116cc3fe6e25cc3002
tree602fef288a1e14beec6ca36de299a79cefce8fac
parenta4a0693aef2a0fd22c840ccf05f175a0a96e12ff
C++-ify analysis nbsearch more.

- Use static_cast instead of C casts throughout.
- Use more C++-style comments.
- Use the same pattern for startPairSearch() as for initSearch() to
  allow multiple concurrent searches.
- Use std::vector instead of explicit memory allocation for the grid
  cell management.  Memory is still managed explicitly for rvec and ivec
  arrays, since they can't be put into std::vector.
- Make most members of the implementation classes private to clarify the
  code (the implementation class is now explicitly responsible for
  managing its internal state).

Part of #866.

Change-Id: I2af5552b696644f954957f96705baabd891284a9
src/gromacs/selection/nbsearch.cpp
src/gromacs/selection/nbsearch.h
src/gromacs/selection/tests/nbsearch.cpp