From 1e89c19a03ddd654e063e26fbb5223247212822d Mon Sep 17 00:00:00 2001 From: Ilya Fedin Date: Fri, 9 Jun 2023 03:54:50 +0400 Subject: [PATCH] Make window check in Wayland integration assert This function was always requiring a window handle but after latest refactor it would fail silently --- ui/platform/linux/ui_linux_wayland_integration.cpp | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/ui/platform/linux/ui_linux_wayland_integration.cpp b/ui/platform/linux/ui_linux_wayland_integration.cpp index c87a2dc..c76a293 100644 --- a/ui/platform/linux/ui_linux_wayland_integration.cpp +++ b/ui/platform/linux/ui_linux_wayland_integration.cpp @@ -92,9 +92,7 @@ void WaylandIntegration::setWindowExtents( not_null widget, const QMargins &extents) { const auto window = widget->windowHandle(); - if (!window) { - return; - } + Expects(window != nullptr); const auto native = window->nativeInterface(); if (!native) { @@ -106,9 +104,7 @@ void WaylandIntegration::setWindowExtents( void WaylandIntegration::unsetWindowExtents(not_null widget) { const auto window = widget->windowHandle(); - if (!window) { - return; - } + Expects(window != nullptr); const auto native = window->nativeInterface(); if (!native) { @@ -122,9 +118,7 @@ void WaylandIntegration::showWindowMenu( not_null widget, const QPoint &point) { const auto window = widget->windowHandle(); - if (!window) { - return; - } + Expects(window != nullptr); const auto native = qApp->nativeInterface(); const auto nativeWindow = window->nativeInterface();