Skip to content

Commit 30df687

Browse files
Enchufa2eddelbuettel
Enchufa2
and
eddelbuettel
authored
Remove macros for unordered maps/sets (#1372)
* remove macros for unordered maps/sets * do not deprecate yet * update Changelog * Roll micro version and date Used by #1372 --------- Co-authored-by: Dirk Eddelbuettel <[email protected]>
1 parent d654e80 commit 30df687

File tree

9 files changed

+43
-27
lines changed

9 files changed

+43
-27
lines changed

‎ChangeLog

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,19 @@
1+
2025-03-26 Dirk Eddelbuettel <[email protected]>
2+
3+
* DESCRIPTION (Version, Date): Roll micro version and date
4+
* inst/include/Rcpp/config.h: Idem
5+
6+
2025-03-26 Iñaki Ucar <[email protected]>
7+
8+
* inst/include/Rcpp/platform/compiler.h: Remove definitions of
9+
RCPP_USING_UNORDERED_MAP and RCPP_USING_UNORDERED_SET
10+
* inst/include/Rcpp/sugar/sets.h: Remove redefinition of macros
11+
* inst/include/Rcpp/sugar/functions/self_match.h: Replace macros with
12+
std::unordered_map and std::unordered_set respectively
13+
* inst/include/Rcpp/sugar/functions/setdiff.h: Idem
14+
* inst/include/Rcpp/sugar/functions/table.h: Idem
15+
* inst/tinytest/cpp/wrap.cpp: Idem
16+
117
2025-03-24 Dirk Eddelbuettel <[email protected]>
218

319
* DESCRIPTION (Version, Date): Roll micro version and date

‎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.14.10
4-
Date: 2025-03-24
3+
Version: 1.0.14.11
4+
Date: 2025-03-26
55
Authors@R: c(person("Dirk", "Eddelbuettel", role = c("aut", "cre"), email = "[email protected]",
66
comment = c(ORCID = "0000-0001-6419-907X")),
77
person("Romain", "Francois", role = "aut",

‎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.14"
3131

3232
// the current source snapshot (using four components, if a fifth is used in DESCRIPTION we ignore it)
33-
#define RCPP_DEV_VERSION RcppDevVersion(1,0,14,10)
34-
#define RCPP_DEV_VERSION_STRING "1.0.14.10"
33+
#define RCPP_DEV_VERSION RcppDevVersion(1,0,14,11)
34+
#define RCPP_DEV_VERSION_STRING "1.0.14.11"
3535

3636
#endif

‎inst/include/Rcpp/platform/compiler.h

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
// compiler.h: Rcpp R/C++ interface class library -- check compiler
22
//
33
// Copyright (C) 2012 - 2025 Dirk Eddelbuettel, Romain Francois, and Kevin Ushey
4+
// Copyright (C) 2025 Dirk Eddelbuettel, Romain Francois, Kevin Ushey and Iñaki Ucar
45
//
56
// This file is part of Rcpp.
67
//
@@ -36,11 +37,9 @@
3637
#include <cmath>
3738
#include <initializer_list>
3839
#include <unordered_map>
39-
#define RCPP_USING_UNORDERED_MAP
40-
#define RCPP_UNORDERED_MAP std::unordered_map
40+
#define RCPP_UNORDERED_MAP std::unordered_map // TODO deprecate
4141
#include <unordered_set>
42-
#define RCPP_USING_UNORDERED_SET
43-
#define RCPP_UNORDERED_SET std::unordered_set
42+
#define RCPP_UNORDERED_SET std::unordered_set // TODO deprecate
4443

4544
#if defined(__GNUC__)
4645
#define RCPP_HAS_DEMANGLING

‎inst/include/Rcpp/sugar/functions/self_match.h

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,8 @@
22
//
33
// self_match.h: Rcpp R/C++ interface class library -- self match
44
//
5-
// Copyright (C) 2012 Dirk Eddelbuettel and Romain Francois
5+
// Copyright (C) 2012 - 2024 Dirk Eddelbuettel and Romain Francois
6+
// Copyright (C) 2025 Dirk Eddelbuettel, Romain Francois and Iñaki Ucar
67
//
78
// This file is part of Rcpp.
89
//
@@ -57,7 +58,7 @@ class SelfMatch {
5758
inline operator IntegerVector() const { return result ; }
5859

5960
private:
60-
typedef RCPP_UNORDERED_MAP<STORAGE, int> HASH ;
61+
typedef std::unordered_map<STORAGE, int> HASH ;
6162
typedef SelfInserter<HASH,STORAGE> Inserter ;
6263
HASH hash ;
6364
IntegerVector result ;

‎inst/include/Rcpp/sugar/functions/setdiff.h

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,8 @@
22
//
33
// setdiff.h: Rcpp R/C++ interface class library -- setdiff
44
//
5-
// Copyright (C) 2012 - 2014 Dirk Eddelbuettel and Romain Francois
5+
// Copyright (C) 2012 - 2024 Dirk Eddelbuettel and Romain Francois
6+
// Copyright (C) 2025 Dirk Eddelbuettel, Romain Francois and Iñaki Ucar
67
//
78
// This file is part of Rcpp.
89
//
@@ -60,7 +61,7 @@ namespace sugar{
6061
}
6162

6263
private:
63-
typedef RCPP_UNORDERED_SET<STORAGE> SET ;
64+
typedef std::unordered_set<STORAGE> SET ;
6465
typedef typename SET::const_iterator ITERATOR ;
6566
SET lhs_set ;
6667
SET rhs_set ;
@@ -89,7 +90,7 @@ namespace sugar{
8990
}
9091

9192
private:
92-
typedef RCPP_UNORDERED_SET<STORAGE> SET ;
93+
typedef std::unordered_set<STORAGE> SET ;
9394
typedef typename SET::const_iterator ITERATOR ;
9495
SET lhs_set ;
9596
SET rhs_set ;
@@ -123,7 +124,7 @@ namespace sugar{
123124
}
124125

125126
private:
126-
typedef RCPP_UNORDERED_SET<STORAGE> SET ;
127+
typedef std::unordered_set<STORAGE> SET ;
127128
typedef typename SET::const_iterator ITERATOR ;
128129
SET intersect ;
129130

@@ -148,7 +149,7 @@ namespace sugar{
148149
}
149150

150151
private:
151-
typedef RCPP_UNORDERED_SET<STORAGE> SET ;
152+
typedef std::unordered_set<STORAGE> SET ;
152153
typedef typename SET::const_iterator ITERATOR ;
153154
SET result ;
154155

‎inst/include/Rcpp/sugar/functions/table.h

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
// table.h: Rcpp R/C++ interface class library -- table match
22
//
33
// Copyright (C) 2012 - 2025 Dirk Eddelbuettel, Romain Francois, and Kevin Ushey
4+
// Copyright (C) 2025 Dirk Eddelbuettel, Romain Francois, Kevin Ushey and Iñaki Ucar
45
//
56
// This file is part of Rcpp.
67
//
@@ -77,7 +78,7 @@ class Table {
7778
}
7879

7980
private:
80-
typedef RCPP_UNORDERED_MAP<STORAGE, int> HASH ;
81+
typedef std::unordered_map<STORAGE, int> HASH ;
8182
typedef CountInserter<HASH,STORAGE> Inserter ;
8283
HASH hash ;
8384

‎inst/include/Rcpp/sugar/sets.h

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,9 +20,6 @@
2020
#ifndef Rcpp__sugar__sets_h
2121
#define Rcpp__sugar__sets_h
2222

23-
#define RCPP_UNORDERED_SET std::unordered_set
24-
#define RCPP_UNORDERED_MAP std::unordered_map
25-
2623
namespace std {
2724
template<>
2825
struct hash<Rcpp::String> {

‎inst/tinytest/cpp/wrap.cpp

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
// wrap.cpp: Rcpp R/C++ interface class library -- wrap unit tests
44
//
55
// Copyright (C) 2013 - 2025 Dirk Eddelbuettel and Romain Francois
6+
// Copyright (C) 2025 Dirk Eddelbuettel, Romain Francois and Iñaki Ucar
67
//
78
// This file is part of Rcpp.
89
//
@@ -170,7 +171,7 @@ SEXP nonnull_const_char(){
170171

171172
// [[Rcpp::export]]
172173
IntegerVector unordered_map_string_int(){
173-
RCPP_UNORDERED_MAP< std::string, int > m ;
174+
std::unordered_map< std::string, int > m ;
174175
m["b"] = 100;
175176
m["a"] = 200;
176177
m["c"] = 300;
@@ -179,7 +180,7 @@ IntegerVector unordered_map_string_int(){
179180

180181
// [[Rcpp::export]]
181182
IntegerVector unordered_map_rcpp_string_int(StringVector v){
182-
RCPP_UNORDERED_MAP< String, int > m ;
183+
std::unordered_map< String, int > m ;
183184
m[v[0]] = 200;
184185
m[v[1]] = 100;
185186
m[v[2]] = 300;
@@ -188,7 +189,7 @@ IntegerVector unordered_map_rcpp_string_int(StringVector v){
188189

189190
// [[Rcpp::export]]
190191
LogicalVector unordered_set_rcpp_string(StringVector x) {
191-
RCPP_UNORDERED_SET<String> seen;
192+
std::unordered_set<String> seen;
192193
LogicalVector out(x.size());
193194
for (int i = 0; i < x.size(); i++) {
194195
out[i] = !seen.insert(x[i]).second;
@@ -198,7 +199,7 @@ LogicalVector unordered_set_rcpp_string(StringVector x) {
198199

199200
// [[Rcpp::export]]
200201
NumericVector unordered_map_string_double(){
201-
RCPP_UNORDERED_MAP<std::string,double> m ;
202+
std::unordered_map<std::string,double> m ;
202203
m["b"] = 100;
203204
m["a"] = 200;
204205
m["c"] = 300;
@@ -207,7 +208,7 @@ NumericVector unordered_map_string_double(){
207208

208209
// [[Rcpp::export]]
209210
LogicalVector unordered_map_string_bool(){
210-
RCPP_UNORDERED_MAP<std::string,bool> m ;
211+
std::unordered_map<std::string,bool> m ;
211212
m["b"] = true;
212213
m["a"] = false;
213214
m["c"] = true;
@@ -216,7 +217,7 @@ LogicalVector unordered_map_string_bool(){
216217

217218
// [[Rcpp::export]]
218219
RawVector unordered_map_string_Rbyte(){
219-
RCPP_UNORDERED_MAP<std::string,Rbyte> m ;
220+
std::unordered_map<std::string,Rbyte> m ;
220221
m["b"] = (Rbyte)0;
221222
m["a"] = (Rbyte)1;
222223
m["c"] = (Rbyte)2;
@@ -225,7 +226,7 @@ RawVector unordered_map_string_Rbyte(){
225226

226227
// [[Rcpp::export]]
227228
CharacterVector unordered_map_string_string(){
228-
RCPP_UNORDERED_MAP<std::string,std::string> m ;
229+
std::unordered_map<std::string,std::string> m ;
229230
m["b"] = "foo" ;
230231
m["a"] = "bar" ;
231232
m["c"] = "bling" ;
@@ -234,7 +235,7 @@ CharacterVector unordered_map_string_string(){
234235

235236
// [[Rcpp::export]]
236237
List unordered_map_string_generic(){
237-
RCPP_UNORDERED_MAP< std::string,std::vector<int> > m ;
238+
std::unordered_map< std::string,std::vector<int> > m ;
238239
std::vector<int> b; b.push_back(1); b.push_back(2); m["b"] = b ;
239240
std::vector<int> a; a.push_back(1); a.push_back(2); a.push_back(2); m["a"] = a;
240241
std::vector<int> c; c.push_back(1); c.push_back(2); c.push_back(2); c.push_back(2); m["c"] = c;

0 commit comments

Comments
 (0)