fix: refined orLog usage

swiftlang · Jun 27, 2024 · 8596b21 · 8596b21
1 parent ecdea04
commit 8596b21
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/Sources/SourceKitLSP/Swift/SwiftLanguageService.swift b/Sources/SourceKitLSP/Swift/SwiftLanguageService.swift
@@ -437,7 +437,7 @@ extension SwiftLanguageService {
     cancelInFlightPublishDiagnosticsTask(for: notification.textDocument.uri)
     await diagnosticReportManager.removeItemsFromCache(with: notification.textDocument.uri)
 
-    guard let snapshot = try? self.documentManager.open(notification) else {
+    guard let snapshot = orLog("Getting document manager", { try self.documentManager.open(notification) }) else {
       // Already logged failure.
       return
     }
@@ -454,8 +454,8 @@ extension SwiftLanguageService {
     inFlightPublishDiagnosticsTasks[notification.textDocument.uri] = nil
     await diagnosticReportManager.removeItemsFromCache(with: notification.textDocument.uri)
 
-    orLog("Connection to the editor closed") {
-      try? self.documentManager.close(notification)
+    orLog("Getting document manager") {
+      try self.documentManager.close(notification)
     }
 
     let req = closeDocumentSourcekitdRequest(uri: notification.textDocument.uri)
@@ -550,7 +550,7 @@ extension SwiftLanguageService {
       let replacement: String
     }
 
-    guard let (preEditSnapshot, postEditSnapshot, edits) = try? self.documentManager.edit(notification) else {
+    guard let (preEditSnapshot, postEditSnapshot, edits) = orLog("Getting document manager", { try self.documentManager.edit(notification) }) else {
       return
     }