From 7f71ab519868e88df3e501e6cd951f3882493520 Mon Sep 17 00:00:00 2001
From: firewave <firewave@users.noreply.github.com>
Date: Sun, 21 Jul 2024 20:21:01 +0200
Subject: [PATCH] programmemory.cpp: avoid duplicated evaluation in
 `ProgramMemoryState::removeModifiedVars()`

Co-authored-by: Paul Fultz II <pfultz2@yahoo.com>
---
 lib/programmemory.cpp | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/lib/programmemory.cpp b/lib/programmemory.cpp
index 2619ed876b2..29ff0c6b480 100644
--- a/lib/programmemory.cpp
+++ b/lib/programmemory.cpp
@@ -505,9 +505,11 @@ void ProgramMemoryState::removeModifiedVars(const Token* tok)
 {
     const ProgramMemory& pm = state;
     auto eval = [&](const Token* cond) -> std::vector<MathLib::bigint> {
-        if (conditionIsTrue(cond, pm, *settings))
+        ProgramMemory pm2 = pm;
+        auto result = execute(cond, pm2, *settings);
+        if (isTrue(result))
             return {1};
-        if (conditionIsFalse(cond, pm, *settings))
+        if (isFalse(result))
             return {0};
         return {};
     };