Re: [PATCH] clang-format: add configuration file

On Wed, Mar 14, 2018 at 1:18 AM, Miguel Ojeda wrote: > > Since we seem to agree on having this, I will send a v2 after I try a > few experiments to try to reduce more the produced diffs, explain > things better in Documentation/, add more comments in the config

Re: [PATCH] clang-format: add configuration file

On Wed, Mar 14, 2018 at 1:18 AM, Miguel Ojeda wrote: > > Since we seem to agree on having this, I will send a v2 after I try a > few experiments to try to reduce more the produced diffs, explain > things better in Documentation/, add more comments in the config file > and send a v2. Alright,

Re: [PATCH] clang-format: add configuration file

On Tue, Mar 13, 2018 at 11:29 PM, Joe Perches wrote: > On Tue, 2018-03-13 at 22:52 +0100, Miguel Ojeda wrote: >> On Tue, Mar 13, 2018 at 10:00 PM, Andrew Morton >> wrote: >> > On Tue, 13 Mar 2018 00:39:52 +0100 Miguel Ojeda >> >

Re: [PATCH] clang-format: add configuration file

On Tue, Mar 13, 2018 at 11:29 PM, Joe Perches wrote: > On Tue, 2018-03-13 at 22:52 +0100, Miguel Ojeda wrote: >> On Tue, Mar 13, 2018 at 10:00 PM, Andrew Morton >> wrote: >> > On Tue, 13 Mar 2018 00:39:52 +0100 Miguel Ojeda >> > wrote: >> > > --- a/Documentation/process/4.Coding.rst >> > >

Re: [PATCH] clang-format: add configuration file

On Tue, 2018-03-13 at 22:52 +0100, Miguel Ojeda wrote: > On Tue, Mar 13, 2018 at 10:00 PM, Andrew Morton > wrote: > > On Tue, 13 Mar 2018 00:39:52 +0100 Miguel Ojeda > > wrote: > > > --- a/Documentation/process/4.Coding.rst > > > +++

Re: [PATCH] clang-format: add configuration file

On Tue, 2018-03-13 at 22:52 +0100, Miguel Ojeda wrote: > On Tue, Mar 13, 2018 at 10:00 PM, Andrew Morton > wrote: > > On Tue, 13 Mar 2018 00:39:52 +0100 Miguel Ojeda > > wrote: > > > --- a/Documentation/process/4.Coding.rst > > > +++ b/Documentation/process/4.Coding.rst > > > @@ -58,6 +58,12

Re: [PATCH] clang-format: add configuration file

On Tue, Mar 13, 2018 at 10:00 PM, Andrew Morton wrote: > On Tue, 13 Mar 2018 00:39:52 +0100 Miguel Ojeda > wrote: > >> --- a/Documentation/process/4.Coding.rst >> +++ b/Documentation/process/4.Coding.rst >> @@ -58,6 +58,12 @@ can never

Re: [PATCH] clang-format: add configuration file

On Tue, Mar 13, 2018 at 10:00 PM, Andrew Morton wrote: > On Tue, 13 Mar 2018 00:39:52 +0100 Miguel Ojeda > wrote: > >> --- a/Documentation/process/4.Coding.rst >> +++ b/Documentation/process/4.Coding.rst >> @@ -58,6 +58,12 @@ can never be transgressed. If there is a good reason to >> go

Re: [PATCH] clang-format: add configuration file

On Tue, 13 Mar 2018 00:39:52 +0100 Miguel Ojeda wrote: > --- a/Documentation/process/4.Coding.rst > +++ b/Documentation/process/4.Coding.rst > @@ -58,6 +58,12 @@ can never be transgressed. If there is a good reason to > go against the > style (a line which

Re: [PATCH] clang-format: add configuration file

On Tue, 13 Mar 2018 00:39:52 +0100 Miguel Ojeda wrote: > --- a/Documentation/process/4.Coding.rst > +++ b/Documentation/process/4.Coding.rst > @@ -58,6 +58,12 @@ can never be transgressed. If there is a good reason to > go against the > style (a line which becomes far less readable if split

Re: [PATCH] clang-format: add configuration file

On Tue, 2018-03-13 at 00:39 +0100, Miguel Ojeda wrote: > clang-format is a tool to format C/C++/... code according to a set > of rules and heuristics. Like all tools, it is not perfect nor > covers every single case, but it is good enough to be helpful. Yes, thanks. It'd also be nice to remove

Re: [PATCH] clang-format: add configuration file

On Tue, 2018-03-13 at 00:39 +0100, Miguel Ojeda wrote: > clang-format is a tool to format C/C++/... code according to a set > of rules and heuristics. Like all tools, it is not perfect nor > covers every single case, but it is good enough to be helpful. Yes, thanks. It'd also be nice to remove