diff options
author | Andrei Karas <akaras@inbox.ru> | 2017-06-06 23:34:34 +0300 |
---|---|---|
committer | Andrei Karas <akaras@inbox.ru> | 2017-06-07 19:23:40 +0300 |
commit | 36ba43d6ea38062b17f7e63ef659962bfc51c64d (patch) | |
tree | 190156cb88b13a38a6d13c69ee0742cc078065a1 /src/fs/mkdir.cpp | |
parent | f1518dd8476c968a43fa57cfb06198e290a4f77a (diff) | |
download | mv-36ba43d6ea38062b17f7e63ef659962bfc51c64d.tar.gz mv-36ba43d6ea38062b17f7e63ef659962bfc51c64d.tar.bz2 mv-36ba43d6ea38062b17f7e63ef659962bfc51c64d.tar.xz mv-36ba43d6ea38062b17f7e63ef659962bfc51c64d.zip |
Fix clang-tidy check readability-implicit-bool-cast.
Diffstat (limited to 'src/fs/mkdir.cpp')
-rw-r--r-- | src/fs/mkdir.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/fs/mkdir.cpp b/src/fs/mkdir.cpp index e84ab5f28..d45810b7c 100644 --- a/src/fs/mkdir.cpp +++ b/src/fs/mkdir.cpp @@ -105,7 +105,7 @@ int mkdir_r(const char *const pathname) /// Create a directory, making leading components first if necessary int mkdir_r(const char *const pathname) { - if (!pathname) + if (pathname == nullptr) return -1; const size_t len = CAST_SIZE(strlen(pathname)); @@ -121,7 +121,7 @@ int mkdir_r(const char *const pathname) tmp[len + 1] = '\0'; } - for (p = tmp; *p; p++) + for (p = tmp; *p != 0; p++) { if (*p == '/') { @@ -135,7 +135,7 @@ int mkdir_r(const char *const pathname) // check if the name already exists, but not as directory struct stat statbuf; - if (!stat(tmp, &statbuf)) + if (stat(tmp, &statbuf) == 0) { if (S_ISDIR(statbuf.st_mode)) { @@ -149,7 +149,7 @@ int mkdir_r(const char *const pathname) } } - if (mkdir(tmp, S_IRWXU | S_IRWXG | S_IROTH | S_IXOTH)) + if (mkdir(tmp, S_IRWXU | S_IRWXG | S_IROTH | S_IXOTH) != 0) { delete []tmp; return -1; |