From c9b963e82b751e5af9f6105cf1cbae46c537829e Mon Sep 17 00:00:00 2001 From: dmitry Date: Tue, 4 Jul 2023 01:17:26 +0300 Subject: [PATCH 1/3] fix segfault --- include/modules/hyprland/workspaces.hpp | 7 ++- src/modules/hyprland/workspaces.cpp | 67 ++++++++++++++++++------- 2 files changed, 55 insertions(+), 19 deletions(-) diff --git a/include/modules/hyprland/workspaces.hpp b/include/modules/hyprland/workspaces.hpp index 0b8a452..5033b8d 100644 --- a/include/modules/hyprland/workspaces.hpp +++ b/include/modules/hyprland/workspaces.hpp @@ -1,5 +1,6 @@ #include #include +#include #include "AModule.hpp" #include "bar.hpp" @@ -44,12 +45,16 @@ class Workspaces : public AModule, public EventHandler { private: void onEvent(const std::string&) override; void sort_workspaces(); + void create_workspace(int id); + void remove_workspace(int id); std::string format_; std::map icons_map_; bool with_icon_; int active_workspace_id; - std::vector workspaces_; + std::vector> workspaces_; + std::vector workspaces_to_create_; + std::vector workspaces_to_remove_; std::mutex mutex_; const Bar& bar_; Gtk::Box box_; diff --git a/src/modules/hyprland/workspaces.cpp b/src/modules/hyprland/workspaces.cpp index 4321e22..23a323a 100644 --- a/src/modules/hyprland/workspaces.cpp +++ b/src/modules/hyprland/workspaces.cpp @@ -5,6 +5,7 @@ #include #include +#include #include namespace waybar::modules::hyprland { @@ -45,22 +46,34 @@ Workspaces::Workspaces(const std::string &id, const Bar &bar, const Json::Value } auto Workspaces::update() -> void { - std::lock_guard lock(mutex_); - for (Workspace &workspace : workspaces_) { - workspace.set_active(workspace.id() == active_workspace_id); + for (int &workspace_to_remove : workspaces_to_remove_) { + remove_workspace(workspace_to_remove); + } + + workspaces_to_remove_.clear(); + + for (int &workspace_to_create : workspaces_to_create_) { + create_workspace(workspace_to_create); + } + + workspaces_to_create_.clear(); + + for (std::unique_ptr &workspace : workspaces_) { + workspace->set_active(workspace->id() == active_workspace_id); std::string &workspace_icon = icons_map_[""]; if (with_icon_) { - workspace_icon = workspace.select_icon(icons_map_); + workspace_icon = workspace->select_icon(icons_map_); } - workspace.update(format_, workspace_icon); + workspace->update(format_, workspace_icon); } AModule::update(); } void Workspaces::onEvent(const std::string &ev) { + std::lock_guard lock(mutex_); std::string eventName(begin(ev), begin(ev) + ev.find_first_of('>')); std::string payload = ev.substr(eventName.size() + 2); if (eventName == "workspace") { @@ -68,33 +81,49 @@ void Workspaces::onEvent(const std::string &ev) { } else if (eventName == "destroyworkspace") { int deleted_workspace_id; std::from_chars(payload.data(), payload.data() + payload.size(), deleted_workspace_id); - auto workspace = std::find_if(workspaces_.begin(), workspaces_.end(), - [&](Workspace &x) { return x.id() == deleted_workspace_id; }); - box_.remove(workspace->button()); - workspaces_.erase(workspace); + workspaces_to_remove_.push_back(deleted_workspace_id); } else if (eventName == "createworkspace") { int new_workspace_id; std::from_chars(payload.data(), payload.data() + payload.size(), new_workspace_id); - workspaces_.push_back(new_workspace_id); - Gtk::Button &new_workspace_button = workspaces_.back().button(); - box_.pack_end(new_workspace_button, false, false); - sort_workspaces(); - new_workspace_button.show_all(); + workspaces_to_create_.push_back(new_workspace_id); } dp.emit(); } +void Workspaces::create_workspace(int id) { + workspaces_.push_back(std::make_unique(id)); + Gtk::Button &new_workspace_button = workspaces_.back()->button(); + box_.pack_start(new_workspace_button, false, false); + sort_workspaces(); + new_workspace_button.show_all(); +} + +void Workspaces::remove_workspace(int id) { + auto workspace = std::find_if( + workspaces_.begin(), workspaces_.end(), + [&](std::unique_ptr &x) { return x->id() == id; }); + + if (workspace == workspaces_.end()) { + spdlog::warn("Can't find workspace with id {}", workspace->get()->id()); + return; + } + + box_.remove(workspace->get()->button()); + workspaces_.erase(workspace); +} + void Workspaces::init() { const auto activeWorkspace = WorkspaceDto::parse(gIPC->getSocket1JsonReply("activeworkspace")); active_workspace_id = activeWorkspace.id; const Json::Value workspaces_json = gIPC->getSocket1JsonReply("workspaces"); for (const Json::Value &workspace_json : workspaces_json) { - workspaces_.push_back(Workspace(WorkspaceDto::parse(workspace_json))); + workspaces_.push_back( + std::make_unique(Workspace(WorkspaceDto::parse(workspace_json)))); } for (auto &workspace : workspaces_) { - box_.pack_start(workspace.button(), false, false); + box_.pack_start(workspace->button(), false, false); } sort_workspaces(); @@ -139,10 +168,12 @@ void Workspace::update(const std::string &format, const std::string &icon) { void Workspaces::sort_workspaces() { std::sort(workspaces_.begin(), workspaces_.end(), - [](Workspace &lhs, Workspace &rhs) { return lhs.id() < rhs.id(); }); + [](std::unique_ptr &lhs, std::unique_ptr &rhs) { + return lhs->id() < rhs->id(); + }); for (size_t i = 0; i < workspaces_.size(); ++i) { - box_.reorder_child(workspaces_[i].button(), i); + box_.reorder_child(workspaces_[i]->button(), i); } } From de626dcbbccb5e20b597dec7018a930059ca8ab9 Mon Sep 17 00:00:00 2001 From: dmitry Date: Tue, 4 Jul 2023 01:24:34 +0300 Subject: [PATCH 2/3] format --- include/modules/hyprland/workspaces.hpp | 1 + src/modules/hyprland/workspaces.cpp | 17 ++++++++--------- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/include/modules/hyprland/workspaces.hpp b/include/modules/hyprland/workspaces.hpp index 5033b8d..500bbe3 100644 --- a/include/modules/hyprland/workspaces.hpp +++ b/include/modules/hyprland/workspaces.hpp @@ -1,5 +1,6 @@ #include #include + #include #include "AModule.hpp" diff --git a/src/modules/hyprland/workspaces.cpp b/src/modules/hyprland/workspaces.cpp index 23a323a..f5cc832 100644 --- a/src/modules/hyprland/workspaces.cpp +++ b/src/modules/hyprland/workspaces.cpp @@ -100,17 +100,16 @@ void Workspaces::create_workspace(int id) { } void Workspaces::remove_workspace(int id) { - auto workspace = std::find_if( - workspaces_.begin(), workspaces_.end(), - [&](std::unique_ptr &x) { return x->id() == id; }); + auto workspace = std::find_if(workspaces_.begin(), workspaces_.end(), + [&](std::unique_ptr &x) { return x->id() == id; }); - if (workspace == workspaces_.end()) { - spdlog::warn("Can't find workspace with id {}", workspace->get()->id()); - return; - } + if (workspace == workspaces_.end()) { + spdlog::warn("Can't find workspace with id {}", workspace->get()->id()); + return; + } - box_.remove(workspace->get()->button()); - workspaces_.erase(workspace); + box_.remove(workspace->get()->button()); + workspaces_.erase(workspace); } void Workspaces::init() { From f26a125d150735286f855b15b1634231dfedf8d4 Mon Sep 17 00:00:00 2001 From: dmitry Date: Tue, 4 Jul 2023 01:26:16 +0300 Subject: [PATCH 3/3] format --- src/AModule.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/AModule.cpp b/src/AModule.cpp index b84f551..2626cd8 100644 --- a/src/AModule.cpp +++ b/src/AModule.cpp @@ -102,7 +102,7 @@ AModule::SCROLL_DIR AModule::getScrollDir(GdkEventScroll* e) { bool reverse_mouse = config_["reverse-mouse-scrolling"].asBool(); // ignore reverse-scrolling if event comes from a mouse wheel - GdkDevice* device = gdk_event_get_source_device((GdkEvent *)e); + GdkDevice* device = gdk_event_get_source_device((GdkEvent*)e); if (device != NULL && gdk_device_get_source(device) == GDK_SOURCE_MOUSE) { reverse = reverse_mouse; }