From 9040dab76182d1a019ca7fef7b29733d2c199e61 Mon Sep 17 00:00:00 2001 From: Tiago Vignatti Date: Fri, 28 Aug 2009 15:50:50 +0300 Subject: 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 --- xfixes/cursor.c | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) (limited to 'xfixes') 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, -- cgit v1.2.3