Re: [PATCH v3] checkpatch: add dedicated checker for 'Fixes:' tag

2020-04-29 Thread Wang YanQing
On Tue, Apr 28, 2020 at 12:52:59PM +0200, Markus Elfring wrote: > > And there is no 'Fixes:' tag format checker in checkpatch > > I have taken another look at corresponding implementation details. > Will programming challenges get any more attention? > > > > to check the commit id length too, >

Re: [PATCH v3] checkpatch: add dedicated checker for 'Fixes:' tag

2020-04-28 Thread Joe Perches
On Tue, 2020-04-28 at 09:10 -0700, Joe Perches wrote: > On Tue, 2020-04-28 at 10:02 +0800, Wang YanQing wrote: > > According to submitting-patches.rst, 'Fixes:' tag has a little > > stricter condition about the one line summary: > > ... > > Do not split the tag across multiple > > lines, tags are

Re: [PATCH v3] checkpatch: add dedicated checker for 'Fixes:' tag

2020-04-28 Thread Joe Perches
On Tue, 2020-04-28 at 10:02 +0800, Wang YanQing wrote: > According to submitting-patches.rst, 'Fixes:' tag has a little > stricter condition about the one line summary: > ... > Do not split the tag across multiple > lines, tags are exempt from the "wrap at 75 columns" rule in order to simplify >

Re: [PATCH v3] checkpatch: add dedicated checker for 'Fixes:' tag

2020-04-28 Thread Wang YanQing
On Tue, Apr 28, 2020 at 08:21:37AM +0200, Markus Elfring wrote: > > ... > > Do not split the tag across multiple > > lines, tags are exempt from the "wrap at 75 columns" rule in order to > > simplify > > parsing scripts > > ... > > I suggest to reformat the quotation. > > “… > Do not split the

Re: [PATCH v3] checkpatch: add dedicated checker for 'Fixes:' tag

2020-04-28 Thread Markus Elfring
> And there is no 'Fixes:' tag format checker in checkpatch I have taken another look at corresponding implementation details. Will programming challenges get any more attention? > to check the commit id length too, The mentioned script contains the following information.