From 6e9eb56edb2c5ec48e6daac0b46219c8a2bd24c0 Mon Sep 17 00:00:00 2001
From: Tom Ryder <tom@sanctum.geek.nz>
Date: Sun, 12 Nov 2017 14:42:59 +1300
Subject: Move lots of local Vim config into vim/after

This is a relatively drastic change that should have been done
progressively, but I got carried away in ripping everything out and
putting it back in again.

Reading the documentation for writing a Vim script (:help usr_41.txt), I
am convinced that all of the content that was in the vim/ftplugin
directory and some of the vim/indent directory actually belonged in
vim/after/ftplugin and vim/after/indent respectively.

This is because the section on filetypes makes the distinction between
replacing the core filetype or indent plugins and merely adding to or
editing them after the fact; from :help ftplugin:

> If you do want to use the default plugin, but overrule one of the
> settings, you can write the different setting in a script:
>
>         setlocal textwidth=70
>
> Now write this in the "after" directory, so that it gets sourced after
> the distributed "vim.vim" ftplugin after-directory.  For Unix this
> would be "~/.vim/after/ftplugin/vim.vim".  Note that the default
> plugin will have set "b:did_ftplugin", but it is ignored here.

Therefore, I have deleted the user_indent.vim and user_ftplugin.vim
plugins and their documentation that I wrote, and their ftplugin.vim and
indent.vim shims in ~/.vim, in an attempt to make these plugins
elegantly undo-ready, and instead embraced the way the documentation and
$VIMRUNTIME structure seems to suggest.

I broke the ftplugin files up by function and put them under
subdirectories of vim/after named by filetype, as the 'runtimepath'
layout permits. In doing so, I also carefully applied the
documentation's advice:

* Short-circuiting repeated loads
* Checking for existing mappings using the <Plug> prefix approach
* Avoiding repeated function declarations overwriting each other
* Guarding against 'cpotions' mangling things (by simply
  short-circuiting if 'compatible' is set).

I've made the b:undo_ftplugin and b:undo_indent commands less forgiving,
and append commands to it inline with the initial establishment of the
setup they're reversing, including checking that the b:undo_* variable
actually exists in the first place.

For the indentation scripts, however, three of the four files originally
in vim/indent actually do belong there:

1. csv.vim, because it doesn't have an indent file in the core.
2. tsv.vim, because it doesn't have an indent file in the core.
3. php.vim, because it does what ftplugins are allowed to do in
   preventing the core indent rules from running at all.

The indent/vim.vim rules, however, have been moved to
after/indent/vim.vim, because the tweaks it makes for two-space
indentation are designed to supplement the core indent rules, not
replace them.

Finally, I've adjusted Makefile targets accordingly for the above, given
the vim/ftplugin directory is now empty and there are three new
directories in vim/after to install. We wrap these under a single
`install-vim-after` parent target for convenience. The
`install-vim-after-ftplugin` target accommodates the additional level of
filetype directories beneath it.
---
 vim/after/ftplugin/html/lint.vim          | 29 +++++++++++++++++++
 vim/after/ftplugin/html/tidy.vim          | 29 +++++++++++++++++++
 vim/after/ftplugin/html/url_link.vim      | 47 +++++++++++++++++++++++++++++++
 vim/after/ftplugin/mail/format_flowed.vim | 13 +++++++++
 vim/after/ftplugin/markdown/spell.vim     | 15 ++++++++++
 vim/after/ftplugin/perl/check.vim         | 29 +++++++++++++++++++
 vim/after/ftplugin/perl/lint.vim          | 29 +++++++++++++++++++
 vim/after/ftplugin/perl/tidy.vim          | 29 +++++++++++++++++++
 vim/after/ftplugin/sh/bash_han.vim        | 15 ++++++++++
 vim/after/ftplugin/sh/check.vim           | 40 ++++++++++++++++++++++++++
 vim/after/ftplugin/sh/lint.vim            | 40 ++++++++++++++++++++++++++
 vim/after/ftplugin/sh/posix.vim           | 25 ++++++++++++++++
 vim/after/ftplugin/text/spell.vim         | 15 ++++++++++
 vim/after/ftplugin/vim/lint.vim           | 29 +++++++++++++++++++
 vim/after/indent/vim.vim                  |  5 ++++
 15 files changed, 389 insertions(+)
 create mode 100644 vim/after/ftplugin/html/lint.vim
 create mode 100644 vim/after/ftplugin/html/tidy.vim
 create mode 100644 vim/after/ftplugin/html/url_link.vim
 create mode 100644 vim/after/ftplugin/mail/format_flowed.vim
 create mode 100644 vim/after/ftplugin/markdown/spell.vim
 create mode 100644 vim/after/ftplugin/perl/check.vim
 create mode 100644 vim/after/ftplugin/perl/lint.vim
 create mode 100644 vim/after/ftplugin/perl/tidy.vim
 create mode 100644 vim/after/ftplugin/sh/bash_han.vim
 create mode 100644 vim/after/ftplugin/sh/check.vim
 create mode 100644 vim/after/ftplugin/sh/lint.vim
 create mode 100644 vim/after/ftplugin/sh/posix.vim
 create mode 100644 vim/after/ftplugin/text/spell.vim
 create mode 100644 vim/after/ftplugin/vim/lint.vim
 create mode 100644 vim/after/indent/vim.vim

(limited to 'vim/after')

