On Fri, Oct 10, 2008 at 3:18 AM, Maarten Maathuis <[EMAIL PROTECTED]> wrote: > On Thu, Oct 9, 2008 at 9:34 AM, Michel Dänzer > <[EMAIL PROTECTED]> wrote: >> On Wed, 2008-10-08 at 19:20 +0200, Maarten Maathuis wrote: >>> On Tue, Oct 7, 2008 at 8:04 PM, Maarten Maathuis <[EMAIL PROTECTED]> wrote: >>> > These names are supposedly historical, but everyone seems to use them >>> > for the same purpose. >>> > I am aware this may require a few minor changes to (for example) exa >>> > drivers with driver based memory allocations. >>> > I personally believe in having clear names rather than some obscure >>> > remnant of the past. I removed the devPrivate considering the void >>> > pointer is the only useful part anyway. >>> > >>> > It would be nice to know if all !xfree86 has the equivilant of >>> > VIDEODRV_ABI_VERSION. >>> > >>> > Anyway, share your thoughts, comments and such. >>> > >>> > Maarten. >>> > >>> > PS. i can commit these myself, this is just for review, considering >>> > it's a non-trivial change. >>> > >>> >>> Any thoughts about other stuff needing change in the pixmap (or maybe >>> the closely related drawable)? I'm openminded to change, so it seems >>> sensible to check if there are any other problems out there. Without >>> any further complaints i'll probably go ahead with this in a few days. >> >> I'm not sure this is worth the pain for drivers... >> >> >> -- >> Earthling Michel Dänzer | http://tungstengraphics.com >> Libre software enthusiast | Debian, X and DRI developer >> >> > > Are you thinking of anything besides ddx drivers? (i checked ati, it > has about 5 instances, intel about 12, a simple macro could serve as a > drop in replacement)
It totally messes up the ability to do git bisections and such for 0 value other than changing some names. The gain isn't outweighing much. Dave. > > Maarten. > _______________________________________________ > xorg mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/xorg > _______________________________________________ xorg mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/xorg
