remove write-only variables
Here's a bunch of patches to remove variables that are written but never read.
In particular, there's plenty of places where we store the return-code of XLookupString
without ever checking it.
Should this be error-checked instead? I dunno, and I'm not sure I care much either way.