diff --git a/vim/after/ftplugin/html/lint.vim b/vim/after/ftplugin/html/lint.vim
new file mode 100644
index 00000000..f6648056
--- /dev/null
+++ b/vim/after/ftplugin/html/lint.vim
@@ -0,0 +1,29 @@
+" Only do this when not done yet for this buffer
+" Also do nothing if 'compatible' enabled
+if exists('b:did_ftplugin_html_lint') || &compatible
+  finish
+endif
+let b:did_ftplugin_html_lint = 1
+let b:undo_ftplugin = b:undo_ftplugin
+      \ . '|unlet b:did_ftplugin_html_lint'
+
+" Set up a mapping for the linter, if we're allowed
+if !exists('g:no_plugin_maps') && !exists('g:no_html_maps')
+
+  " Define a mapping target
+  nnoremap <buffer> <silent> <unique>
+        \ <Plug>HtmlLint
+        \ :<C-U>write !tidy -errors -quiet<CR>
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|nunmap <buffer> <Plug>HtmlLint'
+
+  " If there isn't a key mapping already, use a default one
+  if !hasmapto('<Plug>HtmlLint')
+    nmap <buffer> <unique>
+          \ <LocalLeader>l
+          \ <Plug>HtmlLint
+    let b:undo_ftplugin = b:undo_ftplugin
+          \ . '|nunmap <buffer> <LocalLeader>l'
+  endif
+
+endif
diff --git a/vim/after/ftplugin/html/tidy.vim b/vim/after/ftplugin/html/tidy.vim
new file mode 100644
index 00000000..e5d24541
--- /dev/null
+++ b/vim/after/ftplugin/html/tidy.vim
@@ -0,0 +1,29 @@
+" Only do this when not done yet for this buffer
+" Also do nothing if 'compatible' enabled
+if exists('b:did_ftplugin_html_tidy') || &compatible
+  finish
+endif
+let b:did_ftplugin_html_tidy = 1
+let b:undo_ftplugin = b:undo_ftplugin
+      \ . '|unlet b:did_ftplugin_html_tidy'
+
+" Set up a mapping for the tidier, if we're allowed
+if !exists('g:no_plugin_maps') && !exists('g:no_html_maps')
+
+  " Define a mapping target
+  nnoremap <buffer> <silent> <unique>
+        \ <Plug>HtmlTidy
+        \ :<C-U>%!tidy -quiet<CR>
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|nunmap <buffer> <Plug>HtmlTidy'
+
+  " If there isn't a key mapping already, use a default one
+  if !hasmapto('<Plug>HtmlTidy')
+    nmap <buffer> <unique>
+          \ <LocalLeader>t
+          \ <Plug>HtmlTidy
+    let b:undo_ftplugin = b:undo_ftplugin
+          \ . '|nunmap <buffer> <LocalLeader>t'
+  endif
+
+endif
diff --git a/vim/after/ftplugin/html/url_link.vim b/vim/after/ftplugin/html/url_link.vim
new file mode 100644
index 00000000..cb0d6253
--- /dev/null
+++ b/vim/after/ftplugin/html/url_link.vim
@@ -0,0 +1,47 @@
+" Only do this when not done yet for this buffer
+" Also do nothing if 'compatible' enabled
+if exists('b:did_ftplugin_html_url_link') || &compatible
+  finish
+endif
+let b:did_ftplugin_html_url_link = 1
+let b:undo_ftplugin = b:undo_ftplugin
+      \ . '|unlet b:did_ftplugin_html_url_link'
+
+" Make a bare URL into a link to itself
+if !exists('*s:UrlLink')
+  function! s:UrlLink()
+
+    " Yank this whole whitespace-separated word
+    normal! yiW
+    " Open a link tag
+    normal! i<a href="">
+    " Paste the URL into the quotes
+    normal! hP
+    " Move to the end of the link text URL
+    normal! E
+    " Close the link tag
+    normal! a</a>
+
+  endfunction
+endif
+
+" Set up a mapping for the function, if we're allowed
+if !exists('g:no_plugin_maps') && !exists('g:no_html_maps')
+
+  " Define a mapping target
+  nnoremap <buffer> <silent> <unique>
+        \ <Plug>HtmlUrlLink
+        \ :<C-U>call <SID>HtmlUrlLink()<CR>
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|nunmap <buffer> <Plug>HtmlUrlLink'
+
+  " If there isn't a key mapping already, use a default one
+  if !hasmapto('<Plug>HtmlUrlLink')
+    nmap <buffer> <unique>
+          \ <LocalLeader>r
+          \ <Plug>HtmlUrlLink
+    let b:undo_ftplugin = b:undo_ftplugin
+          \ . '|nunmap <buffer> <LocalLeader>r'
+  endif
+
+endif
diff --git a/vim/after/ftplugin/mail/format_flowed.vim b/vim/after/ftplugin/mail/format_flowed.vim
new file mode 100644
index 00000000..b1b308fb
--- /dev/null
+++ b/vim/after/ftplugin/mail/format_flowed.vim
@@ -0,0 +1,13 @@
+" Only do this when not done yet for this buffer
+" Also do nothing if 'compatible' enabled
+if exists('b:did_ftplugin_mail_format_flowed') || &compatible
+  finish
+endif
+let b:did_ftplugin_mail_format_flowed = 1
+let b:undo_ftplugin = b:undo_ftplugin
+      \ . '|unlet b:did_ftplugin_mail_format_flowed'
+
+" Use trailing whitespace to denote continued paragraph
+setlocal formatoptions+=w
+let b:undo_ftplugin = b:undo_ftplugin
+      \ . '|setlocal formatoptions<'
diff --git a/vim/after/ftplugin/markdown/spell.vim b/vim/after/ftplugin/markdown/spell.vim
new file mode 100644
index 00000000..8f9ce4a4
--- /dev/null
+++ b/vim/after/ftplugin/markdown/spell.vim
@@ -0,0 +1,15 @@
+" Only do this when not done yet for this buffer
+" Also do nothing if 'compatible' enabled
+if exists('b:did_ftplugin_markdown_spell') || &compatible
+  finish
+endif
+let b:did_ftplugin_markdown_spell = 1
+let b:undo_ftplugin = b:undo_ftplugin
+      \ . '|unlet b:did_ftplugin_markdown_spell'
+
+" Spellcheck documents by default
+if has('syntax')
+  setlocal spell
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|setlocal spell<'
+endif
diff --git a/vim/after/ftplugin/perl/check.vim b/vim/after/ftplugin/perl/check.vim
new file mode 100644
index 00000000..24a174ff
--- /dev/null
+++ b/vim/after/ftplugin/perl/check.vim
@@ -0,0 +1,29 @@
+" Only do this when not done yet for this buffer
+" Also do nothing if 'compatible' enabled
+if exists('b:did_ftplugin_perl_check') || &compatible
+  finish
+endif
+let b:did_ftplugin_perl_check = 1
+let b:undo_ftplugin = b:undo_ftplugin
+      \ . '|unlet b:did_ftplugin_perl_check'
+
+" Set up a mapping for the checker, if we're allowed
+if !exists('g:no_plugin_maps') && !exists('g:no_perl_maps')
+
+  " Define a mapping target
+  nnoremap <buffer> <silent> <unique>
+        \ <Plug>PerlCheck
+        \ :<C-U>write !perl -c<CR>
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|nunmap <buffer> <Plug>PerlCheck'
+
+  " If there isn't a key mapping already, use a default one
+  if !hasmapto('<Plug>PerlCheck')
+    nmap <buffer> <unique>
+          \ <LocalLeader>c
+          \ <Plug>PerlCheck
+    let b:undo_ftplugin = b:undo_ftplugin
+          \ . '|nunmap <buffer> <LocalLeader>c'
+  endif
+
+endif
diff --git a/vim/after/ftplugin/perl/lint.vim b/vim/after/ftplugin/perl/lint.vim
new file mode 100644
index 00000000..8f6915f4
--- /dev/null
+++ b/vim/after/ftplugin/perl/lint.vim
@@ -0,0 +1,29 @@
+" Only do this when not done yet for this buffer
+" Also do nothing if 'compatible' enabled
+if exists('b:did_ftplugin_perl_lint') || &compatible
+  finish
+endif
+let b:did_ftplugin_perl_lint = 1
+let b:undo_ftplugin = b:undo_ftplugin
+      \ . '|unlet b:did_ftplugin_perl_lint'
+
+" Set up a mapping for the linter, if we're allowed
+if !exists('g:no_plugin_maps') && !exists('g:no_perl_maps')
+
+  " Define a mapping target
+  nnoremap <buffer> <silent> <unique>
+        \ <Plug>PerlLint
+        \ :<C-U>write !perlcritic<CR>
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|nunmap <buffer> <Plug>PerlLint'
+
+  " If there isn't a key mapping already, use a default one
+  if !hasmapto('<Plug>PerlLint')
+    nmap <buffer> <unique>
+          \ <LocalLeader>l
+          \ <Plug>PerlLint
+    let b:undo_ftplugin = b:undo_ftplugin
+          \ . '|nunmap <buffer> <LocalLeader>l'
+  endif
+
+endif
diff --git a/vim/after/ftplugin/perl/tidy.vim b/vim/after/ftplugin/perl/tidy.vim
new file mode 100644
index 00000000..b2aa25c3
--- /dev/null
+++ b/vim/after/ftplugin/perl/tidy.vim
@@ -0,0 +1,29 @@
+" Only do this when not done yet for this buffer
+" Also do nothing if 'compatible' enabled
+if exists('b:did_ftplugin_perl_tidy') || &compatible
+  finish
+endif
+let b:did_ftplugin_perl_tidy = 1
+let b:undo_ftplugin = b:undo_ftplugin
+      \ . '|unlet b:did_ftplugin_perl_tidy'
+
+" Set up a mapping for the tidier, if we're allowed
+if !exists('g:no_plugin_maps') && !exists('g:no_perl_maps')
+
+  " Define a mapping target
+  nnoremap <buffer> <silent> <unique>
+        \ <Plug>PerlTidy
+        \ :<C-U>%!perltidy<CR>
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|nunmap <buffer> <Plug>PerlTidy'
+
+  " If there isn't a key mapping already, use a default one
+  if !hasmapto('<Plug>PerlTidy')
+    nmap <buffer> <unique>
+          \ <LocalLeader>t
+          \ <Plug>PerlTidy
+    let b:undo_ftplugin = b:undo_ftplugin
+          \ . '|nunmap <buffer> <LocalLeader>t'
+  endif
+
+endif
diff --git a/vim/after/ftplugin/sh/bash_han.vim b/vim/after/ftplugin/sh/bash_han.vim
new file mode 100644
index 00000000..c1997d7e
--- /dev/null
+++ b/vim/after/ftplugin/sh/bash_han.vim
@@ -0,0 +1,15 @@
+" Only do this when not done yet for this buffer
+" Also do nothing if 'compatible' enabled
+if exists('b:did_ftplugin_sh_bash_han') || &compatible
+  finish
+endif
+let b:did_ftplugin_sh_bash_han = 1
+let b:undo_ftplugin = b:undo_ftplugin
+      \ . '|unlet b:did_ftplugin_sh_bash_han'
+
+" Use han(1df) as a man(1) wrapper for Bash files if available
+if exists('b:is_bash') && executable('han')
+  setlocal keywordprg=han
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|setlocal keywordprg<'
+endif
diff --git a/vim/after/ftplugin/sh/check.vim b/vim/after/ftplugin/sh/check.vim
new file mode 100644
index 00000000..5000137d
--- /dev/null
+++ b/vim/after/ftplugin/sh/check.vim
@@ -0,0 +1,40 @@
+" Only do this when not done yet for this buffer
+" Also do nothing if 'compatible' enabled
+if exists('b:did_ftplugin_sh_check') || &compatible
+  finish
+endif
+let b:did_ftplugin_sh_check = 1
+let b:undo_ftplugin = b:undo_ftplugin
+      \ . '|unlet b:did_ftplugin_sh_check'
+
+" Choose checker based on shell family
+if exists('b:is_bash')
+  let b:sh_check = 'write !bash -n'
+elseif exists('b:is_kornshell')
+  let b:sh_check = 'write !ksh -n'
+else
+  let b:sh_check = 'write !sh -n'
+endif
+let b:undo_ftplugin = b:undo_ftplugin
+      \ . '|unlet b:sh_check'
+
+" Set up a mapping for the checker, if we're allowed
+if !exists('g:no_plugin_maps') && !exists('g:no_sh_maps')
+
+  " Define a mapping target
+  nnoremap <buffer> <silent> <unique>
+        \ <Plug>ShCheck
+        \ :<C-U>execute b:sh_check<CR>
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|nunmap <buffer> <Plug>ShCheck'
+
+  " If there isn't a key mapping already, use a default one
+  if !hasmapto('<Plug>ShCheck')
+    nmap <buffer> <unique>
+          \ <LocalLeader>c
+          \ <Plug>ShCheck
+    let b:undo_ftplugin = b:undo_ftplugin
+          \ . '|nunmap <buffer> <LocalLeader>c'
+  endif
+
+endif
diff --git a/vim/after/ftplugin/sh/lint.vim b/vim/after/ftplugin/sh/lint.vim
new file mode 100644
index 00000000..54f86bea
--- /dev/null
+++ b/vim/after/ftplugin/sh/lint.vim
@@ -0,0 +1,40 @@
+" Only do this when not done yet for this buffer
+" Also do nothing if 'compatible' enabled
+if exists('b:did_ftplugin_sh_lint') || &compatible
+  finish
+endif
+let b:did_ftplugin_sh_lint = 1
+let b:undo_ftplugin = b:undo_ftplugin
+      \ . '|unlet b:did_ftplugin_sh_lint'
+
+" Choose linter based on shell family
+if exists('b:is_bash')
+  let b:sh_lint = 'write !shellcheck -s bash -'
+elseif exists('b:is_kornshell')
+  let b:sh_lint = 'write !shellcheck -s ksh -'
+else
+  let b:sh_lint = 'write !shellcheck -s sh -'
+endif
+let b:undo_ftplugin = b:undo_ftplugin
+      \ . '|unlet b:sh_lint'
+
+" Set up a mapping for the linter, if we're allowed
+if !exists('g:no_plugin_maps') && !exists('g:no_sh_maps')
+
+  " Define a mapping target
+  nnoremap <buffer> <silent> <unique>
+        \ <Plug>ShLint
+        \ :<C-U>execute b:sh_lint<CR>
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|nunmap <buffer> <Plug>ShLint'
+
+  " If there isn't a key mapping already, use a default one
+  if !hasmapto('<Plug>ShLint')
+    nmap <buffer> <unique>
+          \ <LocalLeader>l
+          \ <Plug>ShLint
+    let b:undo_ftplugin = b:undo_ftplugin
+          \ . '|nunmap <buffer> <LocalLeader>l'
+  endif
+
+endif
diff --git a/vim/after/ftplugin/sh/posix.vim b/vim/after/ftplugin/sh/posix.vim
new file mode 100644
index 00000000..ef6b9b93
--- /dev/null
+++ b/vim/after/ftplugin/sh/posix.vim
@@ -0,0 +1,25 @@
+" Only do this when not done yet for this buffer
+" Also do nothing if 'compatible' enabled
+if exists('b:did_ftplugin_sh_posix') || &compatible
+  finish
+endif
+let b:did_ftplugin_sh_posix = 1
+let b:undo_ftplugin = b:undo_ftplugin
+      \ . '|unlet b:did_ftplugin_sh_posix'
+
+"
+" If we have a #!/bin/sh shebang and filetype.vim determined we were neither
+" POSIX nor Bash nor Korn shell, we'll guess POSIX, just because it's far more
+" likely that's what I want to write than plain Bourne shell.
+"
+" You're supposed to be able to do this by setting g:is_posix, but if that's
+" set, the syntax file ends up setting g:is_kornshell for you too, for reasons
+" I don't really understand. This method works though, and is cleaner than
+" the other workaround I had been trying.
+"
+if exists('b:is_sh')
+  unlet b:is_sh
+  if !exists('b:is_bash') && !exists('b:is_kornshell')
+    let b:is_posix = 1
+  endif
+endif
diff --git a/vim/after/ftplugin/text/spell.vim b/vim/after/ftplugin/text/spell.vim
new file mode 100644
index 00000000..107723e5
--- /dev/null
+++ b/vim/after/ftplugin/text/spell.vim
@@ -0,0 +1,15 @@
+" Only do this when not done yet for this buffer
+" Also do nothing if 'compatible' enabled
+if exists('b:did_ftplugin_text_spell') || &compatible
+  finish
+endif
+let b:did_ftplugin_text_spell = 1
+let b:undo_ftplugin = b:undo_ftplugin
+      \ . '|unlet b:did_ftplugin_text_spell'
+
+" Spellcheck documents by default
+if has('syntax')
+  setlocal spell
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|setlocal spell<'
+endif
diff --git a/vim/after/ftplugin/vim/lint.vim b/vim/after/ftplugin/vim/lint.vim
new file mode 100644
index 00000000..5e4ea601
--- /dev/null
+++ b/vim/after/ftplugin/vim/lint.vim
@@ -0,0 +1,29 @@
+" Only do this when not done yet for this buffer
+" Also do nothing if 'compatible' enabled
+if exists('b:did_ftplugin_vim_lint') || &compatible
+  finish
+endif
+let b:did_ftplugin_vim_lint = 1
+let b:undo_ftplugin = b:undo_ftplugin
+      \ . '|unlet b:did_ftplugin_vim_lint'
+
+" Set up a mapping for the linter, if we're allowed
+if !exists('g:no_plugin_maps') && !exists('g:no_vim_maps')
+
+  " Define a mapping target
+  nnoremap <buffer> <silent> <unique>
+        \ <Plug>VimLint
+        \ :<C-U>write !vint -s /dev/stdin<CR>
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|nunmap <buffer> <Plug>VimLint'
+
+  " If there isn't a key mapping already, use a default one
+  if !hasmapto('<Plug>VimLint')
+    nmap <buffer> <unique>
+          \ <LocalLeader>l
+          \ <Plug>VimLint
+    let b:undo_ftplugin = b:undo_ftplugin
+          \ . '|nunmap <buffer> <LocalLeader>l'
+  endif
+
+endif
diff --git a/vim/after/indent/vim.vim b/vim/after/indent/vim.vim
new file mode 100644
index 00000000..b87964a4
--- /dev/null
+++ b/vim/after/indent/vim.vim
@@ -0,0 +1,5 @@
+" Observe VimL conventions for two-space indents
+setlocal shiftwidth=2
+setlocal softtabstop=2
+let b:undo_indent = b:undo_indent
+      \ . '|setlocal shiftwidth< softtabstop<'
-- 
cgit v1.2.3


