[FFmpeg-devel] [PATCH] Adds decode support for formats other than 420

[Carl Eugen Hoyos]

Deb Mukherjee <debargha <at> google.com> writes:

> +    if (avctx->codec_id == AV_CODEC_ID_VP8) {
> +        if (img->fmt != VPX_IMG_FMT_I420)

Wouldn't (avctx->codec_id == ... && img->fmt != ...) 
be simpler?

> +            return 0;

Shouldn't this be return AVERROR_INVALIDDATA?

Thank you, Carl Eugen