diff --git a/poincare/src/layout/binomial_coefficient_layout.cpp b/poincare/src/layout/binomial_coefficient_layout.cpp index e5216be40..fa097578f 100644 --- a/poincare/src/layout/binomial_coefficient_layout.cpp +++ b/poincare/src/layout/binomial_coefficient_layout.cpp @@ -15,7 +15,8 @@ extern "C" { namespace Poincare { ExpressionLayout * BinomialCoefficientLayout::clone() const { - return new BinomialCoefficientLayout(const_cast(this)->nLayout(), const_cast(this)->kLayout(), true); + BinomialCoefficientLayout * layout = new BinomialCoefficientLayout(const_cast(this)->nLayout(), const_cast(this)->kLayout(), true); + return layout; } ExpressionLayoutCursor BinomialCoefficientLayout::cursorLeftOf(ExpressionLayoutCursor cursor, bool * shouldRecomputeLayout) { diff --git a/poincare/src/layout/dynamic_layout_hierarchy.cpp b/poincare/src/layout/dynamic_layout_hierarchy.cpp index 39f48e64f..5e0b9a57c 100644 --- a/poincare/src/layout/dynamic_layout_hierarchy.cpp +++ b/poincare/src/layout/dynamic_layout_hierarchy.cpp @@ -21,7 +21,7 @@ DynamicLayoutHierarchy::DynamicLayoutHierarchy(const ExpressionLayout * const * { assert(children != nullptr); m_children = new const ExpressionLayout * [numberOfChildren]; - for (int i=0; iclone(); @@ -55,10 +55,10 @@ void DynamicLayoutHierarchy::addChildrenAtIndex(const ExpressionLayout * const * const ExpressionLayout ** newOperands = new const ExpressionLayout * [m_numberOfChildren+numberOfOperands]; int currentIndex = 0; assert(indexForInsertion <= m_numberOfChildren); - for (int i=0; iisEmpty() || (i < numberOfOperands-1 && operands[i+1]->mustHaveLeftSibling())) @@ -67,7 +67,7 @@ void DynamicLayoutHierarchy::addChildrenAtIndex(const ExpressionLayout * const * newOperands[currentIndex++] = operands[i]; } } - for (int i=indexForInsertion; i= 0 && index <= m_numberOfChildren); const ExpressionLayout ** newChildren = new const ExpressionLayout * [m_numberOfChildren+1]; int j = 0; - for (int i=0; i<=m_numberOfChildren; i++) { + for (int i = 0; i <= m_numberOfChildren; i++) { if (i == index) { child->setParent(this); newChildren[i] = child; diff --git a/poincare/src/layout/static_layout_hierarchy.cpp b/poincare/src/layout/static_layout_hierarchy.cpp index 224902945..3d69b4384 100644 --- a/poincare/src/layout/static_layout_hierarchy.cpp +++ b/poincare/src/layout/static_layout_hierarchy.cpp @@ -51,7 +51,7 @@ template void StaticLayoutHierarchy::build(const ExpressionLayout * const * operands, int numberOfOperands, bool cloneOperands) { assert(operands != nullptr); assert(numberOfOperands <= T); - for (int i=0; i < numberOfOperands; i++) { + for (int i = 0; i < numberOfOperands; i++) { if (cloneOperands && operands[i] != nullptr) { m_children[i] = operands[i]->clone(); } else {