bl 2.1.1 → 3.0.0

package/.jshintrc

@@ -22,7 +22,7 @@
   , "boss": true
   , "debug": true
   , "eqnull": true
-  , "esnext": true
+  , "esnext": false
   , "evil": true
   , "expr": true
   , "funcscope": false
@@ -56,4 +56,5 @@
   , "nomen": false
   , "onevar": false
   , "passfail": false
+  , "esversion": 3
 }

package/.travis.yml

@@ -1,10 +1,8 @@
 sudo: false
 language: node_js
 node_js:
-  - '4'
   - '6'
   - '8'
-  - '9'
   - '10'
 branches:
   only:

package/README.md

@@ -15,11 +15,11 @@ The original buffers are kept intact and copies are only done as necessary. Any
 const BufferList = require('bl')
 
 var bl = new BufferList()
-bl.append(new Buffer('abcd'))
-bl.append(new Buffer('efg'))
+bl.append(Buffer.from('abcd'))
+bl.append(Buffer.from('efg'))
 bl.append('hi')                     // bl will also accept & convert Strings
-bl.append(new Buffer('j'))
-bl.append(new Buffer([ 0x3, 0x4 ]))
+bl.append(Buffer.from('j'))
+bl.append(Buffer.from([ 0x3, 0x4 ]))
 
 console.log(bl.length) // 12
 
@@ -29,6 +29,9 @@ console.log(bl.slice(3, 6).toString('ascii'))  // 'def'
 console.log(bl.slice(3, 8).toString('ascii'))  // 'defgh'
 console.log(bl.slice(5, 10).toString('ascii')) // 'fghij'
 
+console.log(bl.indexOf('def')) // 3
+console.log(bl.indexOf('asdf')) // -1
+
 // or just use toString!
 console.log(bl.toString())               // 'abcdefghij\u0003\u0004'
 console.log(bl.toString('ascii', 3, 8))  // 'defgh'
@@ -72,10 +75,10 @@ const BufferList = require('bl')
     , fs         = require('fs')
 
 var bl = new BufferList()
-bl.append(new Buffer('abcd'))
-bl.append(new Buffer('efg'))
-bl.append(new Buffer('hi'))
-bl.append(new Buffer('j'))
+bl.append(Buffer.from('abcd'))
+bl.append(Buffer.from('efg'))
+bl.append(Buffer.from('hi'))
+bl.append(Buffer.from('j'))
 
 bl.pipe(fs.createWriteStream('gibberish.txt'))
 ```
@@ -86,6 +89,7 @@ bl.pipe(fs.createWriteStream('gibberish.txt'))
   * <a href="#length"><code>bl.<b>length</b></code></a>
   * <a href="#append"><code>bl.<b>append(buffer)</b></code></a>
   * <a href="#get"><code>bl.<b>get(index)</b></code></a>
+  * <a href="#indexOf"><code>bl.<b>indexOf(value[, byteOffset][, encoding])</b></code></a>
   * <a href="#slice"><code>bl.<b>slice([ start[, end ] ])</b></code></a>
   * <a href="#shallowSlice"><code>bl.<b>shallowSlice([ start[, end ] ])</b></code></a>
   * <a href="#copy"><code>bl.<b>copy(dest, [ destStart, [ srcStart [, srcEnd ] ] ])</b></code></a>
@@ -129,6 +133,12 @@ Get the length of the list in bytes. This is the sum of the lengths of all of th
 ### bl.get(index)
 `get()` will return the byte at the specified index.
 
+--------------------------------------------------------
+<a name="indexOf"></a>
+### bl.indexOf(value[, byteOffset][, encoding])
+`get()` will return the byte at the specified index.
+`indexOf()` method returns the first index at which a given element can be found in the BufferList, or -1 if it is not present.
+
 --------------------------------------------------------
 <a name="slice"></a>
 ### bl.slice([ start, [ end ] ])

package/bl.js

@@ -1,7 +1,6 @@
 'use strict'
 var DuplexStream = require('readable-stream').Duplex
   , util         = require('util')
-  , Buffer       = require('safe-buffer').Buffer
 
 function BufferList (callback) {
   if (!(this instanceof BufferList))
@@ -50,9 +49,9 @@ BufferList.prototype._offset = function _offset (offset) {
 }
 
 BufferList.prototype._reverseOffset = function (blOffset) {
-  const bufferId = blOffset[0]
-  let offset = blOffset[1]
-  for (let i = 0; i < bufferId; i++) {
+  var bufferId = blOffset[0]
+  var offset = blOffset[1]
+  for (var i = 0; i < bufferId; i++) {
     offset += this._bufs[i].length
   }
   return offset
@@ -121,7 +120,7 @@ BufferList.prototype.get = function get (index) {
   if (index > this.length || index < 0) {
     return undefined
   }
-  const offset = this._offset(index)
+  var offset = this._offset(index)
   return this._bufs[offset[0]][offset[1]]
 }
 
@@ -203,13 +202,16 @@ BufferList.prototype.copy = function copy (dst, dstStart, srcStart, srcEnd) {
 
 BufferList.prototype.shallowSlice = function shallowSlice (start, end) {
   start = start || 0
-  end = end || this.length
+  end = typeof end !== 'number' ? this.length : end
 
   if (start < 0)
     start += this.length
   if (end < 0)
     end += this.length
 
+  if (start === end) {
+    return new BufferList()
+  }
   var startOffset = this._offset(start)
     , endOffset = this._offset(end)
     , buffers = this._bufs.slice(startOffset[0], endOffset[0] + 1)
@@ -256,10 +258,10 @@ BufferList.prototype.duplicate = function duplicate () {
 }
 
 
-BufferList.prototype.destroy = function destroy () {
+BufferList.prototype._destroy = function _destroy (err, cb) {
   this._bufs.length = 0
   this.length = 0
-  this.push(null)
+  cb(err)
 }
 
 
@@ -297,23 +299,23 @@ BufferList.prototype.indexOf = function (search, offset, encoding) {
     return offset > this.length ? this.length : offset
   }
 
-  const blOffset = this._offset(offset)
-  let blIndex = blOffset[0] // index of which internal buffer we're working on
-  let buffOffset = blOffset[1] // offset of the internal buffer we're working on
+  var blOffset = this._offset(offset)
+  var blIndex = blOffset[0] // index of which internal buffer we're working on
+  var buffOffset = blOffset[1] // offset of the internal buffer we're working on
 
   // scan over each buffer
   for (blIndex; blIndex < this._bufs.length; blIndex++) {
-    const buff = this._bufs[blIndex]
+    var buff = this._bufs[blIndex]
     while(buffOffset < buff.length) {
-      const availableWindow = buff.length - buffOffset
+      var availableWindow = buff.length - buffOffset
       if (availableWindow >= search.length) {
-        const nativeSearchResult = buff.indexOf(search, buffOffset)
+        var nativeSearchResult = buff.indexOf(search, buffOffset)
         if (nativeSearchResult !== -1) {
           return this._reverseOffset([blIndex, nativeSearchResult])
         }
         buffOffset = buff.length - search.length + 1 // end of native search window
       } else {
-        const revOffset = this._reverseOffset([blIndex, buffOffset])
+        var revOffset = this._reverseOffset([blIndex, buffOffset])
         if (this._match(revOffset, search)) {
           return revOffset
         }
@@ -329,7 +331,7 @@ BufferList.prototype._match = function(offset, search) {
   if (this.length - offset < search.length) {
     return false
   }
-  for (let searchOffset = 0; searchOffset < search.length ; searchOffset++) {
+  for (var searchOffset = 0; searchOffset < search.length ; searchOffset++) {
     if(this.get(offset + searchOffset) !== search[searchOffset]){
       return false
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bl",
-  "version": "2.1.1",
+  "version": "3.0.0",
   "description": "Buffer List: collect buffers and access with a standard readable Buffer interface, streamable too!",
   "main": "bl.js",
   "scripts": {
@@ -24,12 +24,11 @@
   ],
   "license": "MIT",
   "dependencies": {
-    "readable-stream": "^2.3.5",
-    "safe-buffer": "^5.1.1"
+    "readable-stream": "^3.0.1"
   },
   "devDependencies": {
     "faucet": "0.0.1",
     "hash_file": "~0.1.1",
-    "tape": "~4.9.0"
+    "tape": "~4.9.1"
   }
 }

package/test/test.js

@@ -649,6 +649,20 @@ tape('shallow slice does not make a copy', function (t) {
   t.equal(bl.toString(), 'hhhhhhhh')
 })
 
+tape('shallow slice with 0 length', function (t) {
+  t.plan(1)
+  var buffers = [Buffer.from('First'), Buffer.from('Second'), Buffer.from('Third')]
+  var bl = (new BufferList(buffers)).shallowSlice(0, 0)
+  t.equal(bl.length, 0)
+})
+
+tape('shallow slice with 0 length from middle', function (t) {
+  t.plan(1)
+  var buffers = [Buffer.from('First'), Buffer.from('Second'), Buffer.from('Third')]
+  var bl = (new BufferList(buffers)).shallowSlice(10, 10)
+  t.equal(bl.length, 0)
+})
+
 tape('duplicate', function (t) {
   t.plan(2)
 
@@ -669,6 +683,20 @@ tape('destroy no pipe', function (t) {
   t.equal(bl.length, 0)
 })
 
+tape('destroy with error', function (t) {
+  t.plan(3)
+
+  var bl = new BufferList('alsdkfja;lsdkfja;lsdk')
+  var err = new Error('kaboom')
+  bl.destroy(err)
+  bl.on('error', function (_err) {
+    t.equal(_err, err)
+  })
+
+  t.equal(bl._bufs.length, 0)
+  t.equal(bl.length, 0)
+})
+
 !process.browser && tape('destroy with pipe before read end', function (t) {
   t.plan(2)