st/mesa: remove the "Gallium 0.4 on" prefix from GL_RENDERER
authorMarek Olšák <marek.olsak@amd.com>
Wed, 7 Jun 2017 20:00:48 +0000 (22:00 +0200)
committerMarek Olšák <marek.olsak@amd.com>
Mon, 19 Jun 2017 18:09:52 +0000 (20:09 +0200)
If you want to keep it for your driver, please raise your hand.
The prefix will probably have to be added into the driver instead of here.

I cringe when I look at my long renderer string:
  Gallium 0.4 on AMD Radeon R9 Fury Series (DRM 3.17.0 / 4.11.0-staging-01277-gab25a9e, LLVM 5.0.0)

I'm sincerely sorry for all apps that detect Mesa by expecting "Gallium"
in the string.

Reviewed-by: Eric Anholt <eric@anholt.net>
src/mesa/state_tracker/st_cb_strings.c
src/mesa/state_tracker/st_context.h

index fc48fd05e3eb64045fea5b82f2b872643e8d2c6a..99d93a7c8f5185eafcba19630cae8a9a007d686e 100644 (file)
@@ -39,8 +39,6 @@
 #include "st_context.h"
 #include "st_cb_strings.h"
 
-#define ST_VERSION_STRING "0.4"
-
 static const GLubyte *
 st_get_string(struct gl_context * ctx, GLenum name)
 {
@@ -55,11 +53,7 @@ st_get_string(struct gl_context * ctx, GLenum name)
    }
 
    case GL_RENDERER:
-      util_snprintf(st->renderer, sizeof(st->renderer), "Gallium %s on %s", 
-               ST_VERSION_STRING,
-              screen->get_name( screen ));
-
-      return (GLubyte *) st->renderer;
+      return (GLubyte *) screen->get_name(screen);
 
    default:
       return NULL;
index f8e9bf96c5212c5a4f0d399a7d1665c5bb5e8b8e..7d04e892a8b4ee80d858ec37a109422c1e3f3f8f 100644 (file)
@@ -164,7 +164,6 @@ struct st_context
    } state;
 
    char vendor[100];
-   char renderer[100];
 
    uint64_t dirty; /**< dirty states */