From 6f0336f247705aeddc24b6c340e72ce81ca2a500 Mon Sep 17 00:00:00 2001
From: Tom Ryder <tom@sanctum.geek.nz>
Date: Sun, 12 Nov 2017 16:32:44 +1300
Subject: Exclude SC1090 (failed source) shellcheck error

This error seems to be raised when ShellCheck can't source a file
because its filename is not known until runtime. I don't want it to do
that anyway, so I've just excluded it by default.
---
 vim/after/ftplugin/sh/lint.vim | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

(limited to 'vim/after')

diff --git a/vim/after/ftplugin/sh/lint.vim b/vim/after/ftplugin/sh/lint.vim
index 54f86bea..b6377966 100644
--- a/vim/after/ftplugin/sh/lint.vim
+++ b/vim/after/ftplugin/sh/lint.vim
@@ -9,11 +9,11 @@ let b:undo_ftplugin = b:undo_ftplugin
 
 " Choose linter based on shell family
 if exists('b:is_bash')
-  let b:sh_lint = 'write !shellcheck -s bash -'
+  let b:sh_lint = 'write !shellcheck -e SC1090 -s bash -'
 elseif exists('b:is_kornshell')
-  let b:sh_lint = 'write !shellcheck -s ksh -'
+  let b:sh_lint = 'write !shellcheck -e SC1090 -s ksh -'
 else
