lkml.org 
[lkml]   [1999]   [Oct]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] four warnings and a patch
Date
From
"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/

\
 
 \ /
  Last update: 2005-03-22 13:54    [W:0.056 / U:0.392 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site