lkml.org 
[lkml]   [2023]   [Sep]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH v2 0/2] get_maintainer: add patch-only keyword matching
On Thu, Sep 28, 2023 at 2:01 PM Joe Perches <joe@perches.com> wrote:
>
> On Thu, 2023-09-28 at 04:23 +0000, Justin Stitt wrote:
> > Changes in v2:
> > - remove formatting pass (thanks Joe) (but seriously the formatting is
> > bad, is there opportunity to get a formatting pass in here at some
> > point?)
>
> Why? What is it that makes you believe the formatting is bad?
>

Investigating further, it looked especially bad in my editor. There is
a mixture of
tabs and spaces and my vim tabstop is set to 4 for pl files. Setting this to
8 is a whole lot better. But I still see some weird spacing.

Some lines start with a bunch of spaces, other lines start with a tab character.
This interleaving of tabs and spaces to start lines makes some editor configs
render it weirdly.

Notice how any .c file in the kernel only starts with tabs and thus is rendered
consistently.

ctrl+f for "\t" and see that some lines are tabbed and others are not.
I attached a screenshot to this email showing this.

However, now that I understand the issue more clearly this is definitely
not as big an issue as I thought.
[unhandled content-type:image/png]
\
 
 \ /
  Last update: 2023-09-28 07:32    [W:0.079 / U:1.936 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site