[FFmpeg-devel] [PATCH 2/6] rmdec.c: ff_rm_parse_frame()

[Ronald S. Bultje]

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.

Ronald
-------------- next part --------------
A non-text attachment was scrubbed...
Name: rmdec-parse_frame
Type: application/octet-stream
Size: 7866 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071106/f3bb28ff/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: rmdec-remove_double_nested_if.patch
Type: application/octet-stream
Size: 684 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071106/f3bb28ff/attachment-0001.obj>