dbus-cpp

Merge lp:~thomas-voss/dbus-cpp/add-sender-to-match-rule-for-signal-delivery-filtering into lp:dbus-cpp

  1. add-sender-to-match-rule-for-signal-delivery-filtering
  2. Merge into trunk
Proposed by Thomas Voß
Status: Merged
Approved by: Antti Kaijanmäki
Approved revision: 28
Merged at revision: 28
Proposed branch: lp:~thomas-voss/dbus-cpp/add-sender-to-match-rule-for-signal-delivery-filtering
Merge into: lp:dbus-cpp
Prerequisite: lp:~thomas-voss/dbus-cpp/add-test-for-two-different-signals-on-same-object
Diff against target: 12 lines (+1/-1)
1 file modified
src/core/dbus/service.cpp (+1/-1)
To merge this branch: bzr merge lp:~thomas-voss/dbus-cpp/add-sender-to-match-rule-for-signal-delivery-filtering
Reviewer Review Type Date Requested Status
Antti Kaijanmäki (community) Approve
PS Jenkins bot continuous-integration Needs Fixing
Review via email: mp+202608@code.launchpad.net

Commit message

Add sender to match rule for event filtering.

Description of the change

Add sender to match rule for event filtering.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:28
http://jenkins.qa.ubuntu.com/job/dbus-cpp-ci/64/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/dbus-cpp-trusty-amd64-ci/44/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/dbus-cpp-trusty-armhf-ci/43
        deb: http://jenkins.qa.ubuntu.com/job/dbus-cpp-trusty-armhf-ci/43/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/dbus-cpp-trusty-i386-ci/43

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/dbus-cpp-ci/64/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Antti Kaijanmäki (kaijanmaki) wrote :

LGTM.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/core/dbus/service.cpp'
2--- src/core/dbus/service.cpp 2013-11-27 18:57:42 +0000
3+++ src/core/dbus/service.cpp 2014-01-22 07:46:55 +0000
4@@ -74,7 +74,7 @@
5
6 void Service::add_match(const MatchRule& rule)
7 {
8- connection->add_match(rule);
9+ connection->add_match(rule.sender(name));
10 }
11
12 void Service::remove_match(const MatchRule& rule)

Subscribers

People subscribed via source and target branches