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

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

Author: romain
Date: 2012-11-22 12:16:57 +0100 (Thu, 22 Nov 2012)
New Revision: 4016

Modified:
   pkg/Rcpp/ChangeLog
   pkg/Rcpp/inst/include/Rcpp/iostream/Rostream.h
Log:
delete rdbuf()

Modified: pkg/Rcpp/ChangeLog
===================================================================
--- pkg/Rcpp/ChangeLog	2012-11-22 00:30:15 UTC (rev 4015)
+++ pkg/Rcpp/ChangeLog	2012-11-22 11:16:57 UTC (rev 4016)
@@ -1,3 +1,7 @@
+2012-11-22 Romain Francois <romain at r-enthusiasts.com>
+
+        * include/Rcpp/iostream.h: use delete rdbuf() as recommended by Martyn Plummer
+        
 2012-11-21  JJ Allaire <jj at rstudio.org>
 
         * R/Rcpp.package.skeleton.R: add 'attributes' and 'cpp_files'

Modified: pkg/Rcpp/inst/include/Rcpp/iostream/Rostream.h
===================================================================
--- pkg/Rcpp/inst/include/Rcpp/iostream/Rostream.h	2012-11-22 00:30:15 UTC (rev 4015)
+++ pkg/Rcpp/inst/include/Rcpp/iostream/Rostream.h	2012-11-22 11:16:57 UTC (rev 4016)
@@ -34,6 +34,7 @@
     class Rostream : public std::ostream {
     public:
         Rostream(bool output) : std::ostream( new Rstreambuf(output) ) {}
+        ~Rostream(){ delete rdbuf() ; }
     };
     
     // declare global variable