[FFmpeg-devel] [PATCH 6/7] resample: fix set but unused variable (lfe) warning.

Clément Bœsch ubitux at gmail.com
Thu Sep 1 20:31:32 CEST 2011


On Thu, Sep 01, 2011 at 08:29:55PM +0200, Clément Bœsch wrote:
> On Thu, Sep 01, 2011 at 02:50:09AM +0200, Michael Niedermayer wrote:
> > On Thu, Sep 01, 2011 at 02:30:23AM +0200, Clément Bœsch wrote:
> > > ---
> > >  libavcodec/resample.c |    2 +-
> > >  1 files changed, 1 insertions(+), 1 deletions(-)
> > > 
> > > diff --git a/libavcodec/resample.c b/libavcodec/resample.c
> > > index 33413db..a324719 100644
> > > --- a/libavcodec/resample.c
> > > +++ b/libavcodec/resample.c
> > > @@ -121,7 +121,7 @@ static void surround_to_stereo(short **output, short *input, int channels, int s
> > >      short l, r;
> > >  
> > >      for (i = 0; i < samples; i++) {
> > > -        int fl,fr,c,rl,rr,lfe;
> > > +        int fl,fr,c,rl,rr,lfe av_unused;
> > >          fl = input[0];
> > 
> > i think its better to remove the lfe variable and the code writing to
> > it
> > 
> 
> lfe removed and assignation commented out (I think it's still worth
> showing what that byte is), and pushed.
                    ^^^^
                   (short value)

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110901/e5676934/attachment.asc>


More information about the ffmpeg-devel mailing list