Remove useless isdir checks

3 files changed, 9 insertions, 27 deletions

diff --git a/xmake/rules/c++/modules/modules_support/clang.lua b/xmake/rules/c++/modules/modules_support/clang.lua
index 4ab3c3892..cc6e2f927 100644
--- a/xmake/rules/c++/modules/modules_support/clang.lua
+++ b/xmake/rules/c++/modules/modules_support/clang.lua
@@ -184,15 +184,11 @@ function generate_user_headerunits(target, batchcmds, headerunits, opt)
         else
             outdir = path.join(cachedir, path.directory(headerunit.path))
         end
-        if not os.isdir(outdir) then
-            batchcmds:mkdir(outdir)
-        end
+        batchcmds:mkdir(outdir)
 
         local bmifilename = path.basename(objectfile) .. get_bmi_extension()
         local bmifile = (outdir and path.join(outdir, bmifilename) or bmifilename)
-        if not os.isdir(path.directory(objectfile)) then
-            batchcmds:mkdir(path.directory(objectfile))
-        end
+        batchcmds:mkdir(path.directory(objectfile))
 
         local args = { modulecachepathflag .. cachedir, emitmoduleflag, "-c", "-o", bmifile}
         if headerunit.type == ":quote" then
@@ -237,9 +233,7 @@ function build_modules(target, batchcmds, objectfiles, modules, opt)
     for _, objectfile in ipairs(objectfiles) do
         local m = modules[objectfile]
         if m then
-            if not os.isdir(path.directory(objectfile)) then
-                batchcmds:mkdir(path.directory(objectfile))
-            end
+            batchcmds:mkdir(path.directory(objectfile))
 
             local args = { emitmoduleinterfaceflag }
             local bmiflags = {}
diff --git a/xmake/rules/c++/modules/modules_support/gcc.lua b/xmake/rules/c++/modules/modules_support/gcc.lua
index a3ef5c8a8..1c53d22b5 100644
--- a/xmake/rules/c++/modules/modules_support/gcc.lua
+++ b/xmake/rules/c++/modules/modules_support/gcc.lua
@@ -193,15 +193,11 @@ function generate_user_headerunits(target, batchcmds, headerunits, opt)
         else
             outdir = path.join(cachedir, path.directory(headerunit.path))
         end
-        if not os.isdir(outdir) then
-            batchcmds:mkdir(outdir)
-        end
+        batchcmds:mkdir(outdir)
 
         local bmifilename = path.basename(objectfile) .. get_bmi_extension()
         local bmifile = (outdir and path.join(outdir, bmifilename) or bmifilename)
-        if not os.isdir(path.directory(objectfile)) then
-            batchcmds:mkdir(path.directory(objectfile))
-        end
+        batchcmds:mkdir(path.directory(objectfile))
 
         local args = { "-c" }
         local headerunit_path
@@ -238,9 +234,7 @@ function build_modules(target, batchcmds, objectfiles, modules, opt)
     for _, objectfile in ipairs(objectfiles) do
         local m = modules[objectfile]
         if m then
-            if not os.isdir(path.directory(objectfile)) then
-                batchcmds:mkdir(path.directory(objectfile))
-            end
+            batchcmds:mkdir(path.directory(objectfile))
 
             local args = {"-o", objectfile}
             for name, provide in pairs(m.provides) do
diff --git a/xmake/rules/c++/modules/modules_support/msvc.lua b/xmake/rules/c++/modules/modules_support/msvc.lua
index abc7f7319..93eef7c8c 100644
--- a/xmake/rules/c++/modules/modules_support/msvc.lua
+++ b/xmake/rules/c++/modules/modules_support/msvc.lua
@@ -176,15 +176,11 @@ function generate_user_headerunits(target, batchcmds, headerunits, opt)
         else
             outputdir = path.join(cachedir, path.directory(headerunit.path):sub(3))
         end
-        if not os.isdir(outputdir) then
-            batchcmds:mkdir(outputdir)
-        end
+        batchcmds:mkdir(outputdir)
 
         local bmifilename = path.basename(objectfile) .. get_bmi_extension()
         local bmifile = (outputdir and path.join(outputdir, bmifilename) or bmifilename)
-        if not os.isdir(path.directory(objectfile)) then
-            batchcmds:mkdir(path.directory(objectfile))
-        end
+        batchcmds:mkdir(path.directory(objectfile))
 
         local args = {headernameflag .. headerunit.type, headerunit.path, ifcoutputflag, outputdir, "/Fo" .. objectfile}
         batchcmds:show_progress(opt.progress, "${color.build.object}generating.cxx.headerunit.bmi %s", headerunit.name)
@@ -229,9 +225,7 @@ function build_modules(target, batchcmds, objectfiles, modules, opt)
     for _, objectfile in ipairs(objectfiles) do
         local m = modules[objectfile]
         if m then
-            if not os.isdir(path.directory(objectfile)) then
-                batchcmds:mkdir(path.directory(objectfile))
-            end
+            batchcmds:mkdir(path.directory(objectfile))
 
             local args = {"/c", "/Fo" .. objectfile}
             local bmiflags = {}