Move functions into autoload

Adds dependency on Vim >=7.0

1 files changed, 5 insertions, 86 deletions

diff --git a/plugin/toggle_flags.vim b/plugin/toggle_flags.vim
index be0cde4..f300666 100644
--- a/plugin/toggle_flags.vim
+++ b/plugin/toggle_flags.vim
@@ -1,6 +1,6 @@
 "
-" toggle_flags.vim: Provide commands to toggle flags in grouped options
-" like 'formatoptions', 'shortmess', 'complete', 'switchbuf', etc.
+" toggle_flags.vim: Provide commands to toggle flags in grouped options like
+" 'formatoptions', 'shortmess', 'complete', 'switchbuf', etc.
 "
 " Author: Tom Ryder <tom@sanctum.geek.nz>
 " License: Same as Vim itself
@@ -8,96 +8,15 @@
 if exists('loaded_toggle_flags') || &compatible
   finish
 endif
-if !has('user_commands') || v:version < 600
+if v:version < 700
   finish
 endif
 let loaded_toggle_flags = 1
 
-" Show an error-highlighted message and beep, but without a real :echoerr
-function! s:Error(message)
-  execute "normal! \<Esc>"
-  echohl ErrorMsg
-  echomsg a:message
-  echohl None
-endfunction
-
-" Test whether an option currently has a flag as part of its value
-function! s:Has(option, flag)
-
-  " Horrible :execute to get the option's current setting into a variable
-  let current = ''
-  execute 'let current = &' . a:option
-
-  " If the flag we're toggling is longer than one character, this must by
-  " necessity be a delimited option.  I think all of those in VimL are
-  " comma-separated.  Extend the flag and value so that they'll still match at
-  " the start and end.  Otherwise, use them as-is.
-  if strlen(a:flag) > 1
-    let search_flag = ',' . a:flag . ','
-    let search_value = ',' . current . ','
-  else
-    let search_flag = a:flag
-    let search_value = current
-  endif
-
-  " Return whether the flag appears in the value
-  return stridx(search_value, search_flag) > -1
-
-endfunction
-
-" Internal function to do the toggling
-function! s:Toggle(option, flag, local)
-
-  " Check for spurious option strings, we don't want to :execute anything funny
-  if a:option =~# '\L'
-    call s:Error('Illegal option name')
-    return 0
-  endif
-
-  " Check the option actually exists
-  if !exists('&' . a:option)
-    call s:Error('No such option: ' . a:option)
-    return 0
-  endif
-
-  " Choose which set command to use
-  let set = a:local
-        \ ? 'setlocal'
-        \ : 'set'
-
-  " Find whether the flag is set before the change
-  let before = s:Has(a:option, a:flag)
-
-  " Assign -= or += as the operation to run based on whether the flag already
-  " appears in the option value or not
-  let operation = before
-        \ ? '-='
-        \ : '+='
-
-  " Try to set the option; suppress errors, we'll check our work
-  silent! execute set
-        \ . ' '
-        \ . a:option . operation . escape(a:flag, '\ ')
-
-  " Find whether the flag is set after the change
-  let after = s:Has(a:option, a:flag)
-
-  " If we made a difference, report the new value; if we didn't, admit it
-  if before != after
-    execute set . ' ' . a:option . '?'
-  else
-    call s:Error('Unable to toggle '.a:option.' flag '.a:flag)
-  endif
-
-  " Return whether we made a change
-  return before != after
-
-endfunction
-
 " User commands wrapping around calls to the above function
 command -nargs=+ -complete=option
       \ ToggleFlag
-      \ call <SID>Toggle(<f-args>, 0)
+      \ call toggle_flags#Toggle(<f-args>, 0)
 command -nargs=+ -complete=option
       \ ToggleFlagLocal
-      \ call <SID>Toggle(<f-args>, 1)
+      \ call toggle_flags#Toggle(<f-args>, 1)