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

[Stefano Sabatini]

On date Sunday 2013-12-29 22:34:29 +0100, Werner Robitza encoded:
> The "s" keyboard shortcut was previously undocumented in ffplay.
> ---
>  doc/ffplay.texi | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/doc/ffplay.texi b/doc/ffplay.texi
> index 54b6f19..76749a2 100644
> --- a/doc/ffplay.texi
> +++ b/doc/ffplay.texi
> @@ -188,6 +188,9 @@ Cycle program.
>  @item w
>  Show audio waves.
>  

> + at item s
> +Pause the playback if not already the case, and skip to the next video frame.

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.
-- 
FFmpeg = Freak & Fancy Mere Perfectionist Eccentric Gadget