-  let b:sh_lint = 'write !shellcheck -s sh -'
+  let b:sh_lint = 'write !shellcheck -e SC1090 -s sh -'
 endif
 let b:undo_ftplugin = b:undo_ftplugin
       \ . '|unlet b:sh_lint'
-- 
cgit v1.2.3


From a2281bd121949f2215ee9257e28e2c0966f3f003 Mon Sep 17 00:00:00 2001
From: Tom Ryder <tom@sanctum.geek.nz>
Date: Sun, 12 Nov 2017 16:38:09 +1300
Subject: Add guards for presence of b:undo_* var

This variable is not set in older Vims (early 6.x), and I think it's
worth guarding for.
---
 vim/after/ftplugin/html/lint.vim          | 20 ++++++++++++++------
 vim/after/ftplugin/html/tidy.vim          | 18 ++++++++++++------
 vim/after/ftplugin/html/url_link.vim      | 18 ++++++++++++------
 vim/after/ftplugin/mail/format_flowed.vim | 12 ++++++++----
 vim/after/ftplugin/markdown/spell.vim     | 12 ++++++++----
 vim/after/ftplugin/perl/check.vim         | 18 ++++++++++++------
 vim/after/ftplugin/perl/lint.vim          | 18 ++++++++++++------
 vim/after/ftplugin/perl/tidy.vim          | 18 ++++++++++++------
 vim/after/ftplugin/sh/bash_han.vim        | 12 ++++++++----
 vim/after/ftplugin/sh/check.vim           | 24 ++++++++++++++++--------
 vim/after/ftplugin/sh/lint.vim            | 24 ++++++++++++++++--------
 vim/after/ftplugin/sh/posix.vim           |  6 ++++--
 vim/after/ftplugin/text/spell.vim         | 12 ++++++++----
 vim/after/ftplugin/vim/lint.vim           | 18 ++++++++++++------
 vim/after/indent/vim.vim                  |  6 ++++--
 15 files changed, 158 insertions(+), 78 deletions(-)

