aethercast

Merge lp://staging/~morphis/aethercast/small-fixes into lp://staging/aethercast

  1. small-fixes
  2. Merge into trunk
Proposed by Simon Fels
Status: Merged
Approved by: Alfonso Sanchez-Beato
Approved revision: 150
Merged at revision: 152
Proposed branch: lp://staging/~morphis/aethercast/small-fixes
Merge into: lp://staging/aethercast
Diff against target: 33 lines (+4/-5)
2 files modified
src/ac/service.cpp (+3/-3)
src/ac/sourceclient.cpp (+1/-2)
To merge this branch: bzr merge lp://staging/~morphis/aethercast/small-fixes
Reviewer Review Type Date Requested Status
Konrad Zapałowicz (community) code Approve
Matteo Croce (community) Approve
Alfonso Sanchez-Beato Approve
Review via email: mp+300044@code.staging.launchpad.net

Commit message

Fix RTSP debugging and device state update reporting

Due to previous search&replace operations the environment variable we are using to enable RTSP debugging was corrupted and is now correctly named AETHERCAST_RTSP_DEBUG. Furthermore we were only reporting device state updates for the current device but should really report them for every device regardless if its the current one or not.

To post a comment you must log in.
Revision history for this message
Alfonso Sanchez-Beato (alfonsosanchezbeato) wrote :

LGTM

review: Approve
Revision history for this message
Matteo Croce (teknoraver) :
review: Approve
lp://staging/~morphis/aethercast/small-fixes updated
151. By Simon Fels

Use commong ac::utils::IsEnvSet

152. By Simon Fels

Use correct namespace

Revision history for this message
Konrad Zapałowicz (kzapalowicz) wrote :

LGTM

review: Approve (code)

Preview Diff

[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.

Subscribers

People subscribed via source and target branches

to all changes: