On Thu, Dec 03, 2015 at 02:23:22PM -0800, Casey Schaufler wrote: > On 11/3/2015 2:15 PM, Dan Carpenter wrote: > >Also checkpatch complains that we should use kstrtouint() instead of > >sscanf here. > > > >Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > > This no longer parses cipso specifications correctly. > I haven't investigated why. It looks as if it should > work. Does kstrtouint() allow for leading whitespace? No it doesn't. I feel slightly betrayed because checkpatch.pl told me to introduce that bug. I have said so many times that I think checkpatch.pl should not tell people to introduce bugs... regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html