(limited to 'vim/after')

diff --git a/vim/after/ftplugin/html/lint.vim b/vim/after/ftplugin/html/lint.vim
index f6648056..3ac760ed 100644
--- a/vim/after/ftplugin/html/lint.vim
+++ b/vim/after/ftplugin/html/lint.vim
@@ -4,8 +4,12 @@ if exists('b:did_ftplugin_html_lint') || &compatible
   finish
 endif
 let b:did_ftplugin_html_lint = 1
-let b:undo_ftplugin = b:undo_ftplugin
-      \ . '|unlet b:did_ftplugin_html_lint'
+
+" Initialise undo variable if not already done
+if exists('b:undo_ftplugin')
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|unlet b:did_ftplugin_html_lint'
+endif
 
 " Set up a mapping for the linter, if we're allowed
 if !exists('g:no_plugin_maps') && !exists('g:no_html_maps')
@@ -14,16 +18,20 @@ if !exists('g:no_plugin_maps') && !exists('g:no_html_maps')
   nnoremap <buffer> <silent> <unique>
         \ <Plug>HtmlLint
         \ :<C-U>write !tidy -errors -quiet<CR>
-  let b:undo_ftplugin = b:undo_ftplugin
-        \ . '|nunmap <buffer> <Plug>HtmlLint'
+  if exists('b:undo_ftplugin')
+    let b:undo_ftplugin = b:undo_ftplugin
+          \ . '|nunmap <buffer> <Plug>HtmlLint'
+  endif
 
   " If there isn't a key mapping already, use a default one
   if !hasmapto('<Plug>HtmlLint')
     nmap <buffer> <unique>
           \ <LocalLeader>l
           \ <Plug>HtmlLint
-    let b:undo_ftplugin = b:undo_ftplugin
-          \ . '|nunmap <buffer> <LocalLeader>l'
+    if exists('b:undo_ftplugin')
+      let b:undo_ftplugin = b:undo_ftplugin
+            \ . '|nunmap <buffer> <LocalLeader>l'
+    endif
   endif
 
 endif
diff --git a/vim/after/ftplugin/html/tidy.vim b/vim/after/ftplugin/html/tidy.vim
index e5d24541..519a7cd6 100644
--- a/vim/after/ftplugin/html/tidy.vim
+++ b/vim/after/ftplugin/html/tidy.vim
@@ -4,8 +4,10 @@ if exists('b:did_ftplugin_html_tidy') || &compatible
   finish
 endif
 let b:did_ftplugin_html_tidy = 1
-let b:undo_ftplugin = b:undo_ftplugin
-      \ . '|unlet b:did_ftplugin_html_tidy'
+if exists('b:undo_ftplugin')
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|unlet b:did_ftplugin_html_tidy'
+endif
 
 " Set up a mapping for the tidier, if we're allowed
 if !exists('g:no_plugin_maps') && !exists('g:no_html_maps')
@@ -14,16 +16,20 @@ if !exists('g:no_plugin_maps') && !exists('g:no_html_maps')
   nnoremap <buffer> <silent> <unique>
         \ <Plug>HtmlTidy
         \ :<C-U>%!tidy -quiet<CR>
-  let b:undo_ftplugin = b:undo_ftplugin
-        \ . '|nunmap <buffer> <Plug>HtmlTidy'
+  if exists('b:undo_ftplugin')
+    let b:undo_ftplugin = b:undo_ftplugin
+          \ . '|nunmap <buffer> <Plug>HtmlTidy'
+  endif
 
   " If there isn't a key mapping already, use a default one
   if !hasmapto('<Plug>HtmlTidy')
     nmap <buffer> <unique>
           \ <LocalLeader>t
           \ <Plug>HtmlTidy
-    let b:undo_ftplugin = b:undo_ftplugin
-          \ . '|nunmap <buffer> <LocalLeader>t'
+    if exists('b:undo_ftplugin')
+      let b:undo_ftplugin = b:undo_ftplugin
+            \ . '|nunmap <buffer> <LocalLeader>t'
+    endif
   endif
 
 endif
diff --git a/vim/after/ftplugin/html/url_link.vim b/vim/after/ftplugin/html/url_link.vim
index cb0d6253..4f2d2526 100644
--- a/vim/after/ftplugin/html/url_link.vim
+++ b/vim/after/ftplugin/html/url_link.vim
@@ -4,8 +4,10 @@ if exists('b:did_ftplugin_html_url_link') || &compatible
   finish
 endif
 let b:did_ftplugin_html_url_link = 1
-let b:undo_ftplugin = b:undo_ftplugin
-      \ . '|unlet b:did_ftplugin_html_url_link'
+if exists('b:undo_ftplugin')
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|unlet b:did_ftplugin_html_url_link'
+endif
 
 " Make a bare URL into a link to itself
 if !exists('*s:UrlLink')
