[FFmpeg-devel] [PATCH] ffplay: rename stream_pause() to stream_pause_resume()
Michael Niedermayer
michaelni at gmx.at
Sat Apr 23 02:08:33 CEST 2011
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
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
He who knows, does not speak. He who speaks, does not know. -- Lao Tsu
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110423/cedda954/attachment.asc>
More information about the ffmpeg-devel
mailing list