From d35e08b6db911c404d7baf680dab30e8a6831bc4 Mon Sep 17 00:00:00 2001 From: Peter Hutterer Date: Thu, 18 Feb 2010 16:00:15 +1000 Subject: [PATCH] dix: remove now-erroneous comment about frozen slave devices. A direct grab on a slave device through XI2 detaches it, regardless of whether the grab is sync or async. So this comment doesn't apply to XI2 anyway. For XI1, aside from your life being miserable already, it doesn't matter as XI1 does not have a concept of attachment. You can freeze a device and if you don't freeze _all_ other devices at the same time, the master device can still happily send events to the client. Signed-off-by: Peter Hutterer Acked-by: Adam Jackson (cherry picked from commit c0d941913199ed251fceab62fcf65437f6f9bd31) Signed-off-by: Peter Hutterer --- dix/events.c | 5 ----- 1 file changed, 5 deletions(-) diff --git a/dix/events.c b/dix/events.c index 4423c1609..02079bef9 100644 --- a/dix/events.c +++ b/dix/events.c @@ -1414,11 +1414,6 @@ CheckGrabForSyncs(DeviceIntPtr thisDev, Bool thisMode, Bool otherMode) thisDev->deviceGrab.sync.other = NullGrab; } - /* - XXX: Direct slave grab won't freeze the paired master device. - The correct thing to do would be to freeze all SDs attached to the - paired master device. - */ if (IsMaster(thisDev)) { dev = GetPairedDevice(thisDev);