Merge lp:~ted/indicator-messages/seen-applications-list into lp:indicator-messages/0.5
Proposed by
Ted Gould
Status: | Merged |
---|---|
Merged at revision: | not available |
Proposed branch: | lp:~ted/indicator-messages/seen-applications-list |
Merge into: | lp:indicator-messages/0.5 |
Diff against target: |
356 lines (+256/-12) 6 files modified
src/Makefile.am (+2/-0) src/app-menu-item.c (+3/-0) src/launcher-menu-item.c (+40/-12) src/messages-service.c (+3/-0) src/seen-db.c (+177/-0) src/seen-db.h (+31/-0) |
To merge this branch: | bzr merge lp:~ted/indicator-messages/seen-applications-list |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Cody Russell (community) | Approve | ||
Review via email: mp+22100@code.launchpad.net |
Description of the change
Watches for when applications start to know whether they've been seen by
the messaging menu before. If they haven't, and they're a default
application. We replace the default name with "Set Up <blah>..."
instead of just the name of it.
To post a comment you must log in.
180 + if (seendb != NULL) {
181 + return;
182 + }
I kind of wouldn't mind seeing that change to a g_return_if_fail(), because it seems like if you are hitting this multiple times there's probably something screwy going on right?
Approve.