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

Port to Vim9 script #2

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Port to Vim9 script #2

wants to merge 2 commits into from

Conversation

niklasl
Copy link

@niklasl niklasl commented Oct 27, 2024

I love this plugin, and took a quick stab at porting it to Vim9. It worked out well (after some trial and error), and since Vim9 is so heavily optimized, the rainbow got even more blazingly fast for free.

I open this as a PR, but I fully understand and respect that you support both Vim and NeoVim, and that this might not be merged at all. Think of this as much as a "thank you" in code, and hopefully as inspiration. I will in no way be offended if you close with WONTFIX.

(I also recommend to do the porting exercise for anyone to get to know both your plugin and Vim9 script.)

Using the `s:` form was supported in beta but not in released Vim 9.
@sedm0784
Copy link
Owner

Oh hey, that's really cool! Not sure whether or not I'll merge it, but I will certainly try it out and consider it: I was curious as to whether Vim9 script's performance improvements would have much effect on this plugin, and it would be nice if the timing values affected the actual behaviour a bit more predictably.

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