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

unescape characters Grep Tool treats as special but Vim as literal #251

Merged
merged 1 commit into from
Jan 5, 2025

Conversation

Konfekt
Copy link
Contributor

@Konfekt Konfekt commented Aug 27, 2021

addresses #224

@Konfekt Konfekt changed the title unescape braces if necessary unescape characters Grep Tool treats as special but Vim as literal Aug 27, 2021
@Konfekt
Copy link
Contributor Author

Konfekt commented Aug 27, 2021

One could think about using very magic for the Vim search expression to economize much of this (un)escaping.

@Konfekt
Copy link
Contributor Author

Konfekt commented Nov 7, 2021

Anything left to be done? This is useful, for example, if one searched with vim-grepper for lines that match a regular expression containing \b and then wants to jump to the matches in the quickfix list.

@Konfekt
Copy link
Contributor Author

Konfekt commented Mar 12, 2022

Good evening @mhinz, @jez or @ddeville, would one of you mind taking a look at this PR that should ease searching for the grepped string in the obtained quickfix list?
At this occasion, how about @mmrwoods 's one as well?

@Konfekt
Copy link
Contributor Author

Konfekt commented Dec 28, 2024

@mhinz, @jez or @ddeville : Would you mind quick feedback on the current shortcomings of this fix? Happy to brush it up for eventual inclusion as it makes searching in the repo and quickfix list/buffers more seamless

@lbonn
Copy link
Collaborator

lbonn commented Jan 5, 2025

@Konfekt sorry for the crazy wait, I've just tested it and it's looking good.

@lbonn lbonn merged commit 09cb932 into mhinz:master Jan 5, 2025
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.

2 participants