diff options
author | Nikolai Kosjar <nikolai.kosjar@qt.io> | 2019-02-07 12:41:04 +0100 |
---|---|---|
committer | Nikolai Kosjar <nikolai.kosjar@qt.io> | 2019-02-08 07:19:29 +0000 |
commit | 4b26a8c66e6303e090e9e070c7f3a425e26d90bf (patch) | |
tree | 370362094e8c5146f4cf603ea11faf15567b92fe /src/plugins/clangcodemodel | |
parent | ae388a85b98f9004e8e19e72cccac6cf6e8bc282 (diff) |
ClangCodeModel: Fix some clazy level0 warnings
Use QString instead of an empty QStringLiteral
[-Wclazy-empty-qstringliteral]
Use the static QFileInfo::exists() instead. It's documented to be
faster. [-Wclazy-qfileinfo-exists]
Unused CppTools::ClangDiagnosticConfigs
[-Wclazy-unused-non-trivial-variable]
Change-Id: Ia4098d1191d6fcfc6e0774f71c39acdea3f0f36c
Reviewed-by: Orgad Shaneh <orgads@gmail.com>
Reviewed-by: Ivan Donchevskii <ivan.donchevskii@qt.io>
Diffstat (limited to 'src/plugins/clangcodemodel')
-rw-r--r-- | src/plugins/clangcodemodel/clangbackendcommunicator.cpp | 6 | ||||
-rw-r--r-- | src/plugins/clangcodemodel/clangtextmark.cpp | 1 |
2 files changed, 2 insertions, 5 deletions
diff --git a/src/plugins/clangcodemodel/clangbackendcommunicator.cpp b/src/plugins/clangcodemodel/clangbackendcommunicator.cpp index 31e8a85266..28a31f0553 100644 --- a/src/plugins/clangcodemodel/clangbackendcommunicator.cpp +++ b/src/plugins/clangcodemodel/clangbackendcommunicator.cpp @@ -63,9 +63,7 @@ enum { backEndStartTimeOutInMs = 10000 }; static QString backendProcessPath() { - return Core::ICore::libexecPath() - + QStringLiteral("/clangbackend") - + QStringLiteral(QTC_HOST_EXE_SUFFIX); + return Core::ICore::libexecPath() + "/clangbackend" + QTC_HOST_EXE_SUFFIX; } namespace ClangCodeModel { @@ -121,7 +119,7 @@ BackendCommunicator::~BackendCommunicator() void BackendCommunicator::initializeBackend() { const QString clangBackEndProcessPath = backendProcessPath(); - if (!QFileInfo(clangBackEndProcessPath).exists()) { + if (!QFileInfo::exists(clangBackEndProcessPath)) { logExecutableDoesNotExist(); return; } diff --git a/src/plugins/clangcodemodel/clangtextmark.cpp b/src/plugins/clangcodemodel/clangtextmark.cpp index 41ee4c1533..61a4c7716c 100644 --- a/src/plugins/clangcodemodel/clangtextmark.cpp +++ b/src/plugins/clangcodemodel/clangtextmark.cpp @@ -133,7 +133,6 @@ void disableDiagnosticInCurrentProjectConfig(const ClangBackEnd::DiagnosticConta currentConfigId = globalSettings->clangDiagnosticConfigId(); // Get config - const ClangDiagnosticConfigs originalConfigs = globalSettings->clangCustomDiagnosticConfigs(); ClangDiagnosticConfigsModel configsModel(globalSettings->clangCustomDiagnosticConfigs()); QTC_ASSERT(configsModel.hasConfigWithId(currentConfigId), return ); ClangDiagnosticConfig config = configsModel.configWithId(currentConfigId); |