[PATCH] D146827: [diagtool] explain that yellow is used to denote disabled-by-default warnings

[Nick Desaulniers via Phabricator via cfe-commits]

nickdesaulniers created this revision.
Herald added a project: All.
nickdesaulniers requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.

This is already implied, but let's just be explicit about it.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D146827

Files:
  clang/tools/diagtool/TreeView.cpp


Index: clang/tools/diagtool/TreeView.cpp
===================================================================
--- clang/tools/diagtool/TreeView.cpp
+++ clang/tools/diagtool/TreeView.cpp
@@ -127,6 +127,9 @@
   void showKey() {
     out << '\n' << Colors::GREEN << "GREEN" << Colors::RESET
         << " = enabled by default";
+    out << '\n'
+        << Colors::YELLOW << "YELLOW" << Colors::RESET
+        << " = disabled by default";
     out << '\n' << Colors::RED << "RED" << Colors::RESET
         << " = unimplemented (accepted for GCC compatibility)\n\n";
   }



Index: clang/tools/diagtool/TreeView.cpp
===================================================================
--- clang/tools/diagtool/TreeView.cpp
+++ clang/tools/diagtool/TreeView.cpp
@@ -127,6 +127,9 @@
   void showKey() {
     out << '\n' << Colors::GREEN << "GREEN" << Colors::RESET
         << " = enabled by default";
+    out << '\n'
+        << Colors::YELLOW << "YELLOW" << Colors::RESET
+        << " = disabled by default";
     out << '\n' << Colors::RED << "RED" << Colors::RESET
         << " = unimplemented (accepted for GCC compatibility)\n\n";
   }

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits