Skip to content

Commit cd4e2e4

Browse files
committed
Revert "a refactoring for command checking"
This reverts commit 009df09. Conflicts: plugin/clang_format.vim
1 parent d3ffef5 commit cd4e2e4

File tree

2 files changed

+17
-11
lines changed

2 files changed

+17
-11
lines changed

autoload/clang_format.vim

+14-8
Original file line numberDiff line numberDiff line change
@@ -76,23 +76,32 @@ function! clang_format#get_version()
7676
endtry
7777
endfunction
7878

79-
function! clang_format#check_cmd()
79+
function! clang_format#is_invalid()
8080
if !exists('s:command_available')
8181
if ! executable(g:clang_format#command)
82-
return 'clang-format is not found. check g:clang_format#command.'
82+
return 1
8383
endif
8484
let s:command_available = 1
8585
endif
8686

8787
if !exists('s:version')
8888
let v = clang_format#get_version()
8989
if v[0] < 3 || (v[0] == 3 && v[1] < 4)
90-
return 'clang-format 3.3 or earlier is not supported for the lack of aruguments.'
90+
return 2
9191
endif
9292
let s:version = v
9393
endif
9494

95-
return ''
95+
return 0
96+
endfunction
97+
98+
function! s:verify_command()
99+
let invalidity = clang_format#is_invalid()
100+
if invalidity == 1
101+
echoerr "clang-format is not found. check g:clang_format#command."
102+
elseif invalidity == 2
103+
echoerr 'clang-format 3.3 or earlier is not supported for the lack of aruguments'
104+
endif
96105
endfunction
97106
" }}}
98107

@@ -145,10 +154,7 @@ endfunction
145154
" replace buffer {{{
146155
function! clang_format#replace(line1, line2)
147156

148-
let err_msg = clang_format#check_cmd()
149-
if err_msg !=# ''
150-
echoerr err_msg
151-
endif
157+
call s:verify_command()
152158

153159
let pos_save = getpos('.')
154160
let sel_save = &l:selection

plugin/clang_format.vim

+3-3
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,9 @@ command! -range=% -nargs=0 ClangFormatEchoFormattedCode echo clang_format#format
1414

1515
augroup plugin-clang-format-auto-format
1616
autocmd!
17-
autocmd BufWritePre * if &ft =~# '^\%(c\|cpp\|objc\)$' && g:clang_format#auto_format && clang_format#check_cmd() ==# '' | call clang_format#replace(1, line('$')) | endif
18-
autocmd FileType c,cpp,objc if g:clang_format#auto_format_on_insert_leave && clang_format#check_cmd() ==# '' | call clang_format#enable_format_on_insert() | endif
19-
autocmd FileType c,cpp,objc if g:clang_format#auto_formatexpr && clang_format#check_cmd() ==# '' | setlocal formatexpr=clang_format#replace(v:lnum,v:lnum+v:count-1) | endif
17+
autocmd BufWritePre * if &ft =~# '^\%(c\|cpp\|objc\)$' && g:clang_format#auto_format && !clang_format#is_invalid() | call clang_format#replace(1, line('$')) | endif
18+
autocmd FileType c,cpp,objc if g:clang_format#auto_format_on_insert_leave && !clang_format#is_invalid() | call clang_format#enable_format_on_insert() | endif
19+
autocmd FileType c,cpp,objc if g:clang_format#auto_formatexpr && !clang_format#is_invalid() | setlocal formatexpr=clang_format#replace(v:lnum,v:lnum+v:count-1) | endif
2020
augroup END
2121

2222
let g:loaded_clang_format = 1

0 commit comments

Comments
 (0)