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

nom does not handle short form '$val but True' #2494

Closed
p6rt opened this issue Oct 5, 2011 · 6 comments
Closed

nom does not handle short form '$val but True' #2494

p6rt opened this issue Oct 5, 2011 · 6 comments

Comments

@p6rt
Copy link

p6rt commented Oct 5, 2011

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

Searchable as RT100782$

@p6rt
Copy link
Author

p6rt commented Oct 5, 2011

From @japhb

nom does not understand the short form phrases '$val but True' and '$val
but False', only the long form '$val but role { method Bool { True } }'​:

  http://irclog.perlgeek.de/perl6/2011-10-05#i_4524378

jnthn++ said he might be able to make the short forms work if a ticket
was submitted, so here it is. :-)

-'f

@p6rt
Copy link
Author

p6rt commented Oct 8, 2011

From @jnthn

On Wed Oct 05 16​:08​:03 2011, gjb@​sonic.net wrote​:

nom does not understand the short form phrases '$val but True' and '$val
but False', only the long form '$val but role { method Bool { True } }'​:

http://irclog.perlgeek.de/perl6/2011-10-05#i_4524378

jnthn++ said he might be able to make the short forms work if a ticket
was submitted, so here it is. :-)

Now implemented.

say 0 but True
0
say so 0 but True
Bool​::True

Marked as needing tests.

/jnthn

@p6rt
Copy link
Author

p6rt commented Oct 8, 2011

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

@p6rt
Copy link
Author

p6rt commented Nov 15, 2011

From @tadzik

Tests in f362fbd, closable

@p6rt
Copy link
Author

p6rt commented Nov 15, 2011

From @masak

On Tue Nov 15 08​:32​:11 2011, tadzik wrote​:

Tests in f362fbd, closable

Thanks, resolved.

@p6rt
Copy link
Author

p6rt commented Nov 15, 2011

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant