From 24449d2dccbf845543fb7ad9544868f2d68b1934 Mon Sep 17 00:00:00 2001 From: Victor Woeltjen Date: Mon, 14 Sep 2015 11:44:50 -0700 Subject: [PATCH] [Time Controller] Fix series subsetting Fix binary search implementation used to subset telemetry series for time conductor. WTD-1515 --- .../conductor/src/ConductorTelemetrySeries.js | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/platform/features/conductor/src/ConductorTelemetrySeries.js b/platform/features/conductor/src/ConductorTelemetrySeries.js index 044e35ca72..94daeb067a 100644 --- a/platform/features/conductor/src/ConductorTelemetrySeries.js +++ b/platform/features/conductor/src/ConductorTelemetrySeries.js @@ -11,15 +11,9 @@ define( var mid = Math.floor((min + max) / 2), domainValue = series.getDomainValue(mid); - if (min >= max) { - return min; - } - - if (domainValue < value) { - return binSearch(mid + 1, max); - } else { - return binSearch(min, mid - 1); - } + return min >= max ? min : + domainValue < value ? binSearch(mid + 1, max, value) : + binSearch(min, mid - 1, value); } this.startIndex = binSearch(0, max, conductor.displayStart());