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
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.
>
>
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
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
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
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
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
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(-)
8 matches
Mail list logo