<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Jul 22, 2014 at 2:19 PM,  <span dir="ltr"><<a href="mailto:jingham@apple.com" target="_blank">jingham@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
We don't have such a clang-format file.  If somebody can come up with one that will run over the extant source base and produce no changes, then I guess I'd entertain using it for new code.<br></blockquote><div><br>
</div><div>That is exactly what this patch is trying to do.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I'm not really sure I see the point.  The lldb coding conventions as far as braces and the like are pretty straight-forward, and obvious from the code around you.  There are slight variations from person to person, but I'd rather avoid periodic pointless reformatting checkins as folks run these tools over code.</blockquote>
<div><br></div><div>I am certainly not advocating doing that.  clang-format as I, and others, use it in Clang and LLVM development is generally used to format lines that I have changed or added as part of a patch.</div><div>
 </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">More importantly, there's somewhat of an art to laying out code so it looks clear and is easy to read.  These tools tend to make uglify such attempts, in my experience.<br>
</blockquote><div><br></div><div>clang-format has been, by all accounts that I know of, a smashing success in Clang and LLVM.  It is regularly used by many of the contributors and is often explicitly referenced when we see new code up for review that is not compliant with the coding standards.</div>
<div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Jim<br>
<div><div class="h5"><br>
<br>
> On Jul 22, 2014, at 2:01 PM, David Majnemer <<a href="mailto:david.majnemer@gmail.com">david.majnemer@gmail.com</a>> wrote:<br>
><br>
> Hi zturner, tfiala,<br>
><br>
> clang-format is a handy tool that formats code very intelligently.  I'd<br>
> like to use it with LLDB but it requires a .clang-format file to inform<br>
> it about LLDB-specific formatting rules.<br>
><br>
> More information on these rules are here:<br>
> <a href="http://clang.llvm.org/docs/ClangFormatStyleOptions.html" target="_blank">http://clang.llvm.org/docs/ClangFormatStyleOptions.html</a><br>
><br>
> <a href="http://reviews.llvm.org/D4630" target="_blank">http://reviews.llvm.org/D4630</a><br>
><br>
> Files:<br>
>  .clang-format<br>
><br>
> Index: .clang-format<br>
> ===================================================================<br>
> --- /dev/null<br>
> +++ .clang-format<br>
> @@ -0,0 +1,5 @@<br>
> +BasedOnStyle: LLVM<br>
> +IndentWidth:     4<br>
> +ColumnLimit:     120<br>
> +BreakBeforeBraces: Allman<br>
> +PenaltyReturnTypeOnItsOwnLine: 0<br>
</div></div>> <D4630.11784.patch>_______________________________________________<br>
> lldb-commits mailing list<br>
> <a href="mailto:lldb-commits@cs.uiuc.edu">lldb-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits</a><br>
<br>
</blockquote></div><br></div></div>