Merge lp://qastaging/~3v1n0/bamf/dispose-fixes into lp://qastaging/bamf/0.4

Proposed by Marco Trevisan (Treviño)
Status: Merged
Approved by: Jason Smith
Approved revision: 450
Merged at revision: 448
Proposed branch: lp://qastaging/~3v1n0/bamf/dispose-fixes
Merge into: lp://qastaging/bamf/0.4
Diff against target: 155 lines (+35/-33)
4 files modified
lib/libbamf/bamf-application.c (+5/-5)
lib/libbamf/bamf-control.c (+16/-0)
lib/libbamf/bamf-view.c (+14/-2)
lib/libbamf/bamf-window.c (+0/-26)
To merge this branch: bzr merge lp://qastaging/~3v1n0/bamf/dispose-fixes
Reviewer Review Type Date Requested Status
Jason Smith (community) Approve
Review via email: mp+94824@code.qastaging.launchpad.net

Description of the change

Making libbamf correctly free items when disposing BamfView and BamfControl, plus some misc cleanups

To post a comment you must log in.
Revision history for this message
Jason Smith (jassmith) wrote :

+1 all sensible refactors

refactoring does not require testing as this makes no control logic changes

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches