Re: [PATCH] checkpatch: Improve warning message for "needless if" case

From: Joe Perches
Date: Tue Oct 21 2014 - 12:51:29 EST


On Tue, 2014-10-21 at 14:44 -0200, Fabio Estevam wrote:
> On Tue, Oct 21, 2014 at 2:40 PM, Joe Perches <joe@xxxxxxxxxxx> wrote:
>
> >
> > Mixing sentences with and without periods is jarring.
> > I'd prefer a comma or a dash.
>
> Or what about?
>
> "is safe and this check is probably not required"

That'd be fine too

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/