From f9e419dcd623ec723c744701e77aa61ba0ece0b2 Mon Sep 17 00:00:00 2001 From: Ruben Dashyan Date: Thu, 23 Jan 2020 15:26:04 +0100 Subject: [PATCH] [poincare/expression] defaultDeepReduceChildren does not recompute the number of children --- poincare/src/expression.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/poincare/src/expression.cpp b/poincare/src/expression.cpp index ae4d3c1a1..16a0c757f 100644 --- a/poincare/src/expression.cpp +++ b/poincare/src/expression.cpp @@ -312,7 +312,9 @@ Expression Expression::addMissingParentheses() { } void Expression::defaultDeepReduceChildren(ExpressionNode::ReductionContext reductionContext) { - for (int i = 0; i < numberOfChildren(); i++) { + const int childrenCount = numberOfChildren(); + for (int i = 0; i < childrenCount; i++) { + assert(childrenCount == numberOfChildren()); childAtIndex(i).deepReduce(reductionContext); } }