Re: [PATCH] staging: android: binder.c: rename binder_user_error()

From: Joe Perches
Date: Sat Aug 16 2014 - 16:13:29 EST


On Fri, 2014-08-15 at 14:45 +0800, Greg KH wrote:
> On Sun, Aug 10, 2014 at 10:42:53PM +0300, Tair Rzayev wrote:
> > Rename binder_user_error() to shorter but no less readable binder_usr_err()
> > which fixes 34 "line over 80 characters" checkpatch warnings
>
> Those are false-positives due to the fact that we should not wrap
> strings and checkpatch doesn't understand this custom macro.
>
> So don't worry about it, the code is fine as-is.

Using <foo>_err instead of <foo>_error means that
checkpatch can successfully ignore long lines of
the format portion of this macro.


--
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/