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

xfixes: minor clean ups on createInvisibleCursor() 

- unused return value
- no reason to declare static variable given the function is just called
  once
- no reason to declare different type and cast it after.

Signed-off-by: Tiago Vignatti <tiago.vignatti@nokia.com>
This commit is contained in:
Tiago Vignatti 2009-08-28 15:50:50 +03:00
parent 80ed8096f9
commit 9040dab761
1 changed files with 4 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
xfixes/cursor.c
View File

@ -1046,12 +1046,11 @@ static CursorPtr
createInvisibleCursor (void) createInvisibleCursor (void)
{ {
CursorPtr pCursor; CursorPtr pCursor;
static unsigned int *psrcbits, *pmaskbits; unsigned char *psrcbits, *pmaskbits;
CursorMetricRec cm; CursorMetricRec cm;
int rc;
psrcbits = (unsigned int *) xalloc(4); psrcbits = (unsigned char *) xalloc(4);
pmaskbits = (unsigned int *) xalloc(4); pmaskbits = (unsigned char *) xalloc(4);
if (psrcbits == NULL || pmaskbits == NULL) { if (psrcbits == NULL || pmaskbits == NULL) {
return NULL; return NULL;
} }
@ -1063,9 +1062,7 @@ createInvisibleCursor (void)
cm.xhot = 0; cm.xhot = 0;
cm.yhot = 0; cm.yhot = 0;
rc = AllocARGBCursor( AllocARGBCursor(psrcbits, pmaskbits,
(unsigned char *)psrcbits,
(unsigned char *)pmaskbits,
NULL, &cm, NULL, &cm,
0, 0, 0, 0, 0, 0,
0, 0, 0, 0, 0, 0,