[FFmpeg-devel] [PATCH] FFplay speedup if stepping whilst paused

Michael Niedermayer michaelni
Sun Aug 26 15:05:42 CEST 2007


Hi

On Sun, Aug 26, 2007 at 01:38:12PM +0200, Stefano Sabatini wrote:
> On date Sunday 2007-08-26 13:34:44 +0200, Stefano Sabatini encoded:
> > On date Sunday 2007-08-26 13:15:08 +0200, Michael Niedermayer encoded:
> > > Hi
> > > 
> > > On Sun, Aug 26, 2007 at 11:45:22AM +0200, Stefano Sabatini wrote:
> > > [...]
> > > > Since Craig does not seem to come back with a new patch I've done
> > > > that.
> > > > 
> > > > I simply removed the line:
> > > > cur_stream->video_current_pts = get_video_clock(cur_stream);
> > > > 
> > > > since that is already done in stream_pause(), as suggested by Michael.
> > > > 
> > > > BTW the patch should be attributed anyway to Craig in the commit
> > > > message.
> > > [...]
> > > > Index: ffplay.c
> > > > ===================================================================
> > > > --- ffplay.c	(revision 10230)
> > > > +++ ffplay.c	(working copy)
> > > > @@ -2196,9 +2196,9 @@
> > > >  static void step_to_next_frame(void)
> > > >  {
> > > >      if (cur_stream) {
> > > > -        if (cur_stream->paused)
> > > [...]
> > > > +       if (cur_stream->paused)
> > > 
> > > no
> > 
> > Oops, sorry.
> > New try attached.
> 
> New oops and new sorry, this time I hope I'm really going to attach
> it.

looks ok

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

No snowflake in an avalanche ever feels responsible. -- Voltaire
-------------- 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/20070826/0c889a1e/attachment.pgp>



More information about the ffmpeg-devel mailing list