[PATCH 01/15] os: remove superfluous -br option

Alan Coopersmith alan.coopersmith at oracle.com
Thu Oct 28 08:37:38 PDT 2010


Tiago Vignatti wrote:
> There is no effect on using or not such option. Deprecate it.

"Deprecate" can mean stop listing it in the options list or issue a
warning - not completely remove.

NAK to this one since it was widely used in display manager configs
to enable this feature before we turned it on by default, allows them
to keep those configs independent of the Xorg version, and protects
them against us changing our mind about what the default should be.

The cost of removal is simply far far higher than the cost of keeping
it - all removal buys us is removing 2 lines of code out of a million,
and a couple strcmp() calls on startup that no one will ever be able
to measure the performance impact of.


-- 
	-Alan Coopersmith-        alan.coopersmith at oracle.com
	 Oracle Solaris Platform Engineering: X Window System



More information about the xorg-devel mailing list