Changes in src/ResolvExpr/AlternativeFinder.cc [5de1e2c:25fcb84]

File:

• src/ResolvExpr/AlternativeFinder.cc (modified) (3 diffs)

src/ResolvExpr/AlternativeFinder.cc

@@ -324 +324 @@
                 aggInst->lookup( name, members );
 
-                for ( Declaration * decl : members ) {
-                        if ( DeclarationWithType *dwt = dynamic_cast< DeclarationWithType* >( decl ) ) {
-                                // addAnonAlternatives uses vector::push_back, which invalidates references to existing elements, so
-                                // can't construct in place and use vector::back
-                                Alternative newAlt( new MemberExpr( dwt, expr->clone() ), env, newCost );
-                                renameTypes( newAlt.expr );
-                                addAnonConversions( newAlt ); // add anonymous member interpretations whenever an aggregate value type is seen as a member expression.
-                                alternatives.push_back( std::move(newAlt) );
+                for ( std::list< Declaration* >::const_iterator i = members.begin(); i != members.end(); ++i ) {
+                        if ( DeclarationWithType *dwt = dynamic_cast< DeclarationWithType* >( *i ) ) {
+                                alternatives.push_back( Alternative( new MemberExpr( dwt, expr->clone() ), env, newCost ) );
+                                renameTypes( alternatives.back().expr );
+                                addAnonConversions( alternatives.back() ); // add anonymous member interpretations whenever an aggregate value type is seen as a member expression.
                         } else {
                                 assert( false );
@@ -1382 +1379 @@
                         Cost cost = Cost::zero;
                         Expression * newExpr = data.combine( cost );
-
-                        // addAnonAlternatives uses vector::push_back, which invalidates references to existing elements, so
-                        // can't construct in place and use vector::back
-                        Alternative newAlt( newExpr, env, Cost::zero, cost );
+                        alternatives.push_back( Alternative( newExpr, env, Cost::zero, cost ) );
                         PRINT(
                                 std::cerr << "decl is ";
@@ -1394 +1388 @@
                                 std::cerr << std::endl;
                         )
-                        renameTypes( newAlt.expr );
-                        addAnonConversions( newAlt ); // add anonymous member interpretations whenever an aggregate value type is seen as a name expression.
-                        alternatives.push_back( std::move(newAlt) );
+                        renameTypes( alternatives.back().expr );
+                        addAnonConversions( alternatives.back() ); // add anonymous member interpretations whenever an aggregate value type is seen as a name expression.
                 } // for
         }