[FFmpeg-devel] [PATCH]Allow to encode yuvj422 and yuvj444 with libx264

Reimar Döffinger Reimar.Doeffinger at gmx.de
Sat Sep 21 09:35:45 CEST 2013


On Sat, Sep 21, 2013 at 03:11:35AM +0200, Carl Eugen Hoyos wrote:
> Hi!
> 
> Attached patch allows to encode to yuvj422p and yuvj444p just like yuvj420p.
> 
> Please comment, Carl Eugen

Seems correct to me.

> -    x4->params.vui.b_fullrange = avctx->pix_fmt == AV_PIX_FMT_YUVJ420P;
> +    x4->params.vui.b_fullrange = (avctx->pix_fmt == AV_PIX_FMT_YUVJ420P ||
> +                                  avctx->pix_fmt == AV_PIX_FMT_YUVJ422P ||
> +                                  avctx->pix_fmt == AV_PIX_FMT_YUVJ444P);

Nit: IMHO adding the () is a bit overkill.


More information about the ffmpeg-devel mailing list