diff options
| author | Tiago Vignatti <tiago.vignatti@nokia.com> | 2009-08-28 15:50:50 +0300 |
|---|---|---|
| committer | Tiago Vignatti <tiago.vignatti@nokia.com> | 2009-08-28 15:50:50 +0300 |
| commit | 9040dab76182d1a019ca7fef7b29733d2c199e61 (patch) | |
| tree | b9f7fb18b4e48135415519d60360009e2c5797e6 | |
| parent | 80ed8096f9b8e974b556a6c9f3b600fb71b994c3 (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.c | 11 |
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, |