From: Jim Cromie <jcromie@cpan.org>
Date: Wed, 1 Jun 2005 08:01:17 +0000 (-0600)
Subject: Re: [patch] teach B::Concise to see XS code
X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=commitdiff_plain;h=e75702e9a3cd2d1cb16d792e7bd1988fe77177c6;p=p5sagit%2Fp5-mst-13.2.git

Re: [patch] teach B::Concise to see XS code
Message-ID: <429DBFAD.1090308@divsol.com>

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

diff --git a/ext/B/B/Concise.pm b/ext/B/B/Concise.pm
index 2129046..9386e01 100644
--- a/ext/B/B/Concise.pm
+++ b/ext/B/B/Concise.pm
@@ -173,7 +173,7 @@ sub concise_cv_obj {
 	    print $walkHandle "coderef $name has no START\n";
 	}
 	elsif (exists &$name) {
-	    print $walkHandle "subroutine $name exists\n";
+	    print $walkHandle "$name exists in stash, but has no START\n";
 	}
 	else {
 	    print $walkHandle "$name not in symbol table\n";
diff --git a/ext/B/t/concise.t b/ext/B/t/concise.t
index 55a813d..fa696e7 100644
--- a/ext/B/t/concise.t
+++ b/ext/B/t/concise.t
@@ -15,10 +15,9 @@ BEGIN {
     }
     require 'test.pl';		# we use runperl from 'test.pl', so can't use Test::More
     sub diag { print "# @_\n" } # but this is still handy
-
 }
 
-plan tests => 147;
+plan tests => 149;
 
 require_ok("B::Concise");
 
@@ -193,7 +192,7 @@ SKIP: {
 
     my ($res,$err);
     TODO: {
-	local $TODO = "\tdoes this handling make sense ?";
+	#local $TODO = "\tdoes this handling make sense ?";
 
 	sub declared_only;
 	($res,$err) = render('-basic', \&declared_only);
@@ -215,7 +214,7 @@ SKIP: {
 	{
 	    package Bar;
 	    our $AUTOLOAD = 'garbage';
-	    sub AUTOLOAD { print "# in AUTOLOAD: $AUTOLOAD\n" }
+	    sub AUTOLOAD { print "# in AUTOLOAD body: $AUTOLOAD\n" }
 	}
 	($res,$err) = render('-basic', Bar::auto_func);
 	like ($res, qr/unknown function \(Bar::auto_func\)/,
@@ -226,7 +225,7 @@ SKIP: {
 	      "'\&Bar::auto_func' seen as having no START");
 
 	($res,$err) = render('-basic', \&Bar::AUTOLOAD);
-	like ($res, qr/called Bar::AUTOLOAD/, "found body of Bar::AUTOLOAD");
+	like ($res, qr/in AUTOLOAD body: /, "found body of Bar::AUTOLOAD");
 
     }
     ($res,$err) = render('-basic', Foo::bar);
@@ -362,5 +361,30 @@ SKIP: {
     }
 }
 
+
+# test proper NULLING of pointer, derefd by CvSTART, when a coderef is
+# undefd.  W/o this, the pointer can dangle into freed and reused
+# optree mem, which no longer points to opcodes.
+
+# Using B::Concise to render Config::AUTOLOAD's optree at BEGIN-time
+# triggers this obscure bug, cuz AUTOLOAD has a bootstrap version,
+# which is used at load-time then undeffed.  It is normally
+# re-vivified later, but not in time for this (BEGIN/CHECK)-time
+# rendering.
+
+$out = runperl ( switches => ["-MO=Concise,Config::AUTOLOAD"],
+		 prog => 'use Config; BEGIN { $Config{awk} }',
+		 stderr => 1 );
+
+like($out, qr/Config::AUTOLOAD exists in stash, but has no START/,
+    "coderef properly undefined");
+
+$out = runperl ( switches => ["-MO=Concise,Config::AUTOLOAD"],
+		 prog => 'use Config; CHECK { $Config{awk} }',
+		 stderr => 1 );
+
+like($out, qr/Config::AUTOLOAD exists in stash, but has no START/,
+    "coderef properly undefined");
+
 __END__