marc/ale
Archived
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Merge pull request #2700 from elebow/clangcheck-no-color-diagnostics

Browse source 
clangcheck: Add -fno-color-diagnostics (closes #2188)
This commit is contained in:
w0rp 2019-10-28 13:14:14 +00:00 committed by GitHub
commit 0d4dfb61dc
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
ale_linters/cpp/clangcheck.vim
View file

@ -20,7 +20,7 @@ function! ale_linters#cpp#clangcheck#GetCommand(buffer) abort
" being generated. These are only added if no build directory can be " being generated. These are only added if no build directory can be
" detected. " detected.
return '%e -analyze %s' return '%e -analyze %s'
\ . (empty(l:build_dir) ? ' -extra-arg -Xclang -extra-arg -analyzer-output=text' : '') \ . (empty(l:build_dir) ? ' --extra-arg=-Xclang --extra-arg=-analyzer-output=text --extra-arg=-fno-color-diagnostics': '')
\ . ale#Pad(l:user_options) \ . ale#Pad(l:user_options)
\ . (!empty(l:build_dir) ? ' -p ' . ale#Escape(l:build_dir) : '') \ . (!empty(l:build_dir) ? ' -p ' . ale#Escape(l:build_dir) : '')
endfunction endfunction

6
test/command_callback/test_cpp_clangcheck_command_callbacks.vader
View file

@ -7,7 +7,7 @@ After:
Execute(The executable should be configurable): Execute(The executable should be configurable):
AssertLinter 'clang-check', AssertLinter 'clang-check',
\ ale#Escape('clang-check') \ ale#Escape('clang-check')
\ . ' -analyze %s -extra-arg -Xclang -extra-arg -analyzer-output=text' \ . ' -analyze %s --extra-arg=-Xclang --extra-arg=-analyzer-output=text --extra-arg=-fno-color-diagnostics'
let b:ale_cpp_clangcheck_executable = 'foobar' let b:ale_cpp_clangcheck_executable = 'foobar'
@ -15,7 +15,7 @@ Execute(The executable should be configurable):
" being generated. " being generated.
AssertLinter 'foobar', AssertLinter 'foobar',
\ ale#Escape('foobar') \ ale#Escape('foobar')
\ . ' -analyze %s -extra-arg -Xclang -extra-arg -analyzer-output=text' \ . ' -analyze %s --extra-arg=-Xclang --extra-arg=-analyzer-output=text --extra-arg=-fno-color-diagnostics'
Execute(The options should be configurable): Execute(The options should be configurable):
let b:ale_cpp_clangcheck_options = '--something' let b:ale_cpp_clangcheck_options = '--something'
@ -23,7 +23,7 @@ Execute(The options should be configurable):
AssertLinter 'clang-check', AssertLinter 'clang-check',
\ ale#Escape('clang-check') \ ale#Escape('clang-check')
\ . ' -analyze %s' \ . ' -analyze %s'
\ . ' -extra-arg -Xclang -extra-arg -analyzer-output=text' \ . ' --extra-arg=-Xclang --extra-arg=-analyzer-output=text --extra-arg=-fno-color-diagnostics'
\ . ' --something' \ . ' --something'
Execute(The build directory should be used when set): Execute(The build directory should be used when set):