Changeset 9cb2742

Timestamp:

Jul 17, 2023, 1:47:25 PM (10 months ago)

Author:

caparsons <caparson@…>

Branches:

master

Children:

f3c02ea

Parents:

c2c1717

Message:

refactored some waituntil code gen to be more concise

File:

• src/Concurrency/Waituntil.cpp (modified) (3 diffs)

src/Concurrency/Waituntil.cpp

@@ -625 +625 @@
 CompoundStmt * GenerateWaitUntilCore::genStmtBlock( const WhenClause * clause, const ClauseData * data ) {
     const CodeLocation & cLoc = clause->location;
+    // return new CompoundStmt( cLoc, // OLD
+    //     {
+    //         new ExprStmt( cLoc,
+    //             genSelectTraitCall( clause, data, "on_selected" )
+    //         ),
+    //         ast::deepCopy( clause->stmt )
+    //     }
+    // );
     return new CompoundStmt( cLoc,
         {
-            new ExprStmt( cLoc,
-                genSelectTraitCall( clause, data, "on_selected" )
-            ),
-            ast::deepCopy( clause->stmt )
+            new IfStmt( cLoc,
+                genSelectTraitCall( clause, data, "on_selected" ),
+                ast::deepCopy( clause->stmt )
+            )
         }
     );
@@ -819 +827 @@
     );
 }
-
+// C_TODO remove unused param if change kept
 Stmt * GenerateWaitUntilCore::genNoElseClauseBranch( const WaitUntilStmt * stmt, string & satName, string & runName, string & arrName, string & pCountName, vector<ClauseData *> & clauseData ) {
     CompoundStmt * whileBody = new CompoundStmt( stmt->location );
@@ -834 +842 @@
     );
 
-    whileBody->push_back( genStatusCheckFor( stmt, clauseData, satName ) );
+    whileBody->push_back( genStatusCheckFor( stmt, clauseData, runName ) );
+    // whileBody->push_back( genStatusCheckFor( stmt, clauseData, satName ) ); // OLD
 
     return new CompoundStmt( loc,
         {
             new WhileDoStmt( loc,
-                genNotSatExpr( stmt, satName, arrName ),
+                genNotSatExpr( stmt, runName, arrName ),
+                // genNotSatExpr( stmt, satName, arrName ), // OLD
                 whileBody,  // body
                 {}          // no inits
-            ),
-            genStatusCheckFor( stmt, clauseData, runName )
+            )
+            // , // OLD
+            // genStatusCheckFor( stmt, clauseData, runName ) // OLD
         }
     );