diff options
| author | Teddy Wing | 2019-12-14 17:02:03 +0100 |
|---|---|---|
| committer | Teddy Wing | 2019-12-14 17:02:03 +0100 |
| commit | fdee7afc4e9378f0235ef52fb35e63b8219e7ffd (patch) | |
| tree | 4cc42856bd90ed7f63b47c4c8982f0aac3ecab27 | |
| parent | 3cde5aeae893ee5a5f1c782ec38df3de70e30a13 (diff) | |
| download | vim-nohai-fdee7afc4e9378f0235ef52fb35e63b8219e7ffd.tar.bz2 | |
Remove unnecessary code in `CmdlineEnter` command
* The `s:Nohai()` function is unnecessary, as it does the same thing as
`s:AddMapping()`. Call `s:AddMapping()` directly instead.
* Don't bother checking `cmdwin_char`. Because we limit the
`CmdlineEnter` `autocmd` to `/` and `?`, the `s:AddMapping()` function
will only ever be called on `/` and `?`, thus checking its value in a
condition is redundant.
| -rw-r--r-- | plugin/nohai.vim | 14 |
1 files changed, 3 insertions, 11 deletions
diff --git a/plugin/nohai.vim b/plugin/nohai.vim index a6a30a0..dcb7630 100644 --- a/plugin/nohai.vim +++ b/plugin/nohai.vim @@ -1,13 +1,5 @@ -function! s:Nohai(cmdwin_char) - if a:cmdwin_char == '/' || a:cmdwin_char == '?' - call s:AddMapping(a:cmdwin_char) - endif -endfunction - -function! s:AddMapping(cmdwin_char) - if a:cmdwin_char == '/' || a:cmdwin_char == '?' - silent! cnoremap <expr> <CR> <SID>CR() - endif +function! s:AddMapping() + silent! cnoremap <expr> <CR> <SID>CR() endfunction function! s:RemoveMapping() @@ -27,7 +19,7 @@ function! s:AutocmdOn() augroup nohai autocmd! - autocmd CmdlineEnter [/\?] call s:Nohai(expand('<afile>')) + autocmd CmdlineEnter [/\?] call s:AddMapping() autocmd CmdlineLeave [/\?] call s:Deactivate() augroup END endfunction |