averello/sourcekit-lsp
1
0
Fork 0
mirror of https://github.com/apple/sourcekit-lsp.git synced 2026-03-02 18:23:24 +01:00
Code Issues Packages Projects Releases Wiki Activity

Remove clientID from request handling

Browse Source 
The client ID was needed when a `MessageHandler` could handle messages from multiple connections. We don’t support this anymore (because it wasn’t needed) and so the client ID doesn’t need to get passed through as well.
This commit is contained in:
Alex Hoppen
2024-03-12 13:17:05 -07:00
parent 1fe79781df
commit 7c46df3abe
10 changed files with 29 additions and 42 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
Sources/SKCore/BuildServerBuildSystem.swift
View File

@@ -190,7 +190,7 @@ public actor BuildServerBuildSystem: MessageHandler {
/// the build server has sent us a notification.
///
/// We need to notify the delegate about any updated build settings.
public nonisolated func handle(_ params: some NotificationType, from clientID: ObjectIdentifier) {
public nonisolated func handle(_ params: some NotificationType) {
logger.info(
"""
Received notification from build server:
@@ -212,7 +212,6 @@ public actor BuildServerBuildSystem: MessageHandler {
public nonisolated func handle<R: RequestType>(
_ params: R,
id: RequestID,
from clientID: ObjectIdentifier,
reply: @escaping (LSPResult<R.Response>) -> Void
) {
logger.info(