Fix clang-tidy complaint failing our CI lint test
authorAndrey Alekseenko <al42and@gmail.com>
Sat, 6 Mar 2021 14:01:57 +0000 (17:01 +0300)
committerArtem Zhmurov <zhmurov@gmail.com>
Sun, 7 Mar 2021 19:32:56 +0000 (19:32 +0000)
The complaint was about non-const reference, but I also changed an
uninitialized variable nearby.

src/gromacs/mdlib/constr.cpp

index c80e67eb82b577c72055859d8ef92429b212bb13..1cb575ad6a3bf809f88b59aee73248da5aabd047 100644 (file)
@@ -223,13 +223,13 @@ bool Constraints::havePerturbedConstraints() const
 }
 
 //! Clears constraint quantities for atoms in nonlocal region.
-static void clear_constraint_quantity_nonlocal(gmx_domdec_t& dd, ArrayRef<RVec> q)
+static void clear_constraint_quantity_nonlocal(const gmx_domdec_t& dd, ArrayRef<RVec> q)
 {
-    int nonlocal_at_start, nonlocal_at_end, at;
+    int nonlocal_at_start, nonlocal_at_end;
 
     dd_get_constraint_range(dd, &nonlocal_at_start, &nonlocal_at_end);
 
-    for (at = nonlocal_at_start; at < nonlocal_at_end; at++)
+    for (int at = nonlocal_at_start; at < nonlocal_at_end; at++)
     {
         clear_rvec(q[at]);
     }