<div dir="ltr">But what about SVN users? :)<div>Okay, I know format can be applied for modified fragments only. But why did we leave some places unformatted? I thought "clang-format -i `find include source tools -name '*.[cpp|h]'`" ends with empty diff. </div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 8, 2016 at 10:19 AM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">In order to clang-format only the diff, you should use the "git clang-format" extension command.  Search for git-clang-format (yes, that one has an extra dash), and there should be instructions inside <br><div class="HOEnZb"><div class="h5"><div class="gmail_quote"><div dir="ltr">On Thu, Sep 8, 2016 at 12:16 AM Ilia K <<a href="mailto:ki.stfu@gmail.com" target="_blank">ki.stfu@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi!<div><br></div><div>clang-formatting of my patch changed the style outside the patch (diff: <a href="https://reviews.llvm.org/D24331?vs=70653&id=70654" target="_blank">https://reviews.llvm.org/<wbr>D24331?vs=70653&id=70654</a>). Am I doing something wrong?<br></div></div><div class="gmail_extra"></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 7, 2016 at 1:06 PM, Pavel Labath via lldb-dev <span dir="ltr"><<a href="mailto:lldb-dev@lists.llvm.org" target="_blank">lldb-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Windows unit tests passing now. All clear.<br>
<div><div><br>
On 7 September 2016 at 09:54, Pavel Labath <<a href="mailto:labath@google.com" target="_blank">labath@google.com</a>> wrote:<br>
> I am still seeing errors when building windows unittests. I'll have a<br>
> change for fixing that soon. Apart from that everything else looks<br>
> good on our side. I'll send an all clear once the unit tests get green<br>
> on windows.<br>
><br>
> pl<br>
><br>
> On 6 September 2016 at 23:26, Zachary Turner via lldb-dev<br>
> <<a href="mailto:lldb-dev@lists.llvm.org" target="_blank">lldb-dev@lists.llvm.org</a>> wrote:<br>
>> I think Windows is good.<br>
>><br>
>> On Tue, Sep 6, 2016 at 3:10 PM Zachary Turner <<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>> wrote:<br>
>>><br>
>>> Everything compiles on Windows now but all the tests are failing with<br>
>>> ERROR.  I'm looking into this now.<br>
>>><br>
>>> On Tue, Sep 6, 2016 at 2:26 PM Ed Maste via lldb-dev<br>
>>> <<a href="mailto:lldb-dev@lists.llvm.org" target="_blank">lldb-dev@lists.llvm.org</a>> wrote:<br>
>>>><br>
>>>> On 6 September 2016 at 17:17, Kate Stone via lldb-dev<br>
>>>> <<a href="mailto:lldb-dev@lists.llvm.org" target="_blank">lldb-dev@lists.llvm.org</a>> wrote:<br>
>>>> > The storm of commit messages might be a subtle clue, but here it is<br>
>>>> > officially: the reformatting is complete and I’ve verified that no<br>
>>>> > tests<br>
>>>> > regressed locally in our macOS suite.  Please begin any validation<br>
>>>> > process<br>
>>>> > you’ve signed up for on another platform, and if changes are necessary<br>
>>>> > go<br>
>>>> > ahead and land them individually.<br>
>>>><br>
>>>> FreeBSD currently fails to build due to header reordering in<br>
>>>> source/Host/freebsd/Host.cpp which I'll sort out shortly.<br>
>>>><br>
>>>> I'd like to request that we avoid any functional changes other than<br>
>>>> those restoring builds to green, until we get the "all clear" from<br>
>>>> everyone who's signed up to validate other platforms.<br>
>>>><br>
>>>> -Ed<br>
>>>> ______________________________<wbr>_________________<br>
>>>> lldb-dev mailing list<br>
>>>> <a href="mailto:lldb-dev@lists.llvm.org" target="_blank">lldb-dev@lists.llvm.org</a><br>
>>>> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/lldb-dev</a><br>
>><br>
>><br>
>> ______________________________<wbr>_________________<br>
>> lldb-dev mailing list<br>
>> <a href="mailto:lldb-dev@lists.llvm.org" target="_blank">lldb-dev@lists.llvm.org</a><br>
>> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/lldb-dev</a><br>
>><br>
______________________________<wbr>_________________<br>
lldb-dev mailing list<br>
<a href="mailto:lldb-dev@lists.llvm.org" target="_blank">lldb-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/lldb-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div></div><div class="gmail_extra">-- <br><div data-smartmail="gmail_signature"><div dir="ltr">- Ilia</div></div>
</div></blockquote></div>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">- Ilia</div></div>
</div>