util: add todo comments for inspect to add unicode support

PR-URL: #31112
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

Author: BridgeAR

lib/internal/util/inspect.js

@@ -1190,6 +1190,8 @@ function groupArrayElements(ctx, output, value) {
 if(columns<=1){
 returnoutput;
 }
+// TODO(BridgeAR): Add unicode support. Use the readline getStringWidth
+// function.
 consttmp=[];
 constmaxLineLength=[];
 for(leti=0;i<columns;i++){
@@ -1267,6 +1269,8 @@ function formatBigInt(fn, value) {
 functionformatPrimitive(fn,value,ctx){
 if(typeofvalue==='string'){
 if(ctx.compact!==true&&
+// TODO(BridgeAR): Add unicode support. Use the readline getStringWidth
+// function.
 value.length>kMinLineLength&&
 value.length>ctx.breakLength-ctx.indentationLvl-4){
 returnvalue
@@ -1612,6 +1616,9 @@ function isBelowBreakLength(ctx, output, start, base) {
 // Each entry is separated by at least a comma. Thus, we start with a total
 // length of at least `output.length`. In addition, some cases have a
 // whitespace in-between each other that is added to the total as well.
+// TODO(BridgeAR): Add unicode support. Use the readline getStringWidth
+// function. Check the performance overhead and make it an opt-in in case it's
+// significant.
 lettotalLength=output.length+start;
 if(totalLength+output.length>ctx.breakLength)
 returnfalse;