Skip to content

Commit 112228c

Browse files
JonasBatargos
authored andcommitted
fs: remove unnecessary option argument validation
PR-URL: #53958 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
1 parent 783322f commit 112228c

1 file changed

Lines changed: 9 additions & 9 deletions

File tree

‎lib/fs.js‎

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1314,23 +1314,23 @@ function mkdir(path, options, callback) {
13141314
if(typeofoptions==='function'){
13151315
callback=options;
13161316
}elseif(typeofoptions==='number'||typeofoptions==='string'){
1317-
mode=options;
1317+
mode=parseFileMode(options,'mode');
13181318
}elseif(options){
1319-
if(options.recursive!==undefined)
1319+
if(options.recursive!==undefined){
13201320
recursive=options.recursive;
1321-
if(options.mode!==undefined)
1322-
mode=options.mode;
1321+
validateBoolean(recursive,'options.recursive');
1322+
}
1323+
if(options.mode!==undefined){
1324+
mode=parseFileMode(options.mode,'options.mode');
1325+
}
13231326
}
13241327
callback=makeCallback(callback);
1325-
path=getValidatedPath(path);
1326-
1327-
validateBoolean(recursive,'options.recursive');
13281328

13291329
constreq=newFSReqCallback();
13301330
req.oncomplete=callback;
13311331
binding.mkdir(
1332-
path,
1333-
parseFileMode(mode,'mode'),
1332+
getValidatedPath(path),
1333+
mode,
13341334
recursive,
13351335
req,
13361336
);

0 commit comments

Comments
 (0)