Skip to content

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