XkbSelectEventDetails: remove unnecessary assignments

clear & selectAll are set to 0 already a few lines earlier,
affectWhich is set to XkbMapNotifyMask a few lines later.
None are used between the other assignments and the removed ones.

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
_XkbReadGeomOverlay: check for NULL first, then use pointer

Flagged by cppcheck 1.62:
[lib/libX11/src/xkb/XKBGeom.c:479] -> [lib/libX11/src/xkb/XKBGeom.c:480]:
 (warning) Possible null pointer dereference: row - otherwise it is
 redundant to check it against null.

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Remove unused ETEST & ESZTEST macros from XlibInt.c

Left behind when 15e5eaf62897 removed support for building without XCB.

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Jasper St. Pierre <jstpierre@mecheye.net>