Skip to content
This repository was archived by the owner on Apr 14, 2022. It is now read-only.

Support case-insensetive matching #79

Closed
Totktonada opened this issue Mar 13, 2018 · 1 comment
Closed

Support case-insensetive matching #79

Totktonada opened this issue Mar 13, 2018 · 1 comment
Labels
enhancement New feature or request prio1
Milestone

Comments

@Totktonada
Copy link
Member

Totktonada commented Mar 13, 2018

We can use case-insensetive indexes under hood and it will be work while we really use only the indexes. But once we going to use, say, offset argument a postprocessing of objects may needed, so matching will be case-sensetive. It may be unexpected by an user.

We need to add an option to match case-insensetive or, alternatively, add separate set of arguments (like we doing with PCRE, see #74) or add to filter (which is not implemented yet). Also we can allow to redefine the compare function.

Open question: how to consider locales? I think we can use https://github.com/starwing/luautf8 for that.

@Totktonada Totktonada added enhancement New feature or request prio2 labels Mar 13, 2018
Totktonada added a commit that referenced this issue Apr 12, 2018
Totktonada added a commit that referenced this issue Apr 12, 2018
Totktonada added a commit that referenced this issue Apr 13, 2018
@Totktonada Totktonada added this to the 0.0.1 milestone Apr 17, 2018
@Totktonada Totktonada added prio1 and removed prio2 labels Apr 17, 2018
@Totktonada
Copy link
Member Author

I think PCRE with (?i) and UTF-8 support close all our needs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request prio1
Projects
None yet
Development

No branches or pull requests

1 participant