[Rcpp-commits] r3235 - pkg/RcppEigen/src

[noreply at r-forge.r-project.org]

Author: dmbates
Date: 2011-10-28 00:26:14 +0200 (Fri, 28 Oct 2011)
New Revision: 3235

Modified:
   pkg/RcppEigen/src/Makevars
   pkg/RcppEigen/src/Makevars.win
Log:
Explain the use of the -DNDEBUG flag.


Modified: pkg/RcppEigen/src/Makevars
===================================================================
--- pkg/RcppEigen/src/Makevars	2011-10-27 22:25:17 UTC (rev 3234)
+++ pkg/RcppEigen/src/Makevars	2011-10-27 22:26:14 UTC (rev 3235)
@@ -2,7 +2,7 @@
 
 PKG_CXXFLAGS=-I../inst/include -DNDEBUG
 
-## In practice the compiler flags show be
+## In practice the compiler flags should be
 #PKG_CXXFLAGS=-I../inst/include
 ## to enable error checking.  However, checking for errors is flagged
 ## in R CMD check as an undesirable practice.

Modified: pkg/RcppEigen/src/Makevars.win
===================================================================
--- pkg/RcppEigen/src/Makevars.win	2011-10-27 22:25:17 UTC (rev 3234)
+++ pkg/RcppEigen/src/Makevars.win	2011-10-27 22:26:14 UTC (rev 3235)
@@ -2,6 +2,12 @@
 ## Use the R_HOME indirection to support installations of multiple R version
 PKG_LIBS = $(shell "${R_HOME}/bin${R_ARCH_BIN}/Rscript.exe" -e "Rcpp:::LdFlags()")
 
-PKG_CPPFLAGS = -I../inst/include -I.
+PKG_CPPFLAGS = -I../inst/include -I. -DNDEBUG
 
+## For development use the package CXXFLAGS as
+#PKG_CXXFLAGS= -I../inst/include -I.
+## to activate assertions in Eigen.  For the purposes of R CMD check the
+## assertions should be suppressed.
 
+
+