From c3555fc33d6b7897a36b0176aef68a2e7139d51a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tor=20Arne=20Vestb=C3=B8?= Date: Mon, 24 Aug 2020 18:47:33 +0200 Subject: Add note about use of receivedExpose member in QGuiApplication Change-Id: I221586a4fac394a9110d28905a898ab9688c1183 Reviewed-by: Volker Hilsheimer --- src/gui/kernel/qguiapplication.cpp | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'src/gui/kernel') diff --git a/src/gui/kernel/qguiapplication.cpp b/src/gui/kernel/qguiapplication.cpp index 648b128ffd..bac8bb1968 100644 --- a/src/gui/kernel/qguiapplication.cpp +++ b/src/gui/kernel/qguiapplication.cpp @@ -3202,6 +3202,12 @@ void QGuiApplicationPrivate::processExposeEvent(QWindowSystemInterfacePrivate::E p->resizeEventPending = false; } + // FIXME: It would logically make sense to set this _after_ we've sent the + // expose event to the window, to mark that it now has received an expose. + // But some parts of Qt (mis)use this private member to check whether the + // window has been mapped yet, which they do in code that is triggered + // by the very same expose event we send below. To keep the code working + // we need to set the variable up front, until the code has been fixed. p->receivedExpose = true; } -- cgit v1.2.3