On Mon, 27 Aug 2007 03:12:35 +0100, MenTaLguY <mental@...3...> wrote:
On Sat, 2007-08-25 at 19:32 +0100, Thomas Worthington wrote:
If the snooper is not set as the handler then it's never called and can't have any effect other than a little bit of code bloat. So perhaps a #ifdef X11 (assuming such a thing is available) around this would assuage any worries. I don't mean this to be a replacement for testing the code, of course, but perhaps we could at least restrict the code to the platform where it's an issue.
I'd prefer trying it everywhere. Your patch looks like a perfectly reasonable approach to globally remapping the modifiers to me; I just need to give it a final technical review before I merge it (which I plan on doing after I tie off some loose ends with gradient cleanup).
Okay, great. I do find that in any event-driven project there comes a time when a custom handler is useful, and it may pay off in future for other purposes.
Thomas