From 3dc1a716828f512d7a0ff123994ebd254b94edb4 Mon Sep 17 00:00:00 2001 From: Davvex87 Date: Wed, 15 Apr 2026 14:04:08 +0100 Subject: [PATCH] rename variable `isPure` to `hasEffect` We already have a function called isPure so that variable was shadowing it, causing a bunch of issues. How did i not notice this?? --- .../implementations/RemovePureExpressionsImpl.hx | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/reflaxe/preprocessors/implementations/RemovePureExpressionsImpl.hx b/src/reflaxe/preprocessors/implementations/RemovePureExpressionsImpl.hx index ee39c97..f492db1 100644 --- a/src/reflaxe/preprocessors/implementations/RemovePureExpressionsImpl.hx +++ b/src/reflaxe/preprocessors/implementations/RemovePureExpressionsImpl.hx @@ -209,13 +209,13 @@ private class OptimizerTexpr { TSwitch(_), TArrayDecl(_), TBlock(_), TObjectDecl(_), TVar(_): { - var isPure = true; + var hasEffect = false; TypedExprTools.iter(expr, function(subExpr) { if(hasSideEffects(subExpr)) { - isPure = false; + hasEffect = true; } }); - return isPure; + return hasEffect; } case _: { return false;