Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement PVR Addon API changes for Leia #315

Merged
merged 5 commits into from
Jul 8, 2017
Merged

Conversation

ksooo
Copy link
Member

@ksooo ksooo commented Jun 15, 2017

Refer to xbmc/xbmc#12303 for more details.

@Jalle19 this is WIP, review welcome, but WIP wrt other changes that might follow. BTW, I took myself the freedom to cherry-pick from https://github.com/Jalle19/pvr.hts/tree/htsp-descramble-info ;-)

@ksooo
Copy link
Member Author

ksooo commented Jun 15, 2017

BTW, this fixes #106

@Leatherface75
Copy link

Big thx for implementing this.

@piotrasd
Copy link

piotrasd commented Jun 15, 2017

thx for last commit, any chance get information how many min left to end of program ?

screenshot109

that i got over tvheadend api but should be in htsp delivery - i think that basic information.

@ksooo
Copy link
Member Author

ksooo commented Jun 15, 2017

@piotrasd if you don't stop to spam here with unrelated stuff I will ban you. Use the forum or open an issue here for feature requests. BTW, this is not a pvr.hts issue. Ask the skinners in the Kodi forum for support, please.

Copy link
Contributor

@Jalle19 Jalle19 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, thanks for picking this up!

{
CLockObject lock(m_mutex);

info = {0};
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this necessary?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In pvr.hts, we everywhere zero the structs that are passed to the addon from outside, yes. Usually we use memset, but I prefer modern syntax nowadays. btw, memset would overwrite the vtable of the struct if it would have one.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, and main reason is, if a member gets added to a struct by an api change, this prevents that the initialization of the new member must be explicitly added to every addon. As long as the new member's default value is "0" (in memory represantation), this is very handy.

@ksooo ksooo force-pushed the pvr-api-leia branch 3 times, most recently from 8991f14 to 94df0d1 Compare June 20, 2017 18:05
@@ -1,7 +1,7 @@
<?xml version="1.0" encoding="UTF-8"?>
<addon
id="pvr.hts"
version="4.0.16"
version="4.0.17"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we should use 4.1.0 instead since this implements a new verison the PVR API?

@ksooo ksooo force-pushed the pvr-api-leia branch 5 times, most recently from 2d797de to 0b1a3fb Compare July 5, 2017 13:59
@ksooo ksooo merged commit 1653a3a into kodi-pvr:master Jul 8, 2017
@ksooo ksooo deleted the pvr-api-leia branch July 8, 2017 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants