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

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to