This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
[perl #97088] Prevent double get-magic in various cases
[perl5.git] / t / op / gmagic.t
CommitLineData
8d6d96c1
HS
1#!./perl -w
2
3BEGIN {
8d6d96c1
HS
4 chdir 't' if -d 't';
5 @INC = '../lib';
ead3e279 6 require './test.pl';
8d6d96c1
HS
7}
8
ead3e279 9use strict;
8d6d96c1 10
8d6d96c1
HS
11tie my $c => 'Tie::Monitor';
12
ead3e279 13sub expected_tie_calls {
40914f83 14 my ($obj, $rexp, $wexp, $tn) = @_;
ead3e279
NC
15 local $::Level = $::Level + 1;
16 my ($rgot, $wgot) = $obj->init();
40914f83
FC
17 is ($rgot, $rexp, $tn ? "number of fetches when $tn" : ());
18 is ($wgot, $wexp, $tn ? "number of stores when $tn" : ());
8d6d96c1
HS
19}
20
ead3e279 21# Use ok() instead of is(), cmp_ok() etc, to strictly control number of accesses
8d6d96c1 22my($r, $s);
ead3e279
NC
23ok($r = $c + 0 == 0, 'the thing itself');
24expected_tie_calls(tied $c, 1, 0);
25ok($r = "$c" eq '0', 'the thing itself');
26expected_tie_calls(tied $c, 1, 0);
27
28ok($c . 'x' eq '0x', 'concat');
29expected_tie_calls(tied $c, 1, 0);
30ok('x' . $c eq 'x0', 'concat');
31expected_tie_calls(tied $c, 1, 0);
8d6d96c1 32$s = $c . $c;
ead3e279
NC
33ok($s eq '00', 'concat');
34expected_tie_calls(tied $c, 2, 0);
8d6d96c1
HS
35$r = 'x';
36$s = $c = $r . 'y';
ead3e279
NC
37ok($s eq 'xy', 'concat');
38expected_tie_calls(tied $c, 1, 1);
8d6d96c1 39$s = $c = $c . 'x';
ead3e279
NC
40ok($s eq '0x', 'concat');
41expected_tie_calls(tied $c, 2, 1);
8d6d96c1 42$s = $c = 'x' . $c;
ead3e279
NC
43ok($s eq 'x0', 'concat');
44expected_tie_calls(tied $c, 2, 1);
8d6d96c1 45$s = $c = $c . $c;
ead3e279
NC
46ok($s eq '00', 'concat');
47expected_tie_calls(tied $c, 3, 1);
8d6d96c1 48
1e968d83 49$s = chop($c);
ead3e279
NC
50ok($s eq '0', 'multiple magic in core functions');
51expected_tie_calls(tied $c, 1, 1);
1e968d83 52
5cf4b255
FC
53$c = *strat;
54$s = $c;
ead3e279
NC
55ok($s eq *strat,
56 'Assignment should not ignore magic when the last thing assigned was a glob');
074ededa 57expected_tie_calls(tied $c, 1, 1);
5cf4b255 58
835c0338
GG
59# autovivication of aelem, helem, of rv2sv combined with get-magic
60{
61 my $true = 1;
62 my $s;
63 tie $$s, "Tie::Monitor";
64 $$s = undef;
65 $$s->[0] = 73;
66 is($$s->[0], 73);
67 expected_tie_calls(tied $$s, 2, 2);
68
69 my @a;
70 tie $a[0], "Tie::Monitor";
71 $a[0] = undef;
72 $a[0][0] = 73;
73 is($a[0][0], 73);
74 expected_tie_calls(tied $a[0], 2, 2);
75
76 my %h;
77 tie $h{foo}, "Tie::Monitor";
78 $h{foo} = undef;
79 $h{foo}{bar} = 73;
80 is($h{foo}{bar}, 73);
81 expected_tie_calls(tied $h{foo}, 2, 2);
82
83 # Similar tests, but with obscured autovivication by using dummy list or "?:" operator
84 {
85 local our $TODO = "Get magic is executed too often";
86 $$s = undef;
87 ${ (), $$s }[0] = 73;
88 is( $$s->[0], 73);
89 expected_tie_calls(tied $$s, 2, 2);
90 }
91
92 $$s = undef;
93 ( ! $true ? undef : $$s )->[0] = 73;
94 is( $$s->[0], 73);
95 expected_tie_calls(tied $$s, 2, 2);
96
97 {
98 local our $TODO = "Get magic is executed too often";
99 $$s = undef;
100 ( $true ? $$s : undef )->[0] = 73;
101 is( $$s->[0], 73);
102 expected_tie_calls(tied $$s, 2, 2);
103 }
104
105}
106
f64c9ac5
FC
107# A plain *foo should not call get-magic on *foo.
108# This method of scalar-tying an immutable glob relies on details of the
109# current implementation that are subject to change. This test may need to
110# be rewritten if they do change.
111my $tyre = tie $::{gelp} => 'Tie::Monitor';
112# Compilation of this eval autovivifies the *gelp glob.
113eval '$tyre->init(0); () = \*gelp';
114my($rgot, $wgot) = $tyre->init(0);
ead3e279
NC
115ok($rgot == 0, 'a plain *foo causes no get-magic');
116ok($wgot == 0, 'a plain *foo causes no set-magic');
f64c9ac5 117
767eda44
FC
118# get-magic when exiting a non-lvalue sub in potentially autovivify-
119# ing context
40914f83
FC
120{
121 no strict;
122
123 my $tied_to = tie $_{elem}, "Tie::Monitor";
124 () = sub { delete $_{elem} }->()->[3];
125 expected_tie_calls $tied_to, 1, 0,
126 'mortal magic var is implicitly returned in autoviv context';
127
128 $tied_to = tie $_{elem}, "Tie::Monitor";
129 () = sub { return delete $_{elem} }->()->[3];
130 expected_tie_calls $tied_to, 1, 0,
131 'mortal magic var is explicitly returned in autoviv context';
862b2c43
FC
132
133 $tied_to = tie $_{elem}, "Tie::Monitor";
134 my $rsub;
135 $rsub = sub { if ($_[0]) { delete $_{elem} } else { &$rsub(1)->[3] } };
136 &$rsub;
137 expected_tie_calls $tied_to, 1, 0,
138 'mortal magic var is implicitly returned in autoviv context';
139
140 $tied_to = tie $_{elem}, "Tie::Monitor";
141 $rsub = sub {
142 if ($_[0]) { return delete $_{elem} } else { &$rsub(1)->[3] }
143 };
144 &$rsub;
145 expected_tie_calls $tied_to, 1, 0,
146 'mortal magic var is explicitly returned in autoviv context';
40914f83 147}
767eda44 148
ead3e279 149done_testing();
5cf4b255 150
8d6d96c1
HS
151# adapted from Tie::Counter by Abigail
152package Tie::Monitor;
153
154sub TIESCALAR {
155 my($class, $value) = @_;
156 bless {
157 read => 0,
158 write => 0,
159 values => [ 0 ],
160 };
161}
162
163sub FETCH {
164 my $self = shift;
165 ++$self->{read};
166 $self->{values}[$#{ $self->{values} }];
167}
168
169sub STORE {
170 my($self, $value) = @_;
171 ++$self->{write};
172 push @{ $self->{values} }, $value;
173}
174
175sub init {
176 my $self = shift;
177 my @results = ($self->{read}, $self->{write});
178 $self->{read} = $self->{write} = 0;
179 $self->{values} = [ 0 ];
180 @results;
181}