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

Commaless colonpair notation loses all but the first pair in declarations #3584

Closed
p6rt opened this issue Nov 15, 2014 · 5 comments
Closed

Comments

@p6rt
Copy link

p6rt commented Nov 15, 2014

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

Searchable as RT123215$

@p6rt
Copy link
Author

p6rt commented Nov 15, 2014

From @peschwa

14​:20 masak m​: say { :a(2) :b(3) :c(4) }.perl
14​:20 camelia rakudo-moar 7dd729​: OUTPUT«{"a" => 2}␤»
14​:20 masak o.O
14​:21 I postulate that this is wrong and that the output hash should have three items.
14​:21 any second opinion?
14​:21 psch masak​: i think that's already ticketed somewhere
14​:21 i remember asking about something similar some time back

Alas, I was wrong, it's apparently not yet ticketed. I'm unable to find a reference in the spec that says it should work, but a similar issue had been raised previously in [1] and #perl6 thought it was valid then and thinks it is valid still.

[1] http://irclog.perlgeek.de/perl6/2013-12-22#i_8031696

@p6rt
Copy link
Author

p6rt commented Nov 22, 2014

From @FROGGS

That's my first attempt at solving this issue​: https://gist.github.com/FROGGS/199ccc0d6412e6858217

TL;DR​:
$ perl6-m -e 'say( (​:a(2) :b(3) :c(4)) )'
"a" => 2 "b" => Mu "c" => Mu

@p6rt
Copy link
Author

p6rt commented Nov 22, 2014

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

@p6rt
Copy link
Author

p6rt commented Nov 23, 2014

From @FROGGS

Patch for (​:a(1) :b(2))​: rakudo/rakudo@55d59e461a
Patch for {​:a(1) :b(2)}​: rakudo/rakudo@c40c1b861f
Tests​: https://rt-archive.perl.org/perl6//Public/Bug/Display.html?id=123215

@p6rt
Copy link
Author

p6rt commented Nov 23, 2014

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

@p6rt p6rt closed this as completed Nov 23, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant