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
multiple commas in argument list parse as a single comma, should parsefail #103
Comments
From @diakopterwhereas perl5 and pugs both give syntax error. rakudo: sub foo($v, $w?, $x?, $y?){ say $v~"|" gives: OUTPUT[1|3|||] |
From @kylehaI've added a test for this in S06-signature/optional.t #?rakudo todo 'RT# 54804' is rt54804( 1, , 3, ), '1|undef|3|undef', |
@pmichaud - Status changed from 'new' to 'open' |
From @kylehaOn Sun May 25 10:55:44 2008, diakopter.gmail.com wrote:
This should be an error. The test I wrote has changed to an |
From @moritzThe test for this now passes in the ng branch, so the ticket can be |
From martin@martinkjeldsen.dkThis isn't fixed in ng. It was just a problem with the test (it failed because eval_dies_ok wasn't |
From @moritzThe correct answer is to parsefail. |
From @bbkrOn Sat Apr 17 14:51:20 2010, moritz wrote:
On Kiev build: [16:23] <bbkr> rakudo: sub foo($v, $w?, $x?, $y?){ say still no parsefail |
From oha@oha.itSignature.pm must be changed before fixing this: --- a/src/Perl6/Compiler/Signature.pm |
From @cokeOn Sun May 25 10:55:44 2008, diakopter.gmail.com wrote:
Only a minor change in the past year: 15:59 < [Coke]> rakudo: sub foo($v, $w?, $x?, $y?){ say (now it warns on the undefined values) -- |
From @diakopterOn Fri Oct 14 13:01:09 2011, coke wrote:
seems fixed. marking testneeded. 16:57 <diakopter> rakudo: sub foo($v, $w?, $x?, $y?){ say |
From @diakopterOn Tue May 29 14:57:29 2012, diakopter wrote:
test already existed; resolving. |
@diakopter - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#54804 (status was 'resolved')
Searchable as RT54804$
The text was updated successfully, but these errors were encountered: