Changes as per the reviews on the previous patches

As per the reviews on the previous patches, submitting a patch with all the changes suggested in the reviews.

Tested on RD-PQ device

Change-Id: I15e79bdd0331c17908cc62fa523bab188cc28295

diff --git a/src/libsensord/sensor_geomagnetic_rv.h b/src/libsensord/sensor_geomagnetic_rv.h
index 73106aeb309e50f841c6d0e264271fb9552766f7..3deaaa2d61cdea6de20903960858aa0955a188b8 100755 (executable)
--- a/src/libsensord/sensor_geomagnetic_rv.h
+++ b/src/libsensord/sensor_geomagnetic_rv.h
@@ -48,6 +48,4 @@ enum geomagnetic_rv_event_type {
 }
 #endif
 
-#endif
-//! End of a file
-
+#endif /* __SENSOR_GEOMAGNETIC_RV_H__ */

diff --git a/src/sensor_fusion/euler_angles.h b/src/sensor_fusion/euler_angles.h
index 061dfb103ad19e2c0cd22a915e5646e20f547278..9e2f6079a610cc8c49eafe2bb5343b653ae01bc5 100644 (file)
--- a/src/sensor_fusion/euler_angles.h
+++ b/src/sensor_fusion/euler_angles.h
@@ -29,7 +29,7 @@
 template <typename TYPE>
 class euler_angles {
 public:
-       vect<TYPE,EULER_SIZE> m_ang;
+       vect<TYPE, EULER_SIZE> m_ang;
 
        euler_angles();
        euler_angles(const TYPE roll, const TYPE pitch, const TYPE azimuth);

diff --git a/src/sensor_fusion/matrix.cpp b/src/sensor_fusion/matrix.cpp
index 9d483f2961948ffd46e0d1b1b25f97d7061dd144..62f055531983c44df024f42d03ece3c422134b9b 100755 (executable)
--- a/src/sensor_fusion/matrix.cpp
+++ b/src/sensor_fusion/matrix.cpp
@@ -51,9 +51,9 @@ TYPE_ROW_COL matrix<TYPE, ROW, COL> matrix<TYPE, ROW, COL>::operator =(const mat
                return *this;
        }
 
-       for (int p = 0; p < ROW; p++)
-               for (int q = 0; q < COL; q++)
-                       m_mat[p][q] = m.m_mat[p][q];
+       for (int i = 0; i < ROW; i++)
+               for (int j = 0; j < COL; j++)
+                       m_mat[i][j] = m.m_mat[i][j];
 
        return *this;
 }

diff --git a/src/sensor_fusion/orientation_filter.cpp b/src/sensor_fusion/orientation_filter.cpp
index 2bf786563864fd390f1558e86e7abb943d33ac81..76febbb12c196571706e3dea3dc7d0e1ed7c4294 100644 (file)
--- a/src/sensor_fusion/orientation_filter.cpp
+++ b/src/sensor_fusion/orientation_filter.cpp
@@ -84,7 +84,7 @@ inline void orientation_filter<TYPE>::init_accel_gyro_mag_data(const sensor_data
        m_magnetic.m_data = magnetic.m_data;
 
        if (m_gyro.m_time_stamp != 0 && gyro.m_time_stamp != 0)
-               sample_interval_gyro =  gyro.m_time_stamp - m_gyro.m_time_stamp;
+               sample_interval_gyro = gyro.m_time_stamp - m_gyro.m_time_stamp;
 
        m_gyro_dt = sample_interval_gyro * US2S;
 

diff --git a/src/sensor_fusion/quaternion.cpp b/src/sensor_fusion/quaternion.cpp
index caf29509187235698f5a1b405b53750f53079839..9d28f58229967d9df268c8faafe7e8a3250d0fef 100755 (executable)
--- a/src/sensor_fusion/quaternion.cpp
+++ b/src/sensor_fusion/quaternion.cpp
@@ -30,7 +30,7 @@ template <typename T> int sgn(T val) {
 
 template <typename T> T mag(T val) {
        if (val < 0)
-               return val * (T)-1;
+               return val * static_cast<T>(-1);
        else
                return val;
 }

diff --git a/src/sensor_fusion/vector.cpp b/src/sensor_fusion/vector.cpp
index 2fc932aaadb39023afaa7d7334f87a86be2f2ad2..5174035f92791a1bc5437a38e51f84f5df8e59f3 100644 (file)
--- a/src/sensor_fusion/vector.cpp
+++ b/src/sensor_fusion/vector.cpp
@@ -21,7 +21,7 @@
 
 TYPE_SIZE vect<TYPE, SIZE>::vect(void)
 {
-       for(int i=0;i<SIZE;i++)
+       for (int i = 0; i < SIZE; i++)
                m_vec[i] = 0;
 }
 
@@ -46,20 +46,18 @@ TYPE_SIZE vect<TYPE, SIZE>::~vect()
 TYPE_SIZE vect<TYPE, SIZE> vect<TYPE, SIZE>::operator =(const vect<TYPE, SIZE>& v)
 {
        if (this == &v)
-       {
                return *this;
-       }
+
        for (int q = 0; q < SIZE; q++)
                m_vec[q] = v.m_vec[q];
-                       return *this;
+
+       return *this;
 }
 
 T_S ostream& operator <<(ostream& dout, vect<T, S>& v)
 {
        for (int j = 0; j < S; j++)
-       {
                dout << v.m_vec[j] << "\t";
-       }
 
        dout << endl;
 
@@ -114,12 +112,8 @@ T_S_R_C matrix<T, R, S> operator *(const matrix<T, R, C> m, const vect<T, S> v)
        matrix<T,R,S> m1;
 
        for (int i = 0; i < R; i++)
-       {
                for (int j = 0; j < S; j++)
-               {
                        m1.m_mat[i][j] = m.m_mat[i][0] * v.m_vec[j];
-               }
-       }
 
        return m1;
 }