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
Implement \U00000 syntax handling for Unicodable #324
Conversation
t/unicodable.t
Outdated
# https://github.com/perl6/whateverable/issues/234 | ||
$t.test(「\U lookup by code」, | ||
“{$t.bot-nick}: \\U0010ffff”, | ||
/^ <me($t)>‘U+10FFFF <illegal> [] (�)’ $/); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Interesting, are you using rakudo 2018.09 or less? The result is different now: https://gist.github.com/b6d312cb369774379a5c61f5a1d7d080
bin/Unicodable.p6
Outdated
@@ -96,7 +96,7 @@ method from-numerics($query) { | |||
$query ~~ m:ignoremark/^ | |||
:i \s* | |||
[ | |||
[ | ‘u’ (.) <?{ $0[*-1].Str.uniname.match: /PLUS.*SIGN/ }> | |||
[ | [「\U」 |‘u’ (.)] <?{ $0[*-1].Str.uniname.match: /PLUS.*SIGN/ }> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the closing ]
is misplaced and should be at the end of the line, otherwise it expects something like \U+2665
but the tests expect otherwise.
By the way, I've added a helpful error (hopefully) if miniircd can't start: 59b1e4c You can also do |
Most likely an oops.
By adapting the output for rakudo 2017.10 and up.
Sorry for reopening, but I felt like salvaging it :) Thank you for your contribution! |
This probably fixes #234 but I can't figure out how to run the test suite, so I gave up on it.