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.

Sign up for GitHub

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

Jump to bottom

refactor CAddonMgr getters #8919

Merged
merged 1 commit into from
Jan 20, 2016
Merged

refactor CAddonMgr getters #8919

merged 1 commit into from
Jan 20, 2016

Conversation

tamland
Copy link
Member

@tamland tamland commented Jan 19, 2016

die arbitrary boolean parameters!

@tamland
Copy link
Member Author

tamland commented Jan 19, 2016

jenkins build this please

Montellese
Montellese reviewed Jan 19, 2016
View reviewed changes
xbmc/addons/AddonManager.h
bool HasAddons(const TYPE &type);
bool HasInstalledAddons(const TYPE &type);

/*! Returns all installed an enabled addons. */

This comment was marked as spam.

Sign in to view

@tamland
Copy link
Member Author

tamland commented Jan 20, 2016

fixed, thanks;)
jenkins build this please

da-anda
da-anda reviewed Jan 20, 2016
View reviewed changes
xbmc/addons/AddonManager.cpp
return GetAddonsInternal(ADDON_UNKNOWN, addons, false);
}

bool CAddonMgr::GetInstalledAddons(VECADDONS& addons, const TYPE& type)

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

@Paxxi
Copy link
Member

Paxxi commented Jan 20, 2016

+10 die boolean parameters die!

tamland added a commit that referenced this pull request Jan 20, 2016
@tamland
Merge pull request #8919 from tamland/addonmgr_getters
1031b10 
refactor CAddonMgr getters
@tamland tamland merged commit 1031b10 into xbmc:master Jan 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Krypton 17.0-alpha1
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@tamland @Paxxi @Montellese @da-anda @MartijnKaijser