[FFmpeg-devel] [PATCH] rmdec.c: merge old/new packet reading code

Kostya kostya.shishkov
Tue Mar 10 19:10:18 CET 2009


On Tue, Mar 10, 2009 at 11:44:15AM -0400, Ronald S. Bultje wrote:
> Hi,
> 
> On Tue, Mar 10, 2009 at 10:22 AM, Kostya <kostya.shishkov at gmail.com> wrote:
> > Looks ok. And yes, it looks like what I want :)
> 
> Small change, I want to change:
> 
> > +            flags = (seq == 1) ? 2 : 0;
> 
> into
> 
> > +            flags = (seq++ == 1) ? 2 : 0;
> 
> otherwise seq is always one for old_format and thus flags is always 2
> instead of only for the first pkt.

does this have any consequences (i.e. seq value used in other places)?
if no, then ok
 
> Ronald




More information about the ffmpeg-devel mailing list