From 52a4e761a8e4b4440e7b73162495303e0b1e8a24 Mon Sep 17 00:00:00 2001 From: Alexis Date: Sun, 19 Aug 2018 13:43:00 +0200 Subject: [PATCH] fix(workspaces): avoid useless mutex lock --- src/modules/sway/workspaces.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/modules/sway/workspaces.cpp b/src/modules/sway/workspaces.cpp index 69563bdd..3b702bcb 100644 --- a/src/modules/sway/workspaces.cpp +++ b/src/modules/sway/workspaces.cpp @@ -136,7 +136,6 @@ std::string waybar::modules::sway::Workspaces::getIcon(std::string name) bool waybar::modules::sway::Workspaces::handleScroll(GdkEventScroll *e) { - std::lock_guard lock(mutex_); // Avoid concurrent scroll event if (scrolling_) { return false; @@ -144,6 +143,7 @@ bool waybar::modules::sway::Workspaces::handleScroll(GdkEventScroll *e) scrolling_ = true; int id = -1; uint16_t idx = 0; + std::lock_guard lock(mutex_); for (; idx < workspaces_.size(); idx += 1) { if (workspaces_[idx]["focused"].asBool()) { id = workspaces_[idx]["num"].asInt();