Don't save the sign ID on loclist items. This approach won't work.

2 files changed, 1 insertions, 11 deletions

diff --git a/autoload/ale/sign.vim b/autoload/ale/sign.vim
index 1a39ba89..99af2b17 100644
--- a/autoload/ale/sign.vim
+++ b/autoload/ale/sign.vim
@@ -118,20 +118,12 @@ function! ale#sign#SetSigns(buffer, loclist) abort
 
     " Add the new signs,
     for l:index in range(0, len(l:signlist) - 1)
-        let l:sign_id = l:index + g:ale_sign_offset + 1
         let l:sublist = l:signlist[l:index]
         let l:type = !empty(filter(copy(l:sublist), 'v:val.type ==# ''E'''))
         \   ? 'ALEErrorSign'
         \   : 'ALEWarningSign'
 
-        " Save the sign IDs we are setting back on our loclist objects.
-        " These IDs can be used later for changing line numbers of items
-        " we keep, based on what Vim adjusts automatically.
-        for l:obj in l:sublist
-            let l:obj.sign_id = l:sign_id
-        endfor
-
-        let l:sign_line = 'sign place ' . l:sign_id
+        let l:sign_line = 'sign place ' . (l:index + g:ale_sign_offset + 1)
             \. ' line=' . l:sublist[0].lnum
             \. ' name=' . l:type
             \. ' buffer=' . a:buffer
diff --git a/test/test_linting_updates_loclist.vader b/test/test_linting_updates_loclist.vader
index 2e106d05..06f11d8b 100644
--- a/test/test_linting_updates_loclist.vader
+++ b/test/test_linting_updates_loclist.vader
@@ -13,7 +13,6 @@ Before:
   \   'type': 'W',
   \   'col': 10,
   \   'text': 'Infix operators must be spaced. [Warning/space-infix-ops]',
-  \   'sign_id': 1000001,
   \ },
   \ {
   \   'lnum': 2,
@@ -24,7 +23,6 @@ Before:
   \   'type': 'E',
   \   'col': 10,
   \   'text': 'Missing semicolon. [Error/semi]',
-  \   'sign_id': 1000002,
   \ }
   \]