27 Aug
2007
27 Aug
'07
2:12 a.m.
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).
-mental