buffer: remove TODOs in atob / btoa

Refs: #38433 (comment) PR-URL: #38548 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Zijian Liu <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]>
nodejs · Jun 11, 2021 · 3c886e0 · 3c886e0
1 parent e94e0b4
commit 3c886e0
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/lib/buffer.js b/lib/buffer.js
@@ -1212,8 +1212,9 @@ const lazyInvalidCharError = hideStackFrames((message, name) => {
 });
 
 function btoa(input) {
-  // TODO(@jasnell): The implementation here has not been performance
-  // optimized in any way.
+  // The implementation here has not been performance optimized in any way and
+  // should not be.
+  // Refs: https://github.com/nodejs/node/pull/38433#issuecomment-828426932
   input = `${input}`;
   for (let n = 0; n < input.length; n++) {
     if (input[n].charCodeAt(0) > 0xff)
@@ -1227,8 +1228,9 @@ const kBase64Digits =
   'ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789+/=';
 
 function atob(input) {
-  // TODO(@jasnell): The implementation here has not been performance
-  // optimized in any way.
+  // The implementation here has not been performance optimized in any way and
+  // should not be.
+  // Refs: https://github.com/nodejs/node/pull/38433#issuecomment-828426932
   input = `${input}`;
   for (let n = 0; n < input.length; n++) {
     if (!kBase64Digits.includes(input[n]))