Merge pull request #171 from RcppCore/feature/no-vector-from-string

mark two Vector ctors as deprecated

Author: eddelbuettel

ChangeLog

@@ -1,3 +1,8 @@
+2014-08-30  Dirk Eddelbuettel  <[email protected]>
+
+        * inst/include/Rcpp/vector/Vector.h: Mark two 'from string' ctors as
+        deprecated following a suggestion by Yixuan Qiu
+
 2014-08-18  Dirk Eddelbuettel  <[email protected]>
 
         * vignettes/Rcpp-FAQ.Rnw: Added a new entry regarding licensing of

DESCRIPTION

@@ -1,6 +1,6 @@
 Package: Rcpp
 Title: Seamless R and C++ Integration
-Version: 0.11.2.1
+Version: 0.11.2.2
 Date: 2014-06-11
 Author: Dirk Eddelbuettel, Romain Francois, JJ Allaire, Kevin Ushey, 
  Douglas Bates, and John Chambers

inst/NEWS.Rd

@@ -32,6 +32,12 @@
     \itemize{
       \item The \code{Rcpp-FAQ} vignette was updated with respect to OS X issues.
     }
+    \item Deprecation of unused Rcpp featires:
+    \itemize{
+      \item The \code{Vector} class has two constructors from \code{std::string}
+      and \code{const char*} which are not fully functional and will be
+      withdrawn twelve months after this release.
+    }
   }
 }
 

inst/include/Rcpp/vector/Vector.h

@@ -88,12 +88,15 @@ class Vector :
         Storage::set__( Rf_allocVector( RTYPE, size) ) ;
         fill( u ) ;
     }
+
     Vector( const std::string& st ){
         RCPP_DEBUG_2( "Vector<%d>( const std::string& = %s )", RTYPE, st.c_str() )
+        Rf_warning("The Vector(std::string) constructor is deprecated and will be removed in a future release.");
         Storage::set__( internal::vector_from_string<RTYPE>(st) ) ;
     }
     Vector( const char* st ) {
         RCPP_DEBUG_2( "Vector<%d>( const char* = %s )", RTYPE, st )
+        Rf_warning("The Vector(const char*) constructor is deprecated and will be removed in a future release.");
         Storage::set__(internal::vector_from_string<RTYPE>(st) ) ;
     }