diff options
author | Tom Ryder <tom@sanctum.geek.nz> | 2017-11-04 22:41:15 +1300 |
---|---|---|
committer | Tom Ryder <tom@sanctum.geek.nz> | 2017-11-04 22:41:15 +1300 |
commit | 1175a6d4338b43b8521617606f44a3c29a2ec0ff (patch) | |
tree | 2c15a1f3416f4c547fd1834e2991eb60c4cf18c9 /vim/plugin/copy_linebreak.vim | |
parent | Simplify shell linting code with single vars (diff) | |
download | dotfiles-1175a6d4338b43b8521617606f44a3c29a2ec0ff.tar.gz dotfiles-1175a6d4338b43b8521617606f44a3c29a2ec0ff.zip |
Add short-circuit boilerplate to plugins
Set a g:loaded_* flag to prevent repeated reloads, and refuse to load at
all if &compatible is set or if required features are missing.
Some more accommodating plugins avoid the problems 'compatible' causes
by saving its value at startup into a script variable, setting the
option to the Vim default, and then restoring it when the plugin is
done, to prevent any of its flags from interfering in the plugin code:
let s:save_cpo = &cpo
set cpo&vim
...
let &cpo = s:save_cpo
unlet s:save_cpo
I don't want this boilerplate, so I'm going to do what Tim Pope's
modules seem to, and just have the plugin refuse to do a single thing if
'compatible' is set.
Diffstat (limited to 'vim/plugin/copy_linebreak.vim')
-rw-r--r-- | vim/plugin/copy_linebreak.vim | 50 |
1 files changed, 27 insertions, 23 deletions
diff --git a/vim/plugin/copy_linebreak.vim b/vim/plugin/copy_linebreak.vim index 5c8d5f77..faeb1617 100644 --- a/vim/plugin/copy_linebreak.vim +++ b/vim/plugin/copy_linebreak.vim @@ -6,32 +6,36 @@ " Author: Tom Ryder <tom@sanctum.geek.nz> " License: Same as Vim itself " -if has('eval') +if exists('g:loaded_copy_linebreak') + \ || !has('linebreak') + \ || &compatible + finish +endif +let g:loaded_copy_linebreak = 1 - " Define function - function! s:CopyLinebreak() +" Define function +function! s:CopyLinebreak() - " If linebreak is on, turn it off - if &l:linebreak - setlocal nolinebreak linebreak? - setlocal showbreak= - if exists('&breakindent') - setlocal nobreakindent - endif + " If linebreak is on, turn it off + if &l:linebreak + setlocal nolinebreak linebreak? + setlocal showbreak= + if exists('&breakindent') + setlocal nobreakindent + endif - " If it's off, turn it on - else - setlocal linebreak linebreak? - setlocal showbreak< - if exists('&breakindent') - setlocal breakindent - endif + " If it's off, turn it on + else + setlocal linebreak linebreak? + setlocal showbreak< + if exists('&breakindent') + setlocal breakindent endif + endif - endfunction +endfunction - " Provide mapping proxy to the function just defined - noremap <silent> <unique> - \ <Plug>CopyLinebreak - \ :<C-U>call <SID>CopyLinebreak()<CR> -endif +" Provide mapping proxy to the function just defined +noremap <silent> <unique> + \ <Plug>CopyLinebreak + \ :<C-U>call <SID>CopyLinebreak()<CR> |