Fix #1255: Move extra clang-check args to before user options
This commit is contained in:
parent
5285b0b332
commit
ff8d4c5286
@ -24,9 +24,9 @@ function! ale_linters#cpp#clangcheck#GetCommand(buffer) abort
|
||||
" detected.
|
||||
return ale#Escape(ale_linters#cpp#clangcheck#GetExecutable(a:buffer))
|
||||
\ . ' -analyze %s'
|
||||
\ . (empty(l:build_dir) ? ' -extra-arg -Xclang -extra-arg -analyzer-output=text' : '')
|
||||
\ . (!empty(l:user_options) ? ' ' . l:user_options : '')
|
||||
\ . (!empty(l:build_dir) ? ' -p ' . ale#Escape(l:build_dir) : '')
|
||||
\ . (empty(l:build_dir) ? ' -extra-arg -Xanalyzer -extra-arg -analyzer-output=text' : '')
|
||||
endfunction
|
||||
|
||||
call ale#linter#Define('cpp', {
|
||||
|
@ -28,7 +28,7 @@ Execute(The executable should be used in the command):
|
||||
AssertEqual
|
||||
\ ale#Escape('clang-check')
|
||||
\ . ' -analyze %s'
|
||||
\ . ' -extra-arg -Xanalyzer -extra-arg -analyzer-output=text',
|
||||
\ . ' -extra-arg -Xclang -extra-arg -analyzer-output=text',
|
||||
\ ale_linters#cpp#clangcheck#GetCommand(bufnr(''))
|
||||
|
||||
let b:ale_cpp_clangcheck_executable = 'foobar'
|
||||
@ -38,7 +38,7 @@ Execute(The executable should be used in the command):
|
||||
AssertEqual
|
||||
\ ale#Escape('foobar')
|
||||
\ . ' -analyze %s'
|
||||
\ . ' -extra-arg -Xanalyzer -extra-arg -analyzer-output=text',
|
||||
\ . ' -extra-arg -Xclang -extra-arg -analyzer-output=text',
|
||||
\ ale_linters#cpp#clangcheck#GetCommand(bufnr(''))
|
||||
|
||||
Execute(The options should be configurable):
|
||||
@ -46,8 +46,9 @@ Execute(The options should be configurable):
|
||||
|
||||
AssertEqual
|
||||
\ ale#Escape('clang-check')
|
||||
\ . ' -analyze %s --something'
|
||||
\ . ' -extra-arg -Xanalyzer -extra-arg -analyzer-output=text',
|
||||
\ . ' -analyze %s'
|
||||
\ . ' -extra-arg -Xclang -extra-arg -analyzer-output=text',
|
||||
\ . ' --something'
|
||||
\ ale_linters#cpp#clangcheck#GetCommand(bufnr(''))
|
||||
|
||||
Execute(The build directory should be used when set):
|
||||
|
Loading…
Reference in New Issue
Block a user