[MPlayer-dev-eng] [PATCH] dvb input update

Diego Biurrun diego at biurrun.de
Wed Aug 6 02:35:11 CEST 2003


Nico writes:
 > here's the dvb patch updated with documentation in HTML.
 > Diego, can you please take a look at and correct  the docs?.

Trailing whitespace changes applied.  The rest needs a few changes,
there are still some bugs left ;-)

First of all, please try to fit in with the general coding style of
the document, try to break lines at 80 characters, make all tags
uppercase, put every paragraph into <P> tags and indent like the rest
of the document.  Oh, and please remember the spelling of MPlayer and
MEncoder.  It's only lowercase when you are talking about a command
line.  Then all paths should be surrounded by <CODE> tags, as should
be configure options and command lines, etc.  Here come the details:

 > --- MPlayer-20030805/DOCS/en/documentation.html	2003-07-24 08:49:59.000000000 +0200
 > +++ MPlayer-20030805-new/DOCS/en/documentation.html	2003-08-05 23:59:00.000000000 +0200
 > @@ -225,6 +225,13 @@
 >            <LI><A HREF="#edl_making">2.5.2 Making an EDL file</A></LI>
 >          </UL>
 >        </LI>
 > +      <LI><A HREF="#dvbin">2.6 DVB input</A>

I think you should put this in the TV-out section of video.html.  Do
you disagree?

 > +  linux-DVB-1.0.0 or dvb-kernel branch, downloadable from www.linuxtv.org, and

Make the URL a link.

 > +  the included utilities: scan, szap/tzap/czap.<br>
 > +  You need to pass the parameter <code> dvb_shutdown_timeout=0 to the module <code>dvb-core</code>, or mplayer

<code> tag is not closed.

 > +Make sure that in your <code>channels.conf</code> file are present <u>ONLY</u> Free to Air channels,

Hmm, we never use <u>, make that <B>.

 > +<H3><A NAME="dvbin_parameters">2.6.3 Parameters</A></H3>
 > +You can use the following options, preceded by -dvbin, to specify certain parameters:<p/>

Option descriptions belong in the man page, please add this to the man
page instead.

 > +Example:<p/>
 > +mpayer -dvbin card=2:type=TER:file=C.conf dvb://R3<p/>

Careful, you are missing an 'l' in mpayer.

I could correct this myself, but I'm lazy tonight, so please do it.
I'll later port it to XML.

Diego



More information about the MPlayer-dev-eng mailing list