diff mbox

[FFmpeg-devel,1/5] lavc/avrndec: remove AV_CODEC_CAP_DR1, as it's broken

Message ID 20171013165919.7642-2-nfxjfg@googlemail.com
State Accepted
Commit 195dd5c59c89b125b89078c736ae107bcaf23b37
Headers show

Commit Message

wm4 Oct. 13, 2017, 4:59 p.m. UTC 
In the is_mjpeg case, the user's get_buffer2 callback is not called,
thus completely breaking the API.
---
 libavcodec/avrndec.c | 1 -
 1 file changed, 1 deletion(-)

Comments

James Almer Oct. 30, 2017, 12:30 a.m. UTC | #1 
On 10/13/2017 1:59 PM, wm4 wrote:
> In the is_mjpeg case, the user's get_buffer2 callback is not called,
> thus completely breaking the API.
> ---
>  libavcodec/avrndec.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/libavcodec/avrndec.c b/libavcodec/avrndec.c
> index c37f99661b..104ff2d904 100644
> --- a/libavcodec/avrndec.c
> +++ b/libavcodec/avrndec.c
> @@ -168,7 +168,6 @@ AVCodec ff_avrn_decoder = {
>      .init           = init,
>      .close          = end,
>      .decode         = decode_frame,
> -    .capabilities   = AV_CODEC_CAP_DR1,
>      .max_lowres     = 3,
>      .caps_internal  = FF_CODEC_CAP_INIT_THREADSAFE,
>  };

Applied this since it's not part of the currently-blocked stuff.
diff mbox

Patch

diff --git a/libavcodec/avrndec.c b/libavcodec/avrndec.c
index c37f99661b..104ff2d904 100644
--- a/libavcodec/avrndec.c
+++ b/libavcodec/avrndec.c
@@ -168,7 +168,6 @@  AVCodec ff_avrn_decoder = {
     .init           = init,
     .close          = end,
     .decode         = decode_frame,
-    .capabilities   = AV_CODEC_CAP_DR1,
     .max_lowres     = 3,
     .caps_internal  = FF_CODEC_CAP_INIT_THREADSAFE,
 };