[FFmpeg-devel] [PATCH] H.264/AVCHD interlaced fixes

Ivan Schreter schreter
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).

Regards,

Ivan

-------------- next part --------------
A non-text attachment was scrubbed...
Name: h264_parser_0_decodenal.patch
Type: text/x-patch
Size: 4291 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090219/ad9c4691/attachment.bin>



More information about the ffmpeg-devel mailing list