diff options
author | Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2018-01-07 15:42:28 +0100 |
---|---|---|
committer | Lubomir I. Ivanov <neolit123@gmail.com> | 2018-01-11 06:07:13 +0100 |
commit | 6a07ccbad273cdcbea3f984c5c2e9d1b272aae05 (patch) | |
tree | d389058bb3a66b0cce63331ce325ad34fee6ff22 /core/qthelper.cpp | |
parent | e85ecdd9254a8bd222ccb2dfed7d3b3bb96c8294 (diff) | |
download | subsurface-6a07ccbad273cdcbea3f984c5c2e9d1b272aae05.tar.gz |
Use helper function empty_string() instead of manual checks
For code consistency, substitute boolean expressions:
s && *s -> !empty_string(s)
s && s[0] -> !empty_string(s)
!s || !*s -> empty_string(s)
!s || !s[0] -> empty_string(s)
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'core/qthelper.cpp')
-rw-r--r-- | core/qthelper.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/core/qthelper.cpp b/core/qthelper.cpp index 6916b1a68..262399a5f 100644 --- a/core/qthelper.cpp +++ b/core/qthelper.cpp @@ -1163,7 +1163,7 @@ QString localFilePath(const QString originalFilename) QString fileFromHash(const char *hash) { - if (!hash || !*hash) + if (empty_string(hash)) return ""; QMutexLocker locker(&hashOfMutex); |