Skip to content

Commit 3ed9f98

Browse files
authored
fs: remove unnecessary option argument validation
PR-URL: #53958 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
1 parent 78150f3 commit 3ed9f98

File tree

1 file changed

+9
-9
lines changed

1 file changed

+9
-9
lines changed

lib/fs.js

+9-9
Original file line numberDiff line numberDiff line change
@@ -1310,23 +1310,23 @@ function mkdir(path, options, callback) {
13101310
if (typeof options === 'function') {
13111311
callback = options;
13121312
} else if (typeof options === 'number' || typeof options === 'string') {
1313-
mode = options;
1313+
mode = parseFileMode(options, 'mode');
13141314
} else if (options) {
1315-
if (options.recursive !== undefined)
1315+
if (options.recursive !== undefined) {
13161316
recursive = options.recursive;
1317-
if (options.mode !== undefined)
1318-
mode = options.mode;
1317+
validateBoolean(recursive, 'options.recursive');
1318+
}
1319+
if (options.mode !== undefined) {
1320+
mode = parseFileMode(options.mode, 'options.mode');
1321+
}
13191322
}
13201323
callback = makeCallback(callback);
1321-
path = getValidatedPath(path);
1322-
1323-
validateBoolean(recursive, 'options.recursive');
13241324

13251325
const req = new FSReqCallback();
13261326
req.oncomplete = callback;
13271327
binding.mkdir(
1328-
path,
1329-
parseFileMode(mode, 'mode'),
1328+
getValidatedPath(path),
1329+
mode,
13301330
recursive,
13311331
req,
13321332
);

0 commit comments

Comments
 (0)