@@ -32,16 +34,20 @@ if !exists('g:no_plugin_maps') && !exists('g:no_html_maps')
   nnoremap <buffer> <silent> <unique>
         \ <Plug>HtmlUrlLink
         \ :<C-U>call <SID>HtmlUrlLink()<CR>
-  let b:undo_ftplugin = b:undo_ftplugin
-        \ . '|nunmap <buffer> <Plug>HtmlUrlLink'
+  if exists('b:undo_ftplugin')
+    let b:undo_ftplugin = b:undo_ftplugin
+          \ . '|nunmap <buffer> <Plug>HtmlUrlLink'
+  endif
 
   " If there isn't a key mapping already, use a default one
   if !hasmapto('<Plug>HtmlUrlLink')
     nmap <buffer> <unique>
           \ <LocalLeader>r
           \ <Plug>HtmlUrlLink
-    let b:undo_ftplugin = b:undo_ftplugin
-          \ . '|nunmap <buffer> <LocalLeader>r'
+    if exists('b:undo_ftplugin')
+      let b:undo_ftplugin = b:undo_ftplugin
+            \ . '|nunmap <buffer> <LocalLeader>r'
+    endif
   endif
 
 endif
diff --git a/vim/after/ftplugin/mail/format_flowed.vim b/vim/after/ftplugin/mail/format_flowed.vim
index b1b308fb..040a1a51 100644
--- a/vim/after/ftplugin/mail/format_flowed.vim
+++ b/vim/after/ftplugin/mail/format_flowed.vim
@@ -4,10 +4,14 @@ if exists('b:did_ftplugin_mail_format_flowed') || &compatible
   finish
 endif
 let b:did_ftplugin_mail_format_flowed = 1
-let b:undo_ftplugin = b:undo_ftplugin
-      \ . '|unlet b:did_ftplugin_mail_format_flowed'
+if exists('b:undo_ftplugin')
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|unlet b:did_ftplugin_mail_format_flowed'
+endif
 
 " Use trailing whitespace to denote continued paragraph
 setlocal formatoptions+=w
-let b:undo_ftplugin = b:undo_ftplugin
-      \ . '|setlocal formatoptions<'
+if exists('b:undo_ftplugin')
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|setlocal formatoptions<'
+endif
diff --git a/vim/after/ftplugin/markdown/spell.vim b/vim/after/ftplugin/markdown/spell.vim
index 8f9ce4a4..bb344374 100644
--- a/vim/after/ftplugin/markdown/spell.vim
+++ b/vim/after/ftplugin/markdown/spell.vim
@@ -4,12 +4,16 @@ if exists('b:did_ftplugin_markdown_spell') || &compatible
   finish
 endif
 let b:did_ftplugin_markdown_spell = 1
-let b:undo_ftplugin = b:undo_ftplugin
-      \ . '|unlet b:did_ftplugin_markdown_spell'
+if exists('b:undo_ftplugin')
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|unlet b:did_ftplugin_markdown_spell'
+endif
 
 " Spellcheck documents by default
 if has('syntax')
   setlocal spell
-  let b:undo_ftplugin = b:undo_ftplugin
-        \ . '|setlocal spell<'
+  if exists('b:undo_ftplugin')
+    let b:undo_ftplugin = b:undo_ftplugin
+          \ . '|setlocal spell<'
+  endif
 endif
diff --git a/vim/after/ftplugin/perl/check.vim b/vim/after/ftplugin/perl/check.vim
index 24a174ff..6b057c82 100644
--- a/vim/after/ftplugin/perl/check.vim
+++ b/vim/after/ftplugin/perl/check.vim
@@ -4,8 +4,10 @@ if exists('b:did_ftplugin_perl_check') || &compatible
   finish
 endif
 let b:did_ftplugin_perl_check = 1
-let b:undo_ftplugin = b:undo_ftplugin
-      \ . '|unlet b:did_ftplugin_perl_check'
+if exists('b:undo_ftplugin')
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|unlet b:did_ftplugin_perl_check'
+endif
 
 " Set up a mapping for the checker, if we're allowed
 if !exists('g:no_plugin_maps') && !exists('g:no_perl_maps')
@@ -14,16 +16,20 @@ if !exists('g:no_plugin_maps') && !exists('g:no_perl_maps')
   nnoremap <buffer> <silent> <unique>
         \ <Plug>PerlCheck
         \ :<C-U>write !perl -c<CR>
-  let b:undo_ftplugin = b:undo_ftplugin
-        \ . '|nunmap <buffer> <Plug>PerlCheck'
+  if exists('b:undo_ftplugin')
+    let b:undo_ftplugin = b:undo_ftplugin
+          \ . '|nunmap <buffer> <Plug>PerlCheck'
+  endif
 
   " If there isn't a key mapping already, use a default one
   if !hasmapto('<Plug>PerlCheck')
     nmap <buffer> <unique>
           \ <LocalLeader>c
           \ <Plug>PerlCheck
-    let b:undo_ftplugin = b:undo_ftplugin
-          \ . '|nunmap <buffer> <LocalLeader>c'
+    if exists('b:undo_ftplugin')
+      let b:undo_ftplugin = b:undo_ftplugin
+            \ . '|nunmap <buffer> <LocalLeader>c'
+    endif
   endif
 
 endif
diff --git a/vim/after/ftplugin/perl/lint.vim b/vim/after/ftplugin/perl/lint.vim
index 8f6915f4..86740c81 100644
--- a/vim/after/ftplugin/perl/lint.vim
+++ b/vim/after/ftplugin/perl/lint.vim
@@ -4,8 +4,10 @@ if exists('b:did_ftplugin_perl_lint') || &compatible
   finish
 endif
 let b:did_ftplugin_perl_lint = 1
-let b:undo_ftplugin = b:undo_ftplugin
-      \ . '|unlet b:did_ftplugin_perl_lint'
+if exists('b:undo_ftplugin')
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|unlet b:did_ftplugin_perl_lint'
+endif
 
 " Set up a mapping for the linter, if we're allowed
 if !exists('g:no_plugin_maps') && !exists('g:no_perl_maps')
@@ -14,16 +16,20 @@ if !exists('g:no_plugin_maps') && !exists('g:no_perl_maps')
   nnoremap <buffer> <silent> <unique>
         \ <Plug>PerlLint
         \ :<C-U>write !perlcritic<CR>
