From ca1374eb05490d51362364d3de366918ea700454 Mon Sep 17 00:00:00 2001
From: Vitaly Puzrin <vitaly@rcdesign.ru>
Date: Sun, 13 Apr 2014 13:45:12 +0400
Subject: [PATCH] Moved utilities to separate folder

---
 .../deflate-pako-string/index.js              |   2 +-
 .../deflate-pako-untyped/index.js             |   2 +-
 .../inflate-pako-untyped/index.js             |   2 +-
 index.js                                      |   2 +-
 lib/deflate.js                                |   7 +-
 lib/inflate.js                                |   7 +-
 lib/utils/common.js                           | 104 +++++++++++++++++
 lib/{zlib/utils.js => utils/strings.js}       | 108 +-----------------
 lib/zlib/deflate.js                           |   2 +-
 lib/zlib/inflate.js                           |   2 +-
 lib/zlib/inftrees.js                          |   2 +-
 lib/zlib/trees.js                             |   2 +-
 test/chunks.js                                |   2 +-
 test/gzip_specials.js                         |   2 +-
 test/helpers.js                               |   2 +-
 test/strings.js                               |   3 +-
 16 files changed, 129 insertions(+), 122 deletions(-)
 create mode 100644 lib/utils/common.js
 rename lib/{zlib/utils.js => utils/strings.js} (59%)

diff --git a/benchmark/implementations/deflate-pako-string/index.js b/benchmark/implementations/deflate-pako-string/index.js
index e91743d..e94bfd7 100644
--- a/benchmark/implementations/deflate-pako-string/index.js
+++ b/benchmark/implementations/deflate-pako-string/index.js
@@ -1,7 +1,7 @@
 'use strict'
 
 var pako = require('../../../');
-var utils = require('../../../lib/zlib/utils');
+var utils = require('../../../lib/utils/common');
 
 exports.run = function(data, level) {
   pako.deflate(data.string, {
diff --git a/benchmark/implementations/deflate-pako-untyped/index.js b/benchmark/implementations/deflate-pako-untyped/index.js
index 4fd5455..986c3fb 100644
--- a/benchmark/implementations/deflate-pako-untyped/index.js
+++ b/benchmark/implementations/deflate-pako-untyped/index.js
@@ -1,7 +1,7 @@
 'use strict'
 
 var pako = require('../../../');
-var utils = require('../../../lib/zlib/utils');
+var utils = require('../../../lib/utils/common');
 
 exports.run = function(data, level) {
   utils.setTyped(false);
diff --git a/benchmark/implementations/inflate-pako-untyped/index.js b/benchmark/implementations/inflate-pako-untyped/index.js
index bd1b157..5d160de 100644
--- a/benchmark/implementations/inflate-pako-untyped/index.js
+++ b/benchmark/implementations/inflate-pako-untyped/index.js
@@ -1,7 +1,7 @@
 'use strict'
 
 var pako = require('../../../');
-var utils = require('../../../lib/zlib/utils');
+var utils = require('../../../lib/utils/common');
 
 exports.run = function(data, level) {
   utils.setTyped(false);
diff --git a/index.js b/index.js
index 189f577..46cf736 100644
--- a/index.js
+++ b/index.js
@@ -1,7 +1,7 @@
 // Top level file is just a mixin of submodules & constants
 'use strict';
 
-var assign    = require('./lib/zlib/utils').assign;
+var assign    = require('./lib/utils/common').assign;
 
 var deflate   = require('./lib/deflate');
 var inflate   = require('./lib/inflate');
diff --git a/lib/deflate.js b/lib/deflate.js
index d437ebb..b5364fc 100644
--- a/lib/deflate.js
+++ b/lib/deflate.js
@@ -2,7 +2,8 @@
 
 
 var zlib_deflate = require('./zlib/deflate.js');
-var utils = require('./zlib/utils');
+var utils = require('./utils/common');
+var strings = require('./utils/strings');
 var msg = require('./zlib/messages');
 var zstream = require('./zlib/zstream');
 
@@ -187,7 +188,7 @@ Deflate.prototype.push = function(data, mode) {
   // Convert data if needed
   if (typeof data === 'string') {
     // If we need to compress text, change encoding to utf8.
-    strm.next_in = utils.string2buf(data);
+    strm.next_in = strings.string2buf(data);
   } else {
     strm.next_in = data;
   }
@@ -210,7 +211,7 @@ Deflate.prototype.push = function(data, mode) {
     }
     if (strm.avail_out === 0 || (strm.avail_in === 0 && _mode === Z_FINISH)) {
       if (this.options.to === 'string') {
-        this.onData(utils.buf2binstring(utils.shrinkBuf(strm.next_out, strm.next_out_index)));
+        this.onData(strings.buf2binstring(utils.shrinkBuf(strm.next_out, strm.next_out_index)));
       } else {
         this.onData(utils.shrinkBuf(strm.next_out, strm.next_out_index));
       }
diff --git a/lib/inflate.js b/lib/inflate.js
index d75ea0a..a9dfeb5 100644
--- a/lib/inflate.js
+++ b/lib/inflate.js
@@ -2,7 +2,8 @@
 
 
 var zlib_inflate = require('./zlib/inflate.js');
-var utils = require('./zlib/utils');
+var utils = require('./utils/common');
+var strings = require('./utils/strings');
 var c = require('./zlib/constants');
 var msg = require('./zlib/messages');
 var zstream = require('./zlib/zstream');
@@ -187,7 +188,7 @@ Inflate.prototype.push = function(data, mode) {
   // Convert data if needed
   if (typeof data === 'string') {
     // Only binary strings can be decompressed on practice
-    strm.next_in = utils.binstring2buf(data);
+    strm.next_in = strings.binstring2buf(data);
   } else {
     strm.next_in = data;
   }
@@ -231,7 +232,7 @@ Inflate.prototype.push = function(data, mode) {
             tail = 0;
           }
 
-          utf8str = utils.buf2string(strm.next_out, next_out_utf8_index);
+          utf8str = strings.buf2string(strm.next_out, next_out_utf8_index);
 
           // move tail
           strm.next_out_index = tail;
diff --git a/lib/utils/common.js b/lib/utils/common.js
new file mode 100644
index 0000000..6aab672
--- /dev/null
+++ b/lib/utils/common.js
@@ -0,0 +1,104 @@
+'use strict';
+
+
+var TYPED_OK =  (typeof Uint8Array !== 'undefined') &&
+                (typeof Uint16Array !== 'undefined') &&
+                (typeof Int32Array !== 'undefined');
+
+
+exports.assign = function (obj /*from1, from2, from3, ...*/) {
+  var sources = Array.prototype.slice.call(arguments, 1);
+  while (sources.length) {
+    var source = sources.shift();
+    if (!source) { continue; }
+
+    if (typeof(source) !== 'object') {
+      throw new TypeError(source + 'must be non-object');
+    }
+
+    for (var p in source) {
+      if (source.hasOwnProperty(p)) {
+        obj[p] = source[p];
+      }
+    }
+  }
+
+  return obj;
+};
+
+
+// reduce buffer size, avoiding mem copy
+exports.shrinkBuf = function (buf, size) {
+  if (buf.length === size) { return buf; }
+  if (buf.subarray) { return buf.subarray(0, size); }
+  buf.length = size;
+  return buf;
+};
+
+
+var fnTyped = {
+  arraySet: function (dest, src, src_offs, len, dest_offs) {
+    // Suppose, that with typed array support destination is
+    // always typed - don't check it
+    if (src.subarray) {
+      dest.set(src.subarray(src_offs, src_offs+len), dest_offs);
+      return;
+    }
+    // Fallback to ordinary array
+    for(var i=0; i<len; i++) {
+      dest[dest_offs + i] = src[src_offs + i];
+    }
+  },
+  // Join array of chunks to single array.
+  flattenChunks: function(chunks) {
+    var i, l, len, pos, chunk, result;
+
+    // calculate data length
+    len = 0;
+    for (i=0, l=chunks.length; i<l; i++) {
+      len += chunks[i].length;
+    }
+
+    // join chunks
+    result = new Uint8Array(len);
+    pos = 0;
+    for (i=0, l=chunks.length; i<l; i++) {
+      chunk = chunks[i];
+      result.set(chunk, pos);
+      pos += chunk.length;
+    }
+
+    return result;
+  }
+};
+
+var fnUntyped = {
+  arraySet: function (dest, src, src_offs, len, dest_offs) {
+    for(var i=0; i<len; i++) {
+      dest[dest_offs + i] = src[src_offs + i];
+    }
+  },
+  // Join array of chunks to single array.
+  flattenChunks: function(chunks) {
+    return [].concat.apply([], chunks);
+  }
+};
+
+
+// Enable/Disable typed arrays use, for testing
+//
+exports.setTyped = function (on) {
+  if (on) {
+    exports.Buf8  = Uint8Array;
+    exports.Buf16 = Uint16Array;
+    exports.Buf32 = Int32Array;
+    exports.assign(exports, fnTyped);
+  } else {
+    exports.Buf8  = Array;
+    exports.Buf16 = Array;
+    exports.Buf32 = Array;
+    exports.assign(exports, fnUntyped);
+  }
+};
+
+exports.setTyped(TYPED_OK);
\ No newline at end of file
diff --git a/lib/zlib/utils.js b/lib/utils/strings.js
similarity index 59%
rename from lib/zlib/utils.js
rename to lib/utils/strings.js
index 857e291..fe317b5 100644
--- a/lib/zlib/utils.js
+++ b/lib/utils/strings.js
@@ -1,45 +1,15 @@
+// String encode/decode helpers
 'use strict';
 
 
-var TYPED_OK =  (typeof Uint8Array !== 'undefined') &&
-                (typeof Uint16Array !== 'undefined') &&
-                (typeof Int32Array !== 'undefined');
+var utils = require('./common');
+
 
 // Quick check if we can use fast array to bin string conversion
 var STR_APPLY_OK = true;
 try { String.fromCharCode.apply(null, [0]); } catch(__) { STR_APPLY_OK = false; }
 
 
-exports.assign = function (obj /*from1, from2, from3, ...*/) {
-  var sources = Array.prototype.slice.call(arguments, 1);
-  while (sources.length) {
-    var source = sources.shift();
-    if (!source) { continue; }
-
-    if (typeof(source) !== 'object') {
-      throw new TypeError(source + 'must be non-object');
-    }
-
-    for (var p in source) {
-      if (source.hasOwnProperty(p)) {
-        obj[p] = source[p];
-      }
-    }
-  }
-
-  return obj;
-};
-
-
-// reduce buffer size, avoiding mem copy
-exports.shrinkBuf = function (buf, size) {
-  if (buf.length === size) { return buf; }
-  if (buf.subarray) { return buf.subarray(0, size); }
-  buf.length = size;
-  return buf;
-};
-
-
 // convert string to array (typed, when possible)
 // src: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Base64_encoding_and_decoding
 exports.string2buf = function (str) {
@@ -52,7 +22,7 @@ exports.string2buf = function (str) {
     buf_len += c < 0x80 ? 1 : c < 0x800 ? 2 : c < 0x10000 ? 3 : c < 0x200000 ? 4 : c < 0x4000000 ? 5 : 6;
   }
 
-  buf = new exports.Buf8(buf_len);
+  buf = new utils.Buf8(buf_len);
 
   /* transcription... */
 
@@ -143,77 +113,9 @@ exports.buf2binstring = function(buf) {
 
 // Convert binary string (typed, when possible)
 exports.binstring2buf = function(str) {
-  var buf = new exports.Buf8(str.length);
+  var buf = new utils.Buf8(str.length);
   for(var i=0, len=buf.length; i < len; i++) {
     buf[i] = str.charCodeAt(i);
   }
   return buf;
 };
-
-
-var fnTyped = {
-  arraySet: function (dest, src, src_offs, len, dest_offs) {
-    // Suppose, that with typed array support destination is
-    // always typed - don't check it
-    if (src.subarray) {
-      dest.set(src.subarray(src_offs, src_offs+len), dest_offs);
-      return;
-    }
-    // Fallback to ordinary array
-    for(var i=0; i<len; i++) {
-      dest[dest_offs + i] = src[src_offs + i];
-    }
-  },
-  // Join array of chunks to single array.
-  flattenChunks: function(chunks) {
-    var i, l, len, pos, chunk, result;
-
-    // calculate data length
-    len = 0;
-    for (i=0, l=chunks.length; i<l; i++) {
-      len += chunks[i].length;
-    }
-
-    // join chunks
-    result = new Uint8Array(len);
-    pos = 0;
-    for (i=0, l=chunks.length; i<l; i++) {
-      chunk = chunks[i];
-      result.set(chunk, pos);
-      pos += chunk.length;
-    }
-
-    return result;
-  }
-};
-
-var fnUntyped = {
-  arraySet: function (dest, src, src_offs, len, dest_offs) {
-    for(var i=0; i<len; i++) {
-      dest[dest_offs + i] = src[src_offs + i];
-    }
-  },
-  // Join array of chunks to single array.
-  flattenChunks: function(chunks) {
-    return [].concat.apply([], chunks);
-  }
-};
-
-
-// Enable/Disable typed arrays use, for testing
-//
-exports.setTyped = function (on) {
-  if (on) {
-    exports.Buf8  = Uint8Array;
-    exports.Buf16 = Uint16Array;
-    exports.Buf32 = Int32Array;
-    exports.assign(exports, fnTyped);
-  } else {
-    exports.Buf8  = Array;
-    exports.Buf16 = Array;
-    exports.Buf32 = Array;
-    exports.assign(exports, fnUntyped);
-  }
-};
-
-exports.setTyped(TYPED_OK);
\ No newline at end of file
diff --git a/lib/zlib/deflate.js b/lib/zlib/deflate.js
index 6884b32..0a6dfb0 100644
--- a/lib/zlib/deflate.js
+++ b/lib/zlib/deflate.js
@@ -1,6 +1,6 @@
 'use strict';
 
-var utils   = require('./utils');
+var utils   = require('../utils/common');
 var trees   = require('./trees');
 var adler32 = require('./adler32');
 var crc32   = require('./crc32');
diff --git a/lib/zlib/inflate.js b/lib/zlib/inflate.js
index 7fcd0d9..79e3fd7 100644
--- a/lib/zlib/inflate.js
+++ b/lib/zlib/inflate.js
@@ -1,7 +1,7 @@
 'use strict';
 
 
-var utils = require('./utils');
+var utils = require('../utils/common');
 var adler32 = require('./adler32');
 var crc32   = require('./crc32');
 var inflate_fast = require('./inffast');
diff --git a/lib/zlib/inftrees.js b/lib/zlib/inftrees.js
index 4b894b5..193f1ee 100644
--- a/lib/zlib/inftrees.js
+++ b/lib/zlib/inftrees.js
@@ -1,7 +1,7 @@
 'use strict';
 
 
-var utils = require('./utils');
+var utils = require('../utils/common');
 
 var MAXBITS = 15;
 var ENOUGH_LENS = 852;
diff --git a/lib/zlib/trees.js b/lib/zlib/trees.js
index 37645ac..e807c17 100644
--- a/lib/zlib/trees.js
+++ b/lib/zlib/trees.js
@@ -1,7 +1,7 @@
 'use strict';
 
 
-var utils = require('./utils');
+var utils = require('../utils/common');
 
 /* Public constants ==========================================================*/
 /* ===========================================================================*/
diff --git a/test/chunks.js b/test/chunks.js
index 1bc3c86..6ccfca6 100644
--- a/test/chunks.js
+++ b/test/chunks.js
@@ -8,7 +8,7 @@ var assert = require('assert');
 
 var helpers = require('./helpers');
 
-var pako_utils = require('../lib/zlib/utils');
+var pako_utils = require('../lib/utils/common');
 var pako = require('../index');
 
 
diff --git a/test/gzip_specials.js b/test/gzip_specials.js
index c0ad3dc..415624b 100644
--- a/test/gzip_specials.js
+++ b/test/gzip_specials.js
@@ -8,7 +8,7 @@ var fs = require('fs');
 var path  = require('path');
 var assert = require('assert');
 
-var pako_utils = require('../lib/zlib/utils');
+var pako_utils = require('../lib/utils/common');
 var pako  = require('../index');
 
 
diff --git a/test/helpers.js b/test/helpers.js
index fe5e106..d7e5c8d 100644
--- a/test/helpers.js
+++ b/test/helpers.js
@@ -6,7 +6,7 @@ var path  = require('path');
 var _     = require('lodash');
 var async = require('async');
 
-var pako_utils = require('../lib/zlib/utils');
+var pako_utils = require('../lib/utils/common');
 var pako  = require('../index');
 
 // Load fixtures to test
diff --git a/test/strings.js b/test/strings.js
index 2818bff..77908f6 100644
--- a/test/strings.js
+++ b/test/strings.js
@@ -8,7 +8,6 @@ var fs      = require('fs');
 var path    = require('path');
 var assert  = require('assert');
 
-var pako_utils = require('../lib/zlib/utils');
 var pako    = require('../index');
 
 var helpers = require('./helpers');
@@ -31,7 +30,7 @@ describe('Deflate strings', function () {
 
   it('Deflate with binary string output', function () {
     assert.ok(cmp(
-      pako_utils.binstring2buf(pako.deflate(sampleArray, { to: 'string', chunkSize: 99 })),
+      new Buffer(pako.deflate(sampleArray, { to: 'string', chunkSize: 99 }), 'binary'),
       pako.deflate(sampleArray)
     ));
   });