From 9744ef2289b606515b91f39c827a65f55c6ca909 Mon Sep 17 00:00:00 2001 From: Daniel Stenberg Date: Wed, 21 Oct 2015 13:46:03 +0200 Subject: [PATCH] checksrc: add crude // detection --- lib/checksrc.pl | 5 +++++ lib/checksrc.whitelist | 1 + 2 files changed, 6 insertions(+) diff --git a/lib/checksrc.pl b/lib/checksrc.pl index 8fad2cfcb..ddd0b3623 100755 --- a/lib/checksrc.pl +++ b/lib/checksrc.pl @@ -143,6 +143,11 @@ sub scanfile { checkwarn($line, length($1), $file, $l, "Trailing whitespace"); } + # crude attempt to detect // comments without too many false + # positives + if($l =~ /^([^"\*]*)[^:"]\/\//) { + checkwarn($line, length($1), $file, $l, "\/\/ comment"); + } # check spaces after for/if/while if($l =~ /^(.*)(for|if|while) \(/) { if($1 =~ / *\#/) { diff --git a/lib/checksrc.whitelist b/lib/checksrc.whitelist index e261b9da1..d20909bdc 100644 --- a/lib/checksrc.whitelist +++ b/lib/checksrc.whitelist @@ -8,3 +8,4 @@ return fopen(file, "r"); /* VMS */ return fopen(file, "r", "rfm=stmlf", "ctx=stm"); curl_memlog("FILE %s:%d fopen(\"%s\",\"%s\") = %p\n", + a URL as file://localhost//foo must be valid as well, to refer to