Re: [PATCH] four warnings and a patch

Horst von Brand (vonbrand@sleipnir.valparaiso.cl)
Sat, 30 Oct 1999 00:10:19 -0300

"Jones D (ISaCS)" <djones2@glam.ac.uk> said:
> > > 8390.c:1092: warning: unused variable `ei_local'
> > > The patch removes this line.

> > This is a subtle one. It is used on some platforms. You
> > aren't the first to be confused by it.

> And won't be the last. This comes up _so_ often..
> Can't this be ifdef'd for the architectures which do use it ?

Linus hates #ifdef (and I concur, for what it is worth). Why not just place
the answer I was given (and which the gentleman doing the latest patch got,
and countless others, no doubt) as a comment near the offending line?

-- 
Horst von Brand                             vonbrand@sleipnir.valparaiso.cl
Casilla 9G, Viņa del Mar, Chile                               +56 32 672616


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