dbus-cpp

Merge lp://staging/~mandel/dbus-cpp/name-acquired into lp://staging/dbus-cpp

Proposed by Manuel de la Peña on 2015-06-25

Status:	Merged
Approved by:	Alfonso Sanchez-Beato on 2015-06-25
Approved revision:	98
Merged at revision:	100
Proposed branch:	lp://staging/~mandel/dbus-cpp/name-acquired
Merge into:	lp://staging/dbus-cpp
Diff against target:	44 lines (+12/-10) 2 files modified src/core/dbus/message.cpp (+1/-1) src/core/dbus/service_watcher.cpp (+11/-9)
To merge this branch:	bzr merge lp://staging/~mandel/dbus-cpp/name-acquired
Related bugs:	Link a bug report

Reviewer	Review Type	Date Requested	Status
PS Jenkins bot	continuous-integration		Approve on 2015-06-25
Alfonso Sanchez-Beato		2015-06-25	Approve on 2015-06-25
Review via email: mp+262955@code.staging.launchpad.net

Ensure that the signals are correctly emitted for when the services appear and go.

Ensure that the signals are correctly emitted for when the services appear and go.

Revision history for this message

Alfonso Sanchez-Beato (alfonsosanchezbeato) wrote on 2015-06-25:

LGTM

review: Approve

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2015-06-25:

review: Approve (continuous-integration)

99. By Manuel de la Peña on 2015-07-16: No need to add that extra step in the push_boolean method.
100. By Manuel de la Peña on 2015-07-16: Fix tests.
101. By Manuel de la Peña on 2015-07-21: Auto does the right thing and allows to pass the correct value.

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk

The diff is not available at this time. You can reload the page or download it.

People subscribed via source and target branches

to all changes: