<p dir="ltr"><br>
On Apr 7, 2014 6:43 PM, "Samuel Benzaquen" <<a href="mailto:sbenza@google.com">sbenza@google.com</a>> wrote:<br>
><br>
> On Mon, Apr 7, 2014 at 1:14 PM, Manuel Klimek <<a href="mailto:klimek@google.com">klimek@google.com</a>> wrote:<br>
>><br>
>> On Mon, Apr 7, 2014 at 6:29 PM, Samuel Benzaquen <<a href="mailto:sbenza@google.com">sbenza@google.com</a>> wrote:<br>
>>><br>
>>>   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>
>><br>
>><br>
>> Any reason not to use the same pattern in all tests (they get cargo culted a lot).<br>
><br>
><br>
> I am figuring out how FileCheck works as I go.<br>
> 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:<br>
>   // RUN: check_clang_tidy_fix.sh <check-name> %s %t %T<br>
> or something like that. Instead of copying the same N lines everywhere (and changing them slightly).</p>
<p dir="ltr">I think that's a great idea... Go ahead :-) </p>
<p dir="ltr">>  <br>
>><br>
>>  <br>
>>><br>
>>><br>
>>> Hi djasper, klimek-test,<br>
>>><br>
>>> <a href="http://reviews.llvm.org/D3294">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">http://reviews.llvm.org/D3294?vs=8382&id=8411#toc</a><br>
>>><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>
>><br>
>><br>
><br>
</p>