<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Apr 30, 2014 at 11:45 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Wed, Apr 30, 2014 at 10:17 AM, Manuel Klimek <<a href="mailto:klimek@google.com">klimek@google.com</a>> wrote:<br>

> On Wed, Apr 30, 2014 at 6:03 PM, Alex McCarthy <<a href="mailto:alexmc@google.com">alexmc@google.com</a>> wrote:<br>
>><br>
>> Thanks, I'll do that next time. Should I use that instead of git svn<br>
>> dcommit to push the change?<br>
><br>
><br>
> I usually just commit with git svn dcommit (I find the commit message from<br>
> arc commit a bit too wordy for my taste).<br>
<br>
</div>Could we get that fixed?<br>
<br>
Just taking Alexey's recent commit:<br>
<br>
"Summary: No functionality change.<br>
<br>
Test Plan: llvm regression test suite.<br>
<br>
Reviewers: dblaikie<br>
<br>
Reviewed By: dblaikie<br>
<br>
Subscribers: echristo, llvm-commits"<br>
<br>
All seems fairly unnecessary - except maybe the "Reviewed By" field.<br>
(it'd be nice to phrase that as "Approved By" to make it clear that<br>
there was actual sign off)<br></blockquote><div><br></div><div>Yes, I'd love to; Unfortunately, priorities; currently my 20% time is filled up fixing the static analyzer for C++ code. If you want to get your hands dirty with a few lines of PHP (or know somebody who does), I'm happy to point them at the right repository ...</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im HOEnZb"><br>
><br>
>><br>
>> -Alex<br>
>><br>
>><br>
>> On Wed, Apr 30, 2014 at 7:58 AM, Alexander Kornienko <<a href="mailto:alexfh@google.com">alexfh@google.com</a>><br>
>> wrote:<br>
>>><br>
>>> On Wed, Apr 30, 2014 at 4:26 PM, Alex McCarthy <<a href="mailto:alexmc@google.com">alexmc@google.com</a>> wrote:<br>
>>>><br>
>>>> ================<br>
>>>> Comment at: test/clang-tidy/temporaries.cpp:3<br>
>>>> @@ +2,3 @@<br>
>>>> +// FileCheck complains if the input file is empty, so add a dummy line.<br>
>>>> +// RUN: echo foo >> %t.log<br>
>>>> +// RUN: FileCheck %s < %t.log<br>
>>>> ----------------<br>
>>>> Alexander Kornienko wrote:<br>
>>>> > You don't need this now as the correct output is guaranteed to be<br>
>>>> > non-empty. And you can now just pipe clang-tidy output to FileCheck<br>
>>>> Good catch: fixed in <a href="http://llvm-reviews.chandlerc.com/rL207653" target="_blank">llvm-reviews.chandlerc.com/rL207653</a><br>
>>>><br>
>>>> <a href="http://reviews.llvm.org/D3556" target="_blank">http://reviews.llvm.org/D3556</a><br>
>>>><br>
>>><br>
>>> BTW, if you use Phabricator's arc tool, you can also use "arc commit",<br>
>>> which automatically closes the Differential revision in Phab.<br>
>><br>
>><br>
><br>
><br>
</div><div class="HOEnZb"><div class="h5">> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
><br>
</div></div></blockquote></div><br></div></div>