refactor(lsp): split long function into parsing func + logic func

Author: strager

src/quick-lint-js/lsp/lsp-server.cpp

@@ -387,15 +387,23 @@ void linting_lsp_server_handler::handle_text_document_did_close_notification(
     // Ignore invalid notification.
     return;
   }
-  std::string path = parse_file_from_lsp_uri(uri);
+  this->handle_text_document_did_close_notification(
+      lsp_text_document_did_close_notification{
+          .uri = uri,
+      });
+}
+
+void linting_lsp_server_handler::handle_text_document_did_close_notification(
+    const lsp_text_document_did_close_notification& notification) {
+  std::string path = parse_file_from_lsp_uri(notification.uri);
   if (path.empty()) {
     // TODO(strager): Report a warning.
     QLJS_UNIMPLEMENTED();
   }
 
   this->config_loader_.unwatch_file(path);
   this->config_fs_.close_document(path);
-  this->documents_.erase(string8(uri));
+  this->documents_.erase(string8(notification.uri));
   // TODO(strager): Signal to configuration_loader and
   // change_detecting_filesystem_* that we no longer need to track changes to
   // this .js document's config file.

src/quick-lint-js/lsp/lsp-server.h

@@ -160,8 +160,14 @@ class linting_lsp_server_handler final : public lsp_endpoint_handler {
   void handle_initialized_notification();
   void handle_text_document_did_change_notification(
       ::simdjson::ondemand::object& request);
+
+  struct lsp_text_document_did_close_notification {
+    string8_view uri;
+  };
   void handle_text_document_did_close_notification(
       ::simdjson::ondemand::object& request);
+  void handle_text_document_did_close_notification(
+      const lsp_text_document_did_close_notification& notification);
 
   struct lsp_text_document_did_open_notification {
     std::string_view language_id;