diff options
| author | Ran Benita <ran234@gmail.com> | 2014-02-08 15:30:05 +0200 |
|---|---|---|
| committer | Ran Benita <ran234@gmail.com> | 2014-02-08 15:30:05 +0200 |
| commit | 769b91c580832eea279770d8a14f8aa23bf16f34 (patch) | |
| tree | ce640c57d3baef2d4aa80631496cde8832ff0ec9 /src/x11 | |
| parent | 50b73ec0e7d8d592bd56fe0da7c012414acf2748 (diff) | |
| download | xorg-lib-libxkbcommon-769b91c580832eea279770d8a14f8aa23bf16f34.tar.gz | |
Use (1u << idx) instead of (1 << idx) where appropriate
It doesn't matter (I think), since the implicit conversion doesn't have
any effect (e.g. sign-extension). But it's better to be aware of the
type.
Signed-off-by: Ran Benita <ran234@gmail.com>
Diffstat (limited to 'src/x11')
| -rw-r--r-- | src/x11/keymap.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/x11/keymap.c b/src/x11/keymap.c index 9d680b1..8581a79 100644 --- a/src/x11/keymap.c +++ b/src/x11/keymap.c @@ -684,7 +684,7 @@ get_indicators(struct xkb_keymap *keymap, xcb_connection_t *conn, darray_resize0(keymap->leds, msb_pos(reply->which)); for (int i = 0; i < NUM_INDICATORS; i++) { - if (reply->which & (1 << i)) { + if (reply->which & (1u << i)) { xcb_xkb_indicator_map_t *wire = iter.data; struct xkb_led *led = &darray_item(keymap->leds, i); @@ -886,7 +886,7 @@ get_indicator_names(struct xkb_keymap *keymap, xcb_connection_t *conn, FAIL_UNLESS(msb_pos(reply->indicators) <= (int) darray_size(keymap->leds)); for (int i = 0; i < NUM_INDICATORS; i++) { - if (reply->indicators & (1 << i)) { + if (reply->indicators & (1u << i)) { xcb_atom_t wire = *iter; struct xkb_led *led = &darray_item(keymap->leds, i); |