From c9a27b09af4c3a10efb255719e8eceb79994c8fd Mon Sep 17 00:00:00 2001 From: Klaus Schmidinger Date: Sun, 30 Aug 2009 11:32:39 +0200 Subject: [PATCH] Fixed not logging changes for channels that have no number --- CONTRIBUTORS | 4 ++++ HISTORY | 2 ++ channels.c | 13 ++++++++----- 3 files changed, 14 insertions(+), 5 deletions(-) diff --git a/CONTRIBUTORS b/CONTRIBUTORS index 54f27bd3..c38bdf36 100644 --- a/CONTRIBUTORS +++ b/CONTRIBUTORS @@ -2490,3 +2490,7 @@ Martin Neuditschko Mikko Tuumanen for implementing full handling of subtitling descriptors + +Timothy D. Lenz + for reporting a problem with logging changes for channels that + have no number diff --git a/HISTORY b/HISTORY index 83b2498e..da59ac3c 100644 --- a/HISTORY +++ b/HISTORY @@ -6162,3 +6162,5 @@ Video Disk Recorder Revision History - Updated the Italian OSD texts (thanks to Diego Pierotto). - Fixed wrong bracketing in cChannel::SubtitlingType() etc. (thanks to Rolf Ahrenberg). +- Fixed not logging changes for channels that have no number + (reportted by Timothy D. Lenz). diff --git a/channels.c b/channels.c index 590271d6..cca0eee0 100644 --- a/channels.c +++ b/channels.c @@ -4,7 +4,7 @@ * See the main source file 'vdr.c' for copyright information and * how to reach the author. * - * $Id: channels.c 2.7 2009/08/16 15:08:49 kls Exp $ + * $Id: channels.c 2.8 2009/08/30 11:25:50 kls Exp $ */ #include "channels.h" @@ -508,7 +508,8 @@ void cChannel::SetPids(int Vpid, int Ppid, int Vtype, int *Apids, char ALangs[][ q = NewSpidsBuf; q += IntArrayToString(q, Spids, 10, SLangs); *q = 0; - dsyslog("changing pids of channel %d from %d+%d=%d:%s:%s:%d to %d+%d=%d:%s:%s:%d", Number(), vpid, ppid, vtype, OldApidsBuf, OldSpidsBuf, tpid, Vpid, Ppid, Vtype, NewApidsBuf, NewSpidsBuf, Tpid); + if (Number()) + dsyslog("changing pids of channel %d from %d+%d=%d:%s:%s:%d to %d+%d=%d:%s:%s:%d", Number(), vpid, ppid, vtype, OldApidsBuf, OldSpidsBuf, tpid, Vpid, Ppid, Vtype, NewApidsBuf, NewSpidsBuf, Tpid); vpid = Vpid; ppid = Ppid; vtype = Vtype; @@ -558,7 +559,8 @@ void cChannel::SetCaIds(const int *CaIds) char NewCaIdsBuf[MAXCAIDS * 5 + 10]; IntArrayToString(OldCaIdsBuf, caids, 16); IntArrayToString(NewCaIdsBuf, CaIds, 16); - dsyslog("changing caids of channel %d from %s to %s", Number(), OldCaIdsBuf, NewCaIdsBuf); + if (Number()) + dsyslog("changing caids of channel %d from %s to %s", Number(), OldCaIdsBuf, NewCaIdsBuf); for (int i = 0; i <= MAXCAIDS; i++) { // <= to copy the terminating 0 caids[i] = CaIds[i]; if (!CaIds[i]) @@ -574,7 +576,7 @@ void cChannel::SetCaDescriptors(int Level) if (Level > 0) { modification |= CHANNELMOD_CA; Channels.SetModified(); - if (Level > 1) + if (Number() && Level > 1) dsyslog("changing ca descriptors of channel %d", Number()); } } @@ -622,7 +624,8 @@ void cChannel::SetLinkChannels(cLinkChannels *LinkChannels) } else q += sprintf(q, " none"); - dsyslog(buffer); + if (Number()) + dsyslog(buffer); } void cChannel::SetRefChannel(cChannel *RefChannel)