diff options
| author | Alan Coopersmith <alan.coopersmith@oracle.com> | 2011-12-09 10:08:55 -0800 |
|---|---|---|
| committer | Alan Coopersmith <alan.coopersmith@oracle.com> | 2011-12-12 17:03:08 -0800 |
| commit | 424dbde891486ad6a6c00c61a334031ff18f5556 (patch) | |
| tree | 1d1540f9c33ffec4ee606711e6187d47af19e7b7 /dix/cursor.c | |
| parent | b2bc38e4a553c29f49a0284333b34b4d6c8a8c12 (diff) | |
CheckForEmptyMask does not need to declare int n twice
Just use the existing n variable again in the ARGB_CURSOR loop
instead of creating another one.
Fixes gcc -Wshadow warning:
cursor.c: In function 'CheckForEmptyMask':
cursor.c:155:6: warning: declaration of 'n' shadows a previous local
cursor.c:146:9: warning: shadowed declaration is here
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Jamey Sharp <jamey@minilop.net>
Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
Diffstat (limited to 'dix/cursor.c')
| -rw-r--r-- | dix/cursor.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/dix/cursor.c b/dix/cursor.c index f29cb1125..6bff44723 100644 --- a/dix/cursor.c +++ b/dix/cursor.c @@ -152,7 +152,7 @@ CheckForEmptyMask(CursorBitsPtr bits) if (bits->argb) { CARD32 *argb = bits->argb; - int n = bits->width * bits->height; + n = bits->width * bits->height; while (n--) if (*argb++ & 0xff000000) return; } |