Dan Burkert has posted comments on this change.

Change subject: [iwyu] kudu-specific mappings for std::tr1 entities
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/7998/1/README.adoc
File README.adoc:

Line 292: ==== Running include-what-you-use (IWYU) against all source files in 
project
> I don't think it is, but I didn't find a better place to put it.  I thought
I'm fine with keeping it in the awk script and linking to that.  This seems 
like something that we'll want to automate if it ends up being used a lot, at 
which point the directions would become significantly simpler.  A separate 
README is fine with me if you feel that's better, though.


-- 
To view, visit http://gerrit.cloudera.org:8080/7998
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia448dedcbb09ee23b30a922c7143df9036490d84
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-HasComments: Yes

Reply via email to