Alexey Serbin 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
> Is this going to be done commonly?  If not maybe it would be better just to
I don't think it is, but I didn't find a better place to put it.  I thought 
that keeping these notes in the awk script itself was not the best place.

Maybe, I should put that a dedicated README.doc under build-support/iwyu?


-- 
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