[FFmpeg-devel] [PATCH] avcodec/x86/cavsdsp: silence -Wunuse-variable on --disable-mmx

Michael Niedermayer michaelni at gmx.at
Thu Sep 24 04:30:54 CEST 2015


On Sat, Sep 19, 2015 at 10:41:57AM -0400, Ganesh Ajjanagadde wrote:
> On Sat, Sep 19, 2015 at 10:34 AM, Ganesh Ajjanagadde
> <gajjanagadde at gmail.com> wrote:
> > This silences -Wunused-variable when compiled with --disable-mmx, e.g
> > http://fate.ffmpeg.org/log.cgi?time=20150919094617&log=compile&slot=x86_64-archlinux-gcc-disable-mmx.
> > The alternative of header guards will make it far too ugly.
> >
> > Signed-off-by: Ganesh Ajjanagadde <gajjanagadde at gmail.com>
> > ---
> >  libavcodec/x86/cavsdsp.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/libavcodec/x86/cavsdsp.c b/libavcodec/x86/cavsdsp.c
> > index b571163..4b20e65 100644
> > --- a/libavcodec/x86/cavsdsp.c
> > +++ b/libavcodec/x86/cavsdsp.c
> > @@ -563,7 +563,7 @@ static av_cold void cavsdsp_init_3dnow(CAVSDSPContext *c,
> >
> >  av_cold void ff_cavsdsp_init_x86(CAVSDSPContext *c, AVCodecContext *avctx)
> >  {
> > -    int cpu_flags = av_get_cpu_flags();
> > +    av_unused int cpu_flags = av_get_cpu_flags();
> >
> >      cavsdsp_init_mmx(c, avctx);
> >  #if HAVE_AMD3DNOW_INLINE
> > --
> > 2.5.2
> >
> 
> commit message typo: -Wunuse-> -Wunused.

changed
applied

thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Many things microsoft did are stupid, but not doing something just because
microsoft did it is even more stupid. If everything ms did were stupid they
would be bankrupt already.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150924/48e37130/attachment.sig>


More information about the ffmpeg-devel mailing list