fixed naming-varname selfcheck warnings

Author: firewave

lib/color.cpp

@@ -24,7 +24,7 @@
 #include <cstddef>
 #include <sstream> // IWYU pragma: keep
 
-bool DISABLE_COLORS = false;
+bool gDisableColors = false;
 
 #ifdef _WIN32
 std::ostream& operator<<(std::ostream& os, const Color& /*c*/)
@@ -34,7 +34,7 @@ std::ostream& operator<<(std::ostream & os, const Color& c)
 {
     // TODO: handle piping into file as well as other pipes like stderr
     static const bool s_is_tty = isatty(STDOUT_FILENO);
-    if (!DISABLE_COLORS && s_is_tty)
+    if (!gDisableColors && s_is_tty)
         return os << "\033[" << static_cast<std::size_t>(c) << "m";
 #endif
     return os;

lib/color.h

@@ -42,6 +42,6 @@ CPPCHECKLIB std::ostream& operator<<(std::ostream& os, const Color& c);
 
 CPPCHECKLIB std::string toString(const Color& c);
 
-extern CPPCHECKLIB bool DISABLE_COLORS; // for testing
+extern CPPCHECKLIB bool gDisableColors; // for testing
 
 #endif

test/main.cpp

@@ -31,7 +31,7 @@ int main(int argc, char *argv[])
 #endif
 
     Preprocessor::macroChar = '$';     // While macroChar is char(1) per default outside test suite, we require it to be a human-readable character here.
-    DISABLE_COLORS = true;
+    gDisableColors = true;
 
     options args(argc, argv);