Compiz

Merge lp:~3v1n0/compiz/ccs-gsetting-safer-update into lp:compiz/0.9.13

  1. ccs-gsetting-safer-update
  2. Merge into 0.9.13
Proposed by Marco Trevisan (Treviño)
Status: Merged
Approved by: Andrea Azzarone
Approved revision: 4068
Merged at revision: 4069
Proposed branch: lp:~3v1n0/compiz/ccs-gsetting-safer-update
Merge into: lp:compiz/0.9.13
Diff against target: 13 lines (+2/-1)
1 file modified
compizconfig/gsettings/gsettings_backend_shared/ccs_gsettings_backend.c (+2/-1)
To merge this branch: bzr merge lp:~3v1n0/compiz/ccs-gsetting-safer-update
Reviewer Review Type Date Requested Status
Andrea Azzarone Approve
PS Jenkins bot continuous-integration Pending
Review via email: mp+300050@code.launchpad.net

Commit message

ccsGSettingsBackend: don't try to update setting if wrapper is not found

To post a comment you must log in.
Revision history for this message
Andrea Azzarone (azzar1) wrote :

+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'compizconfig/gsettings/gsettings_backend_shared/ccs_gsettings_backend.c'
2--- compizconfig/gsettings/gsettings_backend_shared/ccs_gsettings_backend.c 2016-03-03 14:22:05 +0000
3+++ compizconfig/gsettings/gsettings_backend_shared/ccs_gsettings_backend.c 2016-07-14 09:51:26 +0000
4@@ -93,7 +93,8 @@
5
6 g_value_unset (&schemaNameValue);
7
8- updateSettingWithGSettingsKeyName (backend, wrapper, keyName, backendInterface->updateSetting);
9+ if (wrapper)
10+ updateSettingWithGSettingsKeyName (backend, wrapper, keyName, backendInterface->updateSetting);
11 }
12
13 static CCSGSettingsWrapper *

Subscribers

People subscribed via source and target branches