This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
Make untie check the FAKE flag on globs
authorFather Chrysostomos <sprout@cpan.org>
Mon, 25 Oct 2010 19:28:33 +0000 (12:28 -0700)
committerFather Chrysostomos <sprout@cpan.org>
Mon, 25 Oct 2010 20:02:57 +0000 (13:02 -0700)
This allows untie($scalar) to untie the scalar if the last assigned or
returned happened to be a typeglob.

pp_sys.c
t/op/tie.t

index 7fa9f02..2497ec2 100644 (file)
--- a/pp_sys.c
+++ b/pp_sys.c
@@ -903,7 +903,7 @@ PP(pp_untie)
     const char how = (SvTYPE(sv) == SVt_PVHV || SvTYPE(sv) == SVt_PVAV)
                ? PERL_MAGIC_tied : PERL_MAGIC_tiedscalar;
 
-    if (isGV_with_GP(sv) && !(sv = MUTABLE_SV(GvIOp(sv))))
+    if (isGV_with_GP(sv) && !SvFAKE(sv) && !(sv = MUTABLE_SV(GvIOp(sv))))
        RETPUSHYES;
 
     if ((mg = SvTIED_mg(sv, how))) {
index 5acd9a9..6bad251 100644 (file)
@@ -955,12 +955,20 @@ main
 f
 ########
 
-# tie $glob_copy vs tie *$glob_copy
-sub TIESCALAR { print "TIESCALAR\n" }
-sub TIEHANDLE{ print "TIEHANDLE\n" }
+# (un)tie $glob_copy vs (un)tie *$glob_copy
+sub TIESCALAR { print "TIESCALAR\n"; bless [] }
+sub TIEHANDLE{ print "TIEHANDLE\n"; bless [] }
+sub FETCH { print "never called\n" }
 $f = *foo;
 tie *$f, "";
 tie $f, "";
+untie $f;
+print "ok 1\n" if !tied $f;
+() = $f; # should not call FETCH
+untie *$f;
+print "ok 2\n" if !tied *foo;
 EXPECT
 TIEHANDLE
 TIESCALAR
+ok 1
+ok 2