Revert "treat null/nil server capabilities as false"

This reverts commit 645bcfc6e57181c39dae1f238758e76c1759a765.

A capability of "null" is downright invalid, and must NOT be mistaken
for a value of "{}" (which indicates the presence of the capability)
or "False" (which indicates its asence).  See
https://github.com/microsoft/language-server-protocol/issues/830#issuecomment-537849292
for a clarification from the LSP maintainer.

diff --git a/lisp/progmodes/eglot.el b/lisp/progmodes/eglot.el
index ef7f8f0a3ece14b53d2bba2dbdfc1fdd1f2de34a..d1a1a3d0b20a4abb4233372918435fda1cab9069 100644 (file)
--- a/lisp/progmodes/eglot.el
+++ b/lisp/progmodes/eglot.el
@@ -1113,9 +1113,6 @@ under cursor."
              for probe = (plist-member caps feat)
              if (not probe) do (cl-return nil)
              if (eq (cadr probe) :json-false) do (cl-return nil)
-             ;; If the server specifies null as the value of the capability, it
-             ;; makes sense to treat it like false.
-             if (null (cadr probe)) do (cl-return nil)
              if (not (listp (cadr probe))) do (cl-return (if more nil (cadr probe)))
              finally (cl-return (or (cadr probe) t)))))