Changeset 138e29e for src/Common/SemanticError.cc

Timestamp:

Feb 14, 2017, 3:53:52 PM (8 years ago)

Author:

Thierry Delisle <tdelisle@…>

Branches:

ADT, aaron-thesis, arm-eh, ast-experimental, cleanup-dtors, deferred_resn, demangler, enum, forall-pointer-decay, jacob/cs343-translation, jenkins-sandbox, master, new-ast, new-ast-unique-expr, new-env, no_list, persistent-indexer, pthread-emulation, qualifiedEnum, resolv-new, with_gc

Children:

9bb90a86

Parents:

294647b

Message:

Implemented filename and linenumber errors in most cases, only missing constructor errors apparently

File:

• src/Common/SemanticError.cc (modified) (2 diffs)

src/Common/SemanticError.cc

@@ -22 +22 @@
 #include "SemanticError.h"
 
+#include <unistd.h>
+
+inline const std::string& error_str() {
+        static std::string str = isatty( fileno(stderr) ) ? "\e[31merror:\e[39m " : "error: ";
+        return str;
+}
+
 SemanticError::SemanticError() {
 }
 
 SemanticError::SemanticError( std::string error ) {
-  append( error );
+        append( error );
 }
 
 void SemanticError::append( SemanticError &other ) {
-  errors.splice( errors.end(), other.errors );
+        errors.splice( errors.end(), other.errors );
 }
 
 void SemanticError::append( const std::string & msg ) {
-  using std::to_string;
-  const std::string loc = location.linenumber >= 0 ? "At \"" + to_string(location) + "\" " : "";
-  errors.push_back( loc + "Error: " + msg );
+        errors.emplace_back( error_str() + msg );
 }
 
@@ -44 +49 @@
 
 void SemanticError::print( std::ostream &os ) {
-        std::copy( errors.begin(), errors.end(), std::ostream_iterator< std::string >( os, "\n" ) );
+        using std::to_string;
+        for(auto err : errors) {
+                os << to_string( err.location ) << err.description << '\n';
+        }
 }
 
 void SemanticError::set_location( const CodeLocation& location ) {
-  this->location = location;
-  using std::to_string;
-  const std::string loc = location.linenumber >= 0 ? "At \"" + to_string(location) + "\" " : "";
-  auto& error = *errors.begin();
-  error.insert( 0, loc.c_str());
+        errors.begin()->maybeSet( location );
 }