[PULL] MAXSCREENS removal preparation

Keith Packard keithp at keithp.com
Mon Apr 26 18:56:30 PDT 2010


On Mon, 26 Apr 2010 19:32:39 +0300, "Vignatti Tiago (Nokia-D/Helsinki)" <tiago.vignatti at nokia.com> wrote:

> One thing I find pretty bad with devPrivates logic is the readability of the
> code. To debug it's even worst because we inherited from somewhere this crazy
> idea to use macros in mostly dixLookupPrivate calls.

Yeah, I'm not (currently) planning on changing those macros as I'm
trying to demonstrate a high level of source-level compatibility for the
changes I'm proposing. But, it's certainly possible for us to start
changing this, and I'd encourage it.

I will be making functions like dixLookupPrivate inline though; it's
awfully short now.

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20100426/40ee5b6f/attachment.pgp>


More information about the xorg-devel mailing list