Merge branch 'release/v0.54.0'v0.54.0

* release/v0.54.0:
  Regenerate dotfiles(7) manual
  Bump VERSION
  Add note about compiler scripts
  Arrange for tidiers to hold window position
  Update markdown_autoformat.vim
  Update surround.vim
  Completely overhaul after/ftplugin files
  Use short-circuit for no-mapping check
  Correct g:current_compiler setting
  Consistently require 'nocompatible' for ftplugins
  Upgrade insert_suspend_hlsearch.vim
  Use `function!` consistently in ftplugins
  Fix comments in vim/filetype.vim
  Set 'comments' blank by default
  Add Xresources filenames
  Add more names to Vim Bash types

1 files changed, 42 insertions, 42 deletions

diff --git a/vim/after/ftplugin/html/lint.vim b/vim/after/ftplugin/html/lint.vim
index b24b18c9..a3776d60 100644
--- a/vim/after/ftplugin/html/lint.vim
+++ b/vim/after/ftplugin/html/lint.vim
@@ -1,55 +1,55 @@
-" 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_lint') || &compatible
+" html/lint.vim: Use tidy(1) to lint HTML documents for errors
+
+" 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_lint')
   finish
 endif
-let b:did_ftplugin_html_lint = 1
 
-" 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'
+" Don't load if the primary filetype isn't HTML
+if &filetype !=# 'html'
+  finish
 endif
 
+" Flag as loaded
+let b:did_ftplugin_html_lint = 1
+let b:undo_ftplugin = b:undo_ftplugin
+      \ . '|unlet b:did_ftplugin_html_lint'
+
 " Build function for linter
-if !exists('*s:HtmlLint')
-  function s:HtmlLint()
-    let l:save_makeprg = &l:makeprg
-    let l:save_errorformat = &l:errorformat
-    unlet! g:current_compiler
-    compiler tidy
-    make!
-    let &l:makeprg = l:save_makeprg
-    let &l:errorformat = l:save_errorformat
-    cwindow
-  endfunction
+function! s:HtmlLint()
+  if exists('b:current_compiler')
+    let l:save_compiler = b:current_compiler
+  endif
+  compiler tidy
+  lmake!
+  lwindow
+  if exists('l:save_compiler')
+    execute 'compiler ' . l:save_compiler
+  endif
+endfunction
+
+" Stop here if the user doesn't want ftplugin mappings
+if exists('g:no_plugin_maps') || exists('g:no_html_maps')
+  finish
 endif
 
-" 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>call <SID>HtmlLint()<CR>
+let b:undo_ftplugin = b:undo_ftplugin
+      \ . '|nunmap <buffer> <Plug>HtmlLint'
 
-  " Define a mapping target
-  nnoremap <buffer> <silent> <unique>
+" If there isn't a key mapping already, use a default one
+if !hasmapto('<Plug>HtmlLint')
+  nmap <buffer> <unique>
+        \ <LocalLeader>l
         \ <Plug>HtmlLint
-        \ :<C-U>call <SID>HtmlLint()<CR>
-  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
-    if exists('b:undo_ftplugin')
-      let b:undo_ftplugin = b:undo_ftplugin
-            \ . '|nunmap <buffer> <LocalLeader>l'
-    endif
-  endif
-
+  let b:undo_ftplugin = b:undo_ftplugin
+        \ . '|nunmap <buffer> <LocalLeader>l'
 endif