[libav-devel] [PATCH 1/2] imgutils: Use av_pix_fmt_get_chroma_sub_sample in avcodec_get_chroma_sub_sample

Luca Barbato lu_zero at gentoo.org
Thu Aug 20 09:46:59 CEST 2015


On 20/08/15 09:36, Anton Khirnov wrote:
> How is it any better? It's still a random number.

Ideally who is using an unsupported pixel format will get an error from
the other functions.

If you crash there you just crash, if you return a value the caller will
get an error later if hadn't get it sooner.

Given how unlikely it is the case of having an unsupported pixel format
I can drop the patch.

lu


More information about the libav-devel mailing list