Adar Dembo has posted comments on this change. Change subject: Replace BOOST_FOREACH with c++11 range syntax ......................................................................
Patch Set 1: Code-Review+2 Spot checked some of the calls. There's no danger of an inserted or removed ampersand (i.e. iterating by copy instead of by ref, or vice versa), is there? Anyway, I'm happy if there are no longer any inclusions of foreach.hpp and no usages of BOOST_FOREACH. -- To view, visit http://gerrit.cloudera.org:8080/1786 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I635ba6e83b65eec4427c10175c203c4864b2a22f Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Dan Burkert <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: No
