[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