Merge lp:~dandrader/geis/lp939679 into lp:geis
Proposed by
Daniel d'Andrada
Status: | Merged |
---|---|
Approved by: | Daniel d'Andrada |
Approved revision: | 207 |
Merged at revision: | 208 |
Proposed branch: | lp:~dandrader/geis/lp939679 |
Merge into: | lp:geis |
Diff against target: |
183 lines (+81/-51) 1 file modified
libutouch-geis/backend/grail/geis_grail_backend.c (+81/-51) |
To merge this branch: | bzr merge lp:~dandrader/geis/lp939679 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Stephen M. Webb (community) | Approve | ||
Review via email: mp+94525@code.launchpad.net |
Description of the change
grail_backend: Properly activate a subscription that has no filters.
Fixes bug https:/
To post a comment you must log in.
In the non-filter debug statement (on or around line 432), could you please make the fact that there is no filter on the subscription explicit? This is important debug information and just leaving it out makes it harder to spot during bug analysis without reading the code.
Other than that, this is good.