[MPlayer-dev-eng] [PATCH] Cosmetic upgrade to patches.xml

Diego Biurrun diego at biurrun.de
Sat Oct 8 20:38:15 CEST 2005


On Fri, Oct 07, 2005 at 06:44:06PM -0400, Ergzay wrote:
> The current patches.xml only contains a link to a .txt document. I 
> incorporated the .txt document into the xml page.

Not a good idea IMO, it duplicates information, which then has to be
kept in sync.

In fact I think the patches.xml file is pretty pointless.  I'll remove
it.

> This is my first patch of any kind so I'm not sure how good it is.

Next time please try to break overly long lines and don't manually
increment the revision tag, CVS does it automatically.

Diego




More information about the MPlayer-dev-eng mailing list