Refactor the repeat logic in readBlock function.

In the function repeat the variabe i is not defined in the scope of the
function. This function was from moved by
92fa629d107cf4de1cb486366d783e890e153306 from its original place which had
the i as defined. This fix avoids the scope dependency.

diff --git a/pdf.js b/pdf.js
index ebce3d1b016310d8f5ecfc4f24ad222ffce4dc80..38e4bbdbca3657bdfbd57aa4cca4b4d73f20f7fd 100644 (file)
--- a/pdf.js
+++ b/pdf.js
@@ -556,12 +556,6 @@ var FlateStream = (function() {
   };
 
   constructor.prototype.readBlock = function() {
-    function repeat(stream, array, len, offset, what) {
-      var repeatLength = stream.getBits(len) + offset;
-      while (repeatLength-- > 0)
-        array[i++] = what;
-    }
-
     // read block header
     var hdr = this.getBits(3);
     if (hdr & 1)
@@ -631,14 +625,19 @@ var FlateStream = (function() {
       while (i < codes) {
         var code = this.getCode(codeLenCodeTab);
         if (code == 16) {
-          repeat(this, codeLengths, 2, 3, len);
+          var bitsLength = 2, bitsOffset = 3, what = len;
         } else if (code == 17) {
-          repeat(this, codeLengths, 3, 3, len = 0);
+          var bitsLength = 3, bitsOffset = 3, what = (len = 0);
         } else if (code == 18) {
-          repeat(this, codeLengths, 7, 11, len = 0);
+          var bitsLength = 7, bitsOffset = 11, what = (len = 0);
         } else {
           codeLengths[i++] = len = code;
+          continue;
         }
+
+        var repeatLength = this.getBits(bitsLength) + bitsOffset;
+        while (repeatLength-- > 0)
+          codeLengths[i++] = what;
       }
 
       litCodeTable =