[FFmpeg-devel] [PATCH] DeluxePaint Animation playback system

Peter Ross pross
Mon Oct 12 12:10:22 CEST 2009


On Tue, Sep 22, 2009 at 02:42:32PM +0200, Reimar D?ffinger wrote:
> On Tue, Sep 22, 2009 at 10:22:06PM +1000, Peter Ross wrote:
> > On Wed, Aug 26, 2009 at 02:15:06PM +0200, Reimar D?ffinger wrote:
> > > On Wed, Aug 26, 2009 at 08:45:33PM +1000, Peter Ross wrote:
> > > 
> > > > +    pkt->pts = anm->pts;
> > > > +    if (pkt->pts == 0)
> > > > +        pkt->flags |= PKT_FLAG_KEY;
> > > > +
> > > > +    anm->pts++;
> > > > +    anm->record++;
> > > 
> > > That will probably mean your code can't work with AVFMT_GENERIC_INDEX.
> > > I really think that seeking should be working.
> > 
> > Can you elaborate on this problem Reminar? I don't see a problem with
> > incrementing pts...
> 
> Besides that the pts part is probably just completely useless code, it means you
> are keeping position-dependent state. Which means that you need a
> special seek function that sets this internal state to the correct value
> for the position the seek went to (obviously the generic seek function
> can not know about this demuxer-specific data).

Updated patch. PTS state stuff removed.

-- Peter
(A907 E02F A6E5 0CD2 34CD 20D2 6760 79C5 AC40 DD6B)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: anm-r3.diff
Type: text/x-diff
Size: 16673 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091012/92a0f54d/attachment.diff>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091012/92a0f54d/attachment.pgp>



More information about the ffmpeg-devel mailing list