[PATCH 0/5] ext2: Merge string literals on adjacent lines

[PATCH 0/5] ext2: Merge string literals on adjacent lines

[Artemiy Volkov]

Join lines and merge strings in ext2 files for ease of grep-ability, one file per patch.

Artemiy Volkov (5):
  ext2: super.c: Merge string literals on adjacent lines
  ext2: balloc.c: Merge string literals on adjacent lines
  ext2: dir.c: Merge string literals on adjacent lines
  ext2: inode.c: Merge string literals on adjacent lines
  ext2: xip.c: Merge string literals on adjacent lines

 fs/ext2/balloc.c | 32 +++++++++++-------------------
 fs/ext2/dir.c    | 11 +++++------
 fs/ext2/ialloc.c |  6 ++----
 fs/ext2/super.c  | 60 +++++++++++++++++++++++---------------------------------
 fs/ext2/xip.c    |  3 +--
 5 files changed, 43 insertions(+), 69 deletions(-)

-- 
1.9.1

Re: [PATCH 0/5] ext2: Merge string literals on adjacent lines

[Dan Carpenter]

On Wed, Jul 23, 2014 at 03:58:32AM +1000, Artemiy Volkov wrote:
> Join lines and merge strings in ext2 files for ease of grep-ability, one file per patch.
> 

To be honest, I don't know if the ext2 devs really want to bother with
this...

regards,
dan carpenter

Re: [PATCH 0/5] ext2: Merge string literals on adjacent lines

[Artemiy Volkov]

On Wed, 23 Jul 2014, Dan Carpenter wrote:

> On Wed, Jul 23, 2014 at 03:58:32AM +1000, Artemiy Volkov wrote:
>> Join lines and merge strings in ext2 files for ease of grep-ability, one file per patch.
>>
>
> To be honest, I don't know if the ext2 devs really want to bother with
> this...
>
> regards,
> dan carpenter
>
>

OK, thanks for the info. Is there maybe a way to know in advance if maintainers
of a certain subsystem wouldn't accept this kind of janitorial patches?

Best regards,
Artemiy

Re: [PATCH 0/5] ext2: Merge string literals on adjacent lines

[Dan Carpenter]

On Wed, Jul 23, 2014 at 08:02:12PM +1000, Artemiy Volkov wrote:
> 
> 
> On Wed, 23 Jul 2014, Dan Carpenter wrote:
> 
> >On Wed, Jul 23, 2014 at 03:58:32AM +1000, Artemiy Volkov wrote:
> >>Join lines and merge strings in ext2 files for ease of grep-ability, one file per patch.
> >>
> >
> >To be honest, I don't know if the ext2 devs really want to bother with
> >this...
> >
> >regards,
> >dan carpenter
> >
> >
> 
> OK, thanks for the info. Is there maybe a way to know in advance if maintainers
> of a certain subsystem wouldn't accept this kind of janitorial patches?

This patch mostly feels like something to make checkpatch not complain
and not something with other value than that.  Those kinds of patches
are totally welcome in staging.

Outside of staging it should be something where a human says, "Yes.
This definitely makes the code more readable."

Bug fixes are accepted everywhere.

regards,
dan carpenter