summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorStanislaw Halik <sthalik@misaki.pl>2018-06-05 08:40:32 +0200
committerStanislaw Halik <sthalik@misaki.pl>2018-06-13 07:02:53 +0200
commit3568f283442d9f4f18141f532ddef156e045f370 (patch)
tree9cb8c5366917a96ef81e6a3e5792bf4456671bec
parent200d21d6408494ca68b8e474278cf023734ef413 (diff)
opentrack: fix some Clazy warnings
-rw-r--r--variant/default/main-window.cpp20
-rw-r--r--variant/default/main-window.hpp2
2 files changed, 11 insertions, 11 deletions
diff --git a/variant/default/main-window.cpp b/variant/default/main-window.cpp
index 7b5ea101..24a6feb3 100644
--- a/variant/default/main-window.cpp
+++ b/variant/default/main-window.cpp
@@ -327,17 +327,17 @@ void main_window::register_shortcuts()
t_keys keys
{
- t_key(s.key_start_tracking1, [&](bool) { start_tracker(); }, true),
- t_key(s.key_start_tracking2, [&](bool) { start_tracker(); }, true),
+ t_key(s.key_start_tracking1, [this](bool) { start_tracker(); }, true),
+ t_key(s.key_start_tracking2, [this](bool) { start_tracker(); }, true),
- t_key(s.key_stop_tracking1, [&](bool) { stop_tracker(); }, true),
- t_key(s.key_stop_tracking2, [&](bool) { stop_tracker(); }, true),
+ t_key(s.key_stop_tracking1, [this](bool) { stop_tracker(); }, true),
+ t_key(s.key_stop_tracking2, [this](bool) { stop_tracker(); }, true),
- t_key(s.key_toggle_tracking1, [&](bool) { toggle_tracker(); }, true),
- t_key(s.key_toggle_tracking2, [&](bool) { toggle_tracker(); }, true),
+ t_key(s.key_toggle_tracking1, [this](bool) { toggle_tracker(); }, true),
+ t_key(s.key_toggle_tracking2, [this](bool) { toggle_tracker(); }, true),
- t_key(s.key_restart_tracking1, [&](bool) { restart_tracker(); }, true),
- t_key(s.key_restart_tracking2, [&](bool) { restart_tracker(); }, true),
+ t_key(s.key_restart_tracking1, [this](bool) { restart_tracker(); }, true),
+ t_key(s.key_restart_tracking2, [this](bool) { restart_tracker(); }, true),
};
global_shortcuts.reload(keys);
@@ -354,7 +354,7 @@ void main_window::die_on_config_not_writable()
QMessageBox::critical(this,
tr("The Octopus is sad"),
- tr("Check permissions for your .ini directory:\n\n\"%1\"%2\n\nExiting now.").arg(group::ini_directory()).arg(pad),
+ tr("Check permissions for your .ini directory:\n\n\"%1\"%2\n\nExiting now.").arg(group::ini_directory(), pad),
QMessageBox::Close, QMessageBox::NoButton);
exit(EX_OSFILE);
@@ -725,7 +725,7 @@ inline bool main_window::mk_window(std::unique_ptr<t>& place, Args&&... params)
}
template<typename t>
-bool main_window::mk_dialog(std::shared_ptr<dylib> lib, std::unique_ptr<t>& d)
+bool main_window::mk_dialog(const std::shared_ptr<dylib>& lib, std::unique_ptr<t>& d)
{
const bool just_created = mk_window_common(d, [&]() -> t* {
if (lib && lib->Dialog)
diff --git a/variant/default/main-window.hpp b/variant/default/main-window.hpp
index bc38c517..0e3d8257 100644
--- a/variant/default/main-window.hpp
+++ b/variant/default/main-window.hpp
@@ -102,7 +102,7 @@ class main_window : public QMainWindow, private State
// only use in impl file since no definition in header!
template<typename t>
- bool mk_dialog(std::shared_ptr<dylib> lib, std::unique_ptr<t>& d);
+ bool mk_dialog(const std::shared_ptr<dylib>& lib, std::unique_ptr<t>& d);
// idem
template<typename t, typename... Args>