[FFmpeg-devel] [PATCH 2/6] rmdec.c: ff_rm_parse_frame()
Benoit Fouet
benoit.fouet
Tue Nov 6 17:07:20 CET 2007
Ronald S. Bultje wrote:
> Hi.
>
> On 11/6/07, Aurelien Jacobs <aurel at gnuage.org> wrote:
>
>>> - pkt->pts= timestamp;
>>> - if(flags&2){
>>> - pkt->flags |= PKT_FLAG_KEY;
>>> - if((seq&0x7F) == 1)
>>> - av_add_index_entry(st, pos, timestamp, 0, 0,
>>>
>> AVINDEX_KEYFRAME);
>>
>>> - }
>>> + if(flags&2 && (seq&0x7F) == 1)
>>> + av_add_index_entry(st, pos, timestamp, 0, 0,
>>>
>> AVINDEX_KEYFRAME);
>>
>>> }
>>>
>> Here indentation of av_add_index_entry (and probably also merging of
>> the 2 if) could be done in a separate commit.
>>
>
>
> Attached both.
>
>
could you send updated patches against svn head ?
--
Ben
Purple Labs S.A.
www.purplelabs.com
More information about the ffmpeg-devel
mailing list