Skip to content

Commit 13c849c

Browse files
committed
Add support for the "remark" diagnostic severity
1 parent 8efe6a6 commit 13c849c

File tree

2 files changed

+17
-13
lines changed

2 files changed

+17
-13
lines changed

lib/ASTGen/Sources/ASTGen/Diagnostics.swift

Lines changed: 15 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -14,12 +14,7 @@ fileprivate func emitDiagnosticParts(
1414
let bridgedDiagEngine = BridgedDiagnosticEngine(raw: diagEnginePtr)
1515

1616
// Map severity
17-
let bridgedSeverity: BridgedDiagnosticSeverity
18-
switch severity {
19-
case .error: bridgedSeverity = .error
20-
case .note: bridgedSeverity = .note
21-
case .warning: bridgedSeverity = .warning
22-
}
17+
let bridgedSeverity = severity.bridged
2318

2419
func bridgedSourceLoc(at position: AbsolutePosition) -> BridgedSourceLoc {
2520
return BridgedSourceLoc(at: position, in: sourceFileBuffer)
@@ -119,6 +114,18 @@ func emitDiagnostic(
119114
}
120115
}
121116

117+
extension DiagnosticSeverity {
118+
var bridged: BridgedDiagnosticSeverity {
119+
switch self {
120+
case .error: return .error
121+
case .note: return .note
122+
case .warning: return .warning
123+
case .remark: return .remark
124+
@unknown default: return .error
125+
}
126+
}
127+
}
128+
122129
extension SourceManager {
123130
private func diagnoseSingle<Node: SyntaxProtocol>(
124131
message: String,
@@ -129,12 +136,7 @@ extension SourceManager {
129136
fixItChanges: [FixIt.Change] = []
130137
) {
131138
// Map severity
132-
let bridgedSeverity: BridgedDiagnosticSeverity
133-
switch severity {
134-
case .error: bridgedSeverity = .error
135-
case .note: bridgedSeverity = .note
136-
case .warning: bridgedSeverity = .warning
137-
}
139+
let bridgedSeverity = severity.bridged
138140

139141
// Emit the diagnostic
140142
var mutableMessage = message
@@ -292,7 +294,7 @@ extension BridgedDiagnosticSeverity {
292294
case .fatalError: return .error
293295
case .error: return .error
294296
case .warning: return .warning
295-
case .remark: return .warning // FIXME
297+
case .remark: return .remark
296298
case .note: return .note
297299
@unknown default: return .error
298300
}

lib/ASTGen/Sources/ASTGen/PluginHost.swift

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -269,6 +269,8 @@ class PluginDiagnosticsEngine {
269269
case .error: bridgedSeverity = .error
270270
case .note: bridgedSeverity = .note
271271
case .warning: bridgedSeverity = .warning
272+
case .remark: bridgedSeverity = .remark
273+
@unknown default: bridgedSeverity = .error
272274
}
273275

274276
// Emit the diagnostic

0 commit comments

Comments
 (0)