Re: [patch 3/2] drivers/char/vt.c: remove unnecessary code

[Arjan van de Ven]

\
> > >> + for(npar = NPAR-1; npar < NPAR; npar--)
> > 
> > >How many times do you want this for loop to run?
> > 
> > NPAR times :-). As I stated, npar is unsigned.
> > 
> 
> for (npar = NPAR - 1; npar >= 0; npar--)
> 
> would be more readable and may be even faster on a dumb compiler than
> your variant. Still, I'd have compiler worry about such
> micro-optimizations.

actually that goes wrong for npar unsigned...



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