Re: [PATCH] scripts/Lindent: increase the maximum line length to 100

2020-07-03 Thread Joe Perches
On Fri, 2020-07-03 at 18:22 +0800, Zong Li wrote: > On Fri, Jul 3, 2020 at 5:51 PM Joe Perches wrote: > > On Fri, 2020-07-03 at 11:41 +0200, Miguel Ojeda wrote: > > > On Fri, Jul 3, 2020 at 10:51 AM Joe Perches wrote: > > > > I'd prefer to delete Lindent instead. > > > > > > +1, especially since

Re: [PATCH] scripts/Lindent: increase the maximum line length to 100

2020-07-03 Thread Zong Li
On Fri, Jul 3, 2020 at 5:51 PM Joe Perches wrote: > > On Fri, 2020-07-03 at 11:41 +0200, Miguel Ojeda wrote: > > On Fri, Jul 3, 2020 at 10:51 AM Joe Perches wrote: > > > I'd prefer to delete Lindent instead. > > > > +1, especially since there is `clang-format` now. Agree, it is often used. > >

Re: [PATCH] scripts/Lindent: increase the maximum line length to 100

2020-07-03 Thread Joe Perches
On Fri, 2020-07-03 at 11:41 +0200, Miguel Ojeda wrote: > On Fri, Jul 3, 2020 at 10:51 AM Joe Perches wrote: > > I'd prefer to delete Lindent instead. > > +1, especially since there is `clang-format` now. Awhile back I did send a patch: https://lore.kernel.org/lkml/1360610974.28491.6.camel@joe-AO

Re: [PATCH] scripts/Lindent: increase the maximum line length to 100

2020-07-03 Thread Miguel Ojeda
On Fri, Jul 3, 2020 at 10:51 AM Joe Perches wrote: > > I'd prefer to delete Lindent instead. +1, especially since there is `clang-format` now. Cheers, Miguel

Re: [PATCH] scripts/Lindent: increase the maximum line length to 100

2020-07-03 Thread Miguel Ojeda
On Fri, Jul 3, 2020 at 11:19 AM Zong Li wrote: > > Yes, as you mentioned, the reformatting tool uses up to the maximum line > length, > it seems to me that we don't go this patch if you plan to post the > patch to delete it, > otherwise, we could consider to change it to 100. For reference, we d

Re: [PATCH] scripts/Lindent: increase the maximum line length to 100

2020-07-03 Thread Zong Li
On Fri, Jul 3, 2020 at 4:51 PM Joe Perches wrote: > > On Fri, 2020-07-03 at 16:08 +0800, Zong Li wrote: > > As the patch 'bdc48fa11e46 ("checkpatch/coding-style: deprecate > > 80-column warning")', increase the default limit to 100 characters, > > we also increase the maximum line length to 100 fo

Re: [PATCH] scripts/Lindent: increase the maximum line length to 100

2020-07-03 Thread Joe Perches
On Fri, 2020-07-03 at 16:08 +0800, Zong Li wrote: > As the patch 'bdc48fa11e46 ("checkpatch/coding-style: deprecate > 80-column warning")', increase the default limit to 100 characters, > we also increase the maximum line length to 100 for indent script. I'd prefer to delete Lindent instead. Also

[PATCH] scripts/Lindent: increase the maximum line length to 100

2020-07-03 Thread Zong Li
As the patch 'bdc48fa11e46 ("checkpatch/coding-style: deprecate 80-column warning")', increase the default limit to 100 characters, we also increase the maximum line length to 100 for indent script. Signed-off-by: Zong Li --- scripts/Lindent | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)