[FFmpeg-devel] [PATCH] remove __attribute__((unused))
Mon May 14 16:14:32 CEST 2007
On Mon, May 14, 2007 at 04:12:53PM +0200, Reimar D?ffinger wrote:
> On Sun, May 13, 2007 at 09:49:59PM +0200, Michael Niedermayer wrote:
> > On Sun, May 13, 2007 at 06:51:01PM +0200, Reimar D?ffinger wrote:
> > [...]
> > > > > Index: libavcodec/h264.c
> > > > > ===================================================================
> > > > > --- libavcodec/h264.c (revision 9014)
> > > > > +++ libavcodec/h264.c (working copy)
> > > > > @@ -1800,7 +1800,7 @@
> > > > > const int lt= src[-1-1*stride];
> > > > > LOAD_TOP_EDGE
> > > > > LOAD_LEFT_EDGE
> > > > > - const __attribute__((unused)) int unu= l3;
> > > > > + const int unu= l3;
> > > >
> > > > This and similar cases is to kill a warning about l3 (or whatever)
> > > > being unused. The variable l3 is declared by the macro
> > > > LOAD_LEFT_EDGE, but isn't used everywhere this macro is invoked. I
> > > > think the proper solution is to put attribute_unused on all the
> > > > variables declared by those macros.
> > >
> > > I tend to agree but this is for the maintainers to decide.
> > iam ok with it ...
> Like in attached patch?
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Everything should be made as simple as possible, but not simpler.
-- Albert Einstein
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: not available
More information about the ffmpeg-devel