+# check for pointless casting of kmalloc return + if ($rawline =~ /\*\)[ ]k[czm]alloc/) {
It looks to me like this will catch foo = (char *) kmalloc(512); but not for = (char* )kmalloc(512); I haven't tried it but how about something like: if($rawline =~/\(.*\)\s*k[czm]alloc/){ which if I got it right should match the typecast with any combination of spacing.
+ WARN("No need to cast return value.\n");
Could the warning be more descriptive? This describes what, but it should also describe why; after all if somebody made this error they may not know they why.
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/