Merge lp://qastaging/~azzar1/compiz/dont-crash-if-gsettings-key-not-found into lp://qastaging/compiz/0.9.13

Proposed by Andrea Azzarone
Status: Merged
Approved by: Marco Trevisan (Treviño)
Approved revision: 4075
Merged at revision: 4080
Proposed branch: lp://qastaging/~azzar1/compiz/dont-crash-if-gsettings-key-not-found
Merge into: lp://qastaging/compiz/0.9.13
Diff against target: 150 lines (+43/-18)
4 files modified
compizconfig/gsettings/gsettings_backend_shared/ccs_gsettings_interface_wrapper.c (+27/-3)
compizconfig/gsettings/gsettings_backend_shared/gsettings_util.c (+12/-11)
compizconfig/gsettings/gsettings_backend_shared/gsettings_util.h (+1/-1)
compizconfig/gsettings/tests/test_gsettings_tests.cpp (+3/-3)
To merge this branch: bzr merge lp://qastaging/~azzar1/compiz/dont-crash-if-gsettings-key-not-found
Reviewer Review Type Date Requested Status
Marco Trevisan (Treviño) Approve
Review via email: mp+301174@code.qastaging.launchpad.net

Commit message

Don't crash if gsettings key is not found. Default value will be used.

To post a comment you must log in.
4073. By Andrea Azzarone

Unref the schema.

4074. By Andrea Azzarone

Avoid NULL unref.

4075. By Andrea Azzarone

Fix typo.

Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

+1

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