diff options
| author | Volker Hilsheimer <volker.hilsheimer@qt.io> | 2021-02-23 15:14:05 +0100 |
|---|---|---|
| committer | Volker Hilsheimer <volker.hilsheimer@qt.io> | 2021-02-24 11:51:47 +0100 |
| commit | f6c9dec610238bae0d2d61e733d4c29ecf57aa9d (patch) | |
| tree | fb1c1e2010623e62fba208049589ffe9e01ffbb2 /src | |
| parent | 10b3681c72491cacf9ccf515fa2cbdf2ef131ff9 (diff) | |
Assert that dereferenced pointer is never nullptr
It can't be, since then 'o' (which is never nullptr) would be different,
and we'd return in the previous line.
Since we test for d->popup a few lines above, help clang static analyzer
to not report this with hash
27ff831d79d1779825fd964239cbefe7.
Pick-to: 6.1
Change-Id: I9f0b14bd88418b897cee2d7a63e8a10fc4a0b01c
Reviewed-by: Richard Moe Gustavsen <richard.gustavsen@qt.io>
Diffstat (limited to 'src')
| -rw-r--r-- | src/widgets/util/qcompleter.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/widgets/util/qcompleter.cpp b/src/widgets/util/qcompleter.cpp index 7bfd112f5c..13a28b628c 100644 --- a/src/widgets/util/qcompleter.cpp +++ b/src/widgets/util/qcompleter.cpp @@ -1329,6 +1329,7 @@ bool QCompleter::eventFilter(QObject *o, QEvent *e) if (o != d->popup) return QObject::eventFilter(o, e); + Q_ASSERT(d->popup); switch (e->type()) { case QEvent::KeyPress: { QKeyEvent *ke = static_cast<QKeyEvent *>(e); |