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

Michael Niedermayer michaelni
Mon Feb 9 11:34:44 CET 2009


On Mon, Feb 09, 2009 at 12:52:46AM +0100, Ivan Schreter wrote:
> Hi Michael,
>
> Ivan Schreter wrote:
>> Ivan Schreter wrote:
>>   
>>> Michael Niedermayer wrote:
>>>       
>>>> keyframe |= ...
>>>>             
>>> Do you mean I should add additional "keyframe" flag? I can do that...
>>>       
>> Ah, I see that the key_frame flag is already set on current picture. So it 
>> was OK as it was, the problem is elsewhere. If recovery_frame_cnt >= 0, it 
>> should OR a key frame flag, not set it to recovery_frame_cnt >= 0. I will 
>> correct that.
>>
>>   
> I suppose the attached patch regarding SEI recovery point will now suffice 
> your critical eye :-)
>
> I made it independent of #1, so it can be applied as-is.
>
> Please review & if you are happy with it now, apply it.

[...]
> @@ -7340,6 +7344,8 @@
>      int context_count = 0;
>  
>      h->max_contexts = avctx->thread_count;
> +    h->sei_recovery_frame_cnt = -1;
> +

i suspect that this will misbehave if CODEC_FLAG2_CHUNKS is set

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Its not that you shouldnt use gotos but rather that you should write
readable code and code with gotos often but not always is less readable
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090209/f58da4fa/attachment.pgp>



More information about the ffmpeg-devel mailing list