-
-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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
[addons] change binary callbacks #12596
Conversation
return nullptr; | ||
} | ||
|
||
return strdup(CServiceBroker::GetSettings().GetString(CSettings::SETTING_LOOKANDFEEL_SKIN).c_str()); |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
d590594
to
c1b895b
Compare
jenkins build this please |
c1b895b
to
f2b6002
Compare
jenkins build this please |
This add a callback who also present on python and becomes needed on e.g. Web browser addon to know used skin name. The change is more general and included to a new request.
This change the id usage of strings on new binary addons to override a kodi string id with a own for him itself.
f2b6002
to
8889c7d
Compare
You know that I always get puzzled about this version number stuff, but imo this change breaks all old style binary add-ons, like pvr add-ons. @AlwinEsch @MilhouseVH not sure whether I'm actually right, be in case I am, you are informed... |
Okay, you did not modify the function tables, right? Thus, all should be fine, but I would be happy if you could confirm this. |
PVR still use the old and not affected by them. Also is with a add of callback function (if added to the end) still backward compatible. |
Sure, thanks. |
Description
This add a callback who also present on python and becomes needed
on e.g. Web browser addon to know used skin name.
The change is more general and included to a new request.
This change the id usage of strings on new binary addons to override
a kodi string id with a own for him itself.
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of change
Checklist: