[FFmpeg-devel] [PATCH] H.264 timestamps in h264_parser - complete set

[Ivan Schreter]

Michael Niedermayer wrote:
> On Wed, Feb 18, 2009 at 03:12:24PM +0100, Ivan Schreter wrote:
>   
> [...]
>> How do I express the 
>> dependency?
>>     
>
> well, it does not seem we do express that currently though it could be
> done in libavformat/libavformat.pc i suspect not that i know the syntax or
> ask you to do that.
>   
As I see, libavformat.pc actually already contains correct dependency on 
my machine. So I suppose the dependency is generated automagically by make.

>
> [...]
>   
>> Index: libavformat/utils.c
>> ===================================================================
>> --- libavformat/utils.c	(revision 17429)
>> +++ libavformat/utils.c	(working copy)
>> @@ -899,8 +899,21 @@
>>      else if (pc) {
>>          pkt->flags = 0;
>>          /* keyframe computation */
>>     
>
>   
>> -            if (pc->pict_type == FF_I_TYPE)
>> +        switch (pc->key_frame) {
>> +            case 0:
>> +                // not a key frame
>> +                break;
>> +            case -1:
>> +                // not set, use fallback via picture type
>> +                if (pc->pict_type == FF_I_TYPE) {
>> +                    pkt->flags |= PKT_FLAG_KEY;
>> +                }
>> +                break;
>> +            default:
>> +                // key frame
>>                  pkt->flags |= PKT_FLAG_KEY;
>> +                break;
>> +        }
>>     
>
> if(pc->key_frame==1)
>     pkt->flags |= PKT_FLAG_KEY;
> else if(pc->key_frame==-1 && pc->pict_type == FF_I_TYPE)
>     pkt->flags |= PKT_FLAG_KEY;
>
>   
Updated. Patch attached.

Regards,

Ivan

-------------- next part --------------
A non-text attachment was scrubbed...
Name: avcodec_keyframe.patch
Type: text/x-patch
Size: 2304 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090218/6902503e/attachment.bin>