Re: [PATCH xserver 1/3] glx: Remove True/False defines

2017-08-16 Thread Keith Packard 
Adam Jackson  writes:

> Those are xlib spellings, we say TRUE/FALSE pretty consistently
> elsewhere in the server.
>
> Signed-off-by: Adam Jackson 

Reviewed-by: Keith Packard 

-- 
-keith


signature.asc
Description: PGP signature
___
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel

Re: [PATCH xserver 1/3] glx: Remove True/False defines

2017-08-16 Thread Alex Deucher 
On Wed, Aug 16, 2017 at 2:49 PM, Adam Jackson  wrote:
> Those are xlib spellings, we say TRUE/FALSE pretty consistently
> elsewhere in the server.
>
> Signed-off-by: Adam Jackson 

Series is:
Reviewed-by: Alex Deucher 

> ---
>  glx/createcontext.c| 16 
>  glx/glxdri2.c  | 16 
>  glx/glxext.c   | 16 
>  glx/glxserver.h|  7 ---
>  glx/indirect_program.c |  4 ++--
>  hw/xwin/glx/indirect.c |  2 +-
>  6 files changed, 27 insertions(+), 34 deletions(-)
>
> diff --git a/glx/createcontext.c b/glx/createcontext.c
> index 068b35fa7..1216f9412 100644
> --- a/glx/createcontext.c
> +++ b/glx/createcontext.c
> @@ -37,29 +37,29 @@ static Bool
>  validate_GL_version(int major_version, int minor_version)
>  {
>  if (major_version <= 0 || minor_version < 0)
> -return False;
> +return FALSE;
>
>  switch (major_version) {
>  case 1:
>  if (minor_version > 5)
> -return False;
> +return FALSE;
>  break;
>
>  case 2:
>  if (minor_version > 1)
> -return False;
> +return FALSE;
>  break;
>
>  case 3:
>  if (minor_version > 3)
> -return False;
> +return FALSE;
>  break;
>
>  default:
>  break;
>  }
>
> -return True;
> +return TRUE;
>  }
>
>  static Bool
> @@ -70,9 +70,9 @@ validate_render_type(uint32_t render_type)
>  case GLX_COLOR_INDEX_TYPE:
>  case GLX_RGBA_FLOAT_TYPE_ARB:
>  case GLX_RGBA_UNSIGNED_FLOAT_TYPE_EXT:
> -return True;
> +return TRUE;
>  default:
> -return False;
> +return FALSE;
>  }
>  }
>
> @@ -316,7 +316,7 @@ __glXDisp_CreateContextAttribsARB(__GLXclientState * cl, 
> GLbyte * pc)
>  ctx->config = config;
>  ctx->id = req->context;
>  ctx->share_id = req->shareList;
> -ctx->idExists = True;
> +ctx->idExists = TRUE;
>  ctx->isDirect = req->isDirect;
>  ctx->renderMode = GL_RENDER;
>  ctx->resetNotificationStrategy = reset;
> diff --git a/glx/glxdri2.c b/glx/glxdri2.c
> index 701944283..9961c1bfb 100644
> --- a/glx/glxdri2.c
> +++ b/glx/glxdri2.c
> @@ -344,11 +344,11 @@ dri2_convert_glx_attribs(__GLXDRIscreen *screen, 
> unsigned num_attribs,
>  unsigned i;
>
>  if (num_attribs == 0)
> -return True;
> +return TRUE;
>
>  if (attribs == NULL) {
>  *error = BadImplementation;
> -return False;
> +return FALSE;
>  }
>
>  *major_ver = 1;
> @@ -381,13 +381,13 @@ dri2_convert_glx_attribs(__GLXDRIscreen *screen, 
> unsigned num_attribs,
>  break;
>  default:
>  *error = __glXError(GLXBadProfileARB);
> -return False;
> +return FALSE;
>  }
>  break;
>  case GLX_CONTEXT_RESET_NOTIFICATION_STRATEGY_ARB:
>  if (screen->dri2->base.version >= 4) {
>  *error = BadValue;
> -return False;
> +return FALSE;
>  }
>
>  switch (attribs[i * 2 + 1]) {
> @@ -399,14 +399,14 @@ dri2_convert_glx_attribs(__GLXDRIscreen *screen, 
> unsigned num_attribs,
>  break;
>  default:
>  *error = BadValue;
> -return False;
> +return FALSE;
>  }
>  break;
>  default:
>  /* If an unknown attribute is received, fail.
>   */
>  *error = BadValue;
> -return False;
> +return FALSE;
>  }
>  }
>
> @@ -414,7 +414,7 @@ dri2_convert_glx_attribs(__GLXDRIscreen *screen, unsigned 
> num_attribs,
>   */
>  if ((*flags & ~ALL_DRI_CTX_FLAGS) != 0) {
>  *error = BadValue;
> -return False;
> +return FALSE;
>  }
>
>  /* If the core profile is requested for a GL version is less than 3.2,
> @@ -428,7 +428,7 @@ dri2_convert_glx_attribs(__GLXDRIscreen *screen, unsigned 
> num_attribs,
>  }
>
>  *error = Success;
> -return True;
> +return TRUE;
>  }
>
>  static void
> diff --git a/glx/glxext.c b/glx/glxext.c
> index e88bbd107..9b4d81641 100644
> --- a/glx/glxext.c
> +++ b/glx/glxext.c
> @@ -97,7 +97,7 @@ ContextGone(__GLXcontext * cx, XID id)
>  __glXFreeContext(cx);
>  }
>
> -return True;
> +return TRUE;
>  }
>
>  static __GLXcontext *glxPendingDestroyContexts;
> @@ -146,7 +146,7 @@ DrawableGone(__GLXdrawable * glxPriv, XID xid)
>
>  glxPriv->destroy(glxPriv);
>
> -return True;
> +return TRUE;
>  }
>
>  Bool
> @@ -155,12 +155,12 @@ __glXAddContext(__GLXcontext * cx)
>  /* Register this context as a resource.
>   */
>  if (!AddResource(cx->id, __glXContextRes, (void *)cx)) {
> -   return False;
> +   return FALSE;
>  }
>
>  cx->next =

[PATCH xserver 1/3] glx: Remove True/False defines

2017-08-16 Thread Adam Jackson 
Those are xlib spellings, we say TRUE/FALSE pretty consistently
elsewhere in the server.

Signed-off-by: Adam Jackson 
---
 glx/createcontext.c| 16 
 glx/glxdri2.c  | 16 
 glx/glxext.c   | 16 
 glx/glxserver.h|  7 ---
 glx/indirect_program.c |  4 ++--
 hw/xwin/glx/indirect.c |  2 +-
 6 files changed, 27 insertions(+), 34 deletions(-)

diff --git a/glx/createcontext.c b/glx/createcontext.c
index 068b35fa7..1216f9412 100644
--- a/glx/createcontext.c
+++ b/glx/createcontext.c
@@ -37,29 +37,29 @@ static Bool
 validate_GL_version(int major_version, int minor_version)
 {
 if (major_version <= 0 || minor_version < 0)
-return False;
+return FALSE;
 
 switch (major_version) {
 case 1:
 if (minor_version > 5)
-return False;
+return FALSE;
 break;
 
 case 2:
 if (minor_version > 1)
-return False;
+return FALSE;
 break;
 
 case 3:
 if (minor_version > 3)
-return False;
+return FALSE;
 break;
 
 default:
 break;
 }
 
-return True;
+return TRUE;
 }
 
 static Bool
@@ -70,9 +70,9 @@ validate_render_type(uint32_t render_type)
 case GLX_COLOR_INDEX_TYPE:
 case GLX_RGBA_FLOAT_TYPE_ARB:
 case GLX_RGBA_UNSIGNED_FLOAT_TYPE_EXT:
-return True;
+return TRUE;
 default:
-return False;
+return FALSE;
 }
 }
 
