-
-
Notifications
You must be signed in to change notification settings - Fork 214
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
C multi-line comments #119
Comments
I recently tweaked that. Does it work if you go back one commit? |
No, I also tried out a several other commits going back in history and it's consistently the same. One thing I noticed
So I had a little look at the plugin and made this edit which seems to do it, I'm sure there's a cleaner way though I'm just shit with vim regex.
|
Are you setting a custom |
No, but I do have a few settings that may be related but after removing them it's still the same result, though I do get the whitespace padding, just due to
|
With stock commentstring and no plugin changes
|
I can reproduce the lack of whitespace padding if one of the lines being commented out contains a |
Yea 1.3 was good here as well, here's the bisect between
|
I don't believe so |
I get the exact same behavior myself on the current latest release, 1.3. Any chance for a fix @tpope? I really appreciate the minimalist approach when compared to other similar plugins, but the incorrect behavior with C block comments is a real issue for us low-level programmers. @natemaia bisected it here. |
Similarly, it'd be great to be able to configure line vs block comment for C files:
when running
and
I can instead create a new GitHub issue with this, any preferences? |
This gets super complicated if you want to support things like partial uncommenting. We can't just operate on the given line range; we have to parse the entire file. I don't see this happening. |
Thanks @tpope for the commit, but this is still not functional for me. This is what I get:
Visual select the paragraph and then
Maybe I misunderstood what this commit you pushed was fixing? |
Understood, thanks for the feedback. |
This is the intended behavior. The old result had a syntax error at It's not trying to remove an arbitrary multi-line comment. That's back to needing to parse the entire file. |
Right, as I suspected. Thanks for confirming! |
Currently if you have a comment
When commenting the block using
gca{
it results in a broken sectionI realize this may be asking too much and I should just stick to single line comments or blocking them off myself. Either way thank you for all the plugins you've made over the years.
Cheers, Nate
The text was updated successfully, but these errors were encountered: