Skip to content

Commit

Permalink
Revert the previous workaround which didn't pass all tests
Browse files Browse the repository at this point in the history
Even though the last change fixed the problem where the
property code for 'space' was different than the one for
'White_Space', it failed some of the tests, notably testing
' ' ~~ /<:space>/ which may by this problem here:
Raku/old-design-docs#118
Long discussion here:
https://irclog.perlgeek.de/moarvm/2016-12-27#i_13805707
  • Loading branch information
samcv committed Dec 27, 2016
1 parent 960eefb commit 7bbeb8c
Show file tree
Hide file tree
Showing 4 changed files with 21,380 additions and 21,367 deletions.
2 changes: 1 addition & 1 deletion src/strings/ops.c
Original file line number Diff line number Diff line change
Expand Up @@ -1657,7 +1657,7 @@ static MVMint64 grapheme_is_cclass(MVMThreadContext *tc, MVMint64 cclass, MVMGra
return 0;
}
return MVM_unicode_codepoint_has_property_value(tc, cp,
MVM_UNICODE_PROPERTY_SPACE, 1);
MVM_UNICODE_PROPERTY_WHITE_SPACE, 1);

case MVM_CCLASS_BLANK:
if (cp == '\t')
Expand Down
Loading

0 comments on commit 7bbeb8c

Please sign in to comment.