diff options
author | Marc Mutz <marc.mutz@kdab.com> | 2016-09-29 09:06:12 +0200 |
---|---|---|
committer | Marc Mutz <marc.mutz@kdab.com> | 2016-10-22 10:20:22 +0000 |
commit | e70e168abb4b75dc8d00e2cba12efa1a111d429d (patch) | |
tree | b16be0342ba8f948c551abdc9a35e39be3f54a07 /mkspecs | |
parent | 84dc7d5f5560d69a0d7973f591ce253605a458ac (diff) |
Plug leaks in tst_QWizard
This completely over-engineered piece of code has a hierarchy of
Operation subclasses encapsulating but three actual operations
on a QWizard.
Because these operations and their containers were all allocated
on the heap, but never deleted, asan went crazy and reported over
50 leaks (not the record so far, but a (distant) second).
Since these collections are passed through addColumn/QFETCH, too,
it's nearly impossible to track their lifetimes. So instead of
trying, delegate that to the runtime, ie. pack the Operation
objects into QSharedPointer and pass around those instead.
Change-Id: I8a0fe7a60cd30aed618667affaa030e80cf2b1ac
Reviewed-by: Edward Welbourne <edward.welbourne@qt.io>
Reviewed-by: Sérgio Martins <sergio.martins@kdab.com>
Diffstat (limited to 'mkspecs')
0 files changed, 0 insertions, 0 deletions