Merge pull request #1921 from ahoppen/double-options-logging

ahoppen · web-flow · commit 403f88e293df · 2025-01-15T13:52:31.000-08:00
Avoid logging options for a workspace twice
diff --git a/Sources/SourceKitLSP/SourceKitLSPServer.swift b/Sources/SourceKitLSP/SourceKitLSPServer.swift
@@ -835,7 +835,7 @@ extension SourceKitLSPServer {
           .appendingPathComponent("config.json")
       )
     )
-    logger.log("Creating workspace at \(workspaceFolder.forLogging) with options: \(options.forLogging)")
+    logger.log("Creating workspace at \(workspaceFolder.forLogging)")
     logger.logFullObjectInMultipleLogMessages(header: "Options for workspace", options.loggingProxy)
 
     let workspace = await Workspace(

Original file line number	Diff line number	Diff line change
`@@ -835,7 +835,7 @@ extension SourceKitLSPServer {`
`835`	`835`	`.appendingPathComponent("config.json")`
`836`	`836`	`)`
`837`	`837`	`)`
`838`		`- logger.log("Creating workspace at \(workspaceFolder.forLogging) with options: \(options.forLogging)")`
	`838`	`+ logger.log("Creating workspace at \(workspaceFolder.forLogging)")`
`839`	`839`	`logger.logFullObjectInMultipleLogMessages(header: "Options for workspace", options.loggingProxy)`
`840`	`840`
`841`	`841`	`let workspace = await Workspace(`