danog/psalm
1
0
Fork 0
mirror of https://github.com/danog/psalm.git synced 2025-01-22 05:41:20 +01:00
Code Issues Projects Releases Wiki Activity

Fixes #5997 by intentionally sending Success response from shutdown method

Browse Source
This commit is contained in:
Your Name 2021-06-28 10:02:18 -07:00
parent 8c137527d4
commit 2e05377323
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/Psalm/Internal/LanguageServer/LanguageServer.php
View File

@ -395,8 +395,9 @@ class LanguageServer extends AdvancedJsonRpc\Dispatcher
* The shutdown request is sent from the client to the server. It asks the server to shut down,
* but to not exit (otherwise the response might not be delivered correctly to the client).
* There is a separate exit notification that asks the server to exit.
* @psalm-suppress PossiblyUnusedReturnValue
*/
public function shutdown(): void
public function shutdown(): Promise
{
$this->clientStatus('closing');
$this->verboseLog("Shutting down...");
@ -407,7 +408,7 @@ class LanguageServer extends AdvancedJsonRpc\Dispatcher
$scanned_files
);
$this->clientStatus('closed');
new Success(null);
return new Success(null);
}
/**
@ -439,7 +440,6 @@ class LanguageServer extends AdvancedJsonRpc\Dispatcher
'[Psalm ' .PSALM_VERSION. ' - PHP Language Server] ' . $message,
$type
);
return;
} catch (\Throwable $err) {
// do nothing
}
@ -465,8 +465,9 @@ class LanguageServer extends AdvancedJsonRpc\Dispatcher
'telemetry/event'
);
} catch (\Throwable $err) {
new Success(null);
// do nothing
}
new Success(null);
}
/**