Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/10106 )
Change subject: Modify fix_includes.py for Kudu usage, add a wrapper ...................................................................... Patch Set 1: (1 comment) Ended up redoing this a bit for rev 2 http://gerrit.cloudera.org:8080/#/c/10106/1/build-support/iwyu/fix_includes.py File build-support/iwyu/fix_includes.py: http://gerrit.cloudera.org:8080/#/c/10106/1/build-support/iwyu/fix_includes.py@1220 PS1, Line 1220: # We also never insert a blank line between C and C++-style #includes, > Update this comment. Done -- To view, visit http://gerrit.cloudera.org:8080/10106 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I3c286271a39a0d825fb11e5610d8eb7e5b0729b9 Gerrit-Change-Number: 10106 Gerrit-PatchSet: 1 Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Comment-Date: Thu, 19 Apr 2018 02:14:53 +0000 Gerrit-HasComments: Yes
