diff options
| author | w0rp <devw0rp@gmail.com> | 2017-05-19 09:49:00 +0100 |
|---|---|---|
| committer | w0rp <devw0rp@gmail.com> | 2017-05-20 19:02:36 +0100 |
| commit | 4214832ae263086d1aa1f565067d00e9ed1b820e (patch) | |
| tree | 4378db3624fc4a6e71460fbf1d394107da73b358 | |
| parent | 1f4d1800e0040d7d36d1c19e15c5f0e570122273 (diff) | |
| download | ale-4214832ae263086d1aa1f565067d00e9ed1b820e.zip | |
Remove the code for checking if functions exist. It breaks autoload functions
| -rw-r--r-- | autoload/ale/fix.vim | 20 | ||||
| -rw-r--r-- | test/test_ale_fix.vader | 5 |
2 files changed, 5 insertions, 20 deletions
diff --git a/autoload/ale/fix.vim b/autoload/ale/fix.vim index 89778a16..b2ca2575 100644 --- a/autoload/ale/fix.vim +++ b/autoload/ale/fix.vim @@ -241,29 +241,19 @@ function! s:GetCallbacks() abort return [] endif - let l:problem_list = [] let l:corrected_list = [] for l:item in l:callback_list if type(l:item) == type('') - if exists('*' . l:item) - call add(l:corrected_list, function(l:item)) - else - let l:func = ale#fix#registry#GetFunc(l:item) + let l:func = ale#fix#registry#GetFunc(l:item) - if !empty(l:func) && exists('*' . l:func) - call add(l:corrected_list, function(l:func)) - else - call add(l:problem_list, l:item) - endif + if !empty(l:func) + let l:item = l:func endif endif - endfor - if !empty(l:problem_list) - echoerr 'Invalid fixers used: ' . string(l:problem_list) - return [] - endif + call add(l:corrected_list, function(l:item)) + endfor return l:corrected_list endfunction diff --git a/test/test_ale_fix.vader b/test/test_ale_fix.vader index a872f38e..8e61aefe 100644 --- a/test/test_ale_fix.vader +++ b/test/test_ale_fix.vader @@ -128,11 +128,6 @@ Expect(The first function should be used): ^b ^c -Execute(ALEFix should complain for missing functions): - let g:ale_fixers.testft = ['XXX', 'YYY'] - AssertThrows ALEFix - AssertEqual 'Vim(echoerr):Invalid fixers used: [''XXX'', ''YYY'']', g:vader_exception - Execute(ALEFix should use functions from the registry): call ale#fix#registry#Add('add_carets', 'AddCarets', [], 'Add some carets') let g:ale_fixers.testft = ['add_carets'] |