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

multi-name parameter parsing too strict #3709

Closed
p6rt opened this issue Mar 1, 2015 · 4 comments
Closed

multi-name parameter parsing too strict #3709

p6rt opened this issue Mar 1, 2015 · 4 comments
Labels
Bug LHF Low-hanging fruit, easy to resolve testcommitted

Comments

@p6rt
Copy link

p6rt commented Mar 1, 2015

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

Searchable as RT123956$

@p6rt
Copy link
Author

p6rt commented Mar 1, 2015

From @Mouq

03​:26 < Mouq> m​: sub (​:t(​:$th)) {}
03​:26 <+camelia> rakudo-moar 61c59d​: ( no output )
03​:26 < Mouq> m​: sub (​:t( :$th )) {}
03​:26 <+camelia> rakudo-moar 61c59d​: OUTPUT«===SORRY!=== Error while
compiling /tmp/0yAAc9tKQn␤Unable to parse named parameter; couldnt find
right parenthesis␤at /tmp/0yAAc9tKQn​:1␤------> sub (​:t( :$th⏏ )) {}␤
expecting any of​:␤ shape declaration␤»
03​:26 < Mouq> std​: sub (​:t( :$th )) {}
03​:26 <+camelia> std 28329a7​: OUTPUT«ok 00​:00 138m␤»
03​:26 * Mouq rakudobugs

The bug is that sub (​:t( :$th )) {} should parse the same as sub (​:t(​:$th))
{}

@p6rt
Copy link
Author

p6rt commented Mar 15, 2015

From @usev6

This was fixed with commit rakudo/rakudo@7a9a4cadeb.

I added a test to S06-signature/named-renaming.t with commit Raku/roast@459be82454.

I'm closing this ticket as 'resolved'.

1 similar comment
@p6rt
Copy link
Author

p6rt commented Mar 15, 2015

From @usev6

This was fixed with commit rakudo/rakudo@7a9a4cadeb.

I added a test to S06-signature/named-renaming.t with commit Raku/roast@459be82454.

I'm closing this ticket as 'resolved'.

@p6rt p6rt closed this as completed Mar 15, 2015
@p6rt
Copy link
Author

p6rt commented Mar 15, 2015

@usev6 - Status changed from 'new' to 'resolved'

@p6rt p6rt added Bug LHF Low-hanging fruit, easy to resolve testcommitted labels Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug LHF Low-hanging fruit, easy to resolve testcommitted
Projects
None yet
Development

No branches or pull requests

1 participant