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

[Michael Niedermayer]

On Wed, Feb 18, 2009 at 10:19:58PM +0100, Ivan Schreter wrote:
> 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.

ok

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

While the State exists there can be no freedom; when there is freedom there
will be no State. -- Vladimir Lenin
-------------- 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/20090218/cab592f8/attachment.pgp>