Fix clang-tidy warnings in pme and sim_utils
authorArtem Zhmurov <zhmurov@gmail.com>
Fri, 24 Sep 2021 09:22:09 +0000 (12:22 +0300)
committerArtem Zhmurov <zhmurov@gmail.com>
Fri, 24 Sep 2021 09:22:09 +0000 (12:22 +0300)
commitc95da1913552336123aeeedfe305126793abd655
tree3a6cf0983603f2a2c1a37796456a0b9a8f5e3393
parent4d38a0d65839f951bf275e465f8fb65a35b60b8d
Fix clang-tidy warnings in pme and sim_utils

1. The global non-const variable was temporary introduced to able to
   test the PME SYCL spread kernel independently of the rest of the PME
   (f190fa59b4a72390b2aea2e74b4960941098ce4d). NOLINT should be
   temporary fix as this variable will go away once the rest of PME
   is implemented in SYCL.
2. The conditionals in sim_utils wrap the same copy operation after
   4d38a0d65839f951bf275e465f8fb65a35b60b8d. So these conditional can
   now be combined.
src/gromacs/ewald/pme.cpp
src/gromacs/ewald/pme.h
src/gromacs/mdrun/md.cpp