default --stdio flag to false, don't use nullable

dotnet · Dec 17, 2024 · 171270f · 171270f
1 parent 84ef787
commit 171270f
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/src/LanguageServer/Microsoft.CodeAnalysis.LanguageServer/Program.cs b/src/LanguageServer/Microsoft.CodeAnalysis.LanguageServer/Program.cs
@@ -239,12 +239,14 @@ static CliRootCommand CreateCommandLineParser()
     var serverPipeNameOption = new CliOption<string?>("--pipe")
     {
         Description = "The name of the pipe the server will connect to.",
-        Required = false,
+        Required = false
     };
-    var useStdIoOption = new CliOption<bool?>("--stdio")
+
+    var useStdIoOption = new CliOption<bool>("--stdio")
     {
         Description = "Use stdio for communication with the client.",
-        Required = false
+        Required = false,
+        DefaultValueFactory = _ => false
     };
 
     var rootCommand = new CliRootCommand()
@@ -276,7 +278,7 @@ static CliRootCommand CreateCommandLineParser()
         var razorDesignTimePath = parseResult.GetValue(razorDesignTimePathOption);
         var extensionLogDirectory = parseResult.GetValue(extensionLogDirectoryOption)!;
         var serverPipeName = parseResult.GetValue(serverPipeNameOption);
-        var useStdIo = parseResult.GetValue(useStdIoOption) ?? false;
+        var useStdIo = parseResult.GetValue(useStdIoOption);
 
         var serverConfiguration = new ServerConfiguration(
             LaunchDebugger: launchDebugger,