From e36efd4f542a24fadf689b82418f94f57dd01f1e Mon Sep 17 00:00:00 2001 From: Tom Ryder Date: Mon, 30 Oct 2017 15:54:40 +1300 Subject: Rearrange and better explain ksh syntax workaround Move the rule setting the custom b:is_ksh variable used for this workaround (established in 52615f6) into an ftplugin file, rather than into ftdetect; the latter seems a much more appropriate place since by this point we've definitely decided the file type is "sh". From the revised comment in this changeset: >Setting g:is_posix above also prompts Vim's core syntax/sh.vim script >to set g:is_kornshell and thereby b:is_kornshell to the same value as >g:is_posix. > >That's very confusing, so before it happens we'll copy b:is_kornshell's >value as determined by filetype.vim and ~/.vim/ftdetect/sh.vim into a >custom variable b:is_ksh, before its meaning gets confused. > >b:is_ksh as a name is more inline with b:is_bash and b:is_sh, anyway, >so we'll just treat b:is_kornshell like it's both misnamed and broken. > >We can then switch on our custom variable in ~/.vim/after/syntax/sh.vim >to apply settings that actually *are* unique to Korn shell and its >derivatives. --- vim/ftdetect/sh.vim | 9 --------- 1 file changed, 9 deletions(-) (limited to 'vim/ftdetect') diff --git a/vim/ftdetect/sh.vim b/vim/ftdetect/sh.vim index 236f329c..8b4c10d3 100644 --- a/vim/ftdetect/sh.vim +++ b/vim/ftdetect/sh.vim @@ -20,13 +20,4 @@ augroup dfsh \ let b:is_posix = 1 | \ setfiletype sh - " If we determined something is b:is_kornshell, tack on b:is_ksh as well so - " we can still tease out what is actually a kornshell script after sh.vim is - " done changing our options for us; it conflates POSIX with Korn shell. - autocmd BufNewFile,BufRead - \ * - \ if exists('b:is_kornshell') | - \ let b:is_ksh = 1 | - \ endif - augroup END -- cgit v1.2.3