diff options
| author | Tormod Volden <debian.tormod@gmail.com> | 2011-04-18 22:04:00 +0200 |
|---|---|---|
| committer | Tormod Volden <debian.tormod@gmail.com> | 2011-04-21 21:34:23 +0200 |
| commit | dad240d81ea5fa946d494467e677c6e1a3a12fd7 (patch) | |
| tree | 005a8561fe3c4661d0d089fda8ce6ca9e463f2b8 | |
| parent | a4f93e8860494285444e6b8a6ba801b9db8c9cdc (diff) | |
savage: Fix logic in waitHSync
Bit-masking the result of a boolean negation seems pointless,
so I guess this is what was intended: Only look at bit 0 of the
inStatus result.
Signed-off-by: Tormod Volden <debian.tormod@gmail.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
Reviewed-by: Alex Deucher <alexdeucher@gmail.com>
| -rw-r--r-- | src/savage_cursor.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/savage_cursor.c b/src/savage_cursor.c index 4d193eb..6364293 100644 --- a/src/savage_cursor.c +++ b/src/savage_cursor.c @@ -67,8 +67,8 @@ static void SavageSetCursorColors(ScrnInfoPtr pScrn, int bg, int fg); #define waitHSync(n) { \ int num = n; \ while (num--) { \ - while ((inStatus1()) & 0x01){};\ - while (!(inStatus1()) & 0x01){};\ + while (inStatus1() & 0x01){};\ + while (!(inStatus1() & 0x01)){};\ } \ } #define MAX_CURS 64 |