From 6db324219362cbbc9a7f29c193ba4fa002f08e87 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=89milie=20Feral?= <emilie.feral@numworks.com>
Date: Thu, 12 Oct 2017 10:12:01 +0200
Subject: [PATCH] [poincare] Clean

Change-Id: Ia21be93687035b302286a178a91f4a676f052c5b
---
 poincare/src/multiplication.cpp | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/poincare/src/multiplication.cpp b/poincare/src/multiplication.cpp
index 36b6b6c58..1e03a4a3d 100644
--- a/poincare/src/multiplication.cpp
+++ b/poincare/src/multiplication.cpp
@@ -214,8 +214,7 @@ Expression * Multiplication::immediateBeautify() {
     return e->immediateBeautify();
   }
   assert(e == this);
-  int index = 0;
-  while (index < numberOfOperands()) {
+  for (int index = 0; index < numberOfOperands(); index++) {
     // a*b^(-1)*... -> a*.../b
     if (operand(index)->type() == Type::Power && operand(index)->operand(1)->type() == Type::Rational && static_cast<const Rational *>(operand(index)->operand(1))->isMinusOne()) {
       Power * p = static_cast<Power *>((Expression *)operand(index));
@@ -253,8 +252,8 @@ Expression * Multiplication::immediateBeautify() {
       replaceWith(d, true);
       return d;
     }
-    index++;
   }
+  // -1*A -> -A
   if (operand(0)->type() == Type::Rational && static_cast<const Rational *>(operand(0))->isMinusOne()) {
     removeOperand((Expression *)operand(0), true);
     Expression * e = squashUnaryHierarchy();