Avoid leaving an empty buffer on complete consume

rvagg · Feb 11, 2016 · 4df2f4c · 4df2f4c
1 parent 9f1e7ce
commit 4df2f4c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/bl.js b/bl.js
@@ -162,7 +162,7 @@ BufferList.prototype.toString = function (encoding, start, end) {
 
 BufferList.prototype.consume = function (bytes) {
   while (this._bufs.length) {
-    if (bytes > this._bufs[0].length) {
+    if (bytes >= this._bufs[0].length) {
       bytes -= this._bufs[0].length
       this.length -= this._bufs[0].length
       this._bufs.shift()

diff --git a/test/test.js b/test/test.js
@@ -125,6 +125,10 @@ tape('consuming from multiple buffers', function (t) {
   t.equal(bl.length, 1)
   t.equal(bl.slice(0, 1).toString('ascii'), 'j')
 
+  bl.consume(1)
+  t.equal(bl.length, 0)
+  t.equal(bl._bufs.length, 0)
+
   t.end()
 })