[FFmpeg-devel] [PATCH]Postpone vdpau removal

[Philip Langdale]

On 2015-08-18 15:59, Andreas Cadhalpun wrote:
> On 18.08.2015 23:15, Philip Langdale wrote:
>> On 2015-08-18 14:05, Carl Eugen Hoyos wrote:
>>> Ronald S. Bultje <rsbultje <at> gmail.com> writes:
>>> 
>>>> > Attached patch postpones the vdpau removal.
>>>> > The decoders were only deprecated today, so give
>>>> > users time until the next version bump.
>>>> 
>>>> No.
>>> 
>>> Then please revert a383f226
>>> I am listed as vdpau maintainer and I object to the patch.
>> 
>> Do we have any known applications that still use the old API.
>> Even mplayer, of all things, uses the new API.
> 
> I'm not exactly sure what you meant here, but FF_API_VDPAU
> is mainly about the (AV_)PIX_FMT_VDPAU_* pixel formats, which
> mplayer is still using, e.g. [1].

mplayer contains code that handles these formats, but by using the new
API, it will never actually see content on those formats. This is now
dead code in mplayer.

I can't speak for the other applications.

> There are also libavg [2], mlt [3] and handbrake [4].
> 
> Considering also FF_API_CAP_VDPAU there is additionally 
> gmerlin-avdecoder [5].
> And mplayer is also using this API [6].
> 
> Best regards,
> Andreas
> 
> 
> 1:
> https://sources.debian.net/src/mplayer/2:1.1.1%2Bsvn37434-1/libmpcodecs/vd_ffmpeg.c/?hl=1133#L1133
> 2:
> https://sources.debian.net/src/libavg/1.8.1-1/src/video/VDPAUDecoder.cpp/?hl=214#L214
> 3:
> https://sources.debian.net/src/mlt/0.9.8-1/src/modules/avformat/vdpau.c/?hl=243#L243
> 4:
> https://sources.debian.net/src/handbrake/0.10.2%2Bds1-1/libhb/vadxva2.c/?hl=696#L696
> 5:
> https://sources.debian.net/src/gmerlin-avdecoder/1.2.0~dfsg-5/lib/video_ffmpeg.c/?hl=260#L260
> 6:
> https://sources.debian.net/src/mplayer/2:1.1.1%2Bsvn37434-1/libvo/vo_vdpau.c/?hl=177#L177
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

--phil