<div dir="ltr">On 1 February 2014 22:28, Jordan Justen <span dir="ltr"><<a href="mailto:jljusten@gmail.com" target="_blank">jljusten@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Thu, Jan 9, 2014 at 6:19 PM, Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>> wrote:<br>
> ---<br>
>  src/mesa/main/shaderapi.c | 18 ++++++++++++++++++<br>
>  1 file changed, 18 insertions(+)<br>
><br>
> diff --git a/src/mesa/main/shaderapi.c b/src/mesa/main/shaderapi.c<br>
> index 053f27b..680d449 100644<br>
> --- a/src/mesa/main/shaderapi.c<br>
> +++ b/src/mesa/main/shaderapi.c<br>
> @@ -663,6 +663,24 @@ get_programiv(struct gl_context *ctx, GLuint program, GLenum pname, GLint *param<br>
><br>
>        *params = shProg->NumAtomicBuffers;<br>
>        return;<br>
> +   case GL_COMPUTE_WORK_GROUP_SIZE: {<br>
> +      int i;<br>
> +      if (!_mesa_is_desktop_gl(ctx) || !ctx->Extensions.ARB_compute_shader)<br>
> +         break;<br>
> +      if (!shProg->LinkStatus) {<br>
> +         _mesa_error(ctx, GL_INVALID_OPERATION, "glGetProgramiv(program not "<br>
> +                     "linked successfully)");<br>
<br>
</div>Nit, but dropping " successfully" sounds better to me. It seems more<br>
likely that they tried to get the value before linking than that they<br>
ignored a link error.<br></blockquote><div><br></div><div>Sure, I can go along with that.  I've changed it to just "glGetProgramiv(program not linked)".<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
24-26 Reviewed-by: Jordan Justen <<a href="mailto:jordan.l.justen@intel.com">jordan.l.justen@intel.com</a>><br>
<div class="im"><br>
> +         return;<br>
> +      }<br>
> +      if (shProg->_LinkedShaders[MESA_SHADER_COMPUTE] == NULL) {<br>
> +         _mesa_error(ctx, GL_INVALID_OPERATION, "glGetProgramiv(no compute "<br>
> +                     "shaders)");<br>
> +         return;<br>
> +      }<br>
> +      for (i = 0; i < 3; i++)<br>
> +         params[i] = shProg->Comp.LocalSize[i];<br>
> +      return;<br>
> +   }<br>
>     default:<br>
>        break;<br>
>     }<br>
> --<br>
> 1.8.5.2<br>
><br>
</div>> _______________________________________________<br>
> mesa-dev mailing list<br>
> <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev" target="_blank">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</blockquote></div><br></div></div>