[Mesa-dev] [PATCH 1/3] mesa: remove unused options var in compile_shader()

2015-04-27 Thread Brian Paul
---
 src/mesa/main/shaderapi.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/src/mesa/main/shaderapi.c b/src/mesa/main/shaderapi.c
index cc001ba..a04b287 100644
--- a/src/mesa/main/shaderapi.c
+++ b/src/mesa/main/shaderapi.c
@@ -861,14 +861,11 @@ static void
 compile_shader(struct gl_context *ctx, GLuint shaderObj)
 {
struct gl_shader *sh;
-   struct gl_shader_compiler_options *options;
 
sh = _mesa_lookup_shader_err(ctx, shaderObj, "glCompileShader");
if (!sh)
   return;
 
-   options = &ctx->Const.ShaderCompilerOptions[sh->Stage];
-
if (!sh->Source) {
   /* If the user called glCompileShader without first calling
* glShaderSource, we should fail to compile, but not raise a GL_ERROR.
-- 
1.9.1

___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev


Re: [Mesa-dev] [PATCH 1/3] mesa: remove unused options var in compile_shader()

2015-04-28 Thread Anuj Phogat
On Mon, Apr 27, 2015 at 4:35 PM, Brian Paul  wrote:
> ---
>  src/mesa/main/shaderapi.c | 3 ---
>  1 file changed, 3 deletions(-)
>
> diff --git a/src/mesa/main/shaderapi.c b/src/mesa/main/shaderapi.c
> index cc001ba..a04b287 100644
> --- a/src/mesa/main/shaderapi.c
> +++ b/src/mesa/main/shaderapi.c
> @@ -861,14 +861,11 @@ static void
>  compile_shader(struct gl_context *ctx, GLuint shaderObj)
>  {
> struct gl_shader *sh;
> -   struct gl_shader_compiler_options *options;
>
> sh = _mesa_lookup_shader_err(ctx, shaderObj, "glCompileShader");
> if (!sh)
>return;
>
> -   options = &ctx->Const.ShaderCompilerOptions[sh->Stage];
> -
> if (!sh->Source) {
>/* If the user called glCompileShader without first calling
> * glShaderSource, we should fail to compile, but not raise a GL_ERROR.
> --
> 1.9.1
>
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Series is:
Reviewed-by: Anuj Phogat 
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev