-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Support for equivalence classes #626
Comments
I'm not in principle against it, but this enhancement requires enhancements to the regex engine itself. If this ever happens, it will not be for a very long time. Anywho, this issue isn't really for ripgrep but for the regex engine, so I've filed the issue there instead: rust-lang/regex#404 |
Fair enough. Thanks for take time to do fill that. Let's see if I can provide a patch on that |
Please sketch out an implementation strategy before doing the work. It is a
*significant* task.
…On Oct 7, 2017 4:18 PM, "Alberto Fanjul" ***@***.***> wrote:
Fair enough. Thanks for take time to do fill that. Let's see if I can
provide a patch on that
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<#626 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAb34k0G2-lOHplWu_063isreiwCqWJHks5sp9ytgaJpZM4PxTAP>
.
|
As a resume for anyone looking at this rust tries to be agnostic about charsets, so this concept of equivalence classes is not avaliable. The only thing remotely similar is the normalization form for chars, so provide custom config for this
seems a better option, but I didn't see it as a useful contribution to ripgrep, seing #196, but for rust regex |
Is there any chance to support equivalence classes?
https://www.regular-expressions.info/posixbrackets.html#eq
here is a scce test case:
The text was updated successfully, but these errors were encountered: