diff options
| author | Ikalco <73481042+ikalco@users.noreply.github.com> | 2024-12-01 11:14:35 -0600 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-12-01 17:14:35 +0000 |
| commit | 6d7544458d0fafcae410c1978a0cabce2fb4a346 (patch) | |
| tree | 77bac5cd967326e3f643cbb91fc7e7ae910d401c /src/protocols/XDGShell.cpp | |
| parent | d26439a0fe5594fb26d5a3c01571f9490a9a2d2c (diff) | |
cleanup: use doLater instead of adding idle event handlers (#8624)
Diffstat (limited to 'src/protocols/XDGShell.cpp')
| -rw-r--r-- | src/protocols/XDGShell.cpp | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/src/protocols/XDGShell.cpp b/src/protocols/XDGShell.cpp index 688d0006..a542c394 100644 --- a/src/protocols/XDGShell.cpp +++ b/src/protocols/XDGShell.cpp @@ -3,6 +3,7 @@ #include <algorithm> #include "../Compositor.hpp" #include "../managers/SeatManager.hpp" +#include "../managers/eventLoop/EventLoopManager.hpp" #include "core/Seat.hpp" #include "core/Compositor.hpp" #include <cstring> @@ -469,8 +470,6 @@ CXDGSurfaceResource::CXDGSurfaceResource(SP<CXdgSurface> resource_, SP<CXDGWMBas CXDGSurfaceResource::~CXDGSurfaceResource() { events.destroy.emit(); - if (configureSource) - wl_event_source_remove(configureSource); if (surface) surface->resetRole(); } @@ -484,22 +483,23 @@ SP<CXDGSurfaceResource> CXDGSurfaceResource::fromResource(wl_resource* res) { return data ? data->self.lock() : nullptr; } -static void onConfigure(void* data) { - ((CXDGSurfaceResource*)data)->configure(); -} - uint32_t CXDGSurfaceResource::scheduleConfigure() { - if (configureSource) + if (configureScheduled) return scheduledSerial; - configureSource = wl_event_loop_add_idle(g_pCompositor->m_sWLEventLoop, onConfigure, this); scheduledSerial = wl_display_next_serial(g_pCompositor->m_sWLDisplay); + configureScheduled = true; + g_pEventLoopManager->doLater([this]() { configure(); }); + return scheduledSerial; } void CXDGSurfaceResource::configure() { - configureSource = nullptr; + if (!resource) + return; + + configureScheduled = false; resource->sendConfigure(scheduledSerial); } |
