Messages in this thread | | | Date | Tue, 26 Apr 2011 20:52:43 -0400 | From | Ted Ts'o <> | Subject | Re: [PATCH] linux/string.h: Introduce streq macro. |
| |
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@gmail.com>
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
| |