-
Notifications
You must be signed in to change notification settings - Fork 4
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
UTF8 capture problem #36
Comments
@todd-richmond did you file a ticket with perl for this. This is the first time I've heard about it |
Also the engine does not get the utf8 flag set yet. #15 |
@demerphq I think this is an issue with this module, not perl, but not 100% positive |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
There is a common re::engine::* bug where RXf_MATCH_UTF8 flag is not being set on the perl regex object to ensure that all captures are correctly computed as UTF8 when the input is UTF8. There are 2 critical issues involved that are fixed by this
XS code will need to do something like this
#ifdef RXf_UTF8
if (flags & RXf_UTF8)
extflags |= RXf_MATCH_UTF8;
#else
if (SvUTF8(pattern))
extflags |= RXf_MATCH_UTF8;
#endif
The text was updated successfully, but these errors were encountered: