From 26468eccca71ed81a32abc29d9d8ecfabf4f86f4 Mon Sep 17 00:00:00 2001
From: Axel Kohlmeyer <akohlmey@gmail.com>
Date: Sun, 16 Oct 2022 11:46:38 -0400
Subject: [PATCH] silence compiler warnings

---
 src/compute_slice.cpp   |  2 +-
 src/fix_ave_atom.cpp    |  6 +++---
 src/fix_store_state.cpp | 14 +++++++-------
 3 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/src/compute_slice.cpp b/src/compute_slice.cpp
index 53f79a3275..0e34d0a1e7 100644
--- a/src/compute_slice.cpp
+++ b/src/compute_slice.cpp
@@ -222,7 +222,7 @@ void ComputeSlice::compute_array()
 {
   invoked_array = update->ntimestep;
 
-  for (int m = 0; m < values.size(); m++) extract_one(0, &array[m][0], values.size());
+  for (std::size_t m = 0; m < values.size(); m++) extract_one(0, &array[m][0], values.size());
 }
 
 /* ----------------------------------------------------------------------
diff --git a/src/fix_ave_atom.cpp b/src/fix_ave_atom.cpp
index 303c06d227..bf57d85063 100644
--- a/src/fix_ave_atom.cpp
+++ b/src/fix_ave_atom.cpp
@@ -386,7 +386,7 @@ void FixAveAtom::grow_arrays(int nmax)
 
 void FixAveAtom::copy_arrays(int i, int j, int /*delflag*/)
 {
-  for (int m = 0; m < values.size(); m++)
+  for (std::size_t m = 0; m < values.size(); m++)
     array[j][m] = array[i][m];
 }
 
@@ -396,7 +396,7 @@ void FixAveAtom::copy_arrays(int i, int j, int /*delflag*/)
 
 int FixAveAtom::pack_exchange(int i, double *buf)
 {
-  for (int m = 0; m < values.size(); m++) buf[m] = array[i][m];
+  for (std::size_t m = 0; m < values.size(); m++) buf[m] = array[i][m];
   return values.size();
 }
 
@@ -406,7 +406,7 @@ int FixAveAtom::pack_exchange(int i, double *buf)
 
 int FixAveAtom::unpack_exchange(int nlocal, double *buf)
 {
-  for (int m = 0; m < values.size(); m++) array[nlocal][m] = buf[m];
+  for (std::size_t m = 0; m < values.size(); m++) array[nlocal][m] = buf[m];
   return values.size();
 }
 
diff --git a/src/fix_store_state.cpp b/src/fix_store_state.cpp
index 664aac841a..b9b301158d 100644
--- a/src/fix_store_state.cpp
+++ b/src/fix_store_state.cpp
@@ -338,7 +338,7 @@ FixStoreState::FixStoreState(LAMMPS *lmp, int narg, char **arg) :
 
   int nlocal = atom->nlocal;
   for (int i = 0; i < nlocal; i++)
-    for (int m = 0; m < values.size(); m++)
+    for (std::size_t m = 0; m < values.size(); m++)
       avalues[i][m] = 0.0;
 
   // store current values for keywords but not for compute, fix, variable
@@ -560,7 +560,7 @@ void FixStoreState::grow_arrays(int nmax)
 
 void FixStoreState::copy_arrays(int i, int j, int /*delflag*/)
 {
-  for (int m = 0; m < values.size(); m++) avalues[j][m] = avalues[i][m];
+  for (std::size_t m = 0; m < values.size(); m++) avalues[j][m] = avalues[i][m];
 }
 
 /* ----------------------------------------------------------------------
@@ -569,7 +569,7 @@ void FixStoreState::copy_arrays(int i, int j, int /*delflag*/)
 
 int FixStoreState::pack_exchange(int i, double *buf)
 {
-  for (int m = 0; m < values.size(); m++) buf[m] = avalues[i][m];
+  for (std::size_t m = 0; m < values.size(); m++) buf[m] = avalues[i][m];
   return values.size();
 }
 
@@ -579,7 +579,7 @@ int FixStoreState::pack_exchange(int i, double *buf)
 
 int FixStoreState::unpack_exchange(int nlocal, double *buf)
 {
-  for (int m = 0; m < values.size(); m++) avalues[nlocal][m] = buf[m];
+  for (std::size_t m = 0; m < values.size(); m++) avalues[nlocal][m] = buf[m];
   return values.size();
 }
 
@@ -591,7 +591,7 @@ int FixStoreState::pack_restart(int i, double *buf)
 {
   // pack buf[0] this way because other fixes unpack it
   buf[0] = values.size()+1;
-  for (int m = 0; m < values.size(); m++) buf[m+1] = avalues[i][m];
+  for (std::size_t m = 0; m < values.size(); m++) buf[m+1] = avalues[i][m];
   return values.size()+1;
 }
 
@@ -607,10 +607,10 @@ void FixStoreState::unpack_restart(int nlocal, int nth)
   // unpack the Nth first values this way because other fixes pack them
 
   int m = 0;
-  for (int i = 0; i < nth; i++) m += static_cast<int> (extra[nlocal][m]);
+  for (int i = 0; i < nth; i++) m += static_cast<int>(extra[nlocal][m]);
   m++;
 
-  for (int i = 0; i < values.size(); i++) avalues[nlocal][i] = extra[nlocal][m++];
+  for (std::size_t i = 0; i < values.size(); i++) avalues[nlocal][i] = extra[nlocal][m++];
 }
 
 /* ----------------------------------------------------------------------