Differential D1059

Fix leaks
ClosedPublic
Actions

Authored by ervin on Mar 2 2016, 10:19 PM.

Details

Reviewers
bensi
dfaure
franckarrecot
Commits
R4:c22575a5db2b: Fix leaks

Diff Detail

Repository
R4 Zanshin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ervin updated this revision to Diff 2535.Mar 2 2016, 10:19 PM
ervin retitled this revision from to Fix leaks.
ervin updated this object.
ervin edited the test plan for this revision. (Show Details)
ervin added reviewers: bensi, franckarrecot, dfaure.
dfaure edited edge metadata.Mar 3 2016, 7:56 AM

Is it useful to create 12 review requests with basically the same thing in them? :-)

bensi accepted this revision.Mar 3 2016, 12:05 PM
bensi edited edge metadata.
This revision is now accepted and ready to land.Mar 3 2016, 12:05 PM
ervin added a comment.Mar 3 2016, 12:19 PM

I was torn on that, you seemed to dislike giant patches in the past, so I split this time. :-)

ervin updated this revision to Diff 2547.Mar 3 2016, 7:26 PM
ervin edited edge metadata.

Address dfaure's remark about refcounting

Closed by commit R4:c22575a5db2b: Fix leaks (authored by ervin). · Explain WhyMar 6 2016, 4:22 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 2589

View Options

tests/units/presentation/querytreemodeltest.cpp

Loading...