From 5bb24107d10842e458665a0a6c489a44d6108796 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Christophe=20Thi=C3=A9ry?= <christopho128@gmail.com>
Date: Fri, 16 Sep 2011 11:35:09 +0200
Subject: [PATCH] Fix a uninitialized variable warning and simplify
 surf_cpu_ti_binary_search

---
 src/surf/cpu_ti.c | 31 ++++++++++++++-----------------
 1 file changed, 14 insertions(+), 17 deletions(-)

diff --git a/src/surf/cpu_ti.c b/src/surf/cpu_ti.c
index 30a42b0787..f40d1c62a6 100644
--- a/src/surf/cpu_ti.c
+++ b/src/surf/cpu_ti.c
@@ -1103,24 +1103,21 @@ static double surf_cpu_ti_solve_trace_simple(surf_cpu_ti_trace_t trace,
 static int surf_cpu_ti_binary_search(double *array, double a, int low,
                                      int high)
 {
-  xbt_assert(low<high,"Wrong parameters: low (%d) should be smaller than high (%d)",low,high);
+  xbt_assert(low < high, "Wrong parameters: low (%d) should be smaller than"
+      " high (%d)", low, high);
 
   int mid;
-  while(low<high) {
-	mid = low + (high - low) / 2;
-	XBT_DEBUG("a %lf low %d high %d mid %d value %lf", a, low, high, mid,
-			array[mid]);
-	/* a == array[mid] */
-	if (array[mid] == a)
-		return mid;
-	/* a is between mid and mid+1 */
-	if (array[mid] < a && array[mid + 1] > a)
-		return mid;
-
-	if (array[mid] < a)
-		low = mid + 1;
-	else
-		high = mid - 1;
+  do {
+    mid = low + (high - low) / 2;
+    XBT_DEBUG("a %lf low %d high %d mid %d value %lf", a, low, high, mid,
+        array[mid]);
+
+    if (array[mid] > a)
+      high = mid;
+    else
+      low = mid;
   }
-  return mid;
+  while (low < high - 1);
+
+  return low;
 }
-- 
2.20.1