[MPlayer-dev-eng] [PATCH] Teletext support try3 (2/5, docs)

Diego Biurrun diego at biurrun.de
Fri Jul 20 03:31:48 CEST 2007


On Thu, Jul 19, 2007 at 01:20:05AM +0700, Vladimir Voroshilov wrote:
> 2007/7/19, Diego Biurrun <diego at biurrun.de>:
>> P.S.: Your mails could be more readable if you inserted a blank line
>> between the quote and your reply.
>
> i'll try to take this in mind.

Thanks.

> P.S. latest patch version is attached.
>
> --- DOCS/tech/slave.txt	(revision 23806)
> +++ DOCS/tech/slave.txt	(working copy)
> @@ -322,6 +322,14 @@
>  
> +teletext_add_digit <value>
> +    On/off teletext page number editing mode and append given digit to
> +    previously entered one.

"On/off" is bad, you should use a verb there.  "Enter/leave" or
"enable/disable" or similar come to mind.

> +    0..9 - Append apropriate digit. (Enables editing mode if called from normal mode, and
> +           switches to normal mode when third digit is entered.)
> +    -    - Delete last digit from page number. (Backspace emulation, works only in page number
> +           editing mode.)

Please keep lines below 80 characters.

> --- DOCS/man/en/mplayer.1	(revision 23806)
> +++ DOCS/man/en/mplayer.1	(working copy)
> @@ -1865,6 +1880,21 @@
> +.IPs tpage=<100-899> (default: 100)
> +Specify initial TV teletext page number .

extra space

Diego



More information about the MPlayer-dev-eng mailing list