diff options
| author | Julien Cristau <jcristau@debian.org> | 2008-10-26 13:13:21 +0100 |
|---|---|---|
| committer | Julien Cristau <jcristau@debian.org> | 2008-10-26 13:13:21 +0100 |
| commit | ffaaa1a198a77eb6800c08d4613ee1cc0b068ba0 (patch) | |
| tree | cc38de9e5db00fee850043bb9a6aaf472c2b00a3 | |
| parent | 66fd05acd6118102aaaebc230d5c03e0758c6089 (diff) | |
xfree86: fix compiler warnings in DoModalias()
The precedence of == is higher than that of &, so that code was
probably buggy.
xf86Init.c: In function 'DoModalias':
xf86Init.c:300: warning: suggest parentheses around comparison in operand of &
xf86Init.c:304: warning: suggest parentheses around comparison in operand of &
xf86Init.c:308: warning: suggest parentheses around comparison in operand of &
| -rw-r--r-- | hw/xfree86/common/xf86Init.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/hw/xfree86/common/xf86Init.c b/hw/xfree86/common/xf86Init.c index 7de7a3f49..45c116a8b 100644 --- a/hw/xfree86/common/xf86Init.c +++ b/hw/xfree86/common/xf86Init.c @@ -297,15 +297,15 @@ DoModalias(void) ErrorF("sd%08X", match->subdevice_id); /* Class */ - if (match->device_class_mask >> 16 & 0xFF == 0xFF) + if ((match->device_class_mask >> 16 & 0xFF) == 0xFF) ErrorF("bc%02X", match->device_class >> 16 & 0xFF); else ErrorF("bc*"); - if (match->device_class_mask >> 8 & 0xFF == 0xFF) + if ((match->device_class_mask >> 8 & 0xFF) == 0xFF) ErrorF("sc%02X", match->device_class >> 8 & 0xFF); else ErrorF("sc*"); - if (match->device_class_mask & 0xFF == 0xFF) + if ((match->device_class_mask & 0xFF) == 0xFF) ErrorF("i%02X*", match->device_class & 0xFF); else ErrorF("i*"); |