Skip to content

Commit ea86221

Browse files
committed
Update LocalizationFormat to use diagnosticIDStringFor
1 parent fee8200 commit ea86221

File tree

2 files changed

+5
-8
lines changed

2 files changed

+5
-8
lines changed

lib/Localization/CMakeLists.txt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,3 +4,5 @@ add_swift_host_library(swiftLocalization STATIC
44

55
LLVM_LINK_COMPONENTS
66
support)
7+
target_link_libraries(swiftLocalization PRIVATE
8+
swiftAST)

lib/Localization/LocalizationFormat.cpp

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
//===----------------------------------------------------------------------===//
1616

1717
#include "swift/Localization/LocalizationFormat.h"
18+
#include "swift/AST/DiagnosticEngine.h"
1819
#include "swift/Basic/Range.h"
1920
#include "llvm/ADT/Optional.h"
2021
#include "llvm/ADT/SmallString.h"
@@ -39,12 +40,6 @@ enum LocalDiagID : uint32_t {
3940
NumDiags
4041
};
4142

42-
static constexpr const char *const diagnosticNameStrings[] = {
43-
#define DIAG(KIND, ID, Options, Text, Signature) " [" #ID "]",
44-
#include "swift/AST/DiagnosticsAll.def"
45-
"<not a diagnostic>",
46-
};
47-
4843
} // namespace
4944

5045
namespace llvm {
@@ -100,9 +95,9 @@ LocalizationProducer::getMessageOr(swift::DiagID id,
10095
if (localizedMessage.empty())
10196
return defaultMessage;
10297
if (printDiagnosticNames) {
103-
llvm::StringRef diagnosticName(diagnosticNameStrings[(unsigned)id]);
98+
llvm::StringRef diagnosticID(DiagnosticEngine::diagnosticIDStringFor(id));
10499
auto localizedDebugDiagnosticMessage =
105-
localizationSaver.save(localizedMessage.str() + diagnosticName.str());
100+
localizationSaver.save(localizedMessage.str() + " [" + diagnosticID.str() + "]");
106101
return localizedDebugDiagnosticMessage;
107102
}
108103
return localizedMessage;

0 commit comments

Comments
 (0)