LSP: Don't swallow bufnr argument from callbacks (#12350)

The callbacks for `textDocument/documentSymbol` and `workspace/symbol`
never received the `bufnr` argument because the logic that adds error
validation and logging swallowed the argument.
This commit is contained in:
Mathias Fußenegger
2020-05-21 03:18:35 +02:00
committed by GitHub
parent c6dc397801
commit 044eb56ed2

View File

@@ -242,12 +242,12 @@ end
-- Add boilerplate error validation and logging for all of these.
for k, fn in pairs(M) do
M[k] = function(err, method, params, client_id)
local _ = log.debug() and log.debug('default_callback', method, { params = params, client_id = client_id, err = err })
M[k] = function(err, method, params, client_id, bufnr)
log.debug('default_callback', method, { params = params, client_id = client_id, err = err, bufnr = bufnr })
if err then
error(tostring(err))
end
return fn(err, method, params, client_id)
return fn(err, method, params, client_id, bufnr)
end
end