From eac79b6d2e0a4feeccf7e5936cd972fdc4a74ac7 Mon Sep 17 00:00:00 2001
From: Benoit Jacob <jacob.benoit.1@gmail.com>
Date: Sat, 9 May 2009 03:41:17 +0000
Subject: [PATCH] CREDIT Hauke Heibel, fix MSVC warnings

---
 test/stdvector.cpp                     | 2 +-
 unsupported/Eigen/AlignedVector3       | 2 +-
 unsupported/test/BVH.cpp               | 4 ++--
 unsupported/test/autodiff.cpp          | 4 +---
 unsupported/test/matrixExponential.cpp | 2 +-
 5 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/test/stdvector.cpp b/test/stdvector.cpp
index 10788a77f..e2e73680e 100644
--- a/test/stdvector.cpp
+++ b/test/stdvector.cpp
@@ -22,8 +22,8 @@
 // License and a copy of the GNU General Public License along with
 // Eigen. If not, see <http://www.gnu.org/licenses/>.
 
-#include <Eigen/StdVector>
 #include "main.h"
+#include <Eigen/StdVector>
 #include <Eigen/Geometry>
 
 template<typename MatrixType>
diff --git a/unsupported/Eigen/AlignedVector3 b/unsupported/Eigen/AlignedVector3
index 19217693f..e7c6ed2cc 100644
--- a/unsupported/Eigen/AlignedVector3
+++ b/unsupported/Eigen/AlignedVector3
@@ -92,7 +92,7 @@ template<typename _Scalar> class AlignedVector3
     {}
     
     template<typename XprType, int Size=XprType::SizeAtCompileTime>
-    struct generic_assign_selector;
+    struct generic_assign_selector {};
     
     template<typename XprType> struct generic_assign_selector<XprType,4>
     {
diff --git a/unsupported/test/BVH.cpp b/unsupported/test/BVH.cpp
index 879daacc2..ea03c2d24 100644
--- a/unsupported/test/BVH.cpp
+++ b/unsupported/test/BVH.cpp
@@ -22,8 +22,8 @@
 // License and a copy of the GNU General Public License along with
 // Eigen. If not, see <http://www.gnu.org/licenses/>.
 
-#include <Eigen/StdVector>
 #include "main.h"
+#include <Eigen/StdVector>
 #include <unsupported/Eigen/BVH>
 
 inline double SQR(double x) { return x * x; }
@@ -43,7 +43,7 @@ EIGEN_MAKE_ALIGNED_OPERATOR_NEW_IF_VECTORIZABLE_FIXED_SIZE(double, Dim)
 };
 
 
-template<int Dim> AlignedBox<double, Dim> ei_bounding_box(const Matrix<double, Dim, 1> &v) { return AlignedBox<double, Dim>(v); }
+template<typename Scalar, int Dim> AlignedBox<Scalar, Dim> ei_bounding_box(const Matrix<Scalar, Dim, 1> &v) { return AlignedBox<Scalar, Dim>(v); }
 template<int Dim> AlignedBox<double, Dim> ei_bounding_box(const Ball<Dim> &b)
 { return AlignedBox<double, Dim>(b.center.cwise() - b.radius, b.center.cwise() + b.radius); }
 
diff --git a/unsupported/test/autodiff.cpp b/unsupported/test/autodiff.cpp
index 71ae4cdc4..8d0986125 100644
--- a/unsupported/test/autodiff.cpp
+++ b/unsupported/test/autodiff.cpp
@@ -30,7 +30,7 @@ EIGEN_DONT_INLINE Scalar foo(const Scalar& x, const Scalar& y)
 {
 //   return x+std::sin(y);
   EIGEN_ASM_COMMENT("mybegin");
-  return x*2 - std::pow(x,2) + 2*std::sqrt(y*y) - 4 * std::sin(x) + 2 * std::cos(y) - std::exp(-0.5*x*x);
+  return static_cast<Scalar>(x*2 - std::pow(x,2) + 2*std::sqrt(y*y) - 4 * std::sin(x) + 2 * std::cos(y) - std::exp(-0.5*x*x));
 //   return y/x;// - y*2;
   EIGEN_ASM_COMMENT("myend");
 }
@@ -136,8 +136,6 @@ template<typename Func> void adolc_forward_jacobian(const Func& f)
 
 void test_autodiff()
 {
-  std::sqrt(3);
-  std::sin(3);
   std::cerr << foo<float>(1,2) << "\n";
   AutoDiffScalar<Vector2f> ax(1,Vector2f::UnitX());
   AutoDiffScalar<Vector2f> ay(2,Vector2f::UnitY());
diff --git a/unsupported/test/matrixExponential.cpp b/unsupported/test/matrixExponential.cpp
index 6dcd80334..7f191731b 100644
--- a/unsupported/test/matrixExponential.cpp
+++ b/unsupported/test/matrixExponential.cpp
@@ -22,8 +22,8 @@
 // License and a copy of the GNU General Public License along with
 // Eigen. If not, see <http://www.gnu.org/licenses/>.
 
-#include <Eigen/StdVector>
 #include "main.h"
+#include <Eigen/StdVector>
 #include <unsupported/Eigen/MatrixFunctions>
 
 double binom(int n, int k)