Mesa (master): swrast: swrastFillInModes doesn't do 8-bit modes, so don' t try

Ian Romanick idr at kemper.freedesktop.org
Sat Nov 10 02:38:46 UTC 2012


Module: Mesa
Branch: master
Commit: 3ec3201f31f6f26029aca66d1967512c8c159245
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=3ec3201f31f6f26029aca66d1967512c8c159245

Author: Ian Romanick <ian.d.romanick at intel.com>
Date:   Tue Oct 30 10:38:54 2012 -0700

swrast: swrastFillInModes doesn't do 8-bit modes, so don't try

Support for 8-bit modes was removed in commits 0398a26 and bda208a4.
However, I didn't notice code in dri_init_screen that explicitly tries
to create this modes.  This is structurally different from other drivers
(that only create modes that match the display color depth).

Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=56555
Cc: Vinson Lee <vlee at freedesktop.org>

---

 src/mesa/drivers/dri/swrast/swrast.c |    4 +---
 1 files changed, 1 insertions(+), 3 deletions(-)

diff --git a/src/mesa/drivers/dri/swrast/swrast.c b/src/mesa/drivers/dri/swrast/swrast.c
index e639049..45ef3c2 100644
--- a/src/mesa/drivers/dri/swrast/swrast.c
+++ b/src/mesa/drivers/dri/swrast/swrast.c
@@ -192,18 +192,16 @@ swrastFillInModes(__DRIscreen *psp,
 static const __DRIconfig **
 dri_init_screen(__DRIscreen * psp)
 {
-    __DRIconfig **configs8, **configs16, **configs24, **configs32;
+    __DRIconfig **configs16, **configs24, **configs32;
 
     TRACE;
 
     psp->extensions = dri_screen_extensions;
 
-    configs8  = swrastFillInModes(psp,  8,  8, 0, 1);
     configs16 = swrastFillInModes(psp, 16, 16, 0, 1);
     configs24 = swrastFillInModes(psp, 24, 24, 8, 1);
     configs32 = swrastFillInModes(psp, 32, 24, 8, 1);
 
-    configs16 = driConcatConfigs(configs8, configs16);
     configs24 = driConcatConfigs(configs16, configs24);
     configs32 = driConcatConfigs(configs24, configs32);
 




More information about the mesa-commit mailing list