stream-read - npm Package Compare versions

Comparing version 1.1.1 to 1.1.2

index.js

@@ -43,2 +43,3 @@ var dz = require('dezalgo');
   if (!stream.readable) return cb();
+  var ended = false;
 
@@ -52,3 +53,6 @@ function onreadable() {
     cleanup();
-    cb(null, null);
+    if (!ended) {
+      ended = true;
+      cb(null, null);
+    }
   }
@@ -83,2 +87,3 @@
     var buf = stream.read();
+    stream.removeListener('error', _onerror);
     if (errored) return;
@@ -85,0 +90,0 @@ if (buf) cb(null, buf);

package.json

 {
   "name": "stream-read",
   "description": "Read from a stream, callback style",
-  "version": "1.1.1",
+  "version": "1.1.2",
   "repository": "micnews/stream-read",
@@ -6,0 +6,0 @@ "license": "MIT",

README.md

@@ -6,2 +6,4 @@
 
+  [![build status](https://secure.travis-ci.org/micnews/stream-read.svg)](http://travis-ci.org/micnews/stream-read)
+
 ## Example
@@ -8,0 +10,0 @@

test.js

@@ -162,1 +162,11 @@ var Readable = require('stream').Readable;
 
+test('pull double end', function(t){
+  t.plan(1);
+  var stream = Readable();
+  stream._read = function(){ this.push(null) };
+  read(stream, function(err, val){
+    t.equal(val, null);
+  });
+  stream.emit('end');
+});
+

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

6120

increased by8.61%

Number of package files

7

increased by16.67%

Lines of code

226

increased by6.6%

Number of lines in readme file

32

increased by6.67%

No dependency changes