Skip to content

Commit 2f3b098

Browse files
committed
BADNUM values sorted at the end - no need for first - tested in IE11
1 parent 8a4bae3 commit 2f3b098

File tree

1 file changed

+4
-10
lines changed

1 file changed

+4
-10
lines changed

src/lib/search.js

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -73,25 +73,19 @@ exports.sorterDes = function(a, b) { return b - a; };
7373
*/
7474
exports.distinctVals = function(valsIn) {
7575
var vals = valsIn.slice(); // otherwise we sort the original array...
76-
vals.sort(exports.sorterAsc);
77-
78-
var first;
79-
for(first = 0; first < vals.length; first++) {
80-
if(vals[first] !== BADNUM) break;
81-
}
76+
vals.sort(exports.sorterAsc); // undefined listed in the end - also works on IE11
8277

8378
var last;
8479
for(last = vals.length - 1; last > -1; last--) {
8580
if(vals[last] !== BADNUM) break;
8681
}
8782

88-
var minDiff = (vals[last] - vals[first]) || 1;
89-
var errDiff = minDiff / ((last - first) || 1) / 10000;
83+
var minDiff = (vals[last] - vals[0]) || 1;
84+
var errDiff = minDiff / (last || 1) / 10000;
9085
var newVals = [];
9186
var preV;
92-
for(var i = first; i <= last; i++) {
87+
for(var i = 0; i <= last; i++) {
9388
var v = vals[i];
94-
if(v === BADNUM) continue;
9589

9690
// make sure values aren't just off by a rounding error
9791
var diff = v - preV;

0 commit comments

Comments
 (0)