Opened 6 years ago

Last modified 2 months ago

#10102 new Patch - Feature

EIT listener changes

Reported by: David Matthews <dm@…> Owned by: stuarta
Priority: minor Milestone: 29.1
Component: MythTV - EIT Version: Master Head
Severity: low Keywords:
Cc: Ticket locked: no


This patch reworks the code that sets up listeners for EIT. While testing the EITpf timing code I found a problem when recording two programmes on the same multiplex and tracked it down to the way listeners are set up. This patch is almost completely independent of the EITpf timing patch (#10101) apart from one place in DVBStreamData::UpdateEITListeners where EIT listening is enabled if _dvb_eit_listeners.size() is non-zero.

The changes move responsibility for adding and removing listeners down from StreamHandler? to the derived classes of MPEGStreamData. I have tidied up the code for DVBStreamData but I've kept the changes to ATSCStreamData to a minimum because I couldn't test them.

The problem with the existing code was primarily that StreamHandler::UpdateListeningForEIT used the _eit_pids variable in StreamHandler? and updated it within the loop, using the new value for the next stream. That meant that EIT filtering could be on or off depending on the order of the streams in _stream_data_list. The fact that passive EIT scanning could be turned off during multirec recordings probably wouldn't make much difference since it is all statistical anyway.


Attachments (1)

eit-multirec.patch (15.8 KB) - added by David Matthews <dm@…> 6 years ago.

Download all attachments as: .zip

Change History (7)

Changed 6 years ago by David Matthews <dm@…>

comment:1 Changed 6 years ago by stuarta

  • Milestone changed from unknown to 0.26
  • Version changed from Unspecified to Master Head

comment:2 Changed 5 years ago by kenni

  • Milestone changed from 0.26 to 0.27

comment:3 Changed 5 years ago by paulh

  • Milestone changed from 0.27 to 0.28

comment:4 Changed 22 months ago by stuarta

  • Milestone changed from 0.28 to 0.29

comment:5 Changed 22 months ago by stuarta

  • Milestone changed from 0.29 to 29.0

Milestone renamed

comment:6 Changed 2 months ago by stuarta

  • Milestone changed from 29.0 to 29.1

Add Comment

Modify Ticket

as new The owner will remain stuarta.

E-mail address and user name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.