sheetjs/sheetjs
24
Fork 24
Code Issues 167 Pull Requests 12 Wiki Activity

Fix potential infinite loop condition

Browse Source 
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.
This commit is contained in:
TeamworkGuy2 2017-07-16 21:52:44 +00:00 committed by GitHub
parent 6a913d3a7b
commit d0df763657
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
test.js
View File

@ -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<x.length; ++i) assert.deepEqual(x[0], x[i]); }
function deepcmp(x,y,k,m,c) {
var s = k.indexOf(".");