[Rcpp-commits] r2898 - in pkg/Rcpp: . inst/include/Rcpp

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

Author: edd
Date: 2011-02-13 21:24:03 +0100 (Sun, 13 Feb 2011)
New Revision: 2898

Modified:
   pkg/Rcpp/ChangeLog
   pkg/Rcpp/inst/include/Rcpp/Module.h
Log:
better solution for Module symbol


Modified: pkg/Rcpp/ChangeLog
===================================================================
--- pkg/Rcpp/ChangeLog	2011-02-13 16:33:11 UTC (rev 2897)
+++ pkg/Rcpp/ChangeLog	2011-02-13 20:24:03 UTC (rev 2898)
@@ -1,7 +1,7 @@
 2011-02-13  Dirk Eddelbuettel  <edd at debian.org>
 
 	* inst/include/Rcpp/Module.h: Cache return of Rf_install("Module") in
-	an inline helper function used by the LOAD_RCPP_MODULE macro
+	a static helper function used by the LOAD_RCPP_MODULE macro
 
 2011-02-11  Dirk Eddelbuettel  <edd at debian.org>
 

Modified: pkg/Rcpp/inst/include/Rcpp/Module.h
===================================================================
--- pkg/Rcpp/inst/include/Rcpp/Module.h	2011-02-13 16:33:11 UTC (rev 2897)
+++ pkg/Rcpp/inst/include/Rcpp/Module.h	2011-02-13 20:24:03 UTC (rev 2898)
@@ -741,8 +741,8 @@
 
 // helper function to cache the result of Rf_install("Module"): once
 // it is allocated and in the symbol table it is safe from gc
-inline SEXP getModuleSym() {
-    static SEXP moduleSym = NULL;
+static SEXP moduleSym = NULL;
+static SEXP getModuleSym() {
     if (moduleSym == NULL) {
 	moduleSym = Rf_install("Module");
     }