From bc7e08471c1884a5100b6e0513a006c263ec3c6b Mon Sep 17 00:00:00 2001 From: Daniel Stenberg Date: Wed, 23 Nov 2016 07:52:38 +0100 Subject: checksrc: add ASTERISKSPACE Verifies a 'char *name' style, with no space after the asterisk. --- lib/checksrc.pl | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) (limited to 'lib/checksrc.pl') diff --git a/lib/checksrc.pl b/lib/checksrc.pl index 83c38b239..9eb76f47e 100755 --- a/lib/checksrc.pl +++ b/lib/checksrc.pl @@ -55,7 +55,8 @@ my %warnings = ( 'COPYRIGHT' => 'file missing a copyright statement', 'BADCOMMAND' => 'bad !checksrc! instruction', 'UNUSEDIGNORE' => 'a warning ignore was not used', - 'OPENCOMMENT' => 'file ended with a /* comment still "open"' + 'OPENCOMMENT' => 'file ended with a /* comment still "open"', + 'ASTERISKSPACE' => 'pointer declared with space after asterisk' ); sub readwhitelist { @@ -471,6 +472,12 @@ sub scanfile { } } + # check for 'char * name' + if(($l =~ /(^.*(char|int|long|void|curl_slist|CURL|CURLM|CURLMsg|curl_httppost) *\*) (\w+)/) && ($3 ne "const")) { + checkwarn("ASTERISKSPACE", + $line, length($1), $file, $ol, + "no space after declarative asterisk"); + } $line++; $prevl = $ol; } -- cgit v1.2.3