From 57b8e56cb3a24d7a4136113b3b820972d47be1f0 Mon Sep 17 00:00:00 2001 From: player-03 Date: Tue, 8 Nov 2022 02:42:03 -0500 Subject: [PATCH] Simplify `prepareSpec()`. (#116) `ExprTools.map()` already iterates through blocks and replaces their contents, so there's no need to type out our own loop. --- src/utest/utils/TestBuilder.hx | 17 ++++------------- 1 file changed, 4 insertions(+), 13 deletions(-) diff --git a/src/utest/utils/TestBuilder.hx b/src/utest/utils/TestBuilder.hx index 17df6e0..09c1f77 100644 --- a/src/utest/utils/TestBuilder.hx +++ b/src/utest/utils/TestBuilder.hx @@ -275,19 +275,10 @@ class TestBuilder { static function prepareSpec(expr:Expr) { return switch(expr.expr) { - case EBlock(exprs): - var newExprs = []; - for(expr in exprs) { - switch(expr.expr) { - case EBinop(op, left, right): - newExprs.push(parseSpecBinop(expr, op, left, right)); - case EUnop(op, prefix, subj): - newExprs.push(parseSpecUnop(expr, op, prefix, subj)); - case _: - newExprs.push(ExprTools.map(expr, prepareSpec)); - } - } - {expr:EBlock(newExprs), pos:expr.pos}; + case EBinop(op, left, right): + parseSpecBinop(expr, op, left, right); + case EUnop(op, prefix, subj): + parseSpecUnop(expr, op, prefix, subj); case _: ExprTools.map(expr, prepareSpec); }