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

Add workaround for text properties #41

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

Conversation

nickspoons
Copy link

@nickspoons nickspoons commented Oct 16, 2020

There is a reasonably long-standing bug in Vim affecting :goto and line2byte() when text properties are present on a line: vim/vim#5930

This results in exe 'goto' line2byte(...) always ending on the column 1 of the expected line, which means breaks vim-cool when there is more than one match on a line.

This PR uses code from this workaround (Apache licensed, so I believe it's fine to copy) on lines containing text properties.

@romainl
Copy link
Owner

romainl commented Oct 16, 2020

Thank you. Could you uncomment the load guard, please?

That said, I am not 100% sure about the compatibility of vim-cool with 7.4. I will have to double check.

@nickspoons
Copy link
Author

Thank you. Could you uncomment the load guard, please?

Whoops! Sorry, yes will do, in a couple of hours.

@nickspoons
Copy link
Author

I uncommented the load guard, and amended the previous commit

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