[PATCH 07/11] glx: Initialize all context fields together

Christopher James Halse Rogers christopher.halse.rogers at canonical.com
Tue Apr 17 03:05:35 PDT 2012


On Fri, 2012-04-13 at 18:11 -0700, Ian Romanick wrote:
> 
> @@ -292,26 +292,10 @@ DoCreateContext(__GLXclientState * cl,
> GLXContextID gcId,
>          return BadAlloc;
>      }
>  
> -    /*
> -     ** Initially, setup the part of the context that could be used
> by
> -     ** a GL core that needs windowing information (e.g., Mesa).
> +    /* Initialize the GLXcontext structure.
>       */
>      glxc->pGlxScreen = pGlxScreen;
>      glxc->config = config;
> -
> -    /*
> -     ** Register this context as a resource.
> -     */
> -    if (!AddResource(gcId, __glXContextRes, (pointer) glxc)) {
> -        (*glxc->destroy) (glxc);
> -        client->errorValue = gcId;
> -        return BadAlloc;
> -    }
> -
> -    /*
> -     ** Finally, now that everything is working, setup the rest of
> the
> -     ** context.
> -     */
>      glxc->id = gcId;
>      glxc->share_id = shareList;
>      glxc->idExists = GL_TRUE;
> @@ -319,6 +303,14 @@ DoCreateContext(__GLXclientState * cl,
> GLXContextID gcId,
>      glxc->isDirect = isDirect;
>      glxc->renderMode = GL_RENDER;
>  
> +    /* Register this context as a resource.
> +     */
> +    if (!AddResource(gcId, __glXContextRes, (pointer)glxc)) {
> +        (*glxc->destroy) (glxc);
> +        client->errorValue = gcId;
> +        return BadAlloc;
> +    }
> +    
^^^ git am, being the pedant that it is, complains about this trailing
whitespace.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part
URL: <http://lists.x.org/archives/xorg-devel/attachments/20120417/7a787159/attachment.pgp>


More information about the xorg-devel mailing list