Re: [PATCH v3 01/11] Add .clang-format with defaults matching what's specified in the style guide

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

 




> On 14 Feb 2018, at 17:34, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote:
> 
> On Wed, Feb 14, 2018 at 10:45:56AM -0500, Frediano Ziglio wrote:
>>> 
>>> Shouldn't this go with a Makefile rule? A few lines in the log what this
>>> is about, what is the goal for having this file, ... would not hurt.
>>> 
>>> Christophe
>>> 
>> 
>> I think this file is supposed to just help developers so should not
>> be in the Makefile.
> 
> Yes, after reading various threads, it's apparently meant to be used
> together with emacs for formatting of small code blocks, it's not usable
> on the whole codebase. So a 'make clang-format' rule apparently would
> not make sense.
> 
>> I think you mean that the intention should be written in the commit message.
> 
> Yes, knowing how it's meant to be used, why we want it in the codebase.

Why would we NOT want it in the codebase?

> 
> Christophe
> 
>> 
>>> On Thu, Feb 08, 2018 at 12:25:21PM +0100, Christophe de Dinechin wrote:
>>>> From: Christophe de Dinechin <dinechin@xxxxxxxxxx>
>>>> 
>>>> Signed-off-by: Christophe de Dinechin <dinechin@xxxxxxxxxx>
>>>> ---
>>>> .clang-format | 23 +++++++++++++++++++++++
>>>> 1 file changed, 23 insertions(+)
>>>> create mode 100644 .clang-format
>>>> 
>>>> diff --git a/.clang-format b/.clang-format
>>>> new file mode 100644
>>>> index 00000000..91203600
>>>> --- /dev/null
>>>> +++ b/.clang-format
>>>> @@ -0,0 +1,23 @@
>>>> +Language:        Cpp
>>>> +# BasedOnStyle:  LLVM
>>>> +
>>>> +# The following is commented out until widely supported
>>>> +# IncludeBlocks: Regroup
>>>> +SortIncludes: true
>>>> +
>>>> +IncludeCategories:
>>>> +  - Regex:           'config.h'
>>>> +    Priority:        -1
>>>> +  - Regex:           '^"spice.*"'
>>>> +    Priority:        1
>>>> +  - Regex:           'glib'
>>>> +    Priority:        4
>>>> +  - Regex:           '^<.*>'
>>>> +    Priority:        3
>>>> +  - Regex:           '^".*"'
>>>> +    Priority:        2
>>>> +
>>>> +ColumnLimit:     100
>>>> +IndentCaseLabels: false
>>>> +IndentWidth:     4
>>>> +BreakBeforeBraces: Linux
>> 
>> Frediano
>> _______________________________________________
>> Spice-devel mailing list
>> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
>> https://lists.freedesktop.org/mailman/listinfo/spice-devel
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/spice-devel

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]