On Fri, Jan 13, 2017 at 06:15:54PM +0100, SF Markus Elfring wrote: > From: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx> > Date: Fri, 13 Jan 2017 14:14:45 +0100 > > The script "checkpatch.pl" pointed information out like the following. > > WARNING: quoted string split across lines > > Thus fix the affected source code places. You should provide more explanation and analysis for changes than just saying "checkpatch" - it's not a 100% reliable tool and sometimes it gives false positives or simply silencing the warning actually makes things worse (as some of the other feedback on the series shows). It is better to provide something in the changelog which explains why the change improves things, giving credit to checkpatch is of course nice to do but something that shows an understanding of what it's saying really helps.
Attachment:
signature.asc
Description: PGP signature
