Skip to content

Commit 6206d2a

Browse files
authored
Merge pull request #1378 from ahoppen/message-handling-signposts-in-single-category
Create all notification/request handling signposts in the same logging category
2 parents 403f550 + 58d448a commit 6206d2a

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

Sources/SourceKitLSP/SourceKitLSPServer.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -541,7 +541,7 @@ extension SourceKitLSPServer: MessageHandler {
541541

542542
let notificationID = notificationIDForLogging.fetchAndIncrement()
543543

544-
let signposter = Logger(subsystem: LoggingScope.subsystem, category: "notification-\(notificationID)")
544+
let signposter = Logger(subsystem: LoggingScope.subsystem, category: "message-handling")
545545
.makeSignposter()
546546
let signpostID = signposter.makeSignpostID()
547547
let state = signposter.beginInterval("Notification", id: signpostID, "\(type(of: params))")
@@ -591,7 +591,7 @@ extension SourceKitLSPServer: MessageHandler {
591591
id: RequestID,
592592
reply: @Sendable @escaping (LSPResult<R.Response>) -> Void
593593
) {
594-
let signposter = Logger(subsystem: LoggingScope.subsystem, category: "request-\(id)").makeSignposter()
594+
let signposter = Logger(subsystem: LoggingScope.subsystem, category: "message-handling").makeSignposter()
595595
let signpostID = signposter.makeSignpostID()
596596
let state = signposter.beginInterval("Request", id: signpostID, "\(R.self)")
597597

0 commit comments

Comments
 (0)