issue with ]c and [c mapping

[mhinz]

issue with ]c and [c mapping

[mhinz]

Well, I included it because users asked for it, but I can see the problem.

I see 2 possible solutions:

1. Like you said: remap to ]g and [g instead.

2. I could add another BufEnter hook which checks if &diff is set. People just would have to use ':diffsplit %' instead of just ':diffsplit', so the buffer actually gets reloaded.

If I can't come up with a better idea until tonight, I'll do #1.

[zhaocai]

2. I could add another BufEnter hook which checks if &diff is set. People just would have to use ':diffsplit %' instead of just ':diffsplit', so the buffer actually gets reloaded.

I prefer this one, and I suggest to only map ]c,[c for signify when it is appropriate - only in a cvs repo, signify is toggled, and &diff == 0.

I hope you could also look at https://github.com/spiiph/vim-space to see if you can make this mapping compatible with motion repeating.

[mhinz]

Does one actually want Sy to be enabled at all, if &diff == 1?

Performance-wise it would be the best to map while starting Sy and unmap while stopping Sy. So if the user switches to diff mode, he'd just have to toggle Sy off.

So I could avoid adding another BufEnter hook. And it would be somewhat futureproof.

[zhaocai]

So I could avoid adding another BufEnter hook. And it would be somewhat futureproof.
turn Sy off in diff mode is good enough for me.

[mhinz]

Okay, I'm making some more refactoring and commit this evening or tomorrow.

[mhinz]

BTW, spiiph used to be a regular on IRC, but got a bit inactive in the last time. I'll look into space.vim.