Hi, the implementation of the Premiere content descriptor in libsi is erroneous. The list of starttimes is a nested structure. The outer structure gives the day and the inner structure give the times for this day.
The attached patch fixes the code.
Regards.
Stefan Huelswitt wrote:
Hi, the implementation of the Premiere content descriptor in libsi is erroneous. The list of starttimes is a nested structure. The outer structure gives the day and the inner structure give the times for this day.
The attached patch fixes the code.
I'll postpone adopting this until an API change is actually unavoidable. Since VDR itself doesn't use these things yet, I prefer to keep the API version constant for now.
Thanks for the fix, though.
Klaus
On 23 May 2006 Klaus Schmidinger Klaus.Schmidinger@cadsoft.de wrote:
Stefan Huelswitt wrote:
The attached patch fixes the code.
I'll postpone adopting this until an API change is actually unavoidable. Since VDR itself doesn't use these things yet, I prefer to keep the API version constant for now.
I plan to release an updated premiereepg plugin soon, which uses this libsi code and thus depends on the fix.
But I can include the patch with the plugin as well.
Regards.
Stefan Huelswitt wrote:
On 23 May 2006 Klaus Schmidinger Klaus.Schmidinger@cadsoft.de wrote:
Stefan Huelswitt wrote:
The attached patch fixes the code.
I'll postpone adopting this until an API change is actually unavoidable. Since VDR itself doesn't use these things yet, I prefer to keep the API version constant for now.
I plan to release an updated premiereepg plugin soon, which uses this libsi code and thus depends on the fix.
But I can include the patch with the plugin as well.
Well, I don't _have_ to keep the API version constant. Just thought I'd save people from having to recompile all their plugins. If the general vote is to bump the API version and rather have this fix included, that's fine with me.
Let's hear it!
Klaus