This should be useful to p9p users. There have been a few quirks irritating most of us for a while. 1) devdraw apps often think that they're the wrong size at startup. 2) When you use Alt as your modkey, and you press an accellerator, devdraw is left expecting a compose. This fixes that in cases where the accellerator causes the devdraw app to lose focus. The first, I fixed quite a long time ago, and have just submitted a patch for.

cvs diff: Diffing .
Index: devdraw.h
===================================================================
RCS file: /cvs/plan9/src/cmd/devdraw/devdraw.h,v
retrieving revision 1.1
diff -r1.1 devdraw.h
4a5,7

int alting;

Index: x11-init.c
===================================================================
RCS file: /cvs/plan9/src/cmd/devdraw/x11-init.c,v
retrieving revision 1.3
diff -r1.3 x11-init.c
62a63
        XEvent ev;
365a367,368
        XSelectInput(_x.display, _x.drawable, StructureNotifyMask);

371a375,377
        do XMaskEvent(_x.display, StructureNotifyMask, &ev);
        while(ev.type != MapNotify);

Index: x11-itrans.c
===================================================================
RCS file: /cvs/plan9/src/cmd/devdraw/x11-itrans.c,v
retrieving revision 1.4
diff -r1.4 x11-itrans.c
25a26
#include "devdraw.h"
144c145
<    static int alting, nk;
---
        static int nk;
Index: x11-srv.c
===================================================================
RCS file: /cvs/plan9/src/cmd/devdraw/x11-srv.c,v
retrieving revision 1.3
diff -r1.3 x11-srv.c
33c33
< #define Mask 
MouseMask|ExposureMask|StructureNotifyMask|KeyPressMask|EnterWindowMask|LeaveWindowMask
---
#define Mask 
MouseMask|ExposureMask|StructureNotifyMask|KeyPressMask|EnterWindowMask|LeaveWindowMask|FocusChangeMask
515a516,518
        case FocusOut:
                alting = 0;
                break;


--
Kris Maglione

The light at the end of the tunnel really is a train.

Reply via email to