path: change posix.join to use array

HBSPS · targos · commit 2859b4ba9ab9 · 2024-10-02T07:57:36.000+02:00
Change posix.join to use array.join instead of additional assignment. PR-URL: #54331 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
diff --git a/lib/path.js b/lib/path.js
@@ -1237,20 +1237,20 @@ const posix = {
 join(...args) {
 if (args.length === 0)
 return '.';
- let joined;
+
+ const path = [];
 for (let i = 0; i < args.length; ++i) {
 const arg = args[i];
 validateString(arg, 'path');
 if (arg.length > 0) {
- if (joined === undefined)
- joined = arg;
- else
- joined += `/${arg}`;
+ path.push(arg);
 }
 }
- if (joined === undefined)
+
+ if (path.length === 0)
 return '.';
- return posix.normalize(joined);
+
+ return posix.normalize(ArrayPrototypeJoin(path, '/'));
 },
 
 /**

-Original file line number
+Diff line change
 join(...args){
 if(args.length===0)
 return'.';
 -letjoined;
++
 +constpath=[];
 for(leti=0;i<args.length;++i){
 constarg=args[i];
 validateString(arg,'path');
 if(arg.length>0){
 -if(joined===undefined)
 -joined=arg;
 -else
 -joined+=`/${arg}`;
 +path.push(arg);
+}
+}
 -if(joined===undefined)
++
 +if(path.length===0)
 return'.';
 -returnposix.normalize(joined);
++
 +returnposix.normalize(ArrayPrototypeJoin(path,'/'));
 },
 /**