-  let b:undo_ftplugin = b:undo_ftplugin
-        \ . '|nunmap <buffer> <Plug>PerlLint'
+  if exists('b:undo_ftplugin')
+    let b:undo_ftplugin = b:undo_ftplugin
+          \ . '|nunmap <buffer> <Plug>PerlLint'
+  endif
 
   " If there isn't a key mapping already, use a default one
   if !hasmapto('<Plug>PerlLint')
     nmap <buffer> <unique>
           \ <LocalLeader>l
           \ <Plug>PerlLint
-    let b:undo_ftplugin = b:undo_ftplugin
-          \ . '|nunmap <buffer> <LocalLeader>l'
+    if exists('b:undo_ftplugin')
+      let b:undo_ftplugin = b:undo_ftplugin
+            \ . '|nunmap <buffer> <LocalLeader>l'
+    endif
   endif
 
 endif
diff --git a/vim/after/ftplugin/perl/tidy.vim b/vim/after/ftplugin/perl/tidy.vim
index b2aa25c3..18033a42 100644
--- a/vim/after/ftplugin/perl/tidy.vim
+++ b/vim/after/ftplugin/perl/tidy.vim
@@ -4,8 +4,10 @@ if exists('b:did_ftplugin_perl_tidy') || &compatible
   finish
 endif
 let b:did_ftplugin_perl_tidy = 1
-let b:undo_ftplugin = b:undo_ftplugin
-      \ . '|unlet b:did_ftplugin_perl_tidy'
+if exists('b:undo_ftplugin')
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|unlet b:did_ftplugin_perl_tidy'
+endif
 
 " Set up a mapping for the tidier, if we're allowed
 if !exists('g:no_plugin_maps') && !exists('g:no_perl_maps')
@@ -14,16 +16,20 @@ if !exists('g:no_plugin_maps') && !exists('g:no_perl_maps')
   nnoremap <buffer> <silent> <unique>
         \ <Plug>PerlTidy
         \ :<C-U>%!perltidy<CR>
-  let b:undo_ftplugin = b:undo_ftplugin
-        \ . '|nunmap <buffer> <Plug>PerlTidy'
+  if exists('b:undo_ftplugin')
+    let b:undo_ftplugin = b:undo_ftplugin
+          \ . '|nunmap <buffer> <Plug>PerlTidy'
+  endif
 
   " If there isn't a key mapping already, use a default one
   if !hasmapto('<Plug>PerlTidy')
     nmap <buffer> <unique>
           \ <LocalLeader>t
           \ <Plug>PerlTidy
-    let b:undo_ftplugin = b:undo_ftplugin
-          \ . '|nunmap <buffer> <LocalLeader>t'
+    if exists('b:undo_ftplugin')
+      let b:undo_ftplugin = b:undo_ftplugin
+            \ . '|nunmap <buffer> <LocalLeader>t'
+    endif
   endif
 
 endif
diff --git a/vim/after/ftplugin/sh/bash_han.vim b/vim/after/ftplugin/sh/bash_han.vim
index c1997d7e..a160165c 100644
--- a/vim/after/ftplugin/sh/bash_han.vim
+++ b/vim/after/ftplugin/sh/bash_han.vim
@@ -4,12 +4,16 @@ if exists('b:did_ftplugin_sh_bash_han') || &compatible
   finish
 endif
 let b:did_ftplugin_sh_bash_han = 1
-let b:undo_ftplugin = b:undo_ftplugin
-      \ . '|unlet b:did_ftplugin_sh_bash_han'
+if exists('b:undo_ftplugin')
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|unlet b:did_ftplugin_sh_bash_han'
+endif
 
 " Use han(1df) as a man(1) wrapper for Bash files if available
 if exists('b:is_bash') && executable('han')
   setlocal keywordprg=han
-  let b:undo_ftplugin = b:undo_ftplugin
-        \ . '|setlocal keywordprg<'
+  if exists('b:undo_ftplugin')
+    let b:undo_ftplugin = b:undo_ftplugin
+          \ . '|setlocal keywordprg<'
+  endif
 endif
diff --git a/vim/after/ftplugin/sh/check.vim b/vim/after/ftplugin/sh/check.vim
index 5000137d..334ec1db 100644
--- a/vim/after/ftplugin/sh/check.vim
+++ b/vim/after/ftplugin/sh/check.vim
@@ -4,8 +4,10 @@ if exists('b:did_ftplugin_sh_check') || &compatible
   finish
 endif
 let b:did_ftplugin_sh_check = 1
-let b:undo_ftplugin = b:undo_ftplugin
-      \ . '|unlet b:did_ftplugin_sh_check'
+if exists('b:undo_ftplugin')
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|unlet b:did_ftplugin_sh_check'
+endif
 
 " Choose checker based on shell family
 if exists('b:is_bash')
@@ -15,8 +17,10 @@ elseif exists('b:is_kornshell')
 else
   let b:sh_check = 'write !sh -n'
 endif
-let b:undo_ftplugin = b:undo_ftplugin
-      \ . '|unlet b:sh_check'
+if exists('b:undo_ftplugin')
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|unlet b:sh_check'
+endif
 
 " Set up a mapping for the checker, if we're allowed
 if !exists('g:no_plugin_maps') && !exists('g:no_sh_maps')
@@ -25,16 +29,20 @@ if !exists('g:no_plugin_maps') && !exists('g:no_sh_maps')
   nnoremap <buffer> <silent> <unique>
         \ <Plug>ShCheck
         \ :<C-U>execute b:sh_check<CR>
-  let b:undo_ftplugin = b:undo_ftplugin
-        \ . '|nunmap <buffer> <Plug>ShCheck'
+  if exists('b:undo_ftplugin')
+    let b:undo_ftplugin = b:undo_ftplugin
+          \ . '|nunmap <buffer> <Plug>ShCheck'
+  endif
 
   " If there isn't a key mapping already, use a default one
   if !hasmapto('<Plug>ShCheck')
     nmap <buffer> <unique>
           \ <LocalLeader>c
           \ <Plug>ShCheck
-    let b:undo_ftplugin = b:undo_ftplugin
-          \ . '|nunmap <buffer> <LocalLeader>c'
+    if exists('b:undo_ftplugin')
+      let b:undo_ftplugin = b:undo_ftplugin
+            \ . '|nunmap <buffer> <LocalLeader>c'
+    endif
   endif
 
 endif
diff --git a/vim/after/ftplugin/sh/lint.vim b/vim/after/ftplugin/sh/lint.vim
index b6377966..8745a31e 100644
--- a/vim/after/ftplugin/sh/lint.vim
+++ b/vim/after/ftplugin/sh/lint.vim
@@ -4,8 +4,10 @@ if exists('b:did_ftplugin_sh_lint') || &compatible
   finish
 endif
 let b:did_ftplugin_sh_lint = 1
