Fix NaN values confusing closest-point search
authorKlaus Weidner <klausw@google.com>
Tue, 28 Feb 2012 04:36:01 +0000 (20:36 -0800)
committerKlaus Weidner <klausw@google.com>
Tue, 28 Feb 2012 04:55:23 +0000 (20:55 -0800)
commita937d03183e33deea11a8d020cb1265c266e7f41
tree25dc83cbcd66a0ca4aa027ceacedd926e794cb60
parenteb96f83382a3b80db03db8b5fd06f02e65cedf7d
Fix NaN values confusing closest-point search

The current algorithm was not handling NaN values right, the "dist >= max"
check is false for dist=NaN and led to the "this point is closer" branch
being taken. Fix this by reversing the if logic, and add a regression
test for it. (I confirmed that the test fails for the original code,
and passes after the change.)
auto_tests/tests/callback.js
dygraph.js