[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <alpine.LFD.0.999.0708161801530.25951@enigma.security.iitk.ac.in>
Date: Thu, 16 Aug 2007 18:11:17 +0530 (IST)
From: Satyam Sharma <satyam@...radead.org>
To: Ilpo Järvinen <ilpo.jarvinen@...sinki.fi>
cc: Heiko Carstens <heiko.carstens@...ibm.com>,
Joe Perches <joe@...ches.com>, Dave Jones <davej@...hat.com>,
Linux Kernel <linux-kernel@...r.kernel.org>, rolandd@...co.com,
Chas Williams <chas@....nrl.navy.mil>,
Paul Mundt <lethal@...ux-sh.org>,
isdn4linux@...tserv.isdn4linux.de, mikep@...uxtr.net,
Netdev <netdev@...r.kernel.org>, swen@...t.ibm.com,
linux390@...ibm.com, linux-s390@...r.kernel.org, jdike@...aya.com,
user-mode-linux-devel@...ts.sourceforge.net,
user-mode-linux-user@...ts.sourceforge.net,
netfilter-devel@...ts.netfilter.org, coreteam@...filter.org
Subject: Re: drivers/infiniband/mlx/mad.c misplaced ;
Hi Ilpo,
On Thu, 16 Aug 2007, Ilpo Järvinen wrote:
>
> ...I guess those guys hunting for broken busyloops in the other thread
> could also benefit from similar searching commands introduced in this
> thread... ...Ccing Satyam to caught their attention too.
>
>
> > On Wed, Aug 15, 2007 at 05:40:11PM -0700, Joe Perches wrote:
> > >
> > > There's more than a few of these (not inspected).
> > >
> > > $ egrep -r --include=*.c "\bif[[:space:]]*\([^\)]*\)[[:space:]]*\;" *
>
> ...Hmm, I plugged in "a preprocessor" too to manage with non compliant
> coding styles :-). Please understand that the line numbers are not an
> exact match due to preprocessor changes:
>
> $ for i in `find . -name '*.[ch]'`; do echo $i; indent -npro -kr -i8 -ts8
> -sob -l8000 -ss -ncs -cp1 -nhnl -st $i | egrep -n "[[:space:]]if [(].*[)] ;$";
> done | grep -B1 "^[^.]"
Thanks, looks useful, will check with this.
Satyam
Powered by blists - more mailing lists