From d0df7636572d91aa6b52bf166d6a12b57a25ac13 Mon Sep 17 00:00:00 2001 From: TeamworkGuy2 Date: Sun, 16 Jul 2017 21:52:44 +0000 Subject: [PATCH] Fix potential infinite loop condition I was doing some testing with js-xlsx's test suite and ran into a test that was hanging. Turns out it was cmparr()'s for-loop condition. Switching from `!=` to `<` fixed the hung test and caused it to error correctly. --- test.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test.js b/test.js index 8dcf36c..66c4942 100644 --- a/test.js +++ b/test.js @@ -683,7 +683,7 @@ function custprop(wb) { assert.equal(wb.Custprops.Counter, -3.14); } -function cmparr(x){ for(var i=1;i!=x.length;++i) assert.deepEqual(x[0], x[i]); } +function cmparr(x){ for(var i=1; i