Merge pull request #1231 from RcppCore/bugfix/rcpp-modules-finalize-materialize

force 'finalize' method to be materialized on Modules

Author: eddelbuettel

ChangeLog

@@ -1,3 +1,7 @@
+2022-09-23  Kevin Ushey  <[email protected]>
+
+	* R/Module.R: Force 'finalize' method to be materialized
+
 2022-09-20  Dirk Eddelbuettel  <[email protected]>
 
 	* DESCRIPTION (Version, Date): Roll minor version

R/Module.R

@@ -131,6 +131,8 @@ new_dummyObject <- function(...)                                # #nocov
 
 # class method for $initialize
 cpp_object_initializer <- function(.self, .refClassDef, ..., .object_pointer){
+    # force finalize method to be materialized
+    invisible(.self$finalize)
     selfEnv <- as.environment(.self)
     ## generate the C++-side object and store its pointer, etc.
     ## access the private fields in the fieldPrototypes env.

inst/NEWS.Rd

@@ -7,6 +7,9 @@
   \itemize{
     \item Changes in Rcpp API:
     \itemize{
+      \item The 'finalize' method for Rcpp Modules is now eagerly materialized,
+      fixing an issue where errors can occur when Module finalizers are run.
+      (Kevin in \ghpr{1231})
       \item Unwind protection is enabled by default (Iñaki in \ghpr{1225}).
       It can be disabled by defining \code{RCPP_NO_UNWIND_PROTECT} before
       including \code{Rcpp.h}. \code{RCPP_USE_UNWIND_PROTECT} is not checked