Changes in src/InitTweak/FixInit.cc [680620d:3c398b6]

File:

• src/InitTweak/FixInit.cc (modified) (4 diffs)

src/InitTweak/FixInit.cc

@@ -390 +390 @@
                         assert( env );
                         CP_CTOR_PRINT( std::cerr << "Type Substitution: " << *env << std::endl; )
-                        assert( arg->result );
+                        assert( arg->has_result() );
                         Type * result = arg->get_result();
                         if ( skipCopyConstruct( result ) ) return; // skip certain non-copyable types
@@ -552 +552 @@
                                 Expression * retExpr = new CommaExpr( assign, new VariableExpr( returnDecl ) );
                                 // move env from callExpr to retExpr
-                                std::swap( retExpr->env, callExpr->env );
+                                retExpr->set_env( callExpr->get_env() );
+                                callExpr->set_env( nullptr );
                                 return retExpr;
                         } else {
@@ -810 +811 @@
                         GuardValue( labelVars );
                         labelVars.clear();
-                        // LabelFinder does not recurse into FunctionDecl, so need to visit
-                        // its children manually.
                         maybeAccept( funcDecl->type, finder );
                         maybeAccept( funcDecl->statements, finder );
@@ -1127 +1126 @@
                         static UniqueName tempNamer( "_tmp_ctor_expr" );
                         // xxx - is the size check necessary?
-                        assert( ctorExpr->result && ctorExpr->get_result()->size() == 1 );
+                        assert( ctorExpr->has_result() && ctorExpr->get_result()->size() == 1 );
 
                         // xxx - ideally we would reuse the temporary generated from the copy constructor passes from within firstArg if it exists and not generate a temporary if it's unnecessary.