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

2015-09-28 Thread Michael Niedermayer
On Mon, Sep 28, 2015 at 10:15:01AM -0400, Ganesh Ajjanagadde wrote: > On Sat, Sep 26, 2015 at 12:09 PM, wm4 wrote: > > On Sat, 26 Sep 2015 10:55:29 -0400 > > Ganesh Ajjanagadde wrote: > > > >> On Sat, Sep 19, 2015 at 5:41 PM, Ganesh Ajjanagadde > >> wrote: > >> > pix_fmt was declared presumably

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

2015-09-28 Thread Ganesh Ajjanagadde
On Sat, Sep 26, 2015 at 12:09 PM, wm4 wrote: > On Sat, 26 Sep 2015 10:55:29 -0400 > Ganesh Ajjanagadde wrote: > >> On Sat, Sep 19, 2015 at 5:41 PM, Ganesh Ajjanagadde >> wrote: >> > pix_fmt was declared presumably to shorten the argument passed to the >> > function. >> > However, it is currentl

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

2015-09-26 Thread wm4
On Sat, 26 Sep 2015 10:55:29 -0400 Ganesh Ajjanagadde wrote: > On Sat, Sep 19, 2015 at 5:41 PM, Ganesh Ajjanagadde > 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 simp

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

2015-09-26 Thread Ganesh Ajjanagadde
On Sat, Sep 19, 2015 at 5:41 PM, Ganesh Ajjanagadde 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

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

2015-09-19 Thread Ganesh Ajjanagadde
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