[FFmpeg-devel] [PATCH] H.264/AVCHD interlaced fixes
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
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-devel