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/resources/loaders/imageloader.cpp | |
parent | f1518dd8476c968a43fa57cfb06198e290a4f77a (diff) | |
download | manaplus-36ba43d6ea38062b17f7e63ef659962bfc51c64d.tar.gz manaplus-36ba43d6ea38062b17f7e63ef659962bfc51c64d.tar.bz2 manaplus-36ba43d6ea38062b17f7e63ef659962bfc51c64d.tar.xz manaplus-36ba43d6ea38062b17f7e63ef659962bfc51c64d.zip |
Fix clang-tidy check readability-implicit-bool-cast.
Diffstat (limited to 'src/resources/loaders/imageloader.cpp')
-rw-r--r-- | src/resources/loaders/imageloader.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/resources/loaders/imageloader.cpp b/src/resources/loaders/imageloader.cpp index 5a1830e15..c5635ef14 100644 --- a/src/resources/loaders/imageloader.cpp +++ b/src/resources/loaders/imageloader.cpp @@ -46,7 +46,7 @@ namespace static Resource *load(const void *const v) { BLOCK_START("DyedImageLoader::load") - if (!v) + if (v == nullptr) { BLOCK_END("DyedImageLoader::load") return nullptr; @@ -64,17 +64,17 @@ namespace path1 = path1.substr(0, p); } SDL_RWops *const rw = VirtFs::rwopsOpenRead(path1); - if (!rw) + if (rw == nullptr) { delete d; reportAlways("Image loading error: %s", path1.c_str()); BLOCK_END("DyedImageLoader::load") return nullptr; } - Resource *const res = d ? imageHelper->load(rw, *d) + Resource *const res = d != nullptr ? imageHelper->load(rw, *d) : imageHelper->load(rw); delete d; - if (!res) + if (res == nullptr) reportAlways("Image loading error: %s", path1.c_str()); BLOCK_END("DyedImageLoader::load") return res; |