Moving average fixes

CMakeLists.txt

@@ -116,7 +116,6 @@ set(sdrbase_SOURCES
     sdrbase/dsp/interpolator.cpp
     sdrbase/dsp/hbfiltertraits.cpp
     sdrbase/dsp/lowpass.cpp
-    sdrbase/dsp/movingaverage.cpp
     sdrbase/dsp/nco.cpp
     sdrbase/dsp/ncof.cpp
     sdrbase/dsp/pidcontroller.cpp

sdrbase/dsp/movingaverage.cpp

@@ -1 +0,0 @@
-#include "dsp/movingaverage.h"

sdrbase/dsp/movingaverage.h

@@ -3,6 +3,7 @@
 
 #include <stdint.h>
 #include <vector>
+#include <algorithm>
 #include "dsp/dsptypes.h"
 
 template<class Type> class MovingAverage {
@@ -24,11 +25,7 @@ public:
 	void resize(int historySize, Type initial)
 	{
 		m_history.resize(historySize);
-
-		for(size_t i = 0; i < m_history.size(); i++) {
-			m_history[i] = initial;
-		}
-
+		std::fill(m_history.begin(), m_history.end(), initial);
 		m_sum = (Type) m_history.size() * initial;
 		m_index = 0;
 	}
@@ -39,19 +36,16 @@ public:
         m_sum += value - oldest;
         oldest = value;
 
-	    m_index++;
-
-	    if(m_index >= m_history.size()) {
-	        m_index = 0;
-	    }
+        if (m_index < m_history.size()) {
+            m_index++;
+        } else {
+            m_index = 0;
+        }
 	}
 
 	void fill(Type value)
 	{
-		for(size_t i = 0; i < m_history.size(); i++) {
-			m_history[i] = value;
-		}
-
+        std::fill(m_history.begin(), m_history.end(), value);
 		m_sum = (Type) m_history.size() * value;
 	}