Changeset fca78f1 for src/Parser/StatementNode.cpp

Timestamp:

Sep 23, 2024, 11:14:56 AM (6 weeks ago)

Author:

Andrew Beach <ajbeach@…>

Branches:

master

Children:

738a9b4

Parents:

b723b63

Message:

Added ForeachStmt? (felt better than ForEachStmt?). This new node is a bit optimistic in that currently it is covering a very narrow case, but with improvements to it and RangeExpr?, it could handle many more kind of loops.

File:

• src/Parser/StatementNode.cpp (modified) (1 diff)

src/Parser/StatementNode.cpp

@@ -213 +213 @@
         if ( forctl->range_over ) {
                 ast::Expr * range_over = maybeMoveBuild( forctl->range_over );
-                auto kind = forctl->kind;                                               // save before delete, used in return
+                bool isIncreasing = forctl->kind == OperKinds::LEThan;
+                // Copy all the data needed before the delete.
                 delete forctl;
-                return new ast::ForStmt( location,
+                return new ast::ForeachStmt( location,
                         std::move( astinit ),
-                        range_over, kind == OperKinds::LEThan,
+                        range_over,
+                        isIncreasing ? ast::IncreasingRange : ast::DecreasingRange,
                         buildMoveSingle( stmt ),
                         buildMoveOptional( else_ )