From 67e7f93d294ecd94114a2ae97232caf4452327f3 Mon Sep 17 00:00:00 2001 From: Klaus Schmidinger Date: Fri, 12 Aug 2011 14:37:25 +0200 Subject: Now scanning new transponders before old ones, to make sure transponder changes are recognized --- CONTRIBUTORS | 2 ++ HISTORY | 2 ++ eitscan.c | 4 ++-- nit.c | 17 +++++++++++++---- 4 files changed, 19 insertions(+), 6 deletions(-) diff --git a/CONTRIBUTORS b/CONTRIBUTORS index 301e1b1c..c8d777c9 100644 --- a/CONTRIBUTORS +++ b/CONTRIBUTORS @@ -1273,6 +1273,8 @@ Reinhard Nissl for avoiding an unecessary call to Recordings.ResetResume() for debugging a problem in handling the bitmap color depth for scaled subtitles for making subtitle PIDs be decrypted + for making cEITScanner process new transponders before old ones, to make sure + transponder changes are recognized Richard Robson for reporting freezing replay if a timer starts while in Transfer Mode from the diff --git a/HISTORY b/HISTORY index 117f2a2b..65f08832 100644 --- a/HISTORY +++ b/HISTORY @@ -6676,3 +6676,5 @@ Video Disk Recorder Revision History - Fixed handling the channelID in cMenuEditChanItem (thanks to Udo Richter). - cStringList::Sort() can now be called with a boolean parameter that controls case insensitive sorting (suggested by Sundararaj Reel). +- Now scanning new transponders before old ones, to make sure transponder changes + are recognized (thanks to Reinhard Nissl). diff --git a/eitscan.c b/eitscan.c index 2b43e718..25b7115c 100644 --- a/eitscan.c +++ b/eitscan.c @@ -4,7 +4,7 @@ * See the main source file 'vdr.c' for copyright information and * how to reach the author. * - * $Id: eitscan.c 2.1 2010/02/07 12:12:05 kls Exp $ + * $Id: eitscan.c 2.2 2011/08/12 14:18:04 kls Exp $ */ #include "eitscan.h" @@ -133,12 +133,12 @@ void cEITScanner::Process(void) if (Channels.Lock(false, 10)) { if (!scanList) { scanList = new cScanList; - scanList->AddTransponders(&Channels); if (transponderList) { scanList->AddTransponders(transponderList); delete transponderList; transponderList = NULL; } + scanList->AddTransponders(&Channels); } bool AnyDeviceSwitched = false; for (int i = 0; i < cDevice::NumDevices(); i++) { diff --git a/nit.c b/nit.c index dfd02076..0603e249 100644 --- a/nit.c +++ b/nit.c @@ -4,7 +4,7 @@ * See the main source file 'vdr.c' for copyright information and * how to reach the author. * - * $Id: nit.c 2.5 2010/02/16 15:37:05 kls Exp $ + * $Id: nit.c 2.6 2011/08/12 14:27:31 kls Exp $ */ #include "nit.h" @@ -150,6 +150,7 @@ void cNitFilter::Process(u_short Pid, u_char Tid, const u_char *Data, int Length } if (Setup.UpdateChannels >= 5) { bool found = false; + bool forceTransponderUpdate = false; for (cChannel *Channel = Channels.First(); Channel; Channel = Channels.Next(Channel)) { if (!Channel->GroupSep() && Channel->Source() == Source && Channel->Nid() == ts.getOriginalNetworkId() && Channel->Tid() == ts.getTransportStreamId()) { int transponder = Channel->Transponder(); @@ -164,9 +165,11 @@ void cNitFilter::Process(u_short Pid, u_char Tid, const u_char *Data, int Length } if (ISTRANSPONDER(cChannel::Transponder(Frequency, dtp.Polarization()), Transponder())) // only modify channels if we're actually receiving this transponder Channel->SetTransponderData(Source, Frequency, SymbolRate, dtp.ToString('S')); + else if (Channel->Srate() != SymbolRate || strcmp(Channel->Parameters(), dtp.ToString('S'))) + forceTransponderUpdate = true; // get us receiving this transponder } } - if (!found) { + if (!found || forceTransponderUpdate) { for (int n = 0; n < NumFrequencies; n++) { cChannel *Channel = new cChannel; Channel->SetId(ts.getOriginalNetworkId(), ts.getTransportStreamId(), 0, 0); @@ -202,6 +205,7 @@ void cNitFilter::Process(u_short Pid, u_char Tid, const u_char *Data, int Length } if (Setup.UpdateChannels >= 5) { bool found = false; + bool forceTransponderUpdate = false; for (cChannel *Channel = Channels.First(); Channel; Channel = Channels.Next(Channel)) { if (!Channel->GroupSep() && Channel->Source() == Source && Channel->Nid() == ts.getOriginalNetworkId() && Channel->Tid() == ts.getTransportStreamId()) { int transponder = Channel->Transponder(); @@ -216,9 +220,11 @@ void cNitFilter::Process(u_short Pid, u_char Tid, const u_char *Data, int Length } if (ISTRANSPONDER(Frequency / 1000, Transponder())) // only modify channels if we're actually receiving this transponder Channel->SetTransponderData(Source, Frequency, SymbolRate, dtp.ToString('C')); + else if (Channel->Srate() != SymbolRate || strcmp(Channel->Parameters(), dtp.ToString('C'))) + forceTransponderUpdate = true; // get us receiving this transponder } } - if (!found) { + if (!found || forceTransponderUpdate) { for (int n = 0; n < NumFrequencies; n++) { cChannel *Channel = new cChannel; Channel->SetId(ts.getOriginalNetworkId(), ts.getTransportStreamId(), 0, 0); @@ -261,6 +267,7 @@ void cNitFilter::Process(u_short Pid, u_char Tid, const u_char *Data, int Length } if (Setup.UpdateChannels >= 5) { bool found = false; + bool forceTransponderUpdate = false; for (cChannel *Channel = Channels.First(); Channel; Channel = Channels.Next(Channel)) { if (!Channel->GroupSep() && Channel->Source() == Source && Channel->Nid() == ts.getOriginalNetworkId() && Channel->Tid() == ts.getTransportStreamId()) { int transponder = Channel->Transponder(); @@ -275,9 +282,11 @@ void cNitFilter::Process(u_short Pid, u_char Tid, const u_char *Data, int Length } if (ISTRANSPONDER(Frequency / 1000000, Transponder())) // only modify channels if we're actually receiving this transponder Channel->SetTransponderData(Source, Frequency, 0, dtp.ToString('T')); + else if (strcmp(Channel->Parameters(), dtp.ToString('T'))) + forceTransponderUpdate = true; // get us receiving this transponder } } - if (!found) { + if (!found || forceTransponderUpdate) { for (int n = 0; n < NumFrequencies; n++) { cChannel *Channel = new cChannel; Channel->SetId(ts.getOriginalNetworkId(), ts.getTransportStreamId(), 0, 0); -- cgit v1.2.3