Re: [Mesa-dev] [PATCH] mesa: Add API debug logging to TexStorage

2013-11-17 Thread Kenneth Graunke 
On 11/05/2013 01:25 PM, Courtney Goeltzenleuchter wrote:
> Give glTexStorage* equivalent debug logging to glTexImage*.
> 
> Signed-off-by: Courtney Goeltzenleuchter 
> ---
>  src/mesa/main/texstorage.c | 7 +++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/src/mesa/main/texstorage.c b/src/mesa/main/texstorage.c
> index 7bd8652..84b8f82 100644
> --- a/src/mesa/main/texstorage.c
> +++ b/src/mesa/main/texstorage.c
> @@ -365,6 +365,13 @@ texstorage(GLuint dims, GLenum target, GLsizei levels, 
> GLenum internalformat,
>  
> GET_CURRENT_CONTEXT(ctx);
>  
> +   if (MESA_VERBOSE & (VERBOSE_API|VERBOSE_TEXTURE))
> +  _mesa_debug(ctx, "glTexStorage%uD %s %d %s %d %d %d\n",
> +  dims,
> +  _mesa_lookup_enum_by_nr(target), levels,
> +  _mesa_lookup_enum_by_nr(internalformat),
> +  width, height, depth);
> +
> if (tex_storage_error_check(ctx, dims, target, levels,
> internalformat, width, height, depth)) {
>return; /* error was recorded */
> 

Reviewed-by: Kenneth Graunke 

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

Re: [Mesa-dev] [PATCH] mesa: Add API debug logging to TexStorage

2013-11-07 Thread Brian Paul 

On 11/05/2013 02:25 PM, Courtney Goeltzenleuchter wrote:

Give glTexStorage* equivalent debug logging to glTexImage*.

Signed-off-by: Courtney Goeltzenleuchter 
---
  src/mesa/main/texstorage.c | 7 +++
  1 file changed, 7 insertions(+)

diff --git a/src/mesa/main/texstorage.c b/src/mesa/main/texstorage.c
index 7bd8652..84b8f82 100644
--- a/src/mesa/main/texstorage.c
+++ b/src/mesa/main/texstorage.c
@@ -365,6 +365,13 @@ texstorage(GLuint dims, GLenum target, GLsizei levels, 
GLenum internalformat,

 GET_CURRENT_CONTEXT(ctx);

+   if (MESA_VERBOSE & (VERBOSE_API|VERBOSE_TEXTURE))
+  _mesa_debug(ctx, "glTexStorage%uD %s %d %s %d %d %d\n",
+  dims,
+  _mesa_lookup_enum_by_nr(target), levels,
+  _mesa_lookup_enum_by_nr(internalformat),
+  width, height, depth);
+
 if (tex_storage_error_check(ctx, dims, target, levels,
 internalformat, width, height, depth)) {
return; /* error was recorded */



Reviewed-by: Brian Paul 

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

[Mesa-dev] [PATCH] mesa: Add API debug logging to TexStorage

2013-11-05 Thread Courtney Goeltzenleuchter 
Give glTexStorage* equivalent debug logging to glTexImage*.

Signed-off-by: Courtney Goeltzenleuchter 
---
 src/mesa/main/texstorage.c | 7 +++
 1 file changed, 7 insertions(+)

diff --git a/src/mesa/main/texstorage.c b/src/mesa/main/texstorage.c
index 7bd8652..84b8f82 100644
--- a/src/mesa/main/texstorage.c
+++ b/src/mesa/main/texstorage.c
@@ -365,6 +365,13 @@ texstorage(GLuint dims, GLenum target, GLsizei levels, 
GLenum internalformat,
 
GET_CURRENT_CONTEXT(ctx);
 
+   if (MESA_VERBOSE & (VERBOSE_API|VERBOSE_TEXTURE))
+  _mesa_debug(ctx, "glTexStorage%uD %s %d %s %d %d %d\n",
+  dims,
+  _mesa_lookup_enum_by_nr(target), levels,
+  _mesa_lookup_enum_by_nr(internalformat),
+  width, height, depth);
+
if (tex_storage_error_check(ctx, dims, target, levels,
internalformat, width, height, depth)) {
   return; /* error was recorded */
-- 
1.8.1.2

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