Skip to content

Commit 2a4c47b

Browse files
authored
Remove unused diagnosticEngine property (#1033)
1 parent f361d5f commit 2a4c47b

File tree

2 files changed

+5
-9
lines changed

2 files changed

+5
-9
lines changed

Sources/SwiftDocC/Model/DocumentationNode.swift

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -430,10 +430,9 @@ public struct DocumentationNode {
430430
let symbolAllowedValues = symbol![mixin: SymbolGraph.Symbol.AllowedValues.self]
431431

432432
if let possibleValues = markupModel.discussionTags?.possiblePropertyListValues, !possibleValues.isEmpty {
433-
let validator = PropertyListPossibleValuesSection.Validator(diagnosticEngine: engine)
434433
guard let symbolAllowedValues else {
435-
possibleValues.forEach {
436-
engine.emit(validator.makeExtraPossibleValueProblem($0, knownPossibleValues: [], symbolName: self.name.plainText))
434+
for value in possibleValues {
435+
engine.emit(PropertyListPossibleValuesSection.Validator.makeExtraPossibleValueProblem(value, knownPossibleValues: [], symbolName: self.name.plainText))
437436
}
438437
return
439438
}
@@ -456,7 +455,7 @@ public struct DocumentationNode {
456455

457456
for unknownValue in unknownPossibleValues {
458457
engine.emit(
459-
validator.makeExtraPossibleValueProblem(unknownValue, knownPossibleValues: knownPossibleValueNames, symbolName: self.name.plainText)
458+
PropertyListPossibleValuesSection.Validator.makeExtraPossibleValueProblem(unknownValue, knownPossibleValues: knownPossibleValueNames, symbolName: self.name.plainText)
460459
)
461460
}
462461

Sources/SwiftDocC/Model/Section/Sections/PropertyListPossibleValuesSection.swift

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -43,10 +43,7 @@ public struct PropertyListPossibleValuesSection {
4343
/// The list of possible values.
4444
public let possibleValues: [PossibleValue]
4545

46-
struct Validator {
47-
/// The engine that collects problems encountered while validating the possible values documentation.
48-
var diagnosticEngine: DiagnosticEngine
49-
46+
enum Validator {
5047
/// Creates a new problem about documentation for a possible value that's not known to that symbol.
5148
///
5249
/// ## Example
@@ -63,7 +60,7 @@ public struct PropertyListPossibleValuesSection {
6360
/// - unknownPossibleValue: The authored documentation for the unknown possible value name.
6461
/// - knownPossibleValues: All known possible value names for that symbol.
6562
/// - Returns: A new problem that suggests that the developer removes the documentation for the unknown possible value.
66-
func makeExtraPossibleValueProblem(_ unknownPossibleValue: PossibleValue, knownPossibleValues: Set<String>, symbolName: String) -> Problem {
63+
static func makeExtraPossibleValueProblem(_ unknownPossibleValue: PossibleValue, knownPossibleValues: Set<String>, symbolName: String) -> Problem {
6764

6865
let source = unknownPossibleValue.range?.source
6966
let summary = """

0 commit comments

Comments
 (0)