Remove iterate_params from AST::CallExpr and AST::MethodCallExpr

These lambda iterators are removed because they make working
with IR more complex. Instead, we are using the get_params ()
to access the parameters with the help of a for loop.

Fixes #722 #723

Signed-off-by: Nirmal Patel <npate012@gmail.com>

1 files changed, 6 insertions, 8 deletions

diff --git a/gcc/rust/resolve/rust-ast-resolve-expr.h b/gcc/rust/resolve/rust-ast-resolve-expr.h
index 48906f5..12a4f8c 100644
--- a/gcc/rust/resolve/rust-ast-resolve-expr.h
+++ b/gcc/rust/resolve/rust-ast-resolve-expr.h
@@ -101,10 +101,9 @@ public:
   void visit (AST::CallExpr &expr) override
   {
     ResolveExpr::go (expr.get_function_expr ().get (), expr.get_node_id ());
-    expr.iterate_params ([&] (AST::Expr *p) mutable -> bool {
-      ResolveExpr::go (p, expr.get_node_id ());
-      return true;
-    });
+    auto const &in_params = expr.get_params ();
+    for (auto &param : in_params)
+      ResolveExpr::go (param.get (), expr.get_node_id ());
   }
 
   void visit (AST::MethodCallExpr &expr) override
@@ -117,10 +116,9 @@ public:
 	ResolveTypeToCanonicalPath::type_resolve_generic_args (args);
       }
 
-    expr.iterate_params ([&] (AST::Expr *p) mutable -> bool {
-      ResolveExpr::go (p, expr.get_node_id ());
-      return true;
-    });
+    auto const &in_params = expr.get_params ();
+    for (auto &param : in_params)
+      ResolveExpr::go (param.get (), expr.get_node_id ());
   }
 
   void visit (AST::AssignmentExpr &expr) override