Re: [PATCH] linux/string.h: Introduce streq macro.

[Ted Ts'o]

On Tue, Apr 26, 2011 at 03:49:49PM -0300, Thiago Farina wrote:
> This macro is arguably more readable than its variants:
> - !strcmp(a, b)
> - strcmp(a, b) == 0
> 
> Signed-off-by: Thiago Farina <tfransosi@xxxxxxxxx>

I don't think this is not a good idea.

First of all, changing 2800 instances of strcmp will induce a huge
amount of code churn, that will cause patches to break, etc.  And
whether streq() looks better is going to be very much a case of
personal preference.  I'm so used to !strcmp(a, b) that streq(a, b)
would be harder for me, just because I'm not used to it.

So I'd NACK a change like this to any parts of the kernel that I'm
maintaining.  If another people feel that way, it's not clear that
having two different conventions in the kernel would necessarily help...

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