kernel32: Remove unneeded casts

Robert Shearman rob at codeweavers.com
Thu Dec 20 10:57:27 CST 2007


Andrew Talbot wrote:
>    /* All local heap allocations are aligned on 4-byte boundaries */
>  #define LALIGN(word)          (((word) + 3) & ~3)
>  
> -#define ARENA_PTR(ptr,arena)       ((LOCALARENA *)((char*)(ptr)+(arena)))
> +#define ARENA_PTR(ptr,arena)       ((LOCALARENA *)((ptr)+(arena)))

Don't remove casts from macros like this as they then won't be safe if 
the type that is passed in is changed to something else (like void *).

-- 
Rob Shearman




More information about the wine-devel mailing list