<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Jul 30, 2014 at 2:04 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="">On Tue, Jul 29, 2014 at 5:01 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>

> If we're going to make a utility function declared in glformats.h, let's<br>
> move the function body to glformats.c.<br>
><br>
</div>or just move the function declaration to teximage.h? That looks like a better<br>
place to declare texture target related functions.<br></blockquote><div><br></div><div>Either way.  I guess it makes sense to keep it in teximage<br></div><div>--Jason<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="HOEnZb"><div class="h5"><br>
><br>
> On Fri, Jun 6, 2014 at 4:57 PM, Anuj Phogat <<a href="mailto:anuj.phogat@gmail.com">anuj.phogat@gmail.com</a>> wrote:<br>
>><br>
>> Cc: <<a href="mailto:mesa-stable@lists.freedesktop.org">mesa-stable@lists.freedesktop.org</a>><br>
>> Signed-off-by: Anuj Phogat <<a href="mailto:anuj.phogat@gmail.com">anuj.phogat@gmail.com</a>><br>
>> ---<br>
>>  src/mesa/main/glformats.h |  3 +++<br>
>>  src/mesa/main/teximage.c  | 12 ++++++------<br>
>>  2 files changed, 9 insertions(+), 6 deletions(-)<br>
>><br>
>> diff --git a/src/mesa/main/glformats.h b/src/mesa/main/glformats.h<br>
>> index af10899..5c3b2e3 100644<br>
>> --- a/src/mesa/main/glformats.h<br>
>> +++ b/src/mesa/main/glformats.h<br>
>> @@ -125,6 +125,9 @@ extern GLenum<br>
>>  _mesa_es3_error_check_format_and_type(GLenum format, GLenum type,<br>
>>                                        GLenum internalFormat);<br>
>><br>
>> +extern GLboolean<br>
>> +_mesa_target_can_be_compressed(const struct gl_context *ctx, GLenum<br>
>> target,<br>
>> +                               GLenum intFormat);<br>
>><br>
>>  #ifdef __cplusplus<br>
>>  }<br>
>> diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c<br>
>> index 845ba80..b8627a3 100644<br>
>> --- a/src/mesa/main/teximage.c<br>
>> +++ b/src/mesa/main/teximage.c<br>
>> @@ -1779,9 +1779,9 @@ compressedteximage_only_format(const struct<br>
>> gl_context *ctx, GLenum format)<br>
>>   * Helper function to determine whether a target and specific compression<br>
>>   * format are supported.<br>
>>   */<br>
>> -static GLboolean<br>
>> -target_can_be_compressed(const struct gl_context *ctx, GLenum target,<br>
>> -                         GLenum intFormat)<br>
>> +GLboolean<br>
>> +_mesa_target_can_be_compressed(const struct gl_context *ctx, GLenum<br>
>> target,<br>
>> +                               GLenum intFormat)<br>
>>  {<br>
>>     (void) intFormat;  /* not used yet */<br>
>><br>
>> @@ -2204,7 +2204,7 @@ texture_error_check( struct gl_context *ctx,<br>
>><br>
>>     /* additional checks for compressed textures */<br>
>>     if (_mesa_is_compressed_format(ctx, internalFormat)) {<br>
>> -      if (!target_can_be_compressed(ctx, target, internalFormat)) {<br>
>> +      if (!_mesa_target_can_be_compressed(ctx, target, internalFormat)) {<br>
>>           _mesa_error(ctx, GL_INVALID_ENUM,<br>
>>                       "glTexImage%dD(target can't be compressed)",<br>
>> dimensions);<br>
>>           return GL_TRUE;<br>
>> @@ -2260,7 +2260,7 @@ compressed_texture_error_check(struct gl_context<br>
>> *ctx, GLint dimensions,<br>
>>     GLenum error = GL_NO_ERROR;<br>
>>     char *reason = ""; /* no error */<br>
>><br>
>> -   if (!target_can_be_compressed(ctx, target, internalFormat)) {<br>
>> +   if (!_mesa_target_can_be_compressed(ctx, target, internalFormat)) {<br>
>>        reason = "target";<br>
>>        error = GL_INVALID_ENUM;<br>
>>        goto error;<br>
>> @@ -2694,7 +2694,7 @@ copytexture_error_check( struct gl_context *ctx,<br>
>> GLuint dimensions,<br>
>>     }<br>
>><br>
>>     if (_mesa_is_compressed_format(ctx, internalFormat)) {<br>
>> -      if (!target_can_be_compressed(ctx, target, internalFormat)) {<br>
>> +      if (!_mesa_target_can_be_compressed(ctx, target, internalFormat)) {<br>
>>           _mesa_error(ctx, GL_INVALID_ENUM,<br>
>>                       "glCopyTexImage%dD(target)", dimensions);<br>
>>           return GL_TRUE;<br>
>> --<br>
>> 1.8.3.1<br>
>><br>
>> _______________________________________________<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>
><br>
><br>
</div></div></blockquote></div><br></div></div>