Skip to content

Commit 90c20bc

Browse files
pgeissMyles Borins
authored and
Myles Borins
committed
buffer: remove unnecessary TODO comments
Refs: #4642 PR-URL: #4719 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Rich Trott <[email protected]>
1 parent e805f50 commit 90c20bc

File tree

1 file changed

+0
-5
lines changed

1 file changed

+0
-5
lines changed

lib/buffer.js

-5
Original file line numberDiff line numberDiff line change
@@ -513,9 +513,6 @@ Buffer.prototype.set = internalUtil.deprecate(function set(offset, v) {
513513
return this[offset] = v;
514514
}, 'Buffer.set is deprecated. Use array indexes instead.');
515515

516-
517-
// TODO(trevnorris): fix these checks to follow new standard
518-
// write(string, offset = 0, length = buffer.length, encoding = 'utf8')
519516
var writeWarned = false;
520517
const writeMsg = 'Buffer.write(string, encoding, offset, length) is ' +
521518
'deprecated. Use write(string[, offset[, length]]' +
@@ -608,8 +605,6 @@ Buffer.prototype.toJSON = function() {
608605
};
609606

610607

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

0 commit comments

Comments
 (0)