[FFmpeg-devel] [PATCH] ffplay: add documentation for "s" key

Stefano Sabatini stefasab at gmail.com
Wed Jan 1 13:07:40 CET 2014


On date Monday 2013-12-30 10:59:23 +0100, Werner Robitza encoded:
> On Mon, Dec 30, 2013 at 10:52 AM, Stefano Sabatini <stefasab at gmail.com> wrote:
> > Sorry to nitpick:
> >
> > Pause the playback if it is not already paused, or step to the next video frame.
> >
> > ("if not already the case" is not very explicit and clear, especially
> > for non fluent/native English readers, "or" seems more logically
> > correct than "and", "step" should be more explicit than "skip").
> >
> > No need to send an updated patch, just tell me if you like the new
> > version and/or propose amendments.
> 
> That's also fine by me. I just wanted the feature to be documented, is all.

Pushed this variant:

@item s
Step to the next frame.

Pause if the stream is not already paused, step to the next video
frame, and pause.

...

Feel free to propose improvements.

Thanks.
-- 
FFmpeg = Fantastic and Fundamental Mind-dumbing Philosofic Earthshaking Gem


More information about the ffmpeg-devel mailing list