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

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

Attachment: signature.asc
Description: PGP signature

_______________________________________________
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]