* xterm.c (handle_one_xevent): Omit unnecessary casts to unsigned.
authorPaul Eggert <eggert@cs.ucla.edu>
Mon, 13 Jun 2011 05:49:35 +0000 (22:49 -0700)
committerPaul Eggert <eggert@cs.ucla.edu>
Mon, 13 Jun 2011 05:49:35 +0000 (22:49 -0700)
src/ChangeLog
src/xterm.c

index f8482b2..8e74e4a 100644 (file)
@@ -1,5 +1,7 @@
 2011-06-13  Paul Eggert  <eggert@cs.ucla.edu>
 
+       * xterm.c (handle_one_xevent): Omit unnecessary casts to unsigned.
+
        * keyboard.c (modify_event_symbol): Don't limit alist len to UINT_MAX.
 
        * lisp.h (CHAR_TABLE_SET): Omit now-redundant test.
index d6d6457..39e4abb 100644 (file)
@@ -6438,8 +6438,7 @@ handle_one_xevent (struct x_display_info *dpyinfo, XEvent *eventptr,
                            keys".  It seems there's no cleaner way.
                            Test IsModifierKey to avoid handling
                            mode_switch incorrectly.  */
-                        || ((unsigned) (keysym) >= XK_Select
-                            && (unsigned)(keysym) < XK_KP_Space)
+                        || (XK_Select <= keysym && keysym < XK_KP_Space)
 #endif
 #ifdef XK_dead_circumflex
                         || orig_keysym == XK_dead_circumflex
@@ -6492,10 +6491,8 @@ handle_one_xevent (struct x_display_info *dpyinfo, XEvent *eventptr,
                                 should be treated similarly to
                                 Mode_switch by Emacs. */
 #if defined XK_ISO_Lock && defined XK_ISO_Last_Group_Lock
-                             || ((unsigned)(orig_keysym)
-                                 >=  XK_ISO_Lock
-                                 && (unsigned)(orig_keysym)
-                                 <= XK_ISO_Last_Group_Lock)
+                             || (XK_ISO_Lock <= orig_keysym
+                                && orig_keysym <= XK_ISO_Last_Group_Lock)
 #endif
                              ))
            {