Re: Patch: Eliminate duplicate events when using xmltv2vdr

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 14.10.2011 16:25, Timo Eskola wrote:
Hi,

I started to use xmltv2vdr for some channels with poor EPG data. I do not want to use xmltv2vdr for all channels so I modified xmltv2vdr.pl <http://xmltv2vdr.pl> script to clear only the channels which will be grabbed with xmltv. The result was multiple EPG events for some programs. I found 2 issues
in VDR code that caused this.

1. CLRE for a channel does not always clear all events. I compared CLRE for clearing all channels and noticed that it also clears events in timers. The patch will add this to CLRE for a channel.

Adopted for version 1.7.23.

2. Second problem was that for some reason there are small differences in events times in EPG and xmltv. The patch find events close to the current event during EPG scan.

I don't see why you are adding 'Duration / 2' here.
The call to

  GetEventAround(StartTime + Around)

that you have introduced in your patch searches for the
event closest to the "middle" of the event in question.
Shouldn't you look for an event closest to the *start*
of the given event? That would obsolete the new 'Around'
parameter and the actual call could simply be

  GetEventAround(StartTime)


What I don't like about this whole thing is that it totally
defeats the purpose of the eventsHashStartTime hash table, which
has been introduced to make this lookup faster. I'm afraid I can't
accept this patch because of this.

Klaus

Now I have proper program data from EPG and xmltv.

http://www.tolleri.net/vdr/vdr/vdr-1.7.21-clre-epgscan.patch

Maybe Klaus can have a look if the changes can be implemented in VDR.

_______________________________________________
vdr mailing list
vdr@xxxxxxxxxxx
http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr


[Index of Archives]     [Linux Media]     [Asterisk]     [DCCP]     [Netdev]     [Xorg]     [Util Linux NG]     [Xfree86]     [Big List of Linux Books]     [Fedora Users]     [Fedora Women]     [ALSA Devel]     [Linux USB]

  Powered by Linux