[FFmpeg-devel] [PATCH 2/2] avformat/yuv4mpegdec: add support for 444alpha

Jan Ekström jeebjp at gmail.com
Thu Mar 19 17:10:12 EET 2020


On Thu, Mar 19, 2020 at 12:19 PM Paul B Mahol <onemda at gmail.com> wrote:
>
> Signed-off-by: Paul B Mahol <onemda at gmail.com>
> ---
>  libavformat/yuv4mpegdec.c | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/libavformat/yuv4mpegdec.c b/libavformat/yuv4mpegdec.c
> index addc03057e..980de7d145 100644
> --- a/libavformat/yuv4mpegdec.c
> +++ b/libavformat/yuv4mpegdec.c
> @@ -124,9 +124,7 @@ static int yuv4_read_header(AVFormatContext *s)
>              } else if (strncmp("422", tokstart, 3) == 0) {
>                  pix_fmt = AV_PIX_FMT_YUV422P;
>              } else if (strncmp("444alpha", tokstart, 8) == 0 ) {
> -                av_log(s, AV_LOG_ERROR, "Cannot handle 4:4:4:4 "
> -                       "YUV4MPEG stream.\n");
> -                return -1;
> +                pix_fmt = AV_PIX_FMT_YUVA444P;
>              } else if (strncmp("444", tokstart, 3) == 0) {
>                  pix_fmt = AV_PIX_FMT_YUV444P;
>              } else if (strncmp("mono16", tokstart, 6) == 0) {
> --
> 2.17.1
>

I had to double-check if the demuxer wasn't hard-coding the amount of
planes, but apparently that was just the muxer :) .If this leads to
correct image being read with av_get_packet && rawvideo's raw_decode,
I think this is alright?

Jan


More information about the ffmpeg-devel mailing list