Skip to content

Commit ce85cf7

Browse files
committed
Resolve a few TODOs in Modules.md
1 parent 2e8637c commit ce85cf7

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

Contributor Documentation/Modules.md

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,7 @@ The SourceKit-LSP package contains the following non-testing modules.
44

55
### BuildServerProtocol
66

7-
Swift types to represent the [Build Server Protocol (BSP) specification, version TODO](TODO). These types should also be usable when implementing a BSP client and thus this module should not have any dependencies other than the LanguageServerProtocol module, with which it shares some types.
8-
9-
FIXME: Add link for BSP and version
7+
Swift types to represent the [Build Server Protocol (BSP) specification](https://build-server-protocol.github.io/docs/specification). These types should also be usable when implementing a BSP client and thus this module should not have any dependencies other than the LanguageServerProtocol module, with which it shares some types.
108

119
### BuildSystemIntegration
1210

@@ -18,11 +16,12 @@ Implementation of atomics for Swift using C. Once we can raise our deployment ta
1816

1917
### CSKTestSupport
2018

21-
FIXME: Can we remove this?
19+
For testing, overrides `__cxa_atexit` to prevent registration of static destructors due to work around https://github.com/swiftlang/swift/issues/55112.
20+
2221

2322
### Csourcekitd
2423

25-
Header file defining the interface to sourcekitd. This should stay in sync with [sourcekitd.h](TODO) in the Swift repository.
24+
Header file defining the interface to sourcekitd. This should stay in sync with [sourcekitd.h](https://github.com/swiftlang/swift/blob/main/tools/SourceKit/tools/sourcekitd/include/sourcekitd/sourcekitd.h) in the Swift repository.
2625

2726
### Diagnose
2827

0 commit comments

Comments
 (0)