[FFmpeg-devel] [PATCH] H.264/AVCHD interlaced fixes
Thu Feb 19 12:51:52 CET 2009
Michael Niedermayer wrote:
> On Tue, Feb 17, 2009 at 10:44:43AM +0100, Ivan Schreter wrote:
>> Michael Niedermayer wrote:
>>> also the whole ff_h264_decode_rbsp_trailing seems unneeded unless i
>>> miss something
>> I'm also unsure, whether it is needed. Byte-exact length should be
>> actually sufficient for the stuff parsed in parser (SEI, SPS, PPS and
>> slice header). I tried removing it and could parse my samples without
>> problem, as it seems. OTOH, in the future, some other NALs might need
>> bit-exact length to decode without warning and then we'll have to search
>> why the warning comes. So maybe we should let it in. What do you think?
> remove it or factorize the code so it is not duplicated
I've factorized decode_nal to actually return bit length. Patch attached.
I'll change the rest of parser patches appropriately, after this is in
(and the convergence duration patch in other thread).
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 4291 bytes
Desc: not available
More information about the ffmpeg-devel