[FFmpeg-devel] [PATCH] vqavideo: return error when vqa_decode_chunk fails.

Nicolas George nicolas.george at normalesup.org
Sun Jan 8 14:31:15 CET 2012


Le nonidi 19 nivôse, an CCXX, Reimar Döffinger a écrit :
> -    vqa_decode_chunk(s);
> +    if (vqa_decode_chunk(s) < 0)
> +        return AVERROR_INVALIDDATA;

I would like it slightly better if vqa_decode_chunk returned directly
INVALIDDATA and tis chunk just passed it through: it makes it more obvious
that the error codes have been thought through.

Apart from that, I do not see a flaw in this patch, but I do not know the
first thing about vqavideo.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120108/78f21252/attachment.asc>


More information about the ffmpeg-devel mailing list