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

[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

[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 =