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

Use keyword_pattern for is_symbol check #1975

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

joshbode
Copy link
Contributor

@joshbode joshbode commented Jul 6, 2024

Use keyword_pattern in place of char.is_symbol to improve matching (especially useful for improving for cmp-path and cmp-cmdline matches in the command-line).

Falls back to original behaviour of using char.is_symbol if no keyword_pattern is provided/specified for a source.

@joshbode joshbode changed the title Use keyword_pattern for is_symbol check Use keyword_pattern for is_symbol check Jul 6, 2024
@joshbode joshbode force-pushed the joshbode/keyword-symbol-check branch from ba7b2b0 to ccd371a Compare July 6, 2024 00:19
@joshbode
Copy link
Contributor Author

joshbode commented Jul 6, 2024

with this change:

image

without the change, no matches

@joshbode joshbode force-pushed the joshbode/keyword-symbol-check branch from ccd371a to 7fcfde7 Compare July 16, 2024 16:14
@joshbode joshbode force-pushed the joshbode/keyword-symbol-check branch from 7fcfde7 to c525b32 Compare August 5, 2024 22:17
@joshbode joshbode force-pushed the joshbode/keyword-symbol-check branch from c525b32 to c124724 Compare October 21, 2024 20:40
@joshbode joshbode force-pushed the joshbode/keyword-symbol-check branch from c124724 to 1d2f68a Compare November 28, 2024 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant