Merge pull request #11970 from shikharvashistha/colors

Added different colors for warning & error

Author: hrkrshnn

Changelog.md

@@ -7,6 +7,7 @@ Language Features:
 Compiler Features:
  * Commandline Interface: Accept nested brackets in step sequences passed to ``--yul-optimizations``.
  * Commandline Interface: Add ``--debug-info`` option for selecting how much extra debug information should be included in the produced EVM assembly and Yul code.
+ * Commandline Interface: Use different colors when printing errors, warnings and infos.
  * SMTChecker: Output values for ``block.*``, ``msg.*`` and ``tx.*`` variables that are present in the called functions.
  * Standard JSON: Accept nested brackets in step sequences passed to ``settings.optimizer.details.yulDetails.optimizerSteps``.
  * Standard JSON: Add ``settings.debug.debugInfo`` option for selecting how much extra debug information should be included in the produced EVM assembly and Yul code.

liblangutil/Exceptions.cpp

@@ -23,6 +23,9 @@
 
 #include <liblangutil/Exceptions.h>
 
+#include <boost/algorithm/string/case_conv.hpp>
+#include <boost/algorithm/string/trim.hpp>
+
 using namespace std;
 using namespace solidity;
 using namespace solidity::langutil;
@@ -71,3 +74,15 @@ Error::Error(
 	if (!_description.empty())
 		*this << util::errinfo_comment(_description);
 }
+
+optional<Error::Severity> Error::severityFromString(string _input)
+{
+	boost::algorithm::to_lower(_input);
+	boost::algorithm::trim(_input);
+
+	for (Severity severity: {Severity::Error, Severity::Warning, Severity::Info})
+		if (_input == formatErrorSeverityLowercase(severity))
+			return severity;
+
+	return nullopt;
+}

liblangutil/Exceptions.h

@@ -258,6 +258,8 @@ class Error: virtual public util::Exception
 		solAssert(false, "");
 	}
 
+	static std::optional<Severity> severityFromString(std::string _input);
+
 private:
 	ErrorId m_errorId;
 	Type m_type;

liblangutil/SourceReferenceFormatter.cpp

@@ -65,9 +65,21 @@ AnsiColorized SourceReferenceFormatter::frameColored() const
 	return AnsiColorized(m_stream, m_colored, {BOLD, BLUE});
 }
 
-AnsiColorized SourceReferenceFormatter::errorColored() const
+AnsiColorized SourceReferenceFormatter::errorColored(optional<Error::Severity> _severity) const
 {
-	return AnsiColorized(m_stream, m_colored, {BOLD, RED});
+	// We used to color messages of any severity as errors so this seems like a good default
+	// for cases where severity cannot be determined.
+	char const* textColor = RED;
+
+	if (_severity.has_value())
+		switch (_severity.value())
+		{
+		case Error::Severity::Error: textColor = RED; break;
+		case Error::Severity::Warning: textColor = YELLOW; break;
+		case Error::Severity::Info: textColor = WHITE; break;
+		}
+
+	return AnsiColorized(m_stream, m_colored, {BOLD, textColor});
 }
 
 AnsiColorized SourceReferenceFormatter::messageColored() const
@@ -164,9 +176,10 @@ void SourceReferenceFormatter::printSourceLocation(SourceReference const& _ref)
 void SourceReferenceFormatter::printExceptionInformation(SourceReferenceExtractor::Message const& _msg)
 {
 	// exception header line
-	errorColored() << _msg.severity;
+	optional<Error::Severity> severity = Error::severityFromString(_msg.severity);
+	errorColored(severity) << _msg.severity;
 	if (m_withErrorIds && _msg.errorId.has_value())
-		errorColored() << " (" << _msg.errorId.value().error << ")";
+		errorColored(severity) << " (" << _msg.errorId.value().error << ")";
 	messageColored() << ": " << _msg.primary.message << '\n';
 
 	printSourceLocation(_msg.primary);

liblangutil/SourceReferenceFormatter.h

@@ -87,7 +87,7 @@ class SourceReferenceFormatter
 private:
 	util::AnsiColorized normalColored() const;
 	util::AnsiColorized frameColored() const;
-	util::AnsiColorized errorColored() const;
+	util::AnsiColorized errorColored(std::optional<langutil::Error::Severity> _severity) const;
 	util::AnsiColorized messageColored() const;
 	util::AnsiColorized secondaryColored() const;
 	util::AnsiColorized highlightColored() const;