Merge pull request #2813 from werneta/master

Update vlog parser to handle new output format

ale_linters/verilog/vlog.vim

@@ -24,6 +24,20 @@ function! ale_linters#verilog#vlog#Handle(buffer, lines) abort
         \})
     endfor
 
+    "Matches patterns like the following:
+    "** Warning: (vlog-2623) add.v(7): Undefined variable: C.
+    "** Error: (vlog-13294) file.v(1): Identifier must be declared with a port mode: C.
+    " let l:pattern = '^**\s\(\w*\):[a-zA-Z0-9\-\.\_\/ ]\+(\(\d\+\)):\s\+\(.*\)'
+    let l:pattern = '^**\s\(\w*\):\s\([^)]*)\)[a-zA-Z0-9\-\.\_\/ ]\+(\(\d\+\)):\s\+\(.*\)'
+
+    for l:match in ale#util#GetMatches(a:lines, l:pattern)
+        call add(l:output, {
+        \   'lnum': l:match[3] + 0,
+        \   'type': l:match[1] is? 'Error' ? 'E' : 'W',
+        \   'text': l:match[2] . ' ' . l:match[4],
+        \})
+    endfor
+
     return l:output
 endfunction
 

test/handler/test_vlog_handler.vader

@@ -4,7 +4,7 @@ Before:
 After:
   call ale#linter#Reset()
 
-Execute(The vlog handler should parse lines correctly):
+Execute(The vlog handler should parse old-style lines correctly):
   AssertEqual
   \ [
   \   {
@@ -22,3 +22,22 @@ Execute(The vlog handler should parse lines correctly):
   \   '** Warning: add.v(7): (vlog-2623) Undefined variable: C.',
   \   '** Error: file.v(1): (vlog-13294) Identifier must be declared with a port mode: C.',
   \ ])
+
+Execute(The vlog handler should parse new-style lines correctly):
+  AssertEqual
+  \ [
+  \   {
+  \     'lnum': 7,
+  \     'type': 'W',
+  \     'text': '(vlog-2623) Undefined variable: C.'
+  \   },
+  \   {
+  \     'lnum': 1,
+  \     'type': 'E',
+  \     'text': '(vlog-13294) Identifier must be declared with a port mode: C.'
+  \   },
+  \ ],
+  \ ale_linters#verilog#vlog#Handle(bufnr(''), [
+  \   '** Warning: (vlog-2623) add.v(7): Undefined variable: C.',
+  \   '** Error: (vlog-13294) file.v(1): Identifier must be declared with a port mode: C.',
+  \ ])