Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

.lines returns string "0" for blank lines #41

Open
sehrgut opened this issue Aug 23, 2013 · 1 comment
Open

.lines returns string "0" for blank lines #41

sehrgut opened this issue Aug 23, 2013 · 1 comment

Comments

@sehrgut
Copy link

sehrgut commented Aug 23, 2013

A minimal case which runs on the latest master.

var Lazy = require('./lazy');
var l = new Lazy();
l.lines.forEach(function (x) { console.log(x); });
l.emit('data', 'abc\n\n');

Actual output:

<Buffer 61 62 63>
<Buffer 30>

Expected output:

<Buffer 61 62 63>
<Buffer >
sehrgut added a commit to sehrgut/node-lazy that referenced this issue Aug 23, 2013
@sehrgut
Copy link
Author

sehrgut commented May 11, 2014

Have you considered pulling this commit? The issue is still in the live version on NPM.

jfremy pushed a commit to jfremy/node-lazy that referenced this issue Feb 11, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant