<p dir="ltr">LG</p>
<div class="gmail_quote">On 8 Apr 2014 16:49, "Samuel Benzaquen" <<a href="mailto:sbenza@google.com">sbenza@google.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
================<br>
Comment at: test/clang-tidy/check_clang_tidy_fix.sh:9<br>
@@ +8,3 @@<br>
+<br>
+grep -v CHECK ${INPUT_FILE} > ${TEMPORARY_FILE} \<br>
+  && clang-tidy ${TEMPORARY_FILE} -fix --checks=${CHECK_TO_RUN} \<br>
----------------<br>
Alexander Kornienko wrote:<br>
> I'd leave the grep as it was in the test, so that it filters out RUN: lines as well, and it only filters out comments:<br>
><br>
>   grep -Ev "// *[A-Z-]+:" ...<br>
><br>
> We'll have some checks for comments, so filtering out special comments can be important there.<br>
Done.<br>
<br>
================<br>
Comment at: test/clang-tidy/check_clang_tidy_output.sh:8<br>
@@ +7,3 @@<br>
+<br>
+clang-tidy --checks=${CHECK_TO_RUN} --disable-checks="" ${INPUT_FILE} \<br>
+  -- --std=c++11 | FileCheck ${INPUT_FILE}<br>
----------------<br>
Alexander Kornienko wrote:<br>
> nit: I'd format this still a bit differently (where's my sh-format? ;), four spaces on the continuation line, and each pipe component from a new line with 2 spaces indentation:<br>
><br>
>   clang-tidy --checks=${CHECK_TO_RUN} --disable-checks="" ${INPUT_FILE} \<br>
>       -- --std=c++11 \<br>
>     | FileCheck ${INPUT_FILE}<br>
Done.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D3294" target="_blank">http://reviews.llvm.org/D3294</a><br>
<br>
BRANCH<br>
  get_==<br>
<br>
ARCANIST PROJECT<br>
  clang-tools-extra<br>
<br>
<br>
</blockquote></div>