The patch titled Subject: checkpatch: improve "no space necessary after cast" test has been added to the -mm tree. Its filename is checkpatch-improve-no-space-necessary-after-cast-test.patch This patch should soon appear at http://ozlabs.org/~akpm/mmots/broken-out/checkpatch-improve-no-space-necessary-after-cast-test.patch and later at http://ozlabs.org/~akpm/mmotm/broken-out/checkpatch-improve-no-space-necessary-after-cast-test.patch Before you just go and hit "reply", please: a) Consider who else should be cc'ed b) Prefer to cc a suitable mailing list as well c) Ideally: find the original patch on the mailing list and do a reply-to-all to that, adding suitable additional cc's *** Remember to use Documentation/SubmitChecklist when testing your code *** The -mm tree is included into linux-next and is updated there every 3-4 working days ------------------------------------------------------ From: Joe Perches <joe@xxxxxxxxxxx> Subject: checkpatch: improve "no space necessary after cast" test Code like: if (a < sizeof(<type>) && and { .len = sizeof(<type>) }, incorrectly emits that warning, so add more exceptions to avoid the warning. Signed-off-by: Joe Perches <joe@xxxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- scripts/checkpatch.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff -puN scripts/checkpatch.pl~checkpatch-improve-no-space-necessary-after-cast-test scripts/checkpatch.pl --- a/scripts/checkpatch.pl~checkpatch-improve-no-space-necessary-after-cast-test +++ a/scripts/checkpatch.pl @@ -2541,7 +2541,7 @@ sub process { } } - if ($line =~ /^\+.*(\w+\s*)?\(\s*$Type\s*\)[ \t]+(?!$Assignment|$Arithmetic|[,;\({\[\<\>])/ && + if ($line =~ /^\+.*(\w+\s*)?\(\s*$Type\s*\)[ \t]+(?!$Assignment|$Arithmetic|[,;:\?\(\{\}\[\<\>]|&&|\|\||\\$)/ && (!defined($1) || $1 !~ /sizeof\s*/)) { if (CHK("SPACING", "No space is necessary after a cast\n" . $herecurr) && _ Patches currently in -mm which might be from joe@xxxxxxxxxxx are printk-correct-timeout-comment-neaten-module_parm_desc.patch mm-utilc-add-kstrimdup.patch checkpatch-emit-an-error-when-using-predefined-timestamp-macros.patch checkpatch-improve-octal-permissions-tests.patch checkpatch-ignore-__pure-attribute.patch checkpatch-fix-unnecessary_kern_level-false-positive.patch checkpatch-add-check-for-keyword-boolean-in-kconfig-definitions.patch checkpatch-allow-comments-in-macros-tested-for-single-statements.patch checkpatch-update-git-commit-message.patch checkpatch-add-likely-unlikely-comparison-misuse-test.patch checkpatch-add-ability-to-coalesce-commit-descriptions-on-multiple-lines.patch checkpatch-add-types-for-other-os-typedefs.patch checkpatch-add-ability-to-fix-unnecessary-blank-lines-around-braces.patch checkpatch-improve-seq_print-seq_puts-suggestion.patch checkpatch-improve-no-space-necessary-after-cast-test.patch checkpatch-neaten-printk_ratelimited-message-position.patch checkpatch-add-strict-test-for-spaces-around-arithmetic.patch checkpatch-make-sure-a-commit-reference-description-uses-parentheses.patch fs-ufs-superc-remove-unnecessary-casting.patch linux-next.patch -- To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html