diff options
author | Giuseppe D'Angelo <giuseppe.dangelo@kdab.com> | 2021-09-16 23:18:10 +0200 |
---|---|---|
committer | Giuseppe D'Angelo <giuseppe.dangelo@kdab.com> | 2021-09-17 16:48:45 +0200 |
commit | d8fd2425fb37b5f745e385552b11dc4cdcffb5ff (patch) | |
tree | 7cc859752c038adda830b467df4e618bf2d6c5d5 /mkspecs | |
parent | 0cea6384ae641b955d0f51bc344b0fb6bb1ae029 (diff) |
QCoreApplication: enforce non-null arguments when sending/posting events
Passing nullptr as receiver and/or as an event parameter to sendEvent,
postEvent, etc. is meaningless. It's also something that users can check
for. Therefore, it should not be allowed. Note that the current code
already relies on the arguments not to be null, albeit "indirectly"
(e.g. they get dereferenced without any null checks).
Hence: add asserts that check for non-null in all the relevant
codepaths, except for the ones in which there's currently just a
warning; for those, add a Qt 7 note.
Change-Id: Ia4c58551de88a5d1003f09efa448c1330b6cb122
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io>
Reviewed-by: Edward Welbourne <edward.welbourne@qt.io>
Diffstat (limited to 'mkspecs')
0 files changed, 0 insertions, 0 deletions