On Wed, Mar 12, 2003 at 10:21:17PM +0100, Michel Dänzer wrote:
This change doesn't seem obvious to me; the RADEONDRIxxx() functions are in radeon_dri.c and directly related to the DRI, whereas RADEONPreInitDRI() (along with other RADEONPreInitxxx() functions) is in radeon_driver.c
Maybe it belongs in radeon_dri.c, then? Coincidence of current location, should not be a reason for avoiding logical naming and grouping. If you like, I'll submit patches for doing the move AND rename.
[snip]
So RADEONPreInitDRI() doesnt actually do any rendering; So what. It is still "DRI related".
I think Michel is saying that this name is more consistent because this function groups more closely with the other RADEONPreInit functions than with the other RADEONDRI functions. I guess it's six of one, half-a-dozen of the other...
------------------------------------------------------- This SF.net email is sponsored by:Crypto Challenge is now open! Get cracking and register here for some mind boggling fun and the chance of winning an Apple iPod: http://ads.sourceforge.net/cgi-bin/redirect.pl?thaw0031en _______________________________________________ Dri-devel mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/dri-devel
