<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Dec 11, 2015 at 2:46 PM, 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"><div dir="ltr">No I'm not talking about whitespace at the end of a line, I'm saying LLVM's style (and what clang-format does) is this:<div><br></div><div>Foo::Foo()</div><div>   : member_1()</div><div>   , member_2()</div><div>   , member_3()</div><div><br></div><div>and what LLDB wants is this:</div><div><br></div><div>Foo::Foo() :</div><div>    member1(),</div><div>    member2(),</div><div>    member3()</div><div></div></div></blockquote><div><br></div><div>Oh I see.</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> <br></div></div></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div></div><div>Neither one have whitespace at the end of a line.</div></div><div class="HOEnZb"><div class="h5"><br><div class="gmail_quote"><div dir="ltr">On Fri, Dec 11, 2015 at 2:44 PM Todd Fiala <<a href="mailto:todd.fiala@gmail.com" target="_blank">todd.fiala@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">Okay.  Ending a line with arbitrary whitespace seems wrong as just about any colorizing editor or diff will flag, but I'll survive :-)</div><div class="gmail_extra"></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Dec 11, 2015 at 2:42 PM, 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"><div dir="ltr">Yes, but as I mentioned, two things are still unsupported due to limitations in clang-format.  They are return-type-on-new-line (only in declarations.  clang-format supports it for definitions) and the constructor initializer list comma at the end (clang-format puts it at the beginning).  <div><br></div><div>That said, the comma at the end of initializer list isn't documented on that page, and where we don't have a clearly documented rule, prefer the LLVM guidelines, so....<div><div><br><br><div class="gmail_quote"><div dir="ltr">On Fri, Dec 11, 2015 at 2:37 PM Todd Fiala <<a href="mailto:todd.fiala@gmail.com" target="_blank">todd.fiala@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">Okay, but does the format match the LLDB-modified format with some kind of configuration file?  We still need to match our guidelines here:<div><br></div><div><a href="http://lldb.llvm.org/lldb-coding-conventions.html" style="font-size:13px" target="_blank">http://lldb.llvm.org/lldb-coding-conventions.html</a><br></div><div><br></div><div>We can achieve that with a config file for it, right?  (Maybe already existing, maybe in the lldb source tree already?)</div></div><div class="gmail_extra"></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Dec 11, 2015 at 2:35 PM, 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"><div dir="ltr">With git you can already run "git clang-format".  You just need `git-clang-format` to be in your PATH (it's under llvm/tools/clang).  Not sure how to hook it into SVN</div><div><div><br><div class="gmail_quote"><div dir="ltr">On Fri, Dec 11, 2015 at 2:32 PM Eugene Zelenko <<a href="mailto:eugene.zelenko@gmail.com" target="_blank">eugene.zelenko@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">At least clang-format should be applied to all newly added files before commit.<br>
<br>
Eugene.<br>
<br>
On Fri, Dec 11, 2015 at 2:30 PM, Zachary Turner <<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>> wrote:<br>
> Back on the topic of clang-format, what would it take to make clang-format a<br>
> regular part of peoples' workflows?<br>
><br>
> On Fri, Dec 11, 2015 at 2:27 PM Todd Fiala <<a href="mailto:todd.fiala@gmail.com" target="_blank">todd.fiala@gmail.com</a>> wrote:<br>
>><br>
>> Yep - sorry.  I had been talking to Greg about this and misunderstood his<br>
>> comment on it. My mistake entirely.  Kate and I just talked and she pointed<br>
>> me to your document, Jim.<br>
>><br>
>> The description was:<br>
>> where we had a clearly adhered to standard, keep it.<br>
>> whee we didn't, we adopted LLVM.<br>
>><br>
>> Sorry for rehashing!<br>
>><br>
>> -Todd<br>
>><br>
>> On Fri, Dec 11, 2015 at 2:12 PM, Jim Ingham <<a href="mailto:jingham@apple.com" target="_blank">jingham@apple.com</a>> wrote:<br>
>>><br>
>>><br>
>>> On Dec 11, 2015, at 2:01 PM, Todd Fiala via lldb-commits<br>
>>> <<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a>> wrote:<br>
>>><br>
>>><br>
>>><br>
>>> On Fri, Dec 11, 2015 at 1:59 PM, Zachary Turner <<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>><br>
>>> wrote:<br>
>>>><br>
>>>> On Fri, Dec 11, 2015 at 1:55 PM Todd Fiala via lldb-commits<br>
>>>> <<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a>> wrote:<br>
>>>>><br>
>>>>> Hey Eugene and Greg,<br>
>>>>><br>
>>>>> I thought we were doing spaces before the open parens in places like<br>
>>>>> this:<br>
>>>>><br>
>>>>> -    BreakpointResolverSP resolver_sp(new BreakpointResolverFileLine<br>
>>>>> (NULL,<br>
>>>>> ...<br>
>>>>> +    BreakpointResolverSP resolver_sp(new<br>
>>>>> BreakpointResolverFileLine(nullptr,<br>
>>>>><br>
>>>>> (see the removal of the space after BreakpointResolverFileLine from the<br>
>>>>> clang-tidy settings I presume).<br>
>>>>><br>
>>>>> Did I misunderstand that?<br>
>>>><br>
>>>><br>
>>>> This was officially removed from the coding standard some months ago,<br>
>>><br>
>>><br>
>>> Okay.  Are we 100% in sync with LLVM coding standard guidelines?  If so I<br>
>>> can just look there to see what we're supposed to be doing.<br>
>>><br>
>>><br>
>>> No, the differences between the lldb and llvm coding standards are<br>
>>> documented in:<br>
>>><br>
>>> <a href="http://lldb.llvm.org/lldb-coding-conventions.html" rel="noreferrer" target="_blank">http://lldb.llvm.org/lldb-coding-conventions.html</a><br>
>>><br>
>>> Jim<br>
>>><br>
>>><br>
>>>><br>
>>>> but not everyone has adopted this unfortunately.  See r228860.  It pains<br>
>>>> me to no end that we differ from LLVM, because it leads to exactly these<br>
>>>> type of problems where people aren't sure what the exact set of rules are.<br>
>>><br>
>>><br>
>>><br>
>>><br>
>>> --<br>
>>> -Todd<br>
>>> _______________________________________________<br>
>>> lldb-commits mailing list<br>
>>> <a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a><br>
>>> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits</a><br>
>>><br>
>>><br>
>><br>
>><br>
>><br>
>> --<br>
>> -Todd<br>
</blockquote></div>
</div></div></blockquote></div><br><br clear="all"><div><br></div></div><div class="gmail_extra">-- <br><div><div dir="ltr">-Todd</div></div>
</div></blockquote></div></div></div></div></div>
</blockquote></div><br><br clear="all"><div><br></div></div><div class="gmail_extra">-- <br><div><div dir="ltr">-Todd</div></div>
</div></blockquote></div>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">-Todd</div></div>
</div></div>