@@ -316,7 +316,7 @@ __glXDisp_CreateContextAttribsARB(__GLXclientState * cl, 
GLbyte * pc)
 ctx->config = config;
 ctx->id = req->context;
 ctx->share_id = req->shareList;
-ctx->idExists = True;
+ctx->idExists = TRUE;
 ctx->isDirect = req->isDirect;
 ctx->renderMode = GL_RENDER;
 ctx->resetNotificationStrategy = reset;
diff --git a/glx/glxdri2.c b/glx/glxdri2.c
index 701944283..9961c1bfb 100644
--- a/glx/glxdri2.c
+++ b/glx/glxdri2.c
@@ -344,11 +344,11 @@ dri2_convert_glx_attribs(__GLXDRIscreen *screen, unsigned 
num_attribs,
 unsigned i;
 
 if (num_attribs == 0)
-return True;
+return TRUE;
 
 if (attribs == NULL) {
 *error = BadImplementation;
-return False;
+return FALSE;
 }
 
 *major_ver = 1;
@@ -381,13 +381,13 @@ dri2_convert_glx_attribs(__GLXDRIscreen *screen, unsigned 
num_attribs,
 break;
 default:
 *error = __glXError(GLXBadProfileARB);
-return False;
+return FALSE;
 }
 break;
 case GLX_CONTEXT_RESET_NOTIFICATION_STRATEGY_ARB:
 if (screen->dri2->base.version >= 4) {
 *error = BadValue;
-return False;
+return FALSE;
 }
 
 switch (attribs[i * 2 + 1]) {
@@ -399,14 +399,14 @@ dri2_convert_glx_attribs(__GLXDRIscreen *screen, unsigned 
num_attribs,
 break;
 default:
 *error = BadValue;
-return False;
+return FALSE;
 }
 break;
 default:
 /* If an unknown attribute is received, fail.
  */
 *error = BadValue;
-return False;
+return FALSE;
 }
 }
 
