Skip to content

Commit 75b3634

Browse files
committed
[PIPEFS] Fix some review issues
1 parent 6643f02 commit 75b3634

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

src/library_pipefs.js

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -74,9 +74,10 @@ mergeInto(LibraryManager.library, {
7474
var pipe = stream.node.pipe;
7575
var currentLength = 0;
7676

77-
pipe.buckets.forEach(function (bucket) {
77+
for (var i = 0; i < pipe.buckets.length; i++) {
78+
var bucket = pipe.buckets[i];
7879
currentLength += bucket.offset - bucket.roffset;
79-
});
80+
}
8081

8182
assert(buffer instanceof ArrayBuffer || ArrayBuffer.isView(buffer));
8283
var data = buffer.subarray(offset, offset + length);
@@ -124,9 +125,7 @@ mergeInto(LibraryManager.library, {
124125
pipe.buckets[toRemove].roffset = 0;
125126
}
126127

127-
while (toRemove--) {
128-
pipe.buckets.shift();
129-
}
128+
pipe.buckets.splice(0, toRemove);
130129

131130
return totalRead;
132131
},

0 commit comments

Comments
 (0)