mirror of
https://github.com/dense-analysis/ale
synced 2024-12-18 12:25:17 +00:00
00eee550ea
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
49 lines
1.3 KiB
Plaintext
49 lines
1.3 KiB
Plaintext
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',
|
|
\ ])
|
|
|