@@ -414,7 +414,7 @@ dri2_convert_glx_attribs(__GLXDRIscreen *screen, unsigned 
num_attribs,
  */
 if ((*flags & ~ALL_DRI_CTX_FLAGS) != 0) {
 *error = BadValue;
-return False;
+return FALSE;
 }
 
 /* If the core profile is requested for a GL version is less than 3.2,
@@ -428,7 +428,7 @@ dri2_convert_glx_attribs(__GLXDRIscreen *screen, unsigned 
num_attribs,
 }
 
 *error = Success;
-return True;
+return TRUE;
 }
 
 static void
diff --git a/glx/glxext.c b/glx/glxext.c
index e88bbd107..9b4d81641 100644
--- a/glx/glxext.c
+++ b/glx/glxext.c
@@ -97,7 +97,7 @@ ContextGone(__GLXcontext * cx, XID id)
 __glXFreeContext(cx);
 }
 
-return True;
+return TRUE;
 }
 
 static __GLXcontext *glxPendingDestroyContexts;
@@ -146,7 +146,7 @@ DrawableGone(__GLXdrawable * glxPriv, XID xid)
 
 glxPriv->destroy(glxPriv);
 
-return True;
+return TRUE;
 }
 
 Bool
@@ -155,12 +155,12 @@ __glXAddContext(__GLXcontext * cx)
 /* Register this context as a resource.
  */
 if (!AddResource(cx->id, __glXContextRes, (void *)cx)) {
-   return False;
+   return FALSE;
 }
 
 cx->next = glxAllContexts;
 glxAllContexts = cx;
-return True;
+return TRUE;
 }
 
 static void
@@ -321,11 +321,11 @@ checkScreenVisuals(void)
 for (j = 0; j < screen->numVisuals; j++) {
 if (screen->visuals[j].class == TrueColor ||
 screen->visuals[j].class == DirectColor)
-return True;
+return TRUE;
 }
 }
 
-return False;
+return FALSE;
 }
 
 static void
@@ -355,7