[Mesa-dev] [PATCH v3 04/18] mesa: return bool instead of GLboolean in compressedteximage_only_format()

Anuj Phogat anuj.phogat at gmail.com
Wed Jun 24 14:12:02 PDT 2015


On Mon, Jun 22, 2015 at 4:02 PM, Nanley Chery <nanleychery at gmail.com> wrote:
> From: Nanley Chery <nanley.g.chery at intel.com>
>
> In agreement with the coding style, functions that aren't directly visible
> to the GL API should prefer the use of bool over GLboolean.
>
> Suggested-by: Ian Romanick <ian.d.romanick at intel.com>
> Signed-off-by: Nanley Chery <nanley.g.chery at intel.com>
> ---
>  src/mesa/main/teximage.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c
> index 86ef407..0e0488a 100644
> --- a/src/mesa/main/teximage.c
> +++ b/src/mesa/main/teximage.c
> @@ -1763,7 +1763,7 @@ _mesa_test_proxy_teximage(struct gl_context *ctx, GLenum target, GLint level,
>  /**
>   * Return true if the format is only valid for glCompressedTexImage.
>   */
> -static GLboolean
> +static bool
>  compressedteximage_only_format(const struct gl_context *ctx, GLenum format)
>  {
>     switch (format) {
> @@ -1806,9 +1806,9 @@ compressedteximage_only_format(const struct gl_context *ctx, GLenum format)
>     case GL_COMPRESSED_SRGB8_ALPHA8_ASTC_10x10_KHR:
>     case GL_COMPRESSED_SRGB8_ALPHA8_ASTC_12x10_KHR:
>     case GL_COMPRESSED_SRGB8_ALPHA8_ASTC_12x12_KHR:
> -      return GL_TRUE;
> +      return true;
>     default:
> -      return GL_FALSE;
> +      return false;
>     }
>  }
>
> --
> 2.4.2
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reviewed-by: Anuj Phogat <anuj.phogat at gmail.com>


More information about the mesa-dev mailing list