wayland: Cleanup an outdated conditional and comment

There may have been a reason for not attaching a null buffer when destroying a popup at some point in the past, but that is unnecessary now, as is the comment about ShowWindow crashing, as ShowWindow assures that a null buffer is attached before (re)creating the window.
This commit is contained in:
Frank Praznik
2025-08-01 10:33:55 -04:00
parent 9034375d2f
commit 550d95e04f

View File

@@ -2164,11 +2164,9 @@ void Wayland_HideWindow(SDL_VideoDevice *_this, SDL_Window *window)
} }
} }
// Be sure to detach after this is done, otherwise ShowWindow crashes! // Attach a null buffer to unmap the surface.
if (wind->shell_surface_type != WAYLAND_SHELL_SURFACE_TYPE_XDG_POPUP) { wl_surface_attach(wind->surface, NULL, 0, 0);
wl_surface_attach(wind->surface, NULL, 0, 0); wl_surface_commit(wind->surface);
wl_surface_commit(wind->surface);
}
SDL_zero(wind->shell_surface); SDL_zero(wind->shell_surface);
wind->show_hide_sync_required = true; wind->show_hide_sync_required = true;