6 files changed, 11 insertions, 4 deletions
diff --git a/Eigen/Regression b/Eigen/Regression
index 8213ff77f..5504f922c 100644
--- a/Eigen/Regression
+++ b/Eigen/Regression
@@ -2,6 +2,7 @@
 #define EIGEN_REGRESSION_MODULE_H
 
 #include "LU"
+#include "QR"
 
 namespace Eigen {
 
diff --git a/Eigen/Sparse b/Eigen/Sparse
index 12ffc0c66..53bc4eccc 100644
--- a/Eigen/Sparse
+++ b/Eigen/Sparse
@@ -4,8 +4,9 @@
 #include "Core"
 #include <vector>
 #include <map>
-#include <stdlib.h>
-#include <string.h>
+#include <cstdlib>
+#include <cstring>
+#include <algorithm>
 
 namespace Eigen {
 
diff --git a/Eigen/src/Sparse/HashMatrix.h b/Eigen/src/Sparse/HashMatrix.h
index a0383574c..992486f5a 100644
--- a/Eigen/src/Sparse/HashMatrix.h
+++ b/Eigen/src/Sparse/HashMatrix.h
@@ -71,7 +71,7 @@ class HashMatrix : public SparseMatrixBase<HashMatrix<_Scalar, _Flags> >
 
   public:
 
-    inline void startFill(int reserveSize = 1000) {}
+    inline void startFill(int /*reserveSize = 1000 --- currenty unused, don't generate a warning*/) {}
 
     inline Scalar& fill(int row, int col) { return coeffRef(row, col); }
 
diff --git a/Eigen/src/Sparse/SparseArray.h b/Eigen/src/Sparse/SparseArray.h
index bd82e6e9a..0315c93d9 100644
--- a/Eigen/src/Sparse/SparseArray.h
+++ b/Eigen/src/Sparse/SparseArray.h
@@ -51,6 +51,7 @@ template<typename Scalar> class SparseArray
       resize(other.size());
       memcpy(m_values, other.m_values, m_size * sizeof(Scalar));
       memcpy(m_indices, other.m_indices, m_size * sizeof(int));
+      return *this;
     }
 
     void swap(SparseArray& other)
diff --git a/Eigen/src/Sparse/SparseMatrix.h b/Eigen/src/Sparse/SparseMatrix.h
index b34ed8fd2..dbc436f80 100644
--- a/Eigen/src/Sparse/SparseMatrix.h
+++ b/Eigen/src/Sparse/SparseMatrix.h
@@ -213,8 +213,8 @@ class SparseMatrix : public SparseMatrixBase<SparseMatrix<_Scalar, _Flags> >
         for (int j=0; j<=m_outerSize; ++j)
           m_outerIndex[j] = other.m_outerIndex[j];
         m_data = other.m_data;
-        return *this;
       }
+      return *this;
     }
 
     template<typename OtherDerived>
diff --git a/Eigen/src/Sparse/SparseMatrixBase.h b/Eigen/src/Sparse/SparseMatrixBase.h
index 508972cbb..051d1cbc2 100644
--- a/Eigen/src/Sparse/SparseMatrixBase.h
+++ b/Eigen/src/Sparse/SparseMatrixBase.h
@@ -73,8 +73,10 @@ class SparseMatrixBase : public MatrixBase<Derived>
         {
           Scalar v = it.value();
           if (v!=Scalar(0))
+          {
             if (OtherDerived::Flags & RowMajorBit) temp.fill(j,it.index()) = v;
             else temp.fill(it.index(),j) = v;
+          }
         }
       }
       temp.endFill();
@@ -100,8 +102,10 @@ class SparseMatrixBase : public MatrixBase<Derived>
           {
             Scalar v = it.value();
             if (v!=Scalar(0))
+            {
               if (RowMajor) derived().fill(j,it.index()) = v;
               else derived().fill(it.index(),j) = v;
+            }
           }
         }
         derived().endFill();