Skip to content

Commit 42004d9

Browse files
committed
Swift: cleanup diagnostic message construction
1 parent d9b3104 commit 42004d9

File tree

1 file changed

+4
-20
lines changed

1 file changed

+4
-20
lines changed

swift/extractor/invocation/SwiftDiagnosticsConsumer.cpp

Lines changed: 4 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -16,10 +16,8 @@ static int diagnosticsKind(const swift::DiagnosticInfo& diagInfo) {
1616
return 1;
1717
case swift::DiagnosticKind::Warning:
1818
return 2;
19-
2019
case swift::DiagnosticKind::Note:
2120
return 3;
22-
2321
case swift::DiagnosticKind::Remark:
2422
return 4;
2523
}
@@ -39,22 +37,8 @@ void SwiftDiagnosticsConsumer::handleDiagnostic(swift::SourceManager& sourceMana
3937

4038
std::string SwiftDiagnosticsConsumer::getDiagMessage(swift::SourceManager& sourceManager,
4139
const swift::DiagnosticInfo& diagInfo) {
42-
// Translate ranges.
43-
llvm::SmallVector<llvm::SMRange, 2> ranges;
44-
for (auto R : diagInfo.Ranges)
45-
ranges.push_back(getRawRange(sourceManager, R));
46-
47-
// Translate fix-its.
48-
llvm::SmallVector<llvm::SMFixIt, 2> fixIts;
49-
for (const swift::DiagnosticInfo::FixIt& F : diagInfo.FixIts)
50-
fixIts.push_back(getRawFixIt(sourceManager, F));
51-
52-
// Actually substitute the diagnostic arguments into the diagnostic text.
53-
llvm::SmallString<256> Text;
54-
{
55-
llvm::raw_svector_ostream Out(Text);
56-
swift::DiagnosticEngine::formatDiagnosticText(Out, diagInfo.FormatString, diagInfo.FormatArgs);
57-
}
58-
59-
return Text.str().str();
40+
llvm::SmallString<256> text;
41+
llvm::raw_svector_ostream out(text);
42+
swift::DiagnosticEngine::formatDiagnosticText(out, diagInfo.FormatString, diagInfo.FormatArgs);
43+
return text.str().str();
6044
}

0 commit comments

Comments
 (0)