Skip to content

Loading…

fix bug for vim <= 7.2 #28

Merged
merged 1 commit into from

2 participants

@munen

Hi there,

let me first say big thanks for forking minibufexpl. Until a few minutes ago I was still using the version from 2004 - but in the process of upgrading all my plugins I realized that your version is far superior.

I found a very small 'bug' - the feature 'colorcolum' was only introduced in VIM 7.3 which won't be preinstalled everywhere for the next few years. I added simple feature detection for enabling it.

Thanks again for your great work!

Best regards,
Alain

@fholgado
Owner

Thanks so much for the pull request! This is something that was brought up on another pull request, and since it worked OK on my environment I went ahead and pushed it. Should have done some research, since I didn't know colorcolumn was only available for > 7.3 :(

Thanks again!

@fholgado fholgado merged commit 74e8194 into fholgado:master
@fholgado fholgado was assigned
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Showing with 4 additions and 2 deletions.
  1. +4 −2 plugin/minibufexpl.vim
View
6 plugin/minibufexpl.vim
@@ -749,8 +749,10 @@ function! <SID>StartExplorer(sticky,delBufNum,currBufName)
" No spell check
setlocal nospell
- " Restore colorcolumn
- setlocal colorcolumn&
+ " Restore colorcolumn for VIM >= 7.3
+ if has("colorcolumn")
+ setlocal colorcolumn&
+ end
if has("syntax")
syn clear
Something went wrong with that request. Please try again.