From eb4ee24a412380c5e316b4dba6f8a60948e5de6d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?L=C3=A9a=20Saviot?= <lea.saviot@numworks.com>
Date: Tue, 26 Jun 2018 13:31:17 +0200
Subject: [PATCH] [apps/shared] Use activeFunctionStoreIndex in
 activeFunctionAtIndex

---
 apps/shared/function_store.cpp | 19 +++++--------------
 1 file changed, 5 insertions(+), 14 deletions(-)

diff --git a/apps/shared/function_store.cpp b/apps/shared/function_store.cpp
index 377d535c0..f321582ce 100644
--- a/apps/shared/function_store.cpp
+++ b/apps/shared/function_store.cpp
@@ -9,23 +9,14 @@ FunctionStore::FunctionStore() :
 }
 
 Function * FunctionStore::activeFunctionAtIndex(int i) {
-  assert(i>=0 && i<m_numberOfModels);
-  int index = 0;
-  for (int k = 0; k < m_numberOfModels; k++) {
-    Function * function = modelAtIndex(k);
-    if (function->isActive() && function->isDefined()) {
-      if (i == index) {
-        return function;
-      }
-      index++;
-    }
-  }
-  assert(false);
-  return nullptr;
+  assert(i >= 0 && i < numberOfActiveFunctions());
+  Function * function = modelAtIndex(activeFunctionStoreIndex(i));
+  assert(function->isActive() && function->isDefined());
+  return function;
 }
 
 int FunctionStore::activeFunctionStoreIndex(int i) {
-  assert(i>=0 && i<numberOfActiveFunctions());
+  assert(i >= 0 && i < numberOfActiveFunctions());
   int index = 0;
   for (int k = 0; k < m_numberOfModels; k++) {
     Function * function = modelAtIndex(k);