lilypond-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Add a tentative .clang-format for LilyPond. (issue 561340043 by addr


From: hanwenn
Subject: Re: Add a tentative .clang-format for LilyPond. (issue 561340043 by address@hidden)
Date: Tue, 04 Feb 2020 14:18:23 -0800

On 2020/02/04 20:35:40, Dan Eble wrote:
> I'm running some of my patches through clang-format as I prepare to
push them.
> 
> This is an example of a kind of change it wants to make:
> 
> -  const array<int, 2> key {column_rank, dir};
> +  const array<int, 2> key{column_rank, dir};
> 
> Note the space after key.  The setting that controls this is
> SpaceBeforeCpp11BracedList.  Am I correct that we should use a space
here for
> consistency with historical style?
> 
> More examples of the effect of the option are in the docs:
>
https://releases.llvm.org/9.0.0/tools/clang/docs/ClangFormatStyleOptions.html

historically, we didn't have C++11, so this wasn't an issue. I'm fine
with whatever you pick.

With the space seems more coherent with the s p a c e y  GNU style. 

https://codereview.appspot.com/561340043/



reply via email to

[Prev in Thread] Current Thread [Next in Thread]