[FFmpeg-devel] [PATCH] y41p encoder and decoder

Carl Eugen Hoyos cehoyos at ag.or.at
Sat Dec 31 21:40:37 CET 2011


Paul B Mahol <onemda <at> gmail.com> writes:

> y41p is a packed 12-bit 4:1:1 YUV format used by Brooktree.

(Fixes issue 1123 / ticket 102)
Just two nits:

> +    avctx->pix_fmt             = PIX_FMT_YUV411P;

> +    avctx->bits_per_raw_sample = 12;

Isn't 12 the default anyway?

[...]

> +    .long_name    = NULL_IF_CONFIG_SMALL("Uncompressed YUV 4:1:1 12-bit"),

Imo, this should be "Uncompressed packed YUV 4:1:1"
(But please feel free to ignore)

Is encoding odd widths allowed?

Thank you, Carl Eugen



More information about the ffmpeg-devel mailing list