2023-09-28 05:07:36

by Joe Perches

[permalink] [raw]
Subject: Re: [PATCH v2 0/2] get_maintainer: add patch-only keyword matching

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?


2023-09-28 09:24:46

by Justin Stitt

[permalink] [raw]
Subject: Re: [PATCH v2 0/2] get_maintainer: add patch-only keyword matching

On Thu, Sep 28, 2023 at 2:01 PM Joe Perches <[email protected]> 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.


Attachments:
Screenshot 2023-09-28 2.24.19 PM.png (133.61 kB)