Messages in this thread | | | Date | Thu, 4 Jun 2015 14:29:03 +0200 | From | Petr Mladek <> | Subject | Re: [PATCH 1/3] checkpatch: Improve output with multiple command-line files |
| |
On Thu 2015-06-04 05:14:39, Joe Perches wrote: > On Thu, 2015-06-04 at 14:03 +0200, Petr Mladek wrote: > > On Wed 2015-06-03 08:53:38, Joe Perches wrote: > > > If there are multiple patches/files on the command line, > > > use a prefix before the patch/file message output like: > > > -------------- > > > patch/filename > > > -------------- > > > to make the identifying which messages go with which > > > file/patch a bit easier to parse. > [] > > Tested-by: Petr Mladek <pmladek@suse.cz> > [] > > > +NOTE: Whitespace errors detected. > > > + You may wish to use scripts/cleanpatch or scripts/cleanfile > > > +EOM > > > > It would make sense to write this message only once as well. > > I don't think so as it applies to each patch/file separately.
IMHO, it advertises scripts/cleanpatch or scripts/cleanfile. I think that we do not need to repeat it for each affected patch.
In fact, using scripts/cleanfile is questionable. IMHO, it does not make sense to fix indentation just because it looks better. It makes problems when backporting fixes. But it might make sense to fix it when you do some real change on that line.
Best Regards, Petr
| |