lkml.org 
[lkml]   [2011]   [Apr]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH] linux/string.h: Introduce streq macro.
From
On Tue, Apr 26, 2011 at 9:58 PM, Steven Rostedt <rostedt@goodmis.org> wrote:
> On Tue, 2011-04-26 at 16:47 -0300, Thiago Farina wrote:
>
>> > $ grep -rP --include=*.[ch] "\bstrcmp.*==\s*0\b" * | wc -l
>> > 1143
>> > $ grep -rP --include=*.[ch] "\!\s*strcmp\s*\(" *  | wc -l
>> > 1663
>> >
>> > Can you count how many misuses of strcmp have been
>> > corrected?
>>
>> > Do you plan to convert the existing 2800?
>>
>> I'd work on this without any problem.
>
> Nothing a perl script can't do either.

If you are really going to do that, please use a coccinelle's semantic
patch (which is designed precisely for that purpose) and document it
so that it can be included in the standard set of semantic patches
applied to the kernel regularly, i.e. something like:

@@ expression E1, E2; @@
- strcmp(E1,E2) == 0
+ streq(E1,E2)

@@ expression E1, E2; @@
- strcmp(E1,E2) != 0
+ !streq(E1,E2)

@@ expression E1, E2; @@
- !strcmp(E1,E2)
+ streq(E1,E2)

@@ expression E1, E2; @@
- strcmp(E1,E2)
+ !streq(E1,E2)

This produces a patch ~1.1 MB in size, which is probably going to
produce some headaches when merging other patches...

$ diffstat streq.patch | tail -1
1073 files changed, 3429 insertions(+), 3419 deletions(-)

$ diffstat streq.patch | sort -nk 3 | tail -20
drivers/sbus/char/envctrl.c | 30 -
fs/reiserfs/bitmap.c | 30 -
sound/pci/hda/hda_eld.c | 30 -
drivers/video/matrox/matroxfb_base.c | 32 -
arch/blackfin/kernel/module.c | 34 +-
tools/perf/util/symbol.c | 35 +-
scripts/mod/modpost.c | 38 +-
net/irda/irlan/irlan_client.c | 40 +-
drivers/staging/hv/vmbus_drv.c | 42 +-
drivers/usb/gadget/gadget_chips.h | 42 +-
drivers/s390/net/qeth_l3_sys.c | 44 +-
drivers/macintosh/windfarm_smu_controls.c | 48 +-
security/selinux/hooks.c | 48 +-
security/smack/smack_lsm.c | 48 +-
arch/x86/pci/common.c | 50 +--
drivers/net/niu.c | 54 +--
fs/xfs/linux-2.6/xfs_super.c | 92 ++---
tools/perf/util/trace-event-parse.c | 108 +++---
drivers/staging/wlags49_h2/wl_profile.c | 121 +++----
net/core/pktgen.c |
164 +++++-----

Miguel Ojeda

>
> -- Steve
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2011-04-27 10:31    [W:0.416 / U:0.504 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site