Changeset 138e29e for src/Common

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

Location:

src/Common

Files:

• SemanticError.cc (modified) (2 diffs)
• SemanticError.h (modified) (2 diffs)
• utility.h (modified) (1 diff)

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 );
 }
 

src/Common/SemanticError.h ¶

@@ -25 +25 @@
 #include "utility.h"
 
+struct error {
+        std::string description;
+        CodeLocation location;
+
+        error() = default;
+        error( const std::string& str ) : description( str ) {}
+
+        void maybeSet( const CodeLocation& location ) {
+                if( this->location.linenumber < 0 ) {
+                        this->location = location;
+                }
+        }
+};
+
 class SemanticError : public std::exception {
   public:
@@ -41 +55 @@
         // representation of the obj (T must have a print method)
   private:
-        std::list< std::string > errors;
-        CodeLocation location;
+        std::list< error > errors;
 };
 

src/Common/utility.h ¶

@@ -320 +320 @@
 
 inline std::string to_string( const CodeLocation& location ) {
-        return location.filename + ":" + std::to_string(location.linenumber);
+        return location.linenumber >= 0 ? location.filename + ":" + std::to_string(location.linenumber) + " " : "";
 }
 #endif // _UTILITY_H