Messages in this thread | | | Subject | Re: [PATCH] four warnings and a patch | Date | Sat, 30 Oct 1999 00:10:19 -0300 | From | Horst von Brand <> |
| |
"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/
| |