Skip to content

Commit e47d2b2

Browse files
committed
stream: add Readable.readv
A faster alternative to Readable.read for certain use cases. e.g. fs.writevSync(fd, readable.readv())
1 parent 6176222 commit e47d2b2

File tree

1 file changed

+55
-6
lines changed

1 file changed

+55
-6
lines changed

lib/internal/streams/readable.js

Lines changed: 55 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -641,8 +641,16 @@ function howMuchToRead(n, state) {
641641
return (state[kState] & kEnded) !== 0 ? state.length : 0;
642642
}
643643

644+
Readable.prototype.readv = function readv () {
645+
return _read.call(this, true);
646+
};
647+
648+
Readable.prototype.read = function read () {
649+
return _read.call(this, false);
650+
}
651+
644652
// You can override either this method, or the async _read(n) below.
645-
Readable.prototype.read = function(n) {
653+
function _read (n, returnArr) {
646654
debug('read', n);
647655
// Same as parseInt(undefined, 10), however V8 7.3 performance regressed
648656
// in this scenario, so we are doing it manually.
@@ -748,7 +756,7 @@ Readable.prototype.read = function(n) {
748756

749757
let ret;
750758
if (n > 0)
751-
ret = fromList(n, state);
759+
ret = fromList(n, state, returnArr);
752760
else
753761
ret = null;
754762

@@ -777,7 +785,13 @@ Readable.prototype.read = function(n) {
777785

778786
if (ret !== null && (state[kState] & (kErrorEmitted | kCloseEmitted)) === 0) {
779787
state[kState] |= kDataEmitted;
780-
this.emit('data', ret);
788+
if (returnArr) {
789+
for (let i = 0; i < ret.length; ++i) {
790+
this.emit('data', ret[i]);
791+
}
792+
} else {
793+
this.emit('data', ret);
794+
}
781795
}
782796

783797
return ret;
@@ -1582,7 +1596,7 @@ Readable._fromList = fromList;
15821596
// Length is the combined lengths of all the buffers in the list.
15831597
// This function is designed to be inlinable, so please take care when making
15841598
// changes to the function body.
1585-
function fromList(n, state) {
1599+
function fromList(n, state, returnArr) {
15861600
// nothing buffered.
15871601
if (state.length === 0)
15881602
return null;
@@ -1594,8 +1608,14 @@ function fromList(n, state) {
15941608
const len = buf.length;
15951609

15961610
if ((state[kState] & kObjectMode) !== 0) {
1597-
ret = buf[idx];
1598-
buf[idx++] = null;
1611+
if (returnArr) {
1612+
ret = buf.slice(idx);
1613+
idx += ret.length;
1614+
} else {
1615+
ret = buf[idx];
1616+
buf[idx++] = null;
1617+
ret = returnArr ? [ret] : ret;
1618+
}
15991619
} else if (!n || n >= state.length) {
16001620
// Read it all, truncate the list.
16011621
if ((state[kState] & kDecoder) !== 0) {
@@ -1604,11 +1624,17 @@ function fromList(n, state) {
16041624
ret += buf[idx];
16051625
buf[idx++] = null;
16061626
}
1627+
ret = returnArr ? [ret] : ret;
16071628
} else if (len - idx === 0) {
16081629
ret = Buffer.alloc(0);
1630+
ret = returnArr ? [ret] : ret;
16091631
} else if (len - idx === 1) {
16101632
ret = buf[idx];
16111633
buf[idx++] = null;
1634+
ret = returnArr ? [ret] : ret;
1635+
} else if (returnArr) {
1636+
ret = buf.slice(idx);
1637+
idx += ret.length;
16121638
} else {
16131639
ret = Buffer.allocUnsafe(state.length);
16141640

@@ -1623,10 +1649,12 @@ function fromList(n, state) {
16231649
// `slice` is the same for buffers and strings.
16241650
ret = buf[idx].slice(0, n);
16251651
buf[idx] = buf[idx].slice(n);
1652+
ret = returnArr ? [ret] : ret;
16261653
} else if (n === buf[idx].length) {
16271654
// First chunk is a perfect match.
16281655
ret = buf[idx];
16291656
buf[idx++] = null;
1657+
ret = returnArr ? [ret] : ret;
16301658
} else if ((state[kState] & kDecoder) !== 0) {
16311659
ret = '';
16321660
while (idx < len) {
@@ -1646,6 +1674,27 @@ function fromList(n, state) {
16461674
break;
16471675
}
16481676
}
1677+
ret = returnArr ? [ret] : ret;
1678+
} else if (returnArr) {
1679+
ret = [];
1680+
const retLen = n;
1681+
while (idx < len) {
1682+
const data = buf[idx];
1683+
if (n > data.length) {
1684+
ret.push(data);
1685+
n -= data.length;
1686+
buf[idx++] = null;
1687+
} else {
1688+
if (n === data.length) {
1689+
ret.push(data);
1690+
buf[idx++] = null;
1691+
} else {
1692+
ret.push(new FastBuffer(data.buffer, data.byteOffset, n));
1693+
buf[idx] = new FastBuffer(data.buffer, data.byteOffset + n, data.length - n);
1694+
}
1695+
break;
1696+
}
1697+
}
16491698
} else {
16501699
ret = Buffer.allocUnsafe(n);
16511700

0 commit comments

Comments
 (0)