From 5d2d7cde8efb14c6b752148e81881c2ce74c5870 Mon Sep 17 00:00:00 2001 From: NRK Date: Thu, 29 Jun 2023 03:02:22 +0600 Subject: x11_common: avoid unnecessary XUnmapWindow() call XDestroyWindow() is called immediately after, which also unmaps window if needed. according to the manpage: > If the window specified by the w argument is mapped, it is unmapped > automatically. --- video/out/x11_common.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) (limited to 'video') diff --git a/video/out/x11_common.c b/video/out/x11_common.c index 03d624ac0c..7ce974d5ce 100644 --- a/video/out/x11_common.c +++ b/video/out/x11_common.c @@ -834,10 +834,8 @@ void vo_x11_uninit(struct vo *vo) set_screensaver(x11, true); - if (x11->window != None && x11->window != x11->rootwin) { - XUnmapWindow(x11->display, x11->window); + if (x11->window != None && x11->window != x11->rootwin) XDestroyWindow(x11->display, x11->window); - } if (x11->xic) XDestroyIC(x11->xic); if (x11->colormap != None) -- cgit v1.2.3