Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rakudo aliases two containers on binding that should be distinct #2417

Closed
p6rt opened this issue Apr 28, 2011 · 5 comments
Closed

rakudo aliases two containers on binding that should be distinct #2417

p6rt opened this issue Apr 28, 2011 · 5 comments

Comments

@p6rt
Copy link

p6rt commented Apr 28, 2011

Migrated from rt.perl.org#89484 (status was 'resolved')

Searchable as RT89484$

@p6rt
Copy link
Author

p6rt commented Apr 28, 2011

From @moritz

18​:01 < moritz> rakudo​: my $x = 5; sub f(Int $y) { $x := 'foo'; say $y
}; f($x); say $x
18​:01 <+p6eval> rakudo 8533c3​: OUTPUT«foo␤foo␤»
18​:02 < TimToady> I think rebinding $x makes $x and $y point to
different things
18​:02 < pmichaud> I can agree with that
18​:03 < pmichaud> Rakudo needs to move to a two-layer system
18​:03 < moritz> so output should be 5\nfoo\n, right?
18​:03 < TimToady> I believe so
18​:03 < pmichaud> $x -> container -> 5
18​:03 * moritz submits rakudobug

@p6rt
Copy link
Author

p6rt commented Oct 19, 2011

From @jnthn

On Thu Apr 28 09​:04​:38 2011, moritz wrote​:

18​:01 < moritz> rakudo​: my $x = 5; sub f(Int $y) { $x := 'foo'; say $y
}; f($x); say $x
18​:01 <+p6eval> rakudo 8533c3​: OUTPUT«foo␤foo␤»
18​:02 < TimToady> I think rebinding $x makes $x and $y point to
different things
18​:02 < pmichaud> I can agree with that
18​:03 < pmichaud> Rakudo needs to move to a two-layer system
18​:03 < moritz> so output should be 5\nfoo\n, right?
18​:03 < TimToady> I believe so
18​:03 < pmichaud> $x -> container -> 5
18​:03 * moritz submits rakudobug

Fixed in nom​:

my $x = 5; sub f(Int $y) { $x := 'foo'; say $y }; f($x); say $x
5
foo

Tagging testneeded.

/jnthn

@p6rt
Copy link
Author

p6rt commented Oct 19, 2011

The RT System itself - Status changed from 'new' to 'open'

@p6rt
Copy link
Author

p6rt commented Feb 3, 2012

From @moritz

now tested in S03-binding/scalars.t.

@p6rt
Copy link
Author

p6rt commented Feb 3, 2012

@moritz - Status changed from 'open' to 'resolved'

@p6rt p6rt closed this as completed Feb 3, 2012
@p6rt p6rt added the testneeded label Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant