[mlpack-git] master: Disable C4519 errors entirely. (ce6d580)

gitdub at big.cc.gt.atl.ga.us gitdub at big.cc.gt.atl.ga.us
Thu Mar 5 22:03:47 EST 2015


Repository : https://github.com/mlpack/mlpack

On branch  : master
Link       : https://github.com/mlpack/mlpack/compare/904762495c039e345beba14c1142fd719b3bd50e...f94823c800ad6f7266995c700b1b630d5ffdcf40

>---------------------------------------------------------------

commit ce6d5809e877844d077c6ebe9b03f37b955da75f
Author: Ryan Curtin <ryan at ratml.org>
Date:   Wed Nov 19 18:32:46 2014 +0000

    Disable C4519 errors entirely.


>---------------------------------------------------------------

ce6d5809e877844d077c6ebe9b03f37b955da75f
 src/mlpack/core.hpp | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/src/mlpack/core.hpp b/src/mlpack/core.hpp
index ae36e77..1bb1401 100644
--- a/src/mlpack/core.hpp
+++ b/src/mlpack/core.hpp
@@ -175,7 +175,7 @@
 #include <mlpack/core/kernels/spherical_kernel.hpp>
 #include <mlpack/core/kernels/triangular_kernel.hpp>
 
-// Use armadillo's C++ version detection
+// Use Armadillo's C++ version detection.
 #ifdef ARMA_USE_CXX11
   #define MLPACK_USE_CX11
 #endif
@@ -193,3 +193,10 @@
     #undef max
   #endif
 #endif
+
+// On Visual Studio, disable C4519 (default arguments for function templates)
+// since it's by default an error, which doesn't even make any sense because
+// it's part of the C++11 standard.
+#ifdef _MSC_VER
+  #pragma warning(disable : 4519)
+#endif



More information about the mlpack-git mailing list