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 _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
