[MPlayer-DOCS] Re: Draft: iPod encoding guide

[Diego Biurrun]

On Thu, Jan 11, 2007 at 07:22:27PM -0500, Mark Pilgrim wrote:
> Oh hell, I found some stupid copy-and-paste errors.  Just ignore draft
> 1 and let's all start the discussion with draft 2 (attached).

Guillaume already gave you some constructive feedback, here are a few
more formalities.

> --- mplayer/DOCS/xml/en/encoding-guide.xml	(revision 21881)
> +++ mplayer/DOCS/xml/en/encoding-guide.xml	(working copy)
> @@ -4395,6 +4395,264 @@
> +  The first video-capable iPod was introduced in 2005 and dubbed the "5G" or
> +  "fifth-generation" iPod.  This guide will focus on the requirements of the
> +  iPod 5G.  Later iPod models may have different limitations.

Please use European-style single spaces after periods.

> +  The iPod 5G supports the H.264 baseline profile with a maximum size of
> +  320 x 240 and a maximum bitrate of 768kbps.

Throughout the documentation we write sizes as AAAxBBB with no spaces in
between, please stick to this for consistency.

> +  <emphasis role="bold">Maximum bitrate</emphasis>: the iPod 5G supports a

Please capitalize complete sentences after a colon.

> +  <application>MPlayer</application>, but of course the iPod 5G can not play

cannot

Nit: There is some trailing whitespace, get rid of that.

Diego