-let b:undo_ftplugin = b:undo_ftplugin
-      \ . '|unlet b:did_ftplugin_sh_lint'
+if exists('b:undo_ftplugin')
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|unlet b:did_ftplugin_sh_lint'
+endif
 
 " Choose linter based on shell family
 if exists('b:is_bash')
@@ -15,8 +17,10 @@ elseif exists('b:is_kornshell')
 else
   let b:sh_lint = 'write !shellcheck -e SC1090 -s sh -'
 endif
-let b:undo_ftplugin = b:undo_ftplugin
-      \ . '|unlet b:sh_lint'
+if exists('b:undo_ftplugin')
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|unlet b:sh_lint'
+endif
 
 " Set up a mapping for the linter, if we're allowed
 if !exists('g:no_plugin_maps') && !exists('g:no_sh_maps')
@@ -25,16 +29,20 @@ if !exists('g:no_plugin_maps') && !exists('g:no_sh_maps')
   nnoremap <buffer> <silent> <unique>
         \ <Plug>ShLint
         \ :<C-U>execute b:sh_lint<CR>
-  let b:undo_ftplugin = b:undo_ftplugin
-        \ . '|nunmap <buffer> <Plug>ShLint'
+  if exists('b:undo_ftplugin')
+    let b:undo_ftplugin = b:undo_ftplugin
+          \ . '|nunmap <buffer> <Plug>ShLint'
+  endif
 
   " If there isn't a key mapping already, use a default one
   if !hasmapto('<Plug>ShLint')
     nmap <buffer> <unique>
           \ <LocalLeader>l
           \ <Plug>ShLint
-    let b:undo_ftplugin = b:undo_ftplugin
-          \ . '|nunmap <buffer> <LocalLeader>l'
+    if exists('b:undo_ftplugin')
+      let b:undo_ftplugin = b:undo_ftplugin
+            \ . '|nunmap <buffer> <LocalLeader>l'
+    endif
   endif
 
 endif
diff --git a/vim/after/ftplugin/sh/posix.vim b/vim/after/ftplugin/sh/posix.vim
index ef6b9b93..a1b2c7ff 100644
--- a/vim/after/ftplugin/sh/posix.vim
+++ b/vim/after/ftplugin/sh/posix.vim
@@ -4,8 +4,10 @@ if exists('b:did_ftplugin_sh_posix') || &compatible
   finish
 endif
 let b:did_ftplugin_sh_posix = 1
-let b:undo_ftplugin = b:undo_ftplugin
-      \ . '|unlet b:did_ftplugin_sh_posix'
+if exists('b:undo_ftplugin')
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|unlet b:did_ftplugin_sh_posix'
+endif
 
 "
 " If we have a #!/bin/sh shebang and filetype.vim determined we were neither
diff --git a/vim/after/ftplugin/text/spell.vim b/vim/after/ftplugin/text/spell.vim
index 107723e5..322090ca 100644
--- a/vim/after/ftplugin/text/spell.vim
+++ b/vim/after/ftplugin/text/spell.vim
@@ -4,12 +4,16 @@ if exists('b:did_ftplugin_text_spell') || &compatible
   finish
 endif
 let b:did_ftplugin_text_spell = 1
-let b:undo_ftplugin = b:undo_ftplugin
-      \ . '|unlet b:did_ftplugin_text_spell'
+if exists('b:undo_ftplugin')
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|unlet b:did_ftplugin_text_spell'
+endif
 
 " Spellcheck documents by default
 if has('syntax')
   setlocal spell
-  let b:undo_ftplugin = b:undo_ftplugin
-        \ . '|setlocal spell<'
+  if exists('b:undo_ftplugin')
+    let b:undo_ftplugin = b:undo_ftplugin
+          \ . '|setlocal spell<'
+  endif
 endif
diff --git a/vim/after/ftplugin/vim/lint.vim b/vim/after/ftplugin/vim/lint.vim
index 5e4ea601..1b557593 100644
--- a/vim/after/ftplugin/vim/lint.vim
+++ b/vim/after/ftplugin/vim/lint.vim
@@ -4,8 +4,10 @@ if exists('b:did_ftplugin_vim_lint') || &compatible
   finish
 endif
 let b:did_ftplugin_vim_lint = 1
-let b:undo_ftplugin = b:undo_ftplugin
-      \ . '|unlet b:did_ftplugin_vim_lint'
+if exists('b:undo_ftplugin')
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|unlet b:did_ftplugin_vim_lint'
+endif
 
 " Set up a mapping for the linter, if we're allowed
 if !exists('g:no_plugin_maps') && !exists('g:no_vim_maps')
@@ -14,16 +16,20 @@ if !exists('g:no_plugin_maps') && !exists('g:no_vim_maps')
   nnoremap <buffer> <silent> <unique>
         \ <Plug>VimLint
         \ :<C-U>write !vint -s /dev/stdin<CR>
-  let b:undo_ftplugin = b:undo_ftplugin
-        \ . '|nunmap <buffer> <Plug>VimLint'
+  if exists('b:undo_ftplugin')
+    let b:undo_ftplugin = b:undo_ftplugin
+          \ . '|nunmap <buffer> <Plug>VimLint'
+  endif
 
   " If there isn't a key mapping already, use a default one
   if !hasmapto('<Plug>VimLint')
     nmap <buffer> <unique>
           \ <LocalLeader>l
           \ <Plug>VimLint
-    let b:undo_ftplugin = b:undo_ftplugin
-          \ . '|nunmap <buffer> <LocalLeader>l'
+    if exists('b:undo_ftplugin')
+      let b:undo_ftplugin = b:undo_ftplugin
+            \ . '|nunmap <buffer> <LocalLeader>l'
+    endif
   endif
 
 endif
diff --git a/vim/after/indent/vim.vim b/vim/after/indent/vim.vim
index b87964a4..0cb1d397 100644
--- a/vim/after/indent/vim.vim
+++ b/vim/after/indent/vim.vim
@@ -1,5 +1,7 @@
 " Observe VimL conventions for two-space indents
 setlocal shiftwidth=2
 setlocal softtabstop=2
-let b:undo_indent = b:undo_indent
-      \ . '|setlocal shiftwidth< softtabstop<'
+if exists('b:undo_indent')
+  let b:undo_indent = b:undo_indent
+        \ . '|setlocal shiftwidth< softtabstop<'
+endif
-- 
cgit v1.2.3