Fix memory leak
ClosedPublic

Authored by gpark on Jul 9 2017, 11:57 AM.

Diff Detail

Repository
R7 Konversation
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
gpark created this revision.Jul 9 2017, 11:57 AM
gpark added a reviewer: hein.Jul 9 2017, 11:59 AM
hein requested changes to this revision.Jul 10 2017, 6:55 AM

Please file separate reviews each logical change. You can repurpose this one for the memory leak.

This revision now requires changes to proceed.Jul 10 2017, 6:55 AM
gpark updated this revision to Diff 16427.Jul 10 2017, 7:48 AM
gpark edited edge metadata.

Fix memory leak

gpark updated this revision to Diff 16428.Jul 10 2017, 7:49 AM

Fix memory leak

gpark updated this revision to Diff 16429.Jul 10 2017, 7:50 AM

Fix memory leak

gpark updated this revision to Diff 16430.Jul 10 2017, 7:52 AM

Fix memory leak

hein requested changes to this revision.Jul 10 2017, 7:54 AM

The diff contains unrelated changes now and still isn't split up.

No worries, starting with git and arc has a learning curve to it ;)

This revision now requires changes to proceed.Jul 10 2017, 7:54 AM
gpark updated this revision to Diff 16431.Jul 10 2017, 7:56 AM
gpark edited edge metadata.

Fix memory leak

hein accepted this revision.Jul 10 2017, 7:57 AM
This revision is now accepted and ready to land.Jul 10 2017, 7:57 AM
gpark edited the summary of this revision. (Show Details)Jul 10 2017, 7:58 AM
Closed by commit R7:cb2c91f06bc9: Fix memory leak (authored by gpark, committed by hein). · Explain WhyJul 10 2017, 8:02 AM
This revision was automatically updated to reflect the committed changes.