Re: [2.6.4-rc2] bogus semicolon behind if()

From: Philippe Elie
Date: Tue Mar 09 2004 - 01:02:22 EST


On Mon, 08 Mar 2004 at 16:29 +0000, Andrew Morton wrote:

> Andreas Schwab <schwab@xxxxxxx> wrote:
> >
> > Thomas Schlichter <thomas.schlichter@xxxxxx> writes:
> >
> > > P.S.: Wouldn't it be nice if gcc complained about these mistakes?
> >
> > Among these 18 cases are 13 false positives. :-)
>
> Rename Thomas's script to crappy-code-detector.sh and its hit rate goes to
> 100% ;)
>

Was this patch so crappy ? http://tinyurl.com/2jbe4 ,

- check_nmi_watchdog();
+ if (check_nmi_watchdog() < 0);
+ timer_ack = !cpu_has_tsc


regards,
Phil

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