[mlpack] 113/149: Disable C4519 in prereqs.hpp not core.hpp.

Barak A. Pearlmutter barak+git at pearlmutter.net
Sat May 2 09:11:15 UTC 2015


This is an automated email from the git hooks/post-receive script.

bap pushed a commit to branch svn-trunk
in repository mlpack.

commit 884c029d4d3e16af93b3aacd08316b4f70ae357f
Author: rcurtin <rcurtin at 9d5b8971-822b-0410-80eb-d18c1038ef23>
Date:   Wed Nov 19 18:38:40 2014 +0000

    Disable C4519 in prereqs.hpp not core.hpp.
    
    
    git-svn-id: http://svn.cc.gatech.edu/fastlab/mlpack/trunk@17394 9d5b8971-822b-0410-80eb-d18c1038ef23
---
 src/mlpack/core.hpp    | 7 -------
 src/mlpack/prereqs.hpp | 7 +++++++
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/mlpack/core.hpp b/src/mlpack/core.hpp
index 1bb1401..4e4a5ec 100644
--- a/src/mlpack/core.hpp
+++ b/src/mlpack/core.hpp
@@ -193,10 +193,3 @@
     #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
diff --git a/src/mlpack/prereqs.hpp b/src/mlpack/prereqs.hpp
index fa18340..50a4f97 100644
--- a/src/mlpack/prereqs.hpp
+++ b/src/mlpack/prereqs.hpp
@@ -48,4 +48,11 @@ problems.  It should only be necessary to include <mlpack/core.hpp> and not\
 // Now include Armadillo through the special mlpack extensions.
 #include <mlpack/core/arma_extend/arma_extend.hpp>
 
+// 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
+
 #endif

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/debian-science/packages/mlpack.git



More information about the debian-science-commits mailing list