Compiz Configuration Library - GConf Backend

Merge lp://qastaging/~compiz-team/compiz-compizconfig-gconf/compiz-compizconfig-gconf.fix_874794 into lp://qastaging/compiz-compizconfig-gconf

  1. compiz-compizconfig-gconf.fix_874794
  2. Merge into 0.9.5
Proposed by Sam Spilsbury
Status: Superseded
Proposed branch: lp://qastaging/~compiz-team/compiz-compizconfig-gconf/compiz-compizconfig-gconf.fix_874794
Merge into: lp://qastaging/compiz-compizconfig-gconf
Diff against target: 173 lines (+19/-19)
1 file modified
src/gconf.c (+19/-19)
To merge this branch: bzr merge lp://qastaging/~compiz-team/compiz-compizconfig-gconf/compiz-compizconfig-gconf.fix_874794
Reviewer Review Type Date Requested Status
Compiz Maintainers Pending
Review via email: mp+79455@code.qastaging.launchpad.net

Description of the change

Bug 874794

To post a comment you must log in.
Revision history for this message
Tim Penhey (thumper) wrote :

The bug and the code don't look like they relate to each other. The bug talks about reading, but the methods you have changed are setting values.

Revision history for this message
Sam Spilsbury (smspillaz) wrote :

Ah, this code writes changes directly to the structure, but changes the arguments so that they are not also written to disk.

See my comment on 874830 on a way to make that more clear.

lp://qastaging/~compiz-team/compiz-compizconfig-gconf/compiz-compizconfig-gconf.fix_874794 updated
163. By Sam Spilsbury

Use more descriptive names

Revision history for this message
Daniel van Vugt (vanvugt) wrote :

Ping compiz-team :)

Unmerged revisions

163. By Sam Spilsbury

Use more descriptive names

162. By Sam Spilsbury

Don't immediately update settings on read

161. By Sam Spilsbury

Merge

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

to all changes: