libsi Premiere content descriptor erroneous - vdr - linuxtv.org

List overview All Threads
Download

libsi Premiere content descriptor erroneous

Recordings order

Use VDR like a sounddevice?

s.huelswitt＠gmx.de

22 May 2006 22 May '06

4:11 p.m.

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 s.huelswitt@gmx.de | http://www.muempf.de/

Attachments:

vdr-1.4.0-libsi.diff (text/plain — 4.9 KB)

Reply

Show replies by date

Klaus Schmidinger

23 May 23 May

6:02 p.m.

Stefan Huelswitt wrote:

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

Reply

s.huelswitt＠gmx.de

8:03 p.m.

On 23 May 2006 Klaus Schmidinger Klaus.Schmidinger@cadsoft.de wrote:

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 s.huelswitt@gmx.de | http://www.muempf.de/

Reply

Klaus Schmidinger

10:24 p.m.

Stefan Huelswitt wrote:

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

Reply

6916

Age (days ago)

6917

Last active (days ago)

vdr@linuxtv.org

3 comments

2 participants

tags (0)

participants (2)

Klaus Schmidinger
s.huelswitt＠gmx.de