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 erroneously parses some optional-parameter question marks as twigils #2081
Comments
From @masak<masak> rakudo: role A[::T?] {}; class B does A[] {}; say "alive" |
From @cokeNo change on original error in a recent-vintage nom. |
@coke - Status changed from 'new' to 'open' |
From @cokeRemoving testneeded tag - this is still erroring. |
From @usev6This looks good to me now: $ perl6-m -e 'sub foo(::T $?) {}; say "alive"' $ perl6-m -e 'role A[::T $?] {}; class B does A[] {}; say "alive"' The following dies, but that's okay (from the IRC log above: "<TimToady> you can't put ? on a type constraint, which ::T is parsed as"). $ perl6-m -e 'role A[::T?] {}; class B does A[] {}; say "alive"' I'll add those three as tests and if nobody objects I'll close this ticket. |
1 similar comment
From @usev6This looks good to me now: $ perl6-m -e 'sub foo(::T $?) {}; say "alive"' $ perl6-m -e 'role A[::T $?] {}; class B does A[] {}; say "alive"' The following dies, but that's okay (from the IRC log above: "<TimToady> you can't put ? on a type constraint, which ::T is parsed as"). $ perl6-m -e 'role A[::T?] {}; class B does A[] {}; say "alive"' I'll add those three as tests and if nobody objects I'll close this ticket. |
From @usev6The tests are in S06-signature/optional.t and S14-roles/parameterized-basic.t, respectively. They were added with the following commit: Raku/roast@2b3bee5223 |
1 similar comment
From @usev6The tests are in S06-signature/optional.t and S14-roles/parameterized-basic.t, respectively. They were added with the following commit: Raku/roast@2b3bee5223 |
@usev6 - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#77338 (status was 'resolved')
Searchable as RT77338$
The text was updated successfully, but these errors were encountered: