From b5ef65159649daaab83bcc9dcffb89a52dec19a5 Mon Sep 17 00:00:00 2001 From: Nathan Peles Date: Thu, 31 Oct 2013 13:03:55 +1100 Subject: [PATCH] Fix warnings from jslint. --- lazywrite.js | 34 +++++++++++++++++++--------------- 1 file changed, 19 insertions(+), 15 deletions(-) diff --git a/lazywrite.js b/lazywrite.js index 1114281..94d9ced 100755 --- a/lazywrite.js +++ b/lazywrite.js @@ -59,9 +59,7 @@ originalWrite = document.write, originalWriteln = document.writeln, originalOnError = window.onerror, - documentWrite = originalWrite.apply - ? function(){ originalWrite.apply(document, arguments); } - : /* handle IE issue */ originalWrite; + documentWrite = (originalWrite.apply) ? function(){ originalWrite.apply(document, arguments); } : originalWrite /* handle IE issue */; // error catcher @@ -206,7 +204,11 @@ // execute the global scripts stack // return continue flag function executeScripts(){ - for (var flag; flag = executeScript(scriptStack.shift());); + var flag; + do { + flag = executeScript(scriptStack.shift()); + } while (flag); + return flag !== false && !scriptBlocker; } @@ -242,10 +244,8 @@ // render in the document if (previousHolder === renderHolder) { // append the stack after last script stack in the global script stack. - scriptStack = ( - // remove executed stack item first - newStack = scriptStack.n.slice(scriptStack.l - scriptStack.length).concat(stack) - ).concat(oldStack = scriptStack.o); + // remove executed stack item first + scriptStack = (newStack = scriptStack.n.slice(scriptStack.l - scriptStack.length).concat(stack)).concat(oldStack = scriptStack.o); scriptStack.n = newStack; scriptStack.o = oldStack; @@ -291,7 +291,10 @@ // render the global write stack function renderStack(){ - while (renderWrite(currentWrite = writeStack.shift())); + var result; + do { + result = renderWrite(currentWrite = writeStack.shift()); + } while (result); if (continued && !writeStack.length) { // remove parallel holder, if it exists @@ -300,11 +303,10 @@ } // destroy objects - scriptBlocker - = scriptHolder - = previousHolder - = parallelHolder - = undef; + scriptBlocker = undef; + scriptHolder = undef; + previousHolder = undef; + parallelHolder = undef; // restore original functions document.write = originalWrite; @@ -371,6 +373,7 @@ return (sHtmlTag = name); } } + return null; } // check html is closed @@ -480,7 +483,8 @@ } for (; script = matches.pop() ;) { replaceElement(script, holder = createHolder()); - if (src = script.getAttribute('src')) { + src = script.getAttribute('src'); + if (src) { appendElement(renderParser, createElement('script')).src = src; } appendElement(renderParser, script);