[MPlayer-DOCS] Bunch of patches for codecs.xml

[Diego Biurrun]

On Sun, Nov 05, 2006 at 03:01:47PM +0100, Torinthiel wrote:
> Here's a bunch of patches for DOCS/xml/en/codecs.xml, with short
> explanations:
> Order of applying is almost irrevelant, except it may cause some
> offsets.
> OK to commit? Comments?
> 
> codecs.option.patch: Fixes a missing <option> tag, around -ac hwmpa

Apply.

> codecs.para.patch:
> Removes </para><para> in a middle of thought. IMHO it logically doesn't
> belong there (although it doesn't visually change resulting HTML).

Apply.

> codecs.screen.patch:
> In two places joins <screen> tag in a single line, thus removing ugly
> indentation in resulting HTML.

Apply.

> codecs.sentence.patch:
> The resulting fragment looks like this:
> #For a complete list of supported codecs please visit the FFmpeg
> #homepage. Supported video and audio codecs.
> with video and audio being hyperlinks. This looks awkward to me.
> I propose to change it to
> #For a complete list of supported (audio and video) codecs please visit
> #the FFmpeg homepage.
> Hyperlinks stay at audio and video, but this time fragments looks better
> when viewed without hyperlinks.

Leave out the parentheses and apply.

> codecs.systemitem.patch:
> Various missing <systemitem class="library"> tags.

Apply.

> codecs.url.patch:
> Better (IMHO at least) and more consistent markup of address for
> uploading codec requests. With <> and <replaceable> tags around the
> replaceable codec name.

Apply.
 
> codecs.configure.patch:
> This one has to go after the systemitem patch, as they overlap (unless
> I get a go for this one and not for systemitem of course).
> It unifies configure script handling, as
> <filename>./configure</filename>

Apply.

Diego