Differential D9156

Fix a memory leak in KuitStaticData
ClosedPublic
Actions

Authored by jtamate on Dec 3 2017, 9:51 PM.

Details

Reviewers
dfaure
Group Reviewers
Frameworks
Commits
R249:170317278f6b: Fix a memory leak in KuitStaticData
Summary

domainSetups QHash was populated in Kuit::setupForDomain, but
the values of the QHash where not deleted in the default destructor.
All the credit goes to mpyne, anthonyfierony, aacid and apol.

Diff Detail

Repository
R249 KI18n
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
jtamate created this revision.Dec 3 2017, 9:51 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptDec 3 2017, 9:51 PM
jtamate requested review of this revision.Dec 3 2017, 9:51 PM
dfaure accepted this revision.Dec 4 2017, 8:29 AM
This revision is now accepted and ready to land.Dec 4 2017, 8:29 AM
Closed by commit R249:170317278f6b: Fix a memory leak in KuitStaticData (authored by jtamate). · Explain WhyDec 4 2017, 12:00 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msrc/kuitmarkup.cpp (7 lines)

Diff 23418

View Options

src/kuitmarkup.cpp

Loading...