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

Angle-quoted Rat literals #392

Merged
merged 4 commits into from Mar 22, 2015
Merged

Angle-quoted Rat literals #392

merged 4 commits into from Mar 22, 2015

Conversation

raydiak
Copy link
Contributor

@raydiak raydiak commented Mar 20, 2015

Implements <1/2> syntax. This patch only supports the bare no-space Rat literal and does not attempt to implement val() semantics or alter the behavior of qw-ish operators. Rough first attempt, feedback welcome.

Spectest clean plus some passing TODOs in t/spec/S32-num/rat.t

http://design.perl6.org/S02.html#Rational_literals
http://irclog.perlgeek.de/perl6/2015-03-20#i_10315220

@Mouq
Copy link
Contributor

Mouq commented Mar 21, 2015

Instead of +~$<nu>, I think $<nu>.ast would be better (and probably faster). Same with $<de>. Otherwise, +1 from me :)

@raydiak
Copy link
Contributor Author

raydiak commented Mar 21, 2015

Thanks for the tip, I've update the PR

@Mouq
Copy link
Contributor

Mouq commented Mar 22, 2015

Looks good, thanks :)

Mouq added a commit that referenced this pull request Mar 22, 2015
Angle-quoted Rat literals
@Mouq Mouq merged commit af6bd02 into rakudo:nom Mar 22, 2015
@raydiak raydiak deleted the rat branch March 26, 2015 01:01
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

Successfully merging this pull request may close these issues.

None yet

2 participants