diff options
author | Friedemann Kleint <Friedemann.Kleint@qt.io> | 2019-06-07 08:58:42 +0200 |
---|---|---|
committer | Friedemann Kleint <Friedemann.Kleint@qt.io> | 2019-06-07 09:48:02 +0200 |
commit | 431c3696e1cc3e15610c9cb4ef55b768f5fc9128 (patch) | |
tree | 7432330ed6fa02f5ffe5082ebeb655db2d89b4e8 /tools | |
parent | de8eb86cc51c2a00abff53f1cafb9db8674f93c2 (diff) |
Cleanup of Clang warnings (with exception of dumpcpp)
- Fix remaining nullptr
- else after return
- Use const ref to avoid copies of variables
- C-Style casts
- Use multi-arg version of QString::arg()
Change-Id: I4525f68bd576f9d6d7249d5153198896b8653fa7
Reviewed-by: Oliver Wolff <oliver.wolff@qt.io>
Diffstat (limited to 'tools')
-rw-r--r-- | tools/testcon/mainwindow.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/testcon/mainwindow.cpp b/tools/testcon/mainwindow.cpp index 243254a..14db741 100644 --- a/tools/testcon/mainwindow.cpp +++ b/tools/testcon/mainwindow.cpp @@ -421,7 +421,7 @@ void MainWindow::on_actionScriptingRun_triggered() QVariant result = m_scripts->call(macro); if (result.isValid()) - logMacros->append(tr("Return value of %1: %2").arg(macro).arg(result.toString())); + logMacros->append(tr("Return value of %1: %2").arg(macro, result.toString())); #endif } |