On 10/20/11 03:44, Dave Airlie wrote:
From: Dave Airlie<airl...@redhat.com>

same as the xnest case.

Signed-off-by: Dave Airlie<airl...@redhat.com>
---
  hw/kdrive/ephyr/ephyr.c |    4 +++-
  1 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/hw/kdrive/ephyr/ephyr.c b/hw/kdrive/ephyr/ephyr.c
index 7ebf1c2..67376cc 100644
--- a/hw/kdrive/ephyr/ephyr.c
+++ b/hw/kdrive/ephyr/ephyr.c
@@ -765,7 +765,7 @@ ephyrUpdateModifierState(unsigned int state)
  {

    DeviceIntPtr pDev = inputInfo.keyboard;
-  KeyClassPtr keyc = pDev->key;
+  KeyClassPtr keyc;
    int i;
    CARD8 mask;
    int xkb_state;
@@ -773,6 +773,8 @@ ephyrUpdateModifierState(unsigned int state)
    if (!pDev)
        return;

+  keyc = pDev->key;
+
    xkb_state = XkbStateFieldFromRec(&pDev->key->xkbInfo->state);
    state = state&  0xff;


Reviewed-by: Alan Coopersmith <alan.coopersm...@oracle.com>

I wouldn't complain should you choose to also fix the typo a few lines
later in that function at the same time:

    /* Modifier shoud be down, but isn't


--
        -Alan Coopersmith-        alan.coopersm...@oracle.com
         Oracle Solaris Platform Engineering: X Window System

_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to