<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Apr 7, 2014 at 1:14 PM, Manuel Klimek <span dir="ltr"><<a href="mailto:klimek@google.com" target="_blank" class="cremed">klimek@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div class="">On Mon, Apr 7, 2014 at 6:29 PM, Samuel Benzaquen <span dir="ltr"><<a href="mailto:sbenza@google.com" target="_blank" class="cremed">sbenza@google.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">  Fixed the typo on shared_ptr and added a shared_ptr to the test.<br>
<br>
  Fixed the -fix.cpp test to actually check the output.<br>
  FileCheck can match the CHECK: lines with themselves if you don't remove them from the input.<br></blockquote><div><br></div></div><div>Any reason not to use the same pattern in all tests (they get cargo culted a lot).</div>
</div></div></div></blockquote><div><br></div><div>I am figuring out how FileCheck works as I go.</div><div>I have been also wondering why we don't have all these as scripts. imo, the test should have a single line at the top:</div>
<div>  // RUN: check_clang_tidy_fix.sh <check-name> %s %t %T</div><div>or something like that. Instead of copying the same N lines everywhere (and changing them slightly).</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div class="">
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Hi djasper, klimek-test,<br>
<br>
<a href="http://reviews.llvm.org/D3294" target="_blank" class="cremed">http://reviews.llvm.org/D3294</a><br>
<br>
CHANGE SINCE LAST DIFF<br>
  <a href="http://reviews.llvm.org/D3294?vs=8382&id=8411#toc" target="_blank" class="cremed">http://reviews.llvm.org/D3294?vs=8382&id=8411#toc</a><br>
<div><div><br>
Files:<br>
  clang-tidy/misc/RedundantSmartptrGet.cpp<br>
  test/clang-tidy/make_compile_commands_json.sh<br>
  test/clang-tidy/redundant-smartptr-get-fix.cpp<br>
  test/clang-tidy/redundant-smartptr-get.cpp<br>
</div></div></blockquote></div></div><br></div></div>
</blockquote></div><br></div></div>