summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTiago Vignatti <tiago.vignatti@nokia.com>2009-08-28 15:50:50 +0300
committerTiago Vignatti <tiago.vignatti@nokia.com>2009-08-28 15:50:50 +0300
commit9040dab76182d1a019ca7fef7b29733d2c199e61 (patch)
treeb9f7fb18b4e48135415519d60360009e2c5797e6
parent80ed8096f9b8e974b556a6c9f3b600fb71b994c3 (diff)
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>
-rw-r--r--xfixes/cursor.c11
1 files changed, 4 insertions, 7 deletions
diff --git a/xfixes/cursor.c b/xfixes/cursor.c
index acc703a42..0c706600e 100644
--- a/xfixes/cursor.c
+++ b/xfixes/cursor.c
@@ -1046,12 +1046,11 @@ static CursorPtr
createInvisibleCursor (void)
{
CursorPtr pCursor;
- static unsigned int *psrcbits, *pmaskbits;
+ unsigned char *psrcbits, *pmaskbits;
CursorMetricRec cm;
- int rc;
- psrcbits = (unsigned int *) xalloc(4);
- pmaskbits = (unsigned int *) xalloc(4);
+ psrcbits = (unsigned char *) xalloc(4);
+ pmaskbits = (unsigned char *) xalloc(4);
if (psrcbits == NULL || pmaskbits == NULL) {
return NULL;
}
@@ -1063,9 +1062,7 @@ createInvisibleCursor (void)
cm.xhot = 0;
cm.yhot = 0;
- rc = AllocARGBCursor(
- (unsigned char *)psrcbits,
- (unsigned char *)pmaskbits,
+ AllocARGBCursor(psrcbits, pmaskbits,
NULL, &cm,
0, 0, 0,
0, 0, 0,