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

degenerates: warning on whitespace with diacritics could be better #5430

Open
p6rt opened this issue Jul 6, 2016 · 4 comments
Open

degenerates: warning on whitespace with diacritics could be better #5430

p6rt opened this issue Jul 6, 2016 · 4 comments
Labels
LTA Less Than Awesome; typically an error message that could be better uni

Comments

@p6rt
Copy link

p6rt commented Jul 6, 2016

Migrated from rt.perl.org#128561 (status was 'new')

Searchable as RT128561$

@p6rt
Copy link
Author

p6rt commented Jul 6, 2016

From zefram@fysh.org

Does a space-with-diacritic grapheme count as whitespace, for the purposes
of the Perl 6 grammar? Sometimes it does, and sometimes it doesn't​:

"<foo \x[308]bar>".EVAL.perl
("foo", "bar")
"+ \x[308]3".EVAL.perl
===SORRY!=== Error while compiling /home/zefram/tmp/EVAL_1
Prefix + requires an argument, but no valid term found
at /home/zefram/tmp/EVAL_1​:1
------> +^ ?3
  expecting any of​:
  prefix
"Q \x[308]foo bar \x[308]".EVAL
===SORRY!=== Error while compiling /home/zefram/tmp/EVAL_2
Whitespace character (0x20) is not allowed as a delimiter
at /home/zefram/tmp/EVAL_2​:1
------> Q^ ?foo bar ?

I think it should be consistent.

-zefram

@p6rt
Copy link
Author

p6rt commented Dec 29, 2016

From @samcv

Looks like this is very related to RT #​130384. This is somewhat the same problem though one is with degenerates involving whitespace and one with non-whitespace characters.

@p6rt
Copy link
Author

p6rt commented Oct 7, 2017

From @samcv

Looking further at this issue, it looks like we are doing things fine. Though this still is LTA since it would be nice if it was revealed that there were combining codepoints.

@p6rt
Copy link
Author

p6rt commented Oct 7, 2017

From @samcv

Using q quoting with a space and combining character(s) will now result in the following error​:

» q ̌(hi)
Whitespace character ‘SPACE’ (0x20) with 1 combining character is not allowed as a delimiter at line 2, near " ̌(hi)\n"

Commit​: Raku/nqp@0bed230bd

@p6rt p6rt added LTA Less Than Awesome; typically an error message that could be better uni labels Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
LTA Less Than Awesome; typically an error message that could be better uni
Projects
None yet
Development

No branches or pull requests

1 participant