CVS Update: xc (branch: trunk)

[Michel Dänzer]

On Mon, 2005-01-31 at 21:10 +0100, Roland Scheidegger wrote:
> Michel Dänzer wrote:
> > On Sun, 2005-01-30 at 17:41 -0800, Felix Kuehling wrote:
> > 
> >>  	- Updated DisableCOB: 3D requires COB, default value is "off", which
> >>  	  means that the COB is enabled.
> > 
> > 
> > If you just called the option "COB", "DisableCOB" would be one of the
> > many cases handled automatically by the option parser.
> 
> Hmm, I think the parser can only handle "No" but not "Disable". 

Indeed, I thought it handled "Enable" and "Disable" as well, but you're
right.

Still, IMHO negative option names are just wrong. :)

> But you've just reminded me I've named that ColorTiling option wrong for the 
> radeon driver, so I've changed that 

Thanks, the style cop in me must have been asleep when I reviewed
that. ;)

> (I think I took EnablePageFlip as an example, which is also named like that).

Oh yeah, there's a lot of bad examples... :\


-- 
Earthling Michel Dänzer      |     Debian (powerpc), X and DRI developer
Libre software enthusiast    |   http://svcs.affero.net/rm.php?r=daenzer