Re: [PATCH] checkpatch giving bogus advice (was staging: goldfish: Fix minor coding style)

From: Joe Perches
Date: Mon Dec 15 2014 - 19:51:08 EST


On Mon, 2014-12-15 at 14:59 +0300, Dan Carpenter wrote:
> I prefer !foo because it is more common in the kernel and I think it's
> easier to read but I don't feel strongly about this.

Me too. But I do prefer consistency.

fyi: for variants of:

"if (!foo)"
vs
"if (foo == NULL)"

a little cocci script shows a preference for "if (!foo)"
of >5:1 in drivers/net/
(actual: 11557:2145)
and a little less (>3:1) in net/
(actual: 10263:3045)

$ cat pointer_style.cocci
@@
type T;
T *a;
@@

* a == NULL

@@
type T;
T *a;
@@

* a != NULL

@@
type T;
T *a;
@@

* !a


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