Changes in / [0766399:7d415b4]

Location:

src

Files:

• AST/Node.hpp (modified) (1 diff)
• AST/Pass.impl.hpp (modified) (3 diffs)
• Common/ErrorObjects.hpp (modified) (1 diff)
• Common/SemanticError.cpp (modified) (1 diff)
• ControlStruct/MultiLevelExit.cpp (modified) (1 diff)
• InitTweak/FixInit.cpp (modified) (2 diffs)
• Parser/DeclarationNode.cpp (modified) (3 diffs)
• Parser/DeclarationNode.hpp (modified) (1 diff)
• ResolvExpr/CandidateFinder.cpp (modified) (1 diff)
• Validate/CheckAssertions.cpp (modified) (2 diffs)
• Validate/ReplaceTypedef.cpp (modified) (1 diff)

src/AST/Node.hpp

@@ -161 +161 @@
                 }
         }
-        errors.throwIfNonEmpty();
+        if ( ! errors.isEmpty() ) {
+                throw errors;
+        }
 }
 

src/AST/Pass.impl.hpp

@@ -250 +250 @@
         }
         pass_visitor_stats.depth--;
-        errors.throwIfNonEmpty();
+        if ( !errors.isEmpty() ) { throw errors; }
 
         return new_kids;
@@ -287 +287 @@
         __pedantic_pass_assert( new_kids.size() == container.size() );
         pass_visitor_stats.depth--;
-        errors.throwIfNonEmpty();
+        if ( !errors.isEmpty() ) { throw errors; }
 
         return ast::__pass::resultN<container_t, node_t>{ mutated, new_kids };
@@ -401 +401 @@
         }
         pass_visitor_stats.depth--;
-        errors.throwIfNonEmpty();
+        if ( !errors.isEmpty() ) { throw errors; }
 }
 

src/Common/ErrorObjects.hpp

@@ -41 +41 @@
         void append( SemanticErrorException & other );
         void append( CodeLocation location, const std::string & );
-        void throwIfNonEmpty();
+        bool isEmpty() const;
         void print();
   private:

src/Common/SemanticError.cpp

@@ -88 +88 @@
 }
 
-void SemanticErrorException::throwIfNonEmpty() {
-        if ( !errors.empty() ) {
-                throw *this;
-        }
+bool SemanticErrorException::isEmpty() const {
+        return errors.empty();
 }
 

src/ControlStruct/MultiLevelExit.cpp

@@ -688 +688 @@
         }
 
-        errors.throwIfNonEmpty();
+        if ( !errors.isEmpty() ) {
+                throw errors;
+        }
         return ret;
 }

src/InitTweak/FixInit.cpp

@@ -301 +301 @@
                 } // try
         } // for
-        errors.throwIfNonEmpty();
+        if ( ! errors.isEmpty() ) {
+                throw errors;
+        } // if
 }
 
@@ -1171 +1173 @@
                 function->stmts = mutStmts;
         }
-        errors.throwIfNonEmpty();
+        if (! errors.isEmpty()) {
+                throw errors;
+        }
         return function;
 }

src/Parser/DeclarationNode.cpp

@@ -829 +829 @@
         } // for
 
-        errors.throwIfNonEmpty();
+        if ( ! errors.isEmpty() ) {
+                throw errors;
+        } // if
 } // buildList
 
@@ -877 +879 @@
         } // for
 
-        errors.throwIfNonEmpty();
+        if ( ! errors.isEmpty() ) {
+                throw errors;
+        } // if
 } // buildList
 
@@ -893 +897 @@
         } // for
 
-        errors.throwIfNonEmpty();
+        if ( ! errors.isEmpty() ) {
+                throw errors;
+        } // if
 } // buildTypeList
 

src/Parser/DeclarationNode.hpp

@@ -156 +156 @@
                 } // try
         } // for
-        errors.throwIfNonEmpty();
+        if ( ! errors.isEmpty() ) {
+                throw errors;
+        } // if
 }
 

src/ResolvExpr/CandidateFinder.cpp

@@ -1115 +1115 @@
                 // Implement SFINAE; resolution errors are only errors if there aren't any non-error
                 // candidates
-                if ( found.empty() ) errors.throwIfNonEmpty();
+                if ( found.empty() && ! errors.isEmpty() ) { throw errors; }
 
                 // only keep the best matching intrinsic result to match C semantics (no unexpected narrowing/widening)

src/Validate/CheckAssertions.cpp

@@ -60 +60 @@
                 makeTypeVarMap( decl, typeVars );
                 checkList( typeVars, errors, decl->assertions );
-                errors.throwIfNonEmpty();
+                if ( !errors.isEmpty() ) { throw errors; }
         }
 
@@ -72 +72 @@
                 }
                 if ( checkMembers ) checkList( typeVars, errors, decl->members );
-                errors.throwIfNonEmpty();
+                if ( !errors.isEmpty() ) { throw errors; }
         }
 

src/Validate/ReplaceTypedef.cpp

@@ -327 +327 @@
         assert( declsToAddBefore.empty() );
         assert( declsToAddAfter.empty() );
-        errors.throwIfNonEmpty();
+        if ( !errors.isEmpty() ) { throw errors; }
 
         mut->members.clear();