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

failing test in S06-multi/subsignature.t: wrong multi candidate called when slurpy and named are passed #4771

Open
p6rt opened this issue Nov 21, 2015 · 4 comments
Labels
JVM Related to Rakudo-JVM

Comments

@p6rt
Copy link

p6rt commented Nov 21, 2015

Migrated from rt.perl.org#126702 (status was 'open')

Searchable as RT126702$

@p6rt
Copy link
Author

p6rt commented Nov 21, 2015

From @usev6

The following code does not give the expected result ('2') on rakudo.jvm​:

$ perl6-j -e 'multi catch(| (*@​all ) ) { 1 }; multi catch(| (*@​all, :$really! ) ) { 2 }; say catch(0, 5, :!really)'
1

@p6rt
Copy link
Author

p6rt commented Jan 28, 2018

From @peschwa

On Sat, 21 Nov 2015 06​:12​:07 -0800, bartolin@​gmx.de wrote​:

The following code does not give the expected result ('2') on
rakudo.jvm​:

$ perl6-j -e 'multi catch(| (*@​all ) ) { 1 }; multi catch(| (*@​all,
:$really! ) ) { 2 }; say catch(0, 5, :!really)'
1

Neither the mentioned test file nor the specific code example seem to fail on current Rakudo as well as the currently availabe camelia build (which is a92950fb4). With confirmation I'd request closing this ticket, as it refers to existing tests that used to fail but don't anymore.

@p6rt
Copy link
Author

p6rt commented Jan 28, 2018

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

@p6rt
Copy link
Author

p6rt commented Mar 10, 2018

From @usev6

On Sat, 27 Jan 2018 17​:19​:18 -0800, peschwa@​gmail.com wrote​:

On Sat, 21 Nov 2015 06​:12​:07 -0800, bartolin@​gmx.de wrote​:

The following code does not give the expected result ('2') on
rakudo.jvm​:

$ perl6-j -e 'multi catch(| (*@​all ) ) { 1 }; multi catch(| (*@​all,
:$really! ) ) { 2 }; say catch(0, 5, :!really)'
1

Neither the mentioned test file nor the specific code example seem to
fail on current Rakudo as well as the currently availabe camelia build
(which is a92950fb4). With confirmation I'd request closing this
ticket, as it refers to existing tests that used to fail but don't
anymore.

rakudo-j is still wrong (also tested with current HEAD (26522e8acd))​:

<bartolin> r​: multi catch(| (*@​all ) ) { 1 }; multi catch(| (*@​all, :$really! ) ) { 2 }; say catch(0, 5, :!really)
<camelia> rakudo-moar 65874b155​: OUTPUT​: «2␤»
<camelia> ..rakudo-jvm a92950fb4​: OUTPUT​: «1␤»

The test in S06-multi/subsignature.t is fudged todo, that's why the testfile passes. Maybe this comment was meant for a different ticket?

@p6rt p6rt added the JVM Related to Rakudo-JVM label Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
JVM Related to Rakudo-JVM
Projects
None yet
Development

No branches or pull requests

1 participant