Merge pull request #1675 from ahoppen/fix-mem-leak

ahoppen · web-flow · commit 4a2d2d07d4b7 · 2024-09-18T11:54:18.000-07:00
Fix a memory leak
diff --git a/Sources/BuildSystemIntegration/BuildSystemManager.swift b/Sources/BuildSystemIntegration/BuildSystemManager.swift
@@ -206,7 +206,7 @@ package actor BuildSystemManager: QueueBasedMessageHandler {
       toolchainRegistry: toolchainRegistry,
       options: options,
       buildSystemTestHooks: buildSystemTestHooks,
-      messagesToSourceKitLSPHandler: self
+      messagesToSourceKitLSPHandler: WeakMessageHandler(self)
     )
     if let buildSystem {
       let connectionToBuildSystem = LocalConnection(receiverName: "Build system")

Original file line number	Diff line number	Diff line change
`@@ -206,7 +206,7 @@ package actor BuildSystemManager: QueueBasedMessageHandler {`
`206`	`206`	`toolchainRegistry: toolchainRegistry,`
`207`	`207`	`options: options,`
`208`	`208`	`buildSystemTestHooks: buildSystemTestHooks,`
`209`		`- messagesToSourceKitLSPHandler: self`
	`209`	`+ messagesToSourceKitLSPHandler: WeakMessageHandler(self)`
`210`	`210`	`)`
`211`	`211`	`if let buildSystem {`
`212`	`212`	`let connectionToBuildSystem = LocalConnection(receiverName: "Build system")`