Opened 13 years ago

Closed 13 years ago

#2064 closed enhancement (fixed)

Share the same instance of RtspComms in Freebox(Recorder|SignalMonitor)

Reported by: mythtv@… Owned by: danielk
Priority: minor Milestone: 0.20
Component: mythtv Version: head
Severity: low Keywords:
Cc: Ticket locked: no

Description

The signal monitor added in [10434] opens a second rtsp stream. Since then, I can watch the first "tuned" channel, but can't change to another one. My problem is that I do not have enough dsl bandwidth to open two concurent streams. The following patch does the following :

  • fix my bandwidth problem by moving the RTSPComms to the FreeboxChannel?, and reuse the same instance for the Recorder and the SignalMonitor?.
  • fix the two "TODO free resources" in rtspcomms.cpp
  • fix a compilation warning in freeboxmediasink.h (similar to #1992)

Attachments (1)

patch-shared-rtsp.diff (12.1 KB) - added by mythtv@… 13 years ago.
The patch

Download all attachments as: .zip

Change History (5)

Changed 13 years ago by mythtv@…

Attachment: patch-shared-rtsp.diff added

The patch

comment:1 Changed 13 years ago by danielk

Milestone: 0.20
Severity: mediumlow
Type: patchenhancement

comment:2 Changed 13 years ago by danielk

(In [10594]) Refs #2064. Frees Live555 environment in RTSPComms when needed (fixes TODOs).

comment:3 Changed 13 years ago by danielk

(In [10595]) Refs #2064. Compiler warning fix.

comment:4 Changed 13 years ago by danielk

Resolution: fixed
Status: newclosed

(In [10596]) Fixes #2064. Refs #1701. Lowers bandwidth requirements for RTSP recording startup by sharing one session for both signal monitoring and recording.

Note: See TracTickets for help on using tickets.