[FFmpeg-devel] [PATCHv2] avcodec/videotoolbox: fix -Wunused-but-set-variable

Ganesh Ajjanagadde gajjanag at mit.edu
Mon Sep 28 16:15:01 CEST 2015


On Sat, Sep 26, 2015 at 12:09 PM, wm4 <nfxjfg at googlemail.com> wrote:
> On Sat, 26 Sep 2015 10:55:29 -0400
> Ganesh Ajjanagadde <gajjanagadde at gmail.com> wrote:
>
>> On Sat, Sep 19, 2015 at 5:41 PM, Ganesh Ajjanagadde
>> <gajjanagadde at gmail.com> wrote:
>> > pix_fmt was declared presumably to shorten the argument passed to the function.
>> > However, it is currently not being used for such a purpose.
>> > This patch simply removes it instead.
>> > This fixes -Wunused-but-set-variable reported at e.g:
>> > http://fate.ffmpeg.org/log.cgi?time=20150919194249&log=compile&slot=x86_64-darwin-gcc-4.9.
>> >
>> > Signed-off-by: Ganesh Ajjanagadde <gajjanagadde at gmail.com>
>> > ---
>> >  libavcodec/videotoolbox.c | 3 ---
>> >  1 file changed, 3 deletions(-)
>> >
>> > diff --git a/libavcodec/videotoolbox.c b/libavcodec/videotoolbox.c
>> > index b78238a..ca44491 100644
>> > --- a/libavcodec/videotoolbox.c
>> > +++ b/libavcodec/videotoolbox.c
>> > @@ -501,7 +501,6 @@ static int videotoolbox_default_init(AVCodecContext *avctx)
>> >      VTDecompressionOutputCallbackRecord decoder_cb;
>> >      CFDictionaryRef decoder_spec;
>> >      CFDictionaryRef buf_attr;
>> > -    int32_t pix_fmt;
>> >
>> >      if (!videotoolbox) {
>> >          av_log(avctx, AV_LOG_ERROR, "hwaccel context is not set\n");
>> > @@ -528,8 +527,6 @@ static int videotoolbox_default_init(AVCodecContext *avctx)
>> >          break;
>> >      }
>> >
>> > -    pix_fmt = videotoolbox->cv_pix_fmt_type;
>> > -
>> >      decoder_spec = videotoolbox_decoder_config_create(videotoolbox->cm_codec_type, avctx);
>> >
>> >      videotoolbox->cm_fmt_desc = videotoolbox_format_desc_create(videotoolbox->cm_codec_type,
>> > --
>> > 2.5.2
>> >
>>
>> ping
>
> LGTM

patch has been reviewed, anyone willing to push?

> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


More information about the ffmpeg-devel mailing list