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

xx sinks when it shouldn't #5368

Closed
p6rt opened this issue Jun 11, 2016 · 6 comments
Closed

xx sinks when it shouldn't #5368

p6rt opened this issue Jun 11, 2016 · 6 comments

Comments

@p6rt
Copy link

p6rt commented Jun 11, 2016

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

Searchable as RT128382$

@p6rt
Copy link
Author

p6rt commented Jun 11, 2016

From @lizmat

<lizmat> m​: class A { method sink() { print "sunk " } }; my @​a = A.new xx 10 # looks like assigning it is sinking?
<camelia> rakudo-moar 915f45​: OUTPUT«sunk sunk sunk sunk sunk sunk sunk sunk sunk sunk »
<lizmat> this shouldn't happen, now should it ?
<jnthn> lizmat​: That sinking looks wrong, yeah. I mean, sinking implies "this value is being thrown away", and storing it in a variable is certainly not throwing it away :)
<lizmat> jnthn​: ok, looking into it
<lizmat> actually, looks like xx is sinking it

@p6rt
Copy link
Author

p6rt commented Jun 15, 2016

From @zoffixznet

Looks like this was fixed​:

<Zoffix> m​: class A { method sink() { print "sunk " } }; my @​a = A.new xx 10
<camelia> rakudo-moar ac0dcd​: ( no output )

@p6rt
Copy link
Author

p6rt commented Jun 15, 2016

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

@p6rt
Copy link
Author

p6rt commented Jun 15, 2016

From @lizmat

On 15 Jun 2016, at 05​:33, Zoffix Znet via RT <perl6-bugs-followup@​perl.org> wrote​:

Looks like this was fixed​:

<Zoffix> m​: class A { method sink() { print "sunk " } }; my @​a = A.new xx 10
<camelia> rakudo-moar ac0dcd​: ( no output )

Indeed, fixed with 0ef135a52207d376df3

Liz

@p6rt
Copy link
Author

p6rt commented Jul 9, 2016

From @zoffixznet

Tests added​: Raku/roast@6fd8bbb9c8

@p6rt
Copy link
Author

p6rt commented Jul 9, 2016

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

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