Messages in this thread | | | Subject | Re: [PATCH] linux/string.h: Introduce streq macro. | From | Steven Rostedt <> | Date | Wed, 27 Apr 2011 16:04:54 -0400 |
| |
On Wed, 2011-04-27 at 22:38 +0300, Pekka Enberg wrote:
> To be honest, I don't think the arguments for streq() are that strong > but I wanted to point out that the arguments against it weren't all > that great either...
And I totally agree with you here. To be honest myself, I'm not set on having streq() in the kernel. I'm perfectly happy doing the strcmp()==0 method. If someone were to get it into the kernel I would be happy to convert to it.
But like you, I don't think the strong NACKs were justified. It was a lot of hand waving why we should not have it in the kernel. If the argument against streq() is simply that the arguments are not strong enough to add it, and there's no real evidence that it fixes bugs, then sure, I'll buy that. And I've been stating from the beginning that this is all a preference thing. I'm guessing that we have probably the same amount for it as against it, so I'm against a full conversion to it. But this talk of it changing the C language is a bunch of bull. It's no different than having printk() instead of printf().
-- Steve
| |