On Sat, Sep 26, 2015 at 12:09 PM, wm4 <nfx...@googlemail.com> wrote: > On Sat, 26 Sep 2015 10:55:29 -0400 > Ganesh Ajjanagadde <gajjanaga...@gmail.com> wrote: > >> On Sat, Sep 19, 2015 at 5:41 PM, Ganesh Ajjanagadde >> <gajjanaga...@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 <gajjanaga...@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@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
