Changes in src/Tuples/TupleAssignment.cc [938dd75:9058414]

File:

• src/Tuples/TupleAssignment.cc (modified) (2 diffs)

src/Tuples/TupleAssignment.cc

@@ -281 +281 @@
         }
 
-        // removes environments from subexpressions within statement exprs, which could throw off later passes like those in Box which rely on PolyMutator, and adds the bindings to the compositeEnv
+        // removes environments from subexpressions within statement exprs, which could throw off later passes like those in Box which rely on PolyMutator.
         // xxx - maybe this should happen in alternative finder for every StmtExpr?
+        // xxx - it's possible that these environments could contain some useful information. Maybe the right thing to do is aggregate the environments and pass the aggregate back to be added into the compositeEnv
         struct EnvRemover {
                 void previsit( ExprStmt * stmt ) {
-                        assert( compositeEnv );
-                        if ( stmt->expr->env ) {
-                                compositeEnv->add( *stmt->expr->env );
-                                delete stmt->expr->env;
-                                stmt->expr->env = nullptr;
-                        }
-                }
-
-                ResolvExpr::TypeEnvironment * compositeEnv = nullptr;
+                        delete stmt->expr->env;
+                        stmt->expr->env = nullptr;
+                }
         };
 
@@ -305 +300 @@
                         ResolvExpr::resolveCtorInit( ctorInit, spotter.currentFinder.get_indexer() ); // resolve ctor/dtors for the new object
                         PassVisitor<EnvRemover> rm; // remove environments from subexpressions of StmtExprs
-                        rm.pass.compositeEnv = &compositeEnv;
                         ctorInit->accept( rm );
                 }