[Cogl] [PATCH 2/4] Remove unused tex_coord_declarations_variable

Robert Bragg robert at sixbynine.org
Mon Sep 2 06:29:14 PDT 2013


This looks good to land to me:

Reviewed-by: Robert Bragg <robert at linux.intel.com>

thanks,
Robert

On Sun, Aug 25, 2013 at 2:42 AM, Neil Roberts <neil at linux.intel.com> wrote:
> This variable was a leftover from before commit e55b64a9cdc93 where
> the texture coord varyings where declared as an array as part of the
> shader boilerplate.
> ---
>  cogl/cogl-glsl-shader.c | 3 ---
>  1 file changed, 3 deletions(-)
>
> diff --git a/cogl/cogl-glsl-shader.c b/cogl/cogl-glsl-shader.c
> index 8e50edb..b469ede 100644
> --- a/cogl/cogl-glsl-shader.c
> +++ b/cogl/cogl-glsl-shader.c
> @@ -54,7 +54,6 @@ _cogl_glsl_shader_set_source_with_boilerplate (CoglContext *ctx,
>    const char **strings = g_alloca (sizeof (char *) * (count_in + 4));
>    GLint *lengths = g_alloca (sizeof (GLint) * (count_in + 4));
>    int count = 0;
> -  char *tex_coord_declarations = NULL;
>
>    vertex_boilerplate = _COGL_VERTEX_SHADER_BOILERPLATE;
>    fragment_boilerplate = _COGL_FRAGMENT_SHADER_BOILERPLATE;
> @@ -119,6 +118,4 @@ _cogl_glsl_shader_set_source_with_boilerplate (CoglContext *ctx,
>
>    GE( ctx, glShaderSource (shader_gl_handle, count,
>                             (const char **) strings, lengths) );
> -
> -  g_free (tex_coord_declarations);
>  }
> --
> 1.8.3.1
>
> _______________________________________________
> Cogl mailing list
> Cogl at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/cogl


More information about the Cogl mailing list