On Thu, Jun 30, 2005 at 04:19:45 +0200, Egbert Eich wrote: > Alan Hourihane writes: > > On Thu, Jun 30, 2005 at 03:23:52 +0200, Egbert Eich wrote: > > > Alan Hourihane writes: > > > > On Thu, Jun 30, 2005 at 09:46:41AM +0100, Alan Hourihane wrote: > > > > > > > > For i810 and i830 it can just be removed as it isn't even used. The > other > > > > drivers just need a little tweaking to remove it's use. > > > > > > > > > > How about casting it to drm_handle_t? > > > > I think we can just remove it in the other cases. I've just looked at > > the via and sis drivers and it doesn't make any real use of it. I'm sure > the > > other drivers don't make any real use of it too. > > > > I think it's best to just remove it's use completely. > > > > Right. Some cleanup doesn't hurt. This drmAddress looks like something > that was introduced without checking if something similar existed already.
The cleanup does hurt, as it breaks backwards compatibility between the DRI clients and the Xserver. But if 7.0 is around the corner I'd rather break things now, and with idr's changes breaking compatibility too, now is the time to do it. Alan. ------------------------------------------------------- SF.Net email is sponsored by: Discover Easy Linux Migration Strategies from IBM. Find simple to follow Roadmaps, straightforward articles, informative Webcasts and more! Get everything you need to get up to speed, fast. http://ads.osdn.com/?ad_id=7477&alloc_id=16492&op=click -- _______________________________________________ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel
