frederik
/
xserver
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

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>
This commit is contained in:
Alan Coopersmith 2011-12-09 10:08:55 -08:00
parent b2bc38e4a5
commit 424dbde891
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
dix/cursor.c
View File

@ -152,7 +152,7 @@ CheckForEmptyMask(CursorBitsPtr bits)
if (bits->argb) if (bits->argb)
{ {
CARD32 *argb = bits->argb; CARD32 *argb = bits->argb;
int n = bits->width * bits->height; n = bits->width * bits->height;
while (n--) while (n--)
if (*argb++ & 0xff000000) return; if (*argb++ & 0xff000000) return;
} }