From 65853812306414b525ea6beb25dc915249dc922e Mon Sep 17 00:00:00 2001
From: Aleksei Bavshin <alebastr89@gmail.com>
Date: Mon, 8 Feb 2021 22:30:01 -0800
Subject: [PATCH] fix(client): remove unnecessary wl_output_roundtrip

At this point we're not awaiting any protocol events and flushing
wayland queue makes little sense. As #1019 shows, it may be even harmful
as an extra roundtrip could process wl_output disappearance and delete
output object right from under our code.
---
 src/client.cpp | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/client.cpp b/src/client.cpp
index 0ad4e6bb..9983bb56 100644
--- a/src/client.cpp
+++ b/src/client.cpp
@@ -124,7 +124,6 @@ void waybar::Client::handleOutputDone(void *data, struct zxdg_output_v1 * /*xdg_
 
     auto configs = client->getOutputConfigs(output);
     if (!configs.empty()) {
-      wl_display_roundtrip(client->wl_display);
       for (const auto &config : configs) {
         client->bars.emplace_back(std::make_unique<Bar>(&output, config));
       }