From: Emil Velikov <emil.velikov@collabora.com>
Date: Tue, 24 Oct 2017 13:35:41 +0000 (+0100)
Subject: mesa: drop unnecessary coping of extra_extensions
X-Git-Tag: upstream/18.1.0~4010
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=444d9e4b081ab18352fc30a3c90829536ddd49fb;p=platform%2Fupstream%2Fmesa.git

mesa: drop unnecessary coping of extra_extensions

The function get_extension_override() returns a copy of a string,
only for it to be copied again ...

Drop the unneeded calloc/strdup/free dance.

Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Brian Paul <brianp@vmware.com>
---

diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c
index 87f60d5..351ed1c 100644
--- a/src/mesa/main/extensions.c
+++ b/src/mesa/main/extensions.c
@@ -198,18 +198,16 @@ set_extension(struct gl_extensions *ext, int i, GLboolean state)
 
 /**
  * The unrecognized extensions from \c MESA_EXTENSION_OVERRIDE.
- * Must be freed, does not return \c NULL.
+ * Returns \c NULL if empty.
  */
-static char *
+static const char *
 get_extension_override( struct gl_context *ctx )
 {
-   if (extra_extensions == NULL) {
-      return calloc(1, sizeof(char));
-   } else {
+   if (extra_extensions)
       _mesa_problem(ctx, "Trying to enable unknown extensions: %s",
                     extra_extensions);
-      return strdup(extra_extensions);
-   }
+
+   return extra_extensions;
 }
 
 
@@ -396,7 +394,7 @@ _mesa_make_extension_string(struct gl_context *ctx)
    /* Indices of the extensions sorted by year */
    extension_index *extension_indices;
    /* String of extra extensions. */
-   char *extra_extensions = get_extension_override(ctx);
+   const char *extra_extensions = get_extension_override(ctx);
    unsigned k;
    unsigned j;
    unsigned maxYear = ~0;
@@ -427,14 +425,12 @@ _mesa_make_extension_string(struct gl_context *ctx)
 
    exts = calloc(ALIGN(length + 1, 4), sizeof(char));
    if (exts == NULL) {
-      free(extra_extensions);
       return NULL;
    }
 
    extension_indices = malloc(count * sizeof(extension_index));
    if (extension_indices == NULL) {
       free(exts);
-      free(extra_extensions);
       return NULL;
    }
 
@@ -464,7 +460,6 @@ _mesa_make_extension_string(struct gl_context *ctx)
    free(extension_indices);
    if (extra_extensions != 0) {
       strcat(exts, extra_extensions);
-      free(extra_extensions);
    }
 
    return (GLubyte *) exts;