Merge pull request #173 from yixuan/erasereturn

fix return value of range eraser in Vector

Author: eddelbuettel

inst/include/Rcpp/vector/Vector.h

@@ -895,13 +895,13 @@ class Vector :
         iterator target_it = target.begin() ;
 
         SEXP names = RCPP_GET_NAMES(Storage::get__()) ;
-        iterator result ;
+        int result = 0;
         if( Rf_isNull(names) ){
             int i=0;
             for( ; it < first; ++it, ++target_it, i++ ){
                 *target_it = *it ;
             }
-            result = begin() + i + 1 ;
+            result = i;
             for( it = last ; it < this_end; ++it, ++target_it ){
                 *target_it = *it ;
             }
@@ -912,7 +912,7 @@ class Vector :
                 *target_it = *it ;
                 SET_STRING_ELT( newnames, i, STRING_ELT(names, i ) );
             }
-            result = begin() + i + 1 ;
+            result = i;
             for( it = last ; it < this_end; ++it, ++target_it, i++ ){
                 *target_it = *it ;
                 SET_STRING_ELT( newnames, i, STRING_ELT(names, i + nremoved ) );
@@ -921,7 +921,7 @@ class Vector :
         }
         Storage::set__( target.get__() ) ;
 
-        return result ;
+        return begin() + result;
 
     }