diff options
author | Tor Arne Vestbø <tor.arne.vestbo@qt.io> | 2020-08-24 18:47:33 +0200 |
---|---|---|
committer | Tor Arne Vestbø <tor.arne.vestbo@qt.io> | 2020-08-26 16:44:49 +0200 |
commit | c3555fc33d6b7897a36b0176aef68a2e7139d51a (patch) | |
tree | faab5db18c2e893e2edcf538fd275d78b19a7b53 | |
parent | 560117351cd1e032c30e2691a2e933e3ebb84edd (diff) |
Add note about use of receivedExpose member in QGuiApplication
Change-Id: I221586a4fac394a9110d28905a898ab9688c1183
Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io>
-rw-r--r-- | src/gui/kernel/qguiapplication.cpp | 6 |
1 files changed, 6 insertions, 0 deletions
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; } |