Changeset 16ba4897 for src/AST

Timestamp:

Oct 9, 2024, 5:07:59 PM (2 weeks ago)

Author:

Andrew Beach <ajbeach@…>

Branches:

master

Children:

0766399

Parents:

1b770e40

Message:

Replaced SemanticErrorException::isEmpty with ...::throwIfNonEmpty. This is how it was used in every context and it saves a bit of text (if not two lines) at every use. I considered putting this function in the header for better inlining, but this should have at least the same preformance as the last version.

Location:

src/AST

Files:

• Node.hpp (modified) (1 diff)
• Pass.impl.hpp (modified) (3 diffs)

src/AST/Node.hpp

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

src/AST/Pass.impl.hpp

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