Hi all,
I notice that quite often EPG data contains events with null or empty titles. These tend to be 'gaps' between programs (especially on movie channels.) Would it be sensible to skip over these when showing the "Whats on Next?" screen?
Looking at the code, the place that needs changed is cSchedule::GetFollowingEvent in epg.c.
Apart from cMenuWhatsOn::cMenuWhatsOn in menu.c, the only other place place this is called from is cSchedule::Dump - would the following patch (attached and included below) cause any problems? I've been running it for a few days and it seems ok.
Or would it be better to make the change to cMenuWhatsOn::cMenuWhatsOn?
Chris
--- vdr-1.3.20/epg.c 2005-01-02 11:25:25.000000000 +0000 +++ vdr-1.3.20-modified/epg.c 2005-02-11 20:26:10.000000000 +0000 @@ -672,8 +672,14 @@ const cEvent *cSchedule::GetPresentEvent const cEvent *cSchedule::GetFollowingEvent(bool CheckRunningStatus) const { const cEvent *p = GetPresentEvent(CheckRunningStatus); - if (p) + if (p) { p = events.Next(p); + + const cEvent *tempp = p; + while(tempp && ((tempp->Title() == NULL) || (strlen(tempp->Title())==0))) tempp = events.Next(tempp); + if(tempp) + return tempp; + } return p; }