diff options
author | Tom Ryder <tom@sanctum.geek.nz> | 2017-10-30 19:11:22 +1300 |
---|---|---|
committer | Tom Ryder <tom@sanctum.geek.nz> | 2017-10-30 19:11:22 +1300 |
commit | 1834c083aa436d59713b0054566e06648d67ac1e (patch) | |
tree | c49418442110da13aef79acfb7af29eee95ab4b1 /vim/config/whitespace.vim | |
parent | Move viminfo conf from spell.vim into new subfile (diff) | |
download | dotfiles-1834c083aa436d59713b0054566e06648d67ac1e.tar.gz dotfiles-1834c083aa436d59713b0054566e06648d67ac1e.zip |
Apply name conventions, scoping to Vim identifiers
The Google VimScript Style Guide says
<https://google.github.io/styleguide/vimscriptguide.xml#Naming>:
>In general, use plugin-names-like-this, FunctionNamesLikeThis,
>CommandNamesLikeThis, augroup_names_like_this,
>variable_names_like_this.
Adjusted variable, function, and `augroup` names accordingly, including
setting script scope for some of the functions and their calls (`s:` and
`<SID>` prefixes).
Initially I tried using `prefix#`, but it turns out that this is a
namespacing contention for publically callable functions like
`pathogen#infect`, and none of these functions need to be publically
callable.
Diffstat (limited to 'vim/config/whitespace.vim')
-rw-r--r-- | vim/config/whitespace.vim | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/vim/config/whitespace.vim b/vim/config/whitespace.vim index 119d2c48..4189d858 100644 --- a/vim/config/whitespace.vim +++ b/vim/config/whitespace.vim @@ -4,12 +4,12 @@ set nojoinspaces " Strip trailing whitespace with \x if has('eval') - function! StripTrailingWhitespace() + function! s:StripTrailingWhitespace() let l:li = 1 for l:line in getline(1,'$') call setline(l:li, substitute(l:line, '\m\C\s\+$', '', 'g')) let l:li = l:li + 1 endfor endfunction - nnoremap <silent> <leader>x :<C-U>call StripTrailingWhitespace()<CR> + nnoremap <silent> <leader>x :<C-U>call <SID>StripTrailingWhitespace()<CR> endif |