On 20.05.2012 13:55, Matti Lehtimäki wrote:
On 05/20/2012 02:45 PM, Klaus Schmidinger wrote:
On 20.05.2012 13:41, Matti Lehtimäki wrote:
On 05/20/2012 02:15 PM, Klaus Schmidinger wrote:
One more thing comes to mind: you also need to make sure that the 'day' is set to the day of the EPG event. Just in case it is past midnight already and the event started "yesterday" ;-)
Indeed that was missing. I think setting the day is relevant only if VPS is used since in otherwise the start time is the current time not the start time of the event. Or should the start time be set to that of the event if VPS is not used?
Well, that's a matter of taste. For VPS it is relevant, otherwise it's up to you.
Attached is a new version of the patch with correct day always set when VPS is used. Could this patch be included in next version of VDR?
While adopting this patch I found that we need to set the timer's 'event' pointer to the actual event here, and that the start time should always be set to the event's start time (even if VPS is not used) in order to keep the event from getting reassigned to a different one later on.
I have attached a revised version of your patch (against VDR 1.7.30). Please take a look and let me know whether it is still OK for you.
Klaus