refactor: get rid of some mutex
parent
340ec7be91
commit
90a9c0e25f
|
@ -27,7 +27,6 @@ class AModule : public IModule {
|
||||||
const Json::Value &config_;
|
const Json::Value &config_;
|
||||||
Gtk::EventBox event_box_;
|
Gtk::EventBox event_box_;
|
||||||
std::string click_param_;
|
std::string click_param_;
|
||||||
std::mutex mutex_;
|
|
||||||
|
|
||||||
virtual bool handleToggle(GdkEventButton *const &ev);
|
virtual bool handleToggle(GdkEventButton *const &ev);
|
||||||
virtual bool handleScroll(GdkEventScroll *);
|
virtual bool handleScroll(GdkEventScroll *);
|
||||||
|
|
|
@ -88,9 +88,6 @@ AModule::SCROLL_DIR AModule::getScrollDir(GdkEventScroll* e) {
|
||||||
}
|
}
|
||||||
|
|
||||||
bool AModule::handleScroll(GdkEventScroll* e) {
|
bool AModule::handleScroll(GdkEventScroll* e) {
|
||||||
// Avoid concurrent scroll event
|
|
||||||
std::lock_guard<std::mutex> lock(mutex_);
|
|
||||||
|
|
||||||
auto dir = getScrollDir(e);
|
auto dir = getScrollDir(e);
|
||||||
if (dir == SCROLL_DIR::UP && config_["on-scroll-up"].isString()) {
|
if (dir == SCROLL_DIR::UP && config_["on-scroll-up"].isString()) {
|
||||||
pid_.push_back(util::command::forkExec(config_["on-scroll-up"].asString()));
|
pid_.push_back(util::command::forkExec(config_["on-scroll-up"].asString()));
|
||||||
|
|
Loading…
Reference in New Issue