On Mon, 2 Dec 2002 18:43:25 -0800 Allen Akin <[EMAIL PROTECTED]> wrote:
> On Mon, Dec 02, 2002 at 02:00:49PM +0100, Felix Kühling wrote: > | .... So if we agree on this, I would make this > | controlled by an environment variable. ... > > The intent of the spec is that drivers should support whichever texture > internal formats they wish to support, and apps should choose between > them (or use the default only if they truly don't care). Environment > variables aren't very portable, and work around the intent of the spec. > Is there some compelling reason to use them, rather than just having the > driver do what was intended? There was previously a dependency on the screen color depth when choosing the texture format in the radeon driver. The environment variable I'm going to add would enable the user to override the bpp dependent default. I had closer look at how the textureformat is selected in radeon_tex.c. The cases in which the screen color depth affects the texture format include the ones in which the application did not specify a color depth in internalformat. In these cases there should be no objection to choosing the preferred texture format with an environment variable. But the choice for the following internalformats also depends on the screen color depth in the current implementation: case GL_RGBA8: case GL_RGB10_A2: case GL_RGBA12: case GL_RGBA16: case GL_RGB8: case GL_RGB10: case GL_RGB12: case GL_RGB16: Would it address your concerns if we would use _mesa_texformat_rgba8888 unconditionally in all these cases? Regards, Felix __\|/__ ___ ___ ___ __Tschüß_______\_6 6_/___/__ \___/__ \___/___\___You can do anything,___ _____Felix_______\Ä/\ \_____\ \_____\ \______U___just not everything____ [EMAIL PROTECTED] >o<__/ \___/ \___/ at the same time! ------------------------------------------------------- This SF.net email is sponsored by: Get the new Palm Tungsten T handheld. Power & Color in a compact size! http://ads.sourceforge.net/cgi-bin/redirect.pl?palm0002en _______________________________________________ Dri-devel mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/dri-devel