On Mon, 6 Dec 2010 14:53:18 -0800, James Jones <[email protected]> wrote:

> In preparation for adding more sync object types
> that will need Await requests of their own, factor
> out some setup and finalization code from
> ProcSyncAwait() into SyncAwaitPrologue() and
> SyncAwaitEpilogue()
> 
> Signed-off-by: James Jones <[email protected]>

> @@ -1525,7 +1569,7 @@ ProcSyncAwait(ClientPtr client)
>       if (pProtocolWaitConds->counter == None) /* XXX protocol change */
>       {
>           /*  this should take care of removing any triggers created by
> -          *  this request that have already been registered on counters
> +          *  this request that have already been registered on sync objects

> @@ -1546,7 +1590,7 @@ ProcSyncAwait(ClientPtr client)
>       if (status != Success)
>       {
>           /*  this should take care of removing any triggers created by
> -          *  this request that have already been registered on counters
> +          *  this request that have already been registered on sync objects

These two changes appear to belong in the previous patch.

Otherwise, this looks like a clean mechanical patch again.

Reviewed-by: Keith Packard <[email protected]>

-- 
[email protected]

Attachment: pgpFlM36R86My.pgp
Description: PGP signature

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to