[Patch] redundant ATTRIBUTE_UNUSED in elf32-bfin.c

[Mike Frysinger]

On Sunday 10 May 2009 19:35:33 Dave Korn wrote:
> Mike Frysinger wrote:
> > On Saturday 09 May 2009 23:03:18 Masaki Muranaka wrote:
> >> There are some redundant ATTRIBUTE_UNUSEDs in elf32-bfin.c
> >> These may be harmless but should be fixed for trusted code.
> >
> > sorry, but how are they redundant ?
>
>   It looks to me like they are redundant because the argument in question
> is, in fact, used, after all, probably as a result of the code evolving
> since the prototype for the function was first written.

i wouldnt describe that as redundant which is why i was asking for more info.  
if the variable is actually used, then having unused attribute marking sounds 
like the code is wrong.

> Given that we have -Werror, surely if this compiles it must be correct?

sure
-mike