diff options
author | Joerg Sonnenberger <joerg@bec.de> | 2021-02-17 22:13:01 +0100 |
---|---|---|
committer | Joerg Sonnenberger <joerg@bec.de> | 2021-02-18 19:43:14 +0100 |
commit | a77e918016043691246d61f045519aaa5bc200f5 (patch) | |
tree | e08ec293a7c7c37e4390f6317b7d7269361adb80 /libcxx/src/filesystem/operations.cpp | |
parent | 612384f6e01194a4689b6663f039145b3d01ddce (diff) | |
download | llvm-a77e918016043691246d61f045519aaa5bc200f5.zip llvm-a77e918016043691246d61f045519aaa5bc200f5.tar.gz llvm-a77e918016043691246d61f045519aaa5bc200f5.tar.bz2 |
libcxx: use early returns
Differential Revision: https://reviews.llvm.org/D96955
Diffstat (limited to 'libcxx/src/filesystem/operations.cpp')
-rw-r--r-- | libcxx/src/filesystem/operations.cpp | 20 |
1 files changed, 9 insertions, 11 deletions
diff --git a/libcxx/src/filesystem/operations.cpp b/libcxx/src/filesystem/operations.cpp index 88a039f..458ba16 100644 --- a/libcxx/src/filesystem/operations.cpp +++ b/libcxx/src/filesystem/operations.cpp @@ -1051,7 +1051,7 @@ bool __create_directory(path const& p, path const& attributes, error_code* ec) { StatT attr_stat; error_code mec; - auto st = detail::posix_stat(attributes, attr_stat, &mec); + file_status st = detail::posix_stat(attributes, attr_stat, &mec); if (!status_known(st)) return err.report(mec); if (!is_directory(st)) @@ -1061,16 +1061,14 @@ bool __create_directory(path const& p, path const& attributes, error_code* ec) { if (detail::mkdir(p.c_str(), attr_stat.st_mode) == 0) return true; - if (errno == EEXIST) { - error_code mec = capture_errno(); - error_code ignored_ec; - const file_status st = status(p, ignored_ec); - if (!is_directory(st)) { - err.report(mec); - } - } else { - err.report(capture_errno()); - } + if (errno != EEXIST) + return err.report(capture_errno()); + + mec = capture_errno(); + error_code ignored_ec; + st = status(p, ignored_ec); + if (!is_directory(st)) + return err.report(mec); return false; } |