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/