Merge lp:~charlesk/indicator-applet/fix-944220 into lp:indicator-applet/0.5
Proposed by
Charles Kerr
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Ted Gould | ||||
Approved revision: | 397 | ||||
Merge reported by: | Charles Kerr | ||||
Merged at revision: | not available | ||||
Proposed branch: | lp:~charlesk/indicator-applet/fix-944220 | ||||
Merge into: | lp:indicator-applet/0.5 | ||||
Diff against target: |
150 lines (+0/-131) 2 files modified
src/eggaccelerators.c (+0/-128) src/eggaccelerators.h (+0/-3) |
||||
To merge this branch: | bzr merge lp:~charlesk/indicator-applet/fix-944220 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Ted Gould (community) | Approve | ||
Review via email:
|
Description of the change
I agree with Ted's assessment that the warnings in Bug #944220 are a low priority.
Happily, since the function generating these warnings is completely unused, we can fix the warnings and avoid the need for testing by just removing the function. :)
To post a comment you must log in.
On Sat, 2012-03-03 at 18:43 +0000, charles wrote:
> I agree with Ted's assessment that the warnings in Bug #944220 are a
> low priority.
>
> Happily, since the function generating these warnings is completely
> unused, we can fix the warnings and avoid the need for testing by just
> removing the function. :)
Heh, I hadn't noticed that! Awesome!
review approve
status approved