[PATCH 05/11] glx: Optionally call DRI2 createContextAttribs from __glXDRIscreenCreateContext

[Jesse Barnes]

On Tue, 03 Jan 2012 16:45:49 -0800
Ian Romanick <idr at freedesktop.org> wrote:
> > Maybe stuff this into a separate function that's a no-op in the<  3
> > case?  That would clean things up a little and save an #ifdef in the
> > middle of a function (always a nice thing).
> >
> > Or just require updated DRI2 bits to build and avoid the ifdefs
> > altogether, since they tend to cause trouble anyway.
> 
> I had thought about both of these, and ultimately we just want at later. 
>   However, that requires a new Mesa release.  I thought it was better to 
> leave the "obvious" clutter of the #ifdef in the code for later clean-up 
> than an extra function.
> 
> What do you think?
> 

I think a separate function is a bit nicer in any case. Having huge
functions with or without ifdefs should probably be avoided
regardless.  That could be an incremental patch though; there are
probably a couple of other version check branches that could be pulled
out for clarity.

Looking forward to the next release though!

-- 
Jesse Barnes, Intel Open Source Technology Center
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20120104/dffd8f15/attachment.pgp>