summaryrefslogtreecommitdiff
path: root/regen/opcodes
diff options
context:
space:
mode:
authorFather Chrysostomos <sprout@cpan.org>2011-12-18 00:00:31 -0800
committerFather Chrysostomos <sprout@cpan.org>2011-12-18 00:00:31 -0800
commit423e8af5fd21022f9107100c8561c5f880121231 (patch)
tree0e115adfbb1f1bd6923853438c9d56e73bf2083e /regen/opcodes
parent3500db162fc97aafc7cfeaa8c61f6c681810e3b4 (diff)
downloadperl-423e8af5fd21022f9107100c8561c5f880121231.tar.gz
Stop seek($glob_copy...) from clearing PL_last_in_gv
seek had the same bug as tell. Here is the commit message from 8dc99089, which fixed tell: ---------------------------------------------------------------------- Stop tell($glob_copy) from clearing PL_last_in_gv This bug is a side effect of rv2gv’s starting to return an incoercible mortal copy of a coercible glob in 5.14: $ perl5.12.4 -le 'open FH, "t/test.pl"; $fh=*FH; tell $fh; print tell' 0 $ perl5.14.0 -le 'open FH, "t/test.pl"; $fh=*FH; tell $fh; print tell' -1 In the first case, tell without arguments is returning the position of the filehandle. In the second case, tell with an explicit argument that happens to be a coercible glob (tell has an implicit rv2gv, so tell $fh is actu- ally tell *$fh) sets PL_last_in_gv to a mortal copy thereof, which is freed at the end of the statement, setting PL_last_in_gv to null. So there is no ‘last used’ handle by the time we get to the tell without arguments. This commit adds a new rv2gv flag that tells it not to copy the glob. By doing it unconditionally on the kidop, this allows tell(*$fh) to work the same way. Let’s hope nobody does tell(*{*$fh}), which will unset PL_last_in_gv because the inner * returns a mortal copy. This whole area is really icky. PL_last_in_gv should be refcounted, but that would cause handles to leak out of scope, breaking programs that rely on the auto-closing ‘feature’.
Diffstat (limited to 'regen/opcodes')
-rw-r--r--regen/opcodes2
1 files changed, 1 insertions, 1 deletions
diff --git a/regen/opcodes b/regen/opcodes
index e3c8767b6d..353bcc68d8 100644
--- a/regen/opcodes
+++ b/regen/opcodes
@@ -350,7 +350,7 @@ syswrite syswrite ck_fun imst@ F S S? S?
eof eof ck_eof is% F?
tell tell ck_tell st% F?
-seek seek ck_fun s@ F S S
+seek seek ck_tell s@ F S S
# truncate really behaves as if it had both "S S" and "F S"
truncate truncate ck_trunc is@ S S