Differential D3941

Fix memory leak in TreeItem
ClosedPublic
Actions

Authored by qi437103 on Jan 3 2017, 10:41 PM.

Details

Reviewers
kfunk
Group Reviewers
KDevelop
Commits
R32:e1e01910d944: Fix memory leak in TreeItem
R33:e1e01910d944: Fix memory leak in TreeItem
Summary

The deletion should be operated on copy rather than the original childItems, which gets cleared already.

Test Plan

Tested with lldb and gdb unittests, with no regression.

Diff Detail

Repository
R33 KDevPlatform
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
qi437103 updated this revision to Diff 9674.Jan 3 2017, 10:41 PM
qi437103 retitled this revision from to Fix memory leak in TreeItem.
qi437103 updated this object.
qi437103 edited the test plan for this revision. (Show Details)
qi437103 added a reviewer: kfunk.
Restricted Application added a subscriber: kdevelop-devel. · View Herald TranscriptJan 3 2017, 10:41 PM
kfunk accepted this revision.Jan 3 2017, 10:53 PM
kfunk edited edge metadata.
This revision is now accepted and ready to land.Jan 3 2017, 10:53 PM
Closed by commit R33:e1e01910d944: Fix memory leak in TreeItem (authored by qi437103). · Explain WhyJan 3 2017, 10:55 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mdebugger/util/treeitem.cpp (5 lines)

Diff 9676

View Options

debugger/util/treeitem.cpp

Loading...