[PATCH 02/22] dix: Make some LBX-era code more obvious

Adam Jackson ajax at redhat.com
Tue Jul 22 08:46:22 PDT 2014


isItTimeToYield in the conditional effectively didn't do anything here.
Take it out, and remove the comment since LBX proxies aren't a thing for
us anymore.

Signed-off-by: Adam Jackson <ajax at redhat.com>
---
 dix/dispatch.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/dix/dispatch.c b/dix/dispatch.c
index 4f830f7..f7a08f8 100644
--- a/dix/dispatch.c
+++ b/dix/dispatch.c
@@ -3188,13 +3188,11 @@ ProcKillClient(ClientPtr client)
     rc = dixLookupClient(&killclient, stuff->id, client, DixDestroyAccess);
     if (rc == Success) {
         CloseDownClient(killclient);
-        /* if an LBX proxy gets killed, isItTimeToYield will be set */
-        if (isItTimeToYield || (client == killclient)) {
+        if (client == killclient) {
             /* force yield and return Success, so that Dispatch()
              * doesn't try to touch client
              */
             isItTimeToYield = TRUE;
-            return Success;
         }
         return Success;
     }
-- 
1.9.3



More information about the xorg-devel mailing list