[MPlayer-dev-eng] [PATCH] Another DOCS patch

Diego Biurrun diego at biurrun.de
Thu Aug 22 00:13:33 CEST 2002


On Wed, Aug 21, 2002 at 11:16:08AM +0200, Andras Mohari wrote:
> 
> I'm not sure if you'll like this, but here it is anyway (:-)):

I like it.

> * Removed "0." before "How to read this documentation".  (0 as a section
>   number?  I think it's OK to leave it unnumbered.)

I agree with Gabucino.  0 is OK as a section number.  Natural numbers
start at 0 in computer science after all :-)

> And a question: what exactly does the rgb2bgr filter do?  Maybe it's
> just me, but I don't understand the DOCS clearly:
> 
> 	Performs RGB 24/32 <-> BGR 24/32 colorspace conversion (default
> 	behavior) or RGB 24/32 <-> RGB 24/32 conversion with R <-> B
> 	swapping (with the 'swap' option).
> 
> To me, both means the same: R<->B swapping.  If I knew it better, I'd
> replace those <-> with something else (or I'd write an other short
> description).

Yes, that section might use some clarification.  Anyone?

Nice work, thanks.  BTW, you made some changes while converting to XML.
Do you remember what they were and could you wrap them up into a patch?

Diego



More information about the MPlayer-dev-eng mailing list