Hello Mahesh Reddy, Kudu Jenkins, Andrew Wong, Bankim Bhavsar,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/17035
to look at the new patch set (#3).
Change subject: [gutil] update map-util to benefit from C++17
......................................................................
[gutil] update map-util to benefit from C++17
This patch updates map-util to benefit from C++17 features such as
try_emplace() and insert_or_assign() for dictionary containers: see
[1], [2], [3], [4] for details. At least, with try_emplace() a new
element isn't ever constructed if the container already contains
an element with the specified key.
[1] https://en.cppreference.com/w/cpp/container/map/try_emplace
[2] https://en.cppreference.com/w/cpp/container/map/insert_or_assign
[3] https://en.cppreference.com/w/cpp/container/unordered_map/try_emplace
[4] https://en.cppreference.com/w/cpp/container/unordered_map/insert_or_assign
Change-Id: I248c03e604855434ea3a0891c311461a455b428e
---
M src/kudu/gutil/map-util.h
1 file changed, 35 insertions(+), 24 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/35/17035/3
--
To view, visit http://gerrit.cloudera.org:8080/17035
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I248c03e604855434ea3a0891c311461a455b428e
Gerrit-Change-Number: 17035
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Bankim Bhavsar <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy <[email protected]>
