Make the use of POSIX signals optional

Signals are not supported on Windows, and gracefully shutting down everything doesn't seem necessary if we're about to exit anyway.
phpactor · Mar 2, 2024 · 18c6336 · 18c6336
1 parent e547efe
commit 18c6336
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/lib/Core/Server/LanguageServer.php b/lib/Core/Server/LanguageServer.php
@@ -93,10 +93,13 @@ public function start(): Promise
      */
     public function run(): void
     {
-        Loop::onSignal(SIGINT, function (string $watcherId) {
-            Loop::cancel($watcherId);
-            yield $this->shutdown();
-        });
+        // Signals are not supported on Windows
+        if(defined('SIGINT')) {
+            Loop::onSignal(SIGINT, function (string $watcherId) {
+                Loop::cancel($watcherId);
+                yield $this->shutdown();
+            });
+        }
 
         Loop::setErrorHandler(function (Throwable $error): void {
             if ($error instanceof ShutdownServer) {