diff options
author | Erik Verbruggen <erik.verbruggen@me.com> | 2021-09-16 16:38:44 +0300 |
---|---|---|
committer | Hannah von Reth <vonreth@kde.org> | 2021-09-17 11:08:15 +0300 |
commit | cc691344a891b8321746d77cc255ebae62b5851b (patch) | |
tree | 26f4bfea31298b47bbaea64ab68a7708723fcdc8 /src/common | |
parent | 0adbdbd27ab71e02feb1ca91fdd51ef5944e0369 (diff) |
Fix some compiler warnings
- comparison of integers of different signs
- usused parameter
- loop variable creates a copy
- unused variable (due to #ifdef)
Diffstat (limited to 'src/common')
-rw-r--r-- | src/common/filesystembase.cpp | 1 | ||||
-rw-r--r-- | src/common/utility.cpp | 2 |
2 files changed, 2 insertions, 1 deletions
diff --git a/src/common/filesystembase.cpp b/src/common/filesystembase.cpp index b57b5d75e..066e20373 100644 --- a/src/common/filesystembase.cpp +++ b/src/common/filesystembase.cpp @@ -484,6 +484,7 @@ bool FileSystem::isFileLocked(const QString &fileName, LockMode mode) } #else Q_UNUSED(fileName); + Q_UNUSED(mode); #endif return false; } diff --git a/src/common/utility.cpp b/src/common/utility.cpp index d5b51215e..18ae8aba3 100644 --- a/src/common/utility.cpp +++ b/src/common/utility.cpp @@ -630,7 +630,7 @@ QString Utility::sanitizeForFileName(const QString &name) const auto invalid = QStringLiteral("/?<>\\:*|\""); QString result; result.reserve(name.size()); - for (const auto c : name) { + for (const auto &c : name) { if (!invalid.contains(c) && c.category() != QChar::Other_Control && c.category() != QChar::Other_Format) { |