[MPlayer-DOCS][PATCH] Documentation for VCD/SVCD/DVD encoding

Guillaume POIRIER poirierg at gmail.com
Wed Jul 6 10:41:14 CEST 2005


Hi,

On 7/6/05, Brendan McCarthy <bmccarthy at iinet.net.au> wrote:
> On Tue, 2005-07-05 at 14:23 +0200, Guillaume POIRIER wrote:
> > Ok, cool. On the nit picking side, it'll be nice if you could add some
> > <option> tags around each encoding option name, like
> > <option>qpel</option> (but not in the "screen" examples).
> > I'd also use the <systemitem> tag when appropriate, like with:
> > <systemitem class="library">libavcodec</systemitem>
> 
> Done
> 
> > On examples, I'd use the <replaceable> tag around input and output
> > file names, and maybe others when appropriate.
> 
> Done
> 
> > Try also to fit the line lengh to less that a terminal size (I think
> > it's 80 characters) so that if someone need to add a word in the
> > middle, the diff won't be too big.
> 
> Done
> 
> > Regarding my personal preferences, I tend to prefer to return to a new
> > line after each sentence, having also into mind to try to minimize the
> > size of the diff.
> 
> Done
> 
> > Pfiou! That may seem like quite a lot of nits here.
> 
> Yes, but I fixed a few more mistakes while reading through it again :)

Don't worry, I'm sure some other guys on this list will find many more! :)

Committed with Minor changes.

Guillaume
-- 
A woman isn't complete without a man.
But where do you find a man - a real man - these
days?
  -- Lauren Bacall
My answer: Only at #mplayerdev on the freenode IRC network.




More information about the MPlayer-DOCS mailing list