Skip to content

Commit a9fde1d

Browse files
pgeissMichael Scovetta
authored and
Michael Scovetta
committed
buffer: remove unnecessary TODO comments
Refs: nodejs#4642 PR-URL: nodejs#4719 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Rich Trott <[email protected]>
1 parent 896dca9 commit a9fde1d

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

lib/buffer.js

-4
Original file line numberDiff line numberDiff line change
@@ -523,8 +523,6 @@ Buffer.prototype.fill = function fill(val, start, end) {
523523
};
524524

525525

526-
// TODO(trevnorris): fix these checks to follow new standard
527-
// write(string, offset = 0, length = buffer.length, encoding = 'utf8')
528526
var writeWarned = false;
529527
const writeMsg = 'Buffer.write(string, encoding, offset, length) is ' +
530528
'deprecated. Use write(string[, offset[, length]]' +
@@ -617,8 +615,6 @@ Buffer.prototype.toJSON = function() {
617615
};
618616

619617

620-
// TODO(trevnorris): currently works like Array.prototype.slice(), which
621-
// doesn't follow the new standard for throwing on out of range indexes.
622618
Buffer.prototype.slice = function slice(start, end) {
623619
const buffer = this.subarray(start, end);
624620
Object.setPrototypeOf(buffer, Buffer.prototype);

0 commit comments

Comments
 (0)