Merge lp:~ted/indicator-session/lp864085 into lp:indicator-session/0.1
Proposed by
Ted Gould
Status: | Merged |
---|---|
Merged at revision: | 224 |
Proposed branch: | lp:~ted/indicator-session/lp864085 |
Merge into: | lp:indicator-session/0.1 |
Diff against target: | 0 lines |
To merge this branch: | bzr merge lp:~ted/indicator-session/lp864085 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Michal Hruby (community) | Approve | ||
Review via email: mp+78733@code.launchpad.net |
Description of the change
Protect against the case where we can't get the settings for whatever reason.
To post a comment you must log in.
Putting function calls into g_return_* macros is quite dangerous, cause you can easily compile those away (using G_DISABLE_CHECKS). Please use a temp variable for what you're doing.