Fix windows warnings

Signed-off-by: Michael Carroll <[email protected]>
gazebosim · Mar 28, 2022 · 6e4d62d · 6e4d62d
1 parent 098bac5
commit 6e4d62d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/include/ignition/math/MovingWindowFilter.hh b/include/ignition/math/MovingWindowFilter.hh
@@ -73,10 +73,10 @@ namespace ignition
 
       /// \brief iterator pointing to current value in buffer
       protected: typename std::vector<T>::iterator valIter;
-      IGN_UTILS_WARN_RESUME__DLL_INTERFACE_MISSING
 
       /// \brief keep track of running sum
       protected: T sum;
+      IGN_UTILS_WARN_RESUME__DLL_INTERFACE_MISSING
 
       /// \brief keep track of number of elements
       protected: unsigned int samples = 0;

diff --git a/src/MovingWindowFilter.cc b/src/MovingWindowFilter.cc
@@ -96,7 +96,9 @@ bool MovingWindowFilter<T>::WindowFilled() const
 template<typename T>
 T MovingWindowFilter<T>::Value() const
 {
-  return static_cast<T>(this->sum / static_cast<double>(this->samples));
+  const double value =
+    static_cast<double>(this->sum) / static_cast<double>(this->samples);
+  return static_cast<T>(value);
 }
 
 template class MovingWindowFilter<int>;