Merge pull request #467 from jspngh/bugfix/mpd_module

Add more locking in mpd module
This commit is contained in:
Alex 2019-09-19 22:03:23 +01:00 committed by GitHub
commit 52db1e8e68
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 13 additions and 1 deletions

View File

@ -65,6 +65,9 @@ class MPD : public ALabel {
unique_status status_; unique_status status_;
mpd_state state_; mpd_state state_;
unique_song song_; unique_song song_;
// To make sure the previous periodic_updater stops before creating a new one
std::mutex periodic_lock_;
}; };
} // namespace waybar::modules } // namespace waybar::modules

View File

@ -41,6 +41,10 @@ auto waybar::modules::MPD::update() -> void {
if (connection_ != nullptr) { if (connection_ != nullptr) {
try { try {
bool wasPlaying = playing(); bool wasPlaying = playing();
if(!wasPlaying) {
// Wait until the periodic_updater has stopped
std::lock_guard periodic_guard(periodic_lock_);
}
fetchState(); fetchState();
if (!wasPlaying && playing()) { if (!wasPlaying && playing()) {
periodic_updater().detach(); periodic_updater().detach();
@ -75,6 +79,7 @@ std::thread waybar::modules::MPD::event_listener() {
std::thread waybar::modules::MPD::periodic_updater() { std::thread waybar::modules::MPD::periodic_updater() {
return std::thread([this] { return std::thread([this] {
std::lock_guard guard(periodic_lock_);
while (connection_ != nullptr && playing()) { while (connection_ != nullptr && playing()) {
dp.emit(); dp.emit();
std::this_thread::sleep_for(std::chrono::seconds(1)); std::this_thread::sleep_for(std::chrono::seconds(1));
@ -297,7 +302,7 @@ void waybar::modules::MPD::waitForEvent() {
// Wait for a player (play/pause), option (random, shuffle, etc.), or playlist // Wait for a player (play/pause), option (random, shuffle, etc.), or playlist
// change // change
if (!mpd_send_idle_mask( if (!mpd_send_idle_mask(
conn, static_cast<mpd_idle>(MPD_IDLE_PLAYER | MPD_IDLE_OPTIONS | MPD_IDLE_PLAYLIST))) { conn, static_cast<mpd_idle>(MPD_IDLE_PLAYER | MPD_IDLE_OPTIONS | MPD_IDLE_QUEUE))) {
checkErrors(conn); checkErrors(conn);
return; return;
} }
@ -306,6 +311,10 @@ void waybar::modules::MPD::waitForEvent() {
// See issue #277: // See issue #277:
// https://github.com/Alexays/Waybar/issues/277 // https://github.com/Alexays/Waybar/issues/277
mpd_recv_idle(conn, /* disable_timeout = */ false); mpd_recv_idle(conn, /* disable_timeout = */ false);
// See issue #281:
// https://github.com/Alexays/Waybar/issues/281
std::lock_guard guard(connection_lock_);
checkErrors(conn); checkErrors(conn);
mpd_response_finish(conn); mpd_response_finish(conn);