From: Jarkko Hietaniemi <jhi@iki.fi>
Date: Fri, 8 Mar 2002 21:52:51 +0000 (+0000)
Subject: New shinier lint.t (and Lint.pm) from Rafael Garcia-Suarez.
X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=commitdiff_plain;h=94011a57d204cc8b19a9e33fa13d66d74bd9ca88;p=p5sagit%2Fp5-mst-13.2.git

New shinier lint.t (and Lint.pm) from Rafael Garcia-Suarez.
(Lint.pm nit noticed by Michael Cook <michael@waxrat.com>)

p4raw-id: //depot/perl@15121
---

diff --git a/MANIFEST b/MANIFEST
index 1afa04f..de50f97 100644
--- a/MANIFEST
+++ b/MANIFEST
@@ -102,6 +102,7 @@ ext/B/t/bblock.t        See if B::Bblock works
 ext/B/t/concise.t	See whether B::Concise works
 ext/B/t/debug.t		See if B::Debug works
 ext/B/t/deparse.t	See if B::Deparse works
+ext/B/t/lint.t		See if B::Lint works
 ext/B/t/showlex.t	See if B::ShowLex works
 ext/B/t/stash.t		See if B::Stash works
 ext/B/t/terse.t		See if B::Terse works
diff --git a/ext/B/B/Lint.pm b/ext/B/B/Lint.pm
index 1510d36..0115d2f 100644
--- a/ext/B/B/Lint.pm
+++ b/ext/B/B/Lint.pm
@@ -111,6 +111,8 @@ include other package names whose subs are then checked by Lint.
 
 This is only a very preliminary version.
 
+This module doesn't work correctly on thread-enabled perls.
+
 =head1 AUTHOR
 
 Malcolm Beattie, mbeattie@sable.ox.ac.uk.
@@ -164,7 +166,7 @@ sub gimme {
     my $op = shift;
     my $flags = $op->flags;
     if ($flags & OPf_WANT) {
-	return(($flags & OPf_WANT == OPf_WANT_LIST) ? 1 : 0);
+	return(($flags & OPf_WANT) == OPf_WANT_LIST ? 1 : 0);
     }
     return undef;
 }
diff --git a/ext/B/t/lint.t b/ext/B/t/lint.t
new file mode 100644
index 0000000..f20d04d
--- /dev/null
+++ b/ext/B/t/lint.t
@@ -0,0 +1,88 @@
+#!./perl -w
+
+BEGIN {
+    chdir 't' if -d 't';
+    @INC = qw(../lib);
+    require './test.pl';
+}
+
+plan tests => 13;
+
+# Runs a separate perl interpreter with the appropriate lint options
+# turned on
+sub runlint ($$$;$) {
+    my ($opts,$prog,$result,$testname) = @_;
+    my $res = runperl(
+	switches => [ "-MO=Lint,$opts" ],
+	prog	 => $prog,
+	stderr	 => 1,
+    );
+    $res =~ s/-e syntax OK\n$//;
+    is( $res, $result, $testname || $opts );
+}
+
+runlint 'context', '$foo = @bar', <<'RESULT';
+Implicit scalar context for array in scalar assignment at -e line 1
+RESULT
+
+runlint 'context', '$foo = length @bar', <<'RESULT';
+Implicit scalar context for array in length at -e line 1
+RESULT
+
+runlint 'implicit-read', '/foo/', <<'RESULT';
+Implicit match on $_ at -e line 1
+RESULT
+
+runlint 'implicit-write', 's/foo/bar/', <<'RESULT';
+Implicit substitution on $_ at -e line 1
+RESULT
+
+SKIP : {
+
+    use Config;
+    skip("Doesn't work with threaded perls",9) if $Config{useithreads};
+
+    runlint 'implicit-read', '1 for @ARGV', <<'RESULT', 'implicit-read in foreach';
+Implicit use of $_ in foreach at -e line 1
+RESULT
+
+    runlint 'dollar-underscore', '$_ = 1', <<'RESULT';
+Use of $_ at -e line 1
+RESULT
+
+    runlint 'dollar-underscore', 'print', <<'RESULT', 'dollar-underscore in print';
+Use of $_ at -e line 1
+RESULT
+
+    runlint 'private-names', 'sub A::_f{};A::_f()', <<'RESULT';
+Illegal reference to private name _f at -e line 1
+RESULT
+
+    runlint 'private-names', '$A::_x', <<'RESULT';
+Illegal reference to private name _x at -e line 1
+RESULT
+
+    {
+	local $TODO = q/doesn't catch methods/;
+	runlint 'private-names', 'sub A::_f{};A->_f()', <<'RESULT',
+Illegal reference to private method name _f at -e line 1
+RESULT
+	'private-names';
+    }
+
+    runlint 'undefined-subs', 'foo()', <<'RESULT';
+Undefined subroutine foo called at -e line 1
+RESULT
+
+    runlint 'regexp-variables', 'print $&', <<'RESULT';
+Use of regexp variable $& at -e line 1
+RESULT
+
+    {
+	local $TODO = 'bug';
+	runlint 'regexp-variables', 's/./$&/', <<'RESULT';
+Use of regexp variable $& at -e line 1
+RESULT
+    }
+
+}