<div dir="ltr">This comment has been addressed.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 30, 2014 at 4:09 PM, Anuj Phogat <span dir="ltr"><<a href="mailto:anuj.phogat@gmail.com" target="_blank">anuj.phogat@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Tue, Dec 16, 2014 at 6:52 AM, Laura Ekstrand <<a href="mailto:laura@jlekstrand.net">laura@jlekstrand.net</a>> wrote:<br>
> This is a convenience function for *Texture*Parameter functions.<br>
> ---<br>
>  src/mesa/main/texparam.c | 38 ++++++++++++++++++++++++++++++++++++++<br>
>  1 file changed, 38 insertions(+)<br>
><br>
> diff --git a/src/mesa/main/texparam.c b/src/mesa/main/texparam.c<br>
> index df2ba3c..6121346 100644<br>
> --- a/src/mesa/main/texparam.c<br>
> +++ b/src/mesa/main/texparam.c<br>
> @@ -147,6 +147,44 @@ get_texobj(struct gl_context *ctx, GLenum target, GLboolean get)<br>
>     return texUnit->CurrentTex[targetIndex];<br>
>  }<br>
><br>
> +/**<br>
> + * Get current texture object for given name.<br>
> + * Return NULL if any error (and record the error).<br>
> + * Note that proxy targets are not accepted.<br>
> + * Only the glGetTexLevelParameter() functions accept proxy targets.<br>
> + */<br>
> +static struct gl_texture_object *<br>
> +get_texobj_by_name(struct gl_context *ctx, GLuint texture, GLboolean get)<br>
> +{<br>
> +   struct gl_texture_object *texObj;<br>
> +<br>
> +   texObj = _mesa_lookup_texture(ctx, texture);<br>
> +   if (!texObj) {<br>
> +      /* User passed a non-generated name. */<br>
> +      /* Throw the error in the caller. */<br>
</div></div>Combine the comments into one.<br>
<div><div class="h5">> +      return NULL;<br>
> +   }<br>
> +<br>
> +   switch (texObj->Target) {<br>
> +   case GL_TEXTURE_1D:<br>
> +   case GL_TEXTURE_1D_ARRAY:<br>
> +   case GL_TEXTURE_2D:<br>
> +   case GL_TEXTURE_2D_ARRAY:<br>
> +   case GL_TEXTURE_2D_MULTISAMPLE:<br>
> +   case GL_TEXTURE_2D_MULTISAMPLE_ARRAY:<br>
> +   case GL_TEXTURE_3D:<br>
> +   case GL_TEXTURE_CUBE_MAP:<br>
> +   case GL_TEXTURE_CUBE_MAP_ARRAY:<br>
> +   case GL_TEXTURE_RECTANGLE:<br>
> +      return texObj;<br>
> +   default:<br>
> +      _mesa_error(ctx, GL_INVALID_ENUM,<br>
> +                  "gl%sTextureParameter(target)", get ? "Get" : "");<br>
> +      return NULL;<br>
> +   }<br>
> +<br>
> +}<br>
> +<br>
><br>
>  /**<br>
>   * Convert GL_RED/GREEN/BLUE/ALPHA/ZERO/ONE to SWIZZLE_X/Y/Z/W/ZERO/ONE.<br>
> --<br>
> 2.1.0<br>
><br>
</div></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>