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

Add definition of c/clangd's language as C (#2791)

Browse source 
* Add definition of c/clangd's language as C
* Update tests for clangd to be called with '-x c'
* Change to use single quotes instead of double quotes
This commit is contained in:
João Borges 2019-10-17 17:28:48 +01:00 committed by w0rp
parent b24fd13423
commit f4070f6c43
2 changed files with 8 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
ale_linters/c/clangd.vim
View file

@ -5,7 +5,7 @@ call ale#Set('c_clangd_executable', 'clangd')
call ale#Set('c_clangd_options', '') call ale#Set('c_clangd_options', '')
function! ale_linters#c#clangd#GetCommand(buffer) abort function! ale_linters#c#clangd#GetCommand(buffer) abort
return '%e' . ale#Pad(ale#Var(a:buffer, 'c_clangd_options')) return '%e -x c' . ale#Pad(ale#Var(a:buffer, 'c_clangd_options'))
endfunction endfunction
call ale#linter#Define('c', { call ale#linter#Define('c', {

10
test/command_callback/test_c_clangd_command_callbacks.vader
View file

@ -4,14 +4,18 @@ Before:
Save &filetype Save &filetype
let &filetype = 'c' let &filetype = 'c'
let b:command_tail = ' -x c'
After: After:
unlet! b:command_tail
call ale#assert#TearDownLinterTest() call ale#assert#TearDownLinterTest()
Execute(The language string should be correct): Execute(The language string should be correct):
AssertLSPLanguage 'c' AssertLSPLanguage 'c'
Execute(The default executable should be correct): Execute(The default executable should be correct):
AssertLinter 'clangd', ale#Escape('clangd') AssertLinter 'clangd', ale#Escape('clangd') . b:command_tail
Execute(The project root should be detected correctly): Execute(The project root should be detected correctly):
call ale#test#SetFilename(tempname() . '/dummy.c') call ale#test#SetFilename(tempname() . '/dummy.c')
@ -25,10 +29,10 @@ Execute(The project root should be detected correctly):
Execute(The executable should be configurable): Execute(The executable should be configurable):
let g:ale_c_clangd_executable = 'foobar' let g:ale_c_clangd_executable = 'foobar'
AssertLinter 'foobar', ale#Escape('foobar') AssertLinter 'foobar', ale#Escape('foobar') . b:command_tail
Execute(The options should be configurable): Execute(The options should be configurable):
let b:ale_c_clangd_options = '-compile-commands-dir=foo' let b:ale_c_clangd_options = '-compile-commands-dir=foo'
AssertLinter 'clangd', ale#Escape('clangd') . ' ' . b:ale_c_clangd_options AssertLinter 'clangd', ale#Escape('clangd') . b:command_tail . ' ' . b:ale_c_clangd_options