From d59047a4c282503dafdc45d9af15ef9584f04969 Mon Sep 17 00:00:00 2001
From: David Schulz <david.schulz@qt.io>
Date: Tue, 8 Feb 2022 15:08:26 +0100
Subject: [PATCH] LanguageClient: do not disconnect client from manager

The previously connected signals should be safe to stay connected while
the server restarts. This fixes restarting crashed clients more than once.

Change-Id: I161d41418e10b7e3bd3e17141720f4a5f399841c
Reviewed-by: Christian Stenger <christian.stenger@qt.io>
---
 src/plugins/languageclient/languageclientmanager.cpp | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/plugins/languageclient/languageclientmanager.cpp b/src/plugins/languageclient/languageclientmanager.cpp
index e76d3ff31b3..cc8b03ea6dc 100644
--- a/src/plugins/languageclient/languageclientmanager.cpp
+++ b/src/plugins/languageclient/languageclientmanager.cpp
@@ -159,7 +159,6 @@ void LanguageClientManager::clientFinished(Client *client)
                 = managerInstance->m_clientForDocument.keys(client);
             if (client->reset()) {
                 qCDebug(Log) << "restart unexpectedly finished client: " << client->name() << client;
-                client->disconnect(managerInstance);
                 client->log(
                     tr("Unexpectedly finished. Restarting in %1 seconds.").arg(restartTimeoutS));
                 QTimer::singleShot(restartTimeoutS * 1000, client, [client]() { client->start(); });