Skip to content

Commit 918863d

Browse files
authored
[clang][Diagnostics] Make 'note' color CYAN (#66997)
Just using BLACK makes it invisible in terminals with a dark background.
1 parent 30fe876 commit 918863d

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

clang/docs/ReleaseNotes.rst

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -212,6 +212,8 @@ Improvements to Clang's diagnostics
212212
(`#51567: <https://github.com/llvm/llvm-project/issues/51567>`_)
213213
- Clang now diagnoses narrowing implicit conversions on variable initializers in immediate
214214
function context and on constexpr variable template initializers.
215+
- Clang now prints its 'note' diagnostic in cyan instead of black, to be more compatible
216+
with terminals with dark background colors. This is also more consistent with GCC.
215217

216218
Bug Fixes in This Version
217219
-------------------------

clang/lib/Frontend/TextDiagnostic.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,7 @@
2424

2525
using namespace clang;
2626

27-
static const enum raw_ostream::Colors noteColor =
28-
raw_ostream::BLACK;
27+
static const enum raw_ostream::Colors noteColor = raw_ostream::CYAN;
2928
static const enum raw_ostream::Colors remarkColor =
3029
raw_ostream::BLUE;
3130
static const enum raw_ostream::Colors fixitColor =

0 commit comments

Comments
 (0)