Re: [FFmpeg-devel] libavcodec/mpegvideo : remove warning introduce by previous patch

2017-11-07 Thread Martin Vignali
2017-11-07 3:38 GMT+01:00 James Almer :

> On 11/6/2017 8:05 PM, Martin Vignali wrote:
> > Hello,
> >
> > Sorry, i put the ret variable declaration in the previous patch, at the
> > wrong place (at the start of the func, instead of inside the
> > FF_API_DEBUG_MV part
> >
> > patch in attach fix warning,
> >
> > Martin
>
> I'm not sure why you bothered to change disabled code in the first
> place, but pushed nonetheless.
>
>
the goal is to remove all use of this func,
Following doc/api changes, avcodec_get_chroma_sub_sample is deprecated
since 2012

Maybe this deprecated function can be remove soon (without depending of
other things)

Martin
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] libavcodec/mpegvideo : remove warning introduce by previous patch

2017-11-06 Thread James Almer
On 11/6/2017 8:05 PM, Martin Vignali wrote:
> Hello,
> 
> Sorry, i put the ret variable declaration in the previous patch, at the
> wrong place (at the start of the func, instead of inside the
> FF_API_DEBUG_MV part
> 
> patch in attach fix warning,
> 
> Martin

I'm not sure why you bothered to change disabled code in the first
place, but pushed nonetheless.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


[FFmpeg-devel] libavcodec/mpegvideo : remove warning introduce by previous patch

2017-11-06 Thread Martin Vignali
Hello,

Sorry, i put the ret variable declaration in the previous patch, at the
wrong place (at the start of the func, instead of inside the
FF_API_DEBUG_MV part

patch in attach fix warning,

Martin


0001-libavcodec-mepgvideo-remove-warning-introduce-by-pre.patch
Description: Binary data
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel