[Lldb-commits] [PATCH] Add a .clang-format file to enhance formatting experience with clang-format

jingham at apple.com jingham at apple.com
Tue Jul 22 14:19:40 PDT 2014


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.  

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.  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.

Jim


> On Jul 22, 2014, at 2:01 PM, David Majnemer <david.majnemer at gmail.com> wrote:
> 
> Hi zturner, tfiala,
> 
> clang-format is a handy tool that formats code very intelligently.  I'd
> like to use it with LLDB but it requires a .clang-format file to inform
> it about LLDB-specific formatting rules.
> 
> More information on these rules are here:
> http://clang.llvm.org/docs/ClangFormatStyleOptions.html
> 
> http://reviews.llvm.org/D4630
> 
> Files:
>  .clang-format
> 
> Index: .clang-format
> ===================================================================
> --- /dev/null
> +++ .clang-format
> @@ -0,0 +1,5 @@
> +BasedOnStyle: LLVM
> +IndentWidth:     4
> +ColumnLimit:     120
> +BreakBeforeBraces: Allman
> +PenaltyReturnTypeOnItsOwnLine: 0
> <D4630.11784.patch>_______________________________________________
> lldb-commits mailing list
> lldb-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits




More information about the lldb-commits mailing list