[Ffmpeg-devel] huffyuv.c: encode_frame() and decode_frame() return values.

Michael Niedermayer michaelni
Thu Mar 30 16:13:57 CEST 2006


Hi

On Sat, Mar 18, 2006 at 07:14:02PM +0100, Tomas Carnecky wrote:
> encode_frame() adds the size of the Huffman tables to the return value,
> eg. the return value is the total from the Huffman tables and the actual
> encoded frame. But decode_frame() returns only the size of the decoded
> frame, without the Huffman tables. This of course only happens when
> avctx->context_model is enabled.
> Is this the correct behavior? I so some sanity checks with the return
> values from these two functions and it fails in that case.
> 
> a possible fix: change the return statement in decode_frame:
> 
> return (get_bits_count(&s->gb)+31)/32*4 + table_size;

yes, fixed that way

[...]

-- 
Michael





More information about the ffmpeg-devel mailing list