Skip to content

Commit 391d48e

Browse files
authored
Merge pull request #1059 from RcppCore/bugfix/more_shields
Two more shields
2 parents 9bf01e7 + 0f809dc commit 391d48e

File tree

4 files changed

+17
-9
lines changed

4 files changed

+17
-9
lines changed

ChangeLog

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,11 @@
1+
2020-03-24 Dirk Eddelbuettel <[email protected]>
2+
3+
* DESCRIPTION (Version, Date): Roll minor version
4+
* inst/include/Rcpp/config.h: Idem
5+
6+
* inst/include/Rcpp/Environment.h: Added two Shield<SEXP> wrappers
7+
around Rf_langX calls
8+
19
2020-03-23 Dirk Eddelbuettel <[email protected]>
210

311
* .travis.yml (script): Run coverage as parallel step

DESCRIPTION

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
Package: Rcpp
22
Title: Seamless R and C++ Integration
3-
Version: 1.0.4.4
4-
Date: 2020-03-22
3+
Version: 1.0.4.5
4+
Date: 2020-03-24
55
Author: Dirk Eddelbuettel, Romain Francois, JJ Allaire, Kevin Ushey, Qiang Kou,
66
Nathan Russell, Douglas Bates and John Chambers
77
Maintainer: Dirk Eddelbuettel <[email protected]>

inst/include/Rcpp/Environment.h

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
// Environment.h: Rcpp R/C++ interface class library -- access R environments
44
//
55
// Copyright (C) 2009 - 2013 Dirk Eddelbuettel and Romain Francois
6-
// Copyright (C) 2014 - 2019 Dirk Eddelbuettel, Romain Francois and Kevin Ushey
6+
// Copyright (C) 2014 - 2020 Dirk Eddelbuettel, Romain Francois and Kevin Ushey
77
//
88
// This file is part of Rcpp.
99
//
@@ -33,8 +33,8 @@ namespace Rcpp{
3333
if( Rf_isEnvironment(x) ) return x ;
3434
SEXP asEnvironmentSym = Rf_install("as.environment");
3535
try {
36-
Shield<SEXP> res(Rcpp_fast_eval(Rf_lang2(asEnvironmentSym, x), R_GlobalEnv));
37-
return res ;
36+
Shield<SEXP> call(Rf_lang2(asEnvironmentSym, x));
37+
return Rcpp_fast_eval(call, R_GlobalEnv);
3838
} catch( const eval_error& ex) {
3939
const char* fmt = "Cannot convert object to an environment: "
4040
"[type=%s; target=ENVSXP].";
@@ -393,10 +393,10 @@ namespace Rcpp{
393393
*/
394394
Environment_Impl new_child(bool hashed) const {
395395
SEXP newEnvSym = Rf_install("new.env");
396-
return Environment_Impl(Rcpp_fast_eval(Rf_lang3(newEnvSym, Rf_ScalarLogical(hashed), Storage::get__()), R_GlobalEnv));
396+
Shield<SEXP> call(Rf_lang3(newEnvSym, Rf_ScalarLogical(hashed), Storage::get__()));
397+
return Environment_Impl(Rcpp_fast_eval(call, R_GlobalEnv));
397398
}
398399

399-
400400
void update(SEXP){}
401401
};
402402

inst/include/Rcpp/config.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@
3030
#define RCPP_VERSION_STRING "1.0.4"
3131

3232
// the current source snapshot
33-
#define RCPP_DEV_VERSION RcppDevVersion(1,0,4,4)
34-
#define RCPP_DEV_VERSION_STRING "1.0.4.4"
33+
#define RCPP_DEV_VERSION RcppDevVersion(1,0,4,5)
34+
#define RCPP_DEV_VERSION_STRING "1.0.4.5"
3535

3636
#endif

0 commit comments

Comments
 (0)