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

make auto escape optional #5

Closed
bstaton1 opened this issue Sep 11, 2019 · 0 comments
Closed

make auto escape optional #5

bstaton1 opened this issue Sep 11, 2019 · 0 comments
Labels
enhancement New feature or request

Comments

@bstaton1
Copy link
Owner

Currently, the usage of regular expressions like this are not supported:

match_p(post, "z[0-9]")

Because the [ and ] are auto-escaped using the ins_regex_bracket() function. It may be desirable to be able to use these characters in regular expressions like those presented here, as it would increase the generality of extracting particular nodes.

This may be accomplished by adding an escape = TRUE argument to all functions that call match_p(), which would also have the escape = TRUE argument. This would trigger whether ins_regex_bracket() should be called or not. The default should be TRUE to maintain compatibility with previously-written code.

@bstaton1 bstaton1 added the enhancement New feature or request label Sep 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant