mirror of
https://github.com/apple/sourcekit-lsp.git
synced 2026-03-02 18:23:24 +01:00
425e1322a1
`IndexStoreDB` moves its index to the `saved` directory when it is deallocated. Because `IndexStoreDB` is primarily owned by `UncheckedIndex`, we rely on deallocating this object to save the index store. This is fairly brittle because various parts of the codebase may hold transient references to that object as reported in https://github.com/swiftlang/sourcekit-lsp/issues/2455#issuecomment-3873561003. Explicitly remove the reference from `UncheckedIndex` to `IndexStoreDB`. While this still isn’t perfect because other parts of the code base may hold references to `IndexStoreDB` but those should be a lot rarer, resulting in a more consistent closing of the index.
47 lines
1.8 KiB
Swift
47 lines
1.8 KiB
Swift
//===----------------------------------------------------------------------===//
|
|
//
|
|
// This source file is part of the Swift.org open source project
|
|
//
|
|
// Copyright (c) 2014 - 2020 Apple Inc. and the Swift project authors
|
|
// Licensed under Apache License v2.0 with Runtime Library Exception
|
|
//
|
|
// See https://swift.org/LICENSE.txt for license information
|
|
// See https://swift.org/CONTRIBUTORS.txt for the list of Swift project authors
|
|
//
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
import BuildServerIntegration
|
|
import Foundation
|
|
import IndexStoreDB
|
|
@_spi(SourceKitLSP) package import LanguageServerProtocol
|
|
@_spi(SourceKitLSP) import SKLogging
|
|
import SemanticIndex
|
|
import SwiftExtensions
|
|
|
|
extension UncheckedIndex: BuildServerIntegration.MainFilesProvider {
|
|
/// - Important: This may return realpaths when the build server might not be using realpaths. Use
|
|
/// `BuildServerManager.mainFiles(containing:)` to work around that problem.
|
|
package func mainFiles(containing uri: DocumentURI, crossLanguage: Bool) -> Set<DocumentURI> {
|
|
let mainFiles: Set<DocumentURI>
|
|
if let filePath = orLog("File path to get main files", { try uri.fileURL?.filePath }) {
|
|
guard let underlyingIndexStoreDB = self.underlyingIndexStoreDB else {
|
|
logger.error("Not checking main files for URI because index has been closed")
|
|
return []
|
|
}
|
|
let mainFilePaths = underlyingIndexStoreDB.mainFilesContainingFile(
|
|
path: filePath,
|
|
crossLanguage: crossLanguage
|
|
)
|
|
mainFiles = Set(
|
|
mainFilePaths
|
|
.filter { FileManager.default.fileExists(atPath: $0) }
|
|
.map({ DocumentURI(filePath: $0, isDirectory: false) })
|
|
)
|
|
} else {
|
|
mainFiles = []
|
|
}
|
|
logger.info("Main files for \(uri.forLogging): \(mainFiles)")
|
|
return mainFiles
|
|
}
|
|
}
|