search

preservim / nerdcommenter

Vim plugin for intensely nerdy commenting powers
Creative Commons Zero v1.0 Universal
4.99k stars 450 forks source link

remove extra space #453

Closed bkbncn closed 3 years ago

bkbncn commented 3 years ago

Hi, there is extra space for python and pyrex, but not for the other languages. We should fix the inconsistence.

alerque commented 3 years ago

I understand the inconsistency and appreciate the desire to clean it up, but unfortunately I don't think we can fix it this easily. We've actually tried before and had to revert it because it broke too many people's workflow without giving any easy way to fix them.

This has been discussed ad nauseum in other issues (see #202, #278, #328, #421), but the TL;DR version is that unlike most languages the space is actually part of the syntax and cannot be optional. NERDComenter doesn't have proper support for this, only for optional spaces. Until this is addressed properly we're left in kind of an awkward situation where there are only workarounds.

I'd be happy to accept a proper fix with per-language defaults for space handling, but this PR would just break the plugin for everybody who hasn't customized their config with a work-around.

bkbncn commented 3 years ago

All right, now I understand how come the inconsistence, and agree not everyone will customize this plugin, including me, I didn't customize before.

And my workaround about python is following, inspired by #202, hopefully help others.

let g:NERDSpaceDelims = 1
let g:NERDAltDelims_python = 1