Completely overhaul after/ftplugin files

Should have done some of this in separate commits; oh well.

* Rewrite headers for each ftplugin
* Require Vim version >= 7.0, and thereby:
    * Switch back to location list
    * Don't check for b:undo_ftplugin existence, assume it
* Save and restore compiler instead of internal options
* Add bash, ksh, sh, and shellcheck compilers
* Rename mail/format_flowed.vim to mail/flowed.vim
* Rename sh/bash_han.vim to sh/han.vim

1 files changed, 20 insertions, 17 deletions

diff --git a/vim/after/ftplugin/html/tidy.vim b/vim/after/ftplugin/html/tidy.vim
index 3d14e79e..e0c43f01 100644
--- a/vim/after/ftplugin/html/tidy.vim
+++ b/vim/after/ftplugin/html/tidy.vim
@@ -1,18 +1,25 @@
-" Only do this when not done yet for this buffer
-" Also do nothing if 'compatible' enabled, or if the current filetype is
-" actually markdown
-if exists('b:did_ftplugin_html_tidy') || &compatible
+" html/tidy.vim: Use tidy(1) to filter HTML documents
+
+" Don't load if running compatible or too old
+if &compatible || v:version < 700
   finish
 endif
-if &filetype ==# 'markdown'
+
+" Don't load if already loaded
+if exists('b:did_ftplugin_html_tidy')
   finish
 endif
-let b:did_ftplugin_html_tidy = 1
-if exists('b:undo_ftplugin')
-  let b:undo_ftplugin = b:undo_ftplugin
-        \ . '|unlet b:did_ftplugin_html_tidy'
+
+" Don't load if the primary filetype isn't HTML
+if &filetype !=# 'html'
+  finish
 endif
 
+" Flag as loaded
+let b:did_ftplugin_html_tidy = 1
+let b:undo_ftplugin = b:undo_ftplugin
+      \ . '|unlet b:did_ftplugin_html_tidy'
+
 " Stop here if the user doesn't want ftplugin mappings
 if exists('g:no_plugin_maps') || exists('g:no_html_maps')
   finish
@@ -22,18 +29,14 @@ endif
 nnoremap <buffer> <silent> <unique>
       \ <Plug>HtmlTidy
       \ :<C-U>%!tidy -quiet<CR>
-if exists('b:undo_ftplugin')
-  let b:undo_ftplugin = b:undo_ftplugin
-        \ . '|nunmap <buffer> <Plug>HtmlTidy'
-endif
+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
-  if exists('b:undo_ftplugin')
-    let b:undo_ftplugin = b:undo_ftplugin
-          \ . '|nunmap <buffer> <LocalLeader>t'
-  endif
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|nunmap <buffer> <LocalLeader>t'
 endif