diff options
| author | Marc Mutz <marc.mutz@qt.io> | 2023-06-07 17:34:59 +0200 |
|---|---|---|
| committer | Marc Mutz <marc.mutz@qt.io> | 2023-06-09 12:32:48 +0000 |
| commit | 0e96dd3464bc2e03d66ebb574a7a5352942ffdb2 (patch) | |
| tree | 73ebeb23d12fc6b4eb2151b7a07994eedcb0945e /src/corelib/kernel/qobject.cpp | |
| parent | 588ca25e4a31642bf8d18cd1948c50fa514ac0a7 (diff) | |
tst_QPixmapCache: QVERIFY a failed replace()
Between the find() == 0 obfuscation (since fixed) and this unchecked
replace(), it took me way too much time to figure out what was going
on: the key passed has been invalidated by the setCacheLimit(0).
Now that we QVERIFY that the replace() _fails_, it's much easier to
backtrack and figure out why it does so and why, consequently, the
following find() is also expected to fail.
As a drive-by, reorder two lines so the grouping becomes clearer
(blocks now both headed by setCacheLimit()).
Pick-to: 6.6 6.5 6.2 5.15
Change-Id: I434b65fc13c3fed6512036efeb98d738eeb2a13d
Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io>
Diffstat (limited to 'src/corelib/kernel/qobject.cpp')
0 files changed, 0 insertions, 0 deletions