mirror of
https://github.com/dense-analysis/ale.git
synced 2025-12-28 06:46:52 +08:00
Adds column number to the verilator verilog linter
Since version 4.032 (04/2020) verilator linter messages also contain the
column number, and look like:
%Error: /tmp/test.sv:3:1: syntax error, unexpected endmodule, expecting ';'
To stay compatible with old versions of the tool, the column number is
optional in the researched pattern regular expression.
See commit:
81c659957e
This commit is contained in:
48
test/handler/test_verilator_handler.vader
Normal file
48
test/handler/test_verilator_handler.vader
Normal file
@@ -0,0 +1,48 @@
|
||||
Before:
|
||||
runtime ale_linters/verilog/verilator.vim
|
||||
|
||||
After:
|
||||
call ale#linter#Reset()
|
||||
|
||||
|
||||
Execute (The verilator handler should parse legacy messages with only line numbers):
|
||||
AssertEqual
|
||||
\ [
|
||||
\ {
|
||||
\ 'lnum': 3,
|
||||
\ 'type': 'E',
|
||||
\ 'text': 'syntax error, unexpected IDENTIFIER'
|
||||
\ },
|
||||
\ {
|
||||
\ 'lnum': 10,
|
||||
\ 'type': 'W',
|
||||
\ 'text': 'Blocking assignments (=) in sequential (flop or latch) block; suggest delayed assignments (<=).'
|
||||
\ },
|
||||
\ ],
|
||||
\ ale_linters#verilog#verilator#Handle(bufnr(''), [
|
||||
\ '%Error: foo_verilator_linted.v:3: syntax error, unexpected IDENTIFIER',
|
||||
\ '%Warning-BLKSEQ: bar_verilator_linted.v:10: Blocking assignments (=) in sequential (flop or latch) block; suggest delayed assignments (<=).',
|
||||
\ ])
|
||||
|
||||
|
||||
Execute (The verilator handler should parse new format messages with line and column numbers):
|
||||
AssertEqual
|
||||
\ [
|
||||
\ {
|
||||
\ 'lnum': 3,
|
||||
\ 'col' : 1,
|
||||
\ 'type': 'E',
|
||||
\ 'text': 'syntax error, unexpected endmodule, expecting ;'
|
||||
\ },
|
||||
\ {
|
||||
\ 'lnum': 4,
|
||||
\ 'col' : 6,
|
||||
\ 'type': 'W',
|
||||
\ 'text': 'Signal is not used: r'
|
||||
\ },
|
||||
\ ],
|
||||
\ ale_linters#verilog#verilator#Handle(bufnr(''), [
|
||||
\ '%Error: bar_verilator_linted.v:3:1: syntax error, unexpected endmodule, expecting ;',
|
||||
\ '%Warning-UNUSED: foo_verilator_linted.v:4:6: Signal is not used: r',
|
||||
\ ])
|
||||
|
||||
Reference in New Issue
Block a user