[FFmpeg-devel] [PATCH] ffplay: rename stream_pause() to stream_pause_resume()

[Stefano Sabatini]

On date Saturday 2011-04-23 02:08:33 +0200, Michael Niedermayer encoded:
> On Fri, Apr 22, 2011 at 06:17:47PM +0200, Stefano Sabatini wrote:
> > On date Friday 2011-04-22 17:44:10 +0200, Alexander Strasser encoded:
> > > Hi
> > > 
> > > Stefano Sabatini wrote:
> > > > The new name is less misleading, since the function will resume the
> > > > stream if it is currently paused.
> > > > 
> > > > Signed-off-by: Stefano Sabatini <stefano.sabatini-lala at poste.it>
> > > > ---
> > > >  ffplay.c |   10 +++++-----
> > > >  1 files changed, 5 insertions(+), 5 deletions(-)
> > > > 
> > > > diff --git a/ffplay.c b/ffplay.c
> > > > index c080cc0..1faa60e 100644
> > > > --- a/ffplay.c
> > > > +++ b/ffplay.c
> > > > @@ -1064,7 +1064,7 @@ static void stream_seek(VideoState *is, int64_t pos, int64_t rel, int seek_by_by
> > > >  }
> > > >  
> > > >  /* pause or resume the video */
> > > > -static void stream_pause(VideoState *is)
> > > > +static void stream_pause_resume(VideoState *is)
> > > >  {
> > > >      if (is->paused) {
> > > [remaining patch snipped]
> > > 
> > >   Maybe using the verb to toggle is more concise, e.g. just rename
> > > the function to "stream_toggle_pause".
> > 
> > Works for me, and it's even better.
> > -- 
> > FFmpeg = Fanciful and Fabulous Magic Plastic Ecumenical Gadget
> 
> >  ffplay.c |   10 +++++-----
> >  1 file changed, 5 insertions(+), 5 deletions(-)
> > c4ac5aebf6a91fc2e5d0b04a37ae5bb3dd806ec5  0003-ffplay-rename-stream_pause-to-stream_toggle_pause.patch
> > From 5f87c8be8becae83260d0413255e86d416aabf55 Mon Sep 17 00:00:00 2001
> > From: Stefano Sabatini <stefano.sabatini-lala at poste.it>
> > Date: Fri, 22 Apr 2011 02:19:48 +0200
> > Subject: [PATCH] ffplay: rename stream_pause() to stream_toggle_pause()
> 
> LGTM

Applied.
-- 
FFmpeg = Fierce and Foolish Most Problematic Exuberant Gigant