summaryrefslogtreecommitdiffstats
path: root/perl_checker
diff options
context:
space:
mode:
authorPascal Rigaux <pixel@mandriva.com>2001-08-25 11:22:06 +0000
committerPascal Rigaux <pixel@mandriva.com>2001-08-25 11:22:06 +0000
commit39bed9aba4ff78ff5f9f98704beaa261c1c7507c (patch)
treec4fce4d71a17deb040d3255fea57302ed52d7147 /perl_checker
parent0557e702a03474955bf16acbb6fff6f5353f8b74 (diff)
downloadperl-MDK-Common-39bed9aba4ff78ff5f9f98704beaa261c1c7507c.tar
perl-MDK-Common-39bed9aba4ff78ff5f9f98704beaa261c1c7507c.tar.gz
perl-MDK-Common-39bed9aba4ff78ff5f9f98704beaa261c1c7507c.tar.bz2
perl-MDK-Common-39bed9aba4ff78ff5f9f98704beaa261c1c7507c.tar.xz
perl-MDK-Common-39bed9aba4ff78ff5f9f98704beaa261c1c7507c.zip
add checking for << and >> operators priority use
Diffstat (limited to 'perl_checker')
-rwxr-xr-xperl_checker4
1 files changed, 4 insertions, 0 deletions
diff --git a/perl_checker b/perl_checker
index fe94764..12cf3a2 100755
--- a/perl_checker
+++ b/perl_checker
@@ -67,6 +67,10 @@ if (/\bsub\b\s*\{[^{}]*\$_(?!\[)/) {
warn_(q(using $_ in a sub is generally bad news), info()) if !/local\s+\$_/;
}
+if ((my ($op) = /([<>]{2})/) && (/[+-]\s*[\w\$]+\s*[<>]{2}/ || /[<>]{2}\s*[\w\$]+\s*[+-]/) && !/\<\<[A-Z]/) { # don't warn for here-document alike
+ err(qq(parentheses needed around operator $op), info());
+}
+
if (/^\s*package ([\w:]+)/) {
$package = $1;
$path{$package} ||= ($ARGV =~ m|(.*/)|)[0];