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

Don't try to generate a thumb for unresolved plugin path #17352

Merged
merged 1 commit into from
Feb 13, 2020

Conversation

rmrector
Copy link
Contributor

@rmrector rmrector commented Feb 9, 2020

Description

Followup to #17202, giving plugins a loader in CFileFactory means they make it to the thumbloader now, but it still can't process them.

Motivation and Context

Fixes a Kodi crash when a generated thumbnail is requested for a plugin path.

How Has This Been Tested?

Quick runtime testing.

@rmrector rmrector added Type: Fix non-breaking change which fixes an issue Component: Video v19 Matrix labels Feb 9, 2020
@enen92 enen92 added this to the Matrix 19.0-alpha 1 milestone Feb 9, 2020
@arnova
Copy link
Member

arnova commented Feb 9, 2020

LGTM

@rmrector rmrector merged commit 5134a95 into xbmc:master Feb 13, 2020
Maven85 pushed a commit to Maven85/kodi that referenced this pull request Feb 17, 2020
Don't try to generate a thumb for unresolved plugin path
Maven85 pushed a commit to Maven85/kodi that referenced this pull request Feb 19, 2020
Don't try to generate a thumb for unresolved plugin path
Maven85 pushed a commit to Maven85/kodi that referenced this pull request Feb 26, 2020
Don't try to generate a thumb for unresolved plugin path
Maven85 pushed a commit to Maven85/kodi that referenced this pull request May 5, 2020
Don't try to generate a thumb for unresolved plugin path
Maven85 pushed a commit to Maven85/kodi that referenced this pull request Aug 4, 2020
Don't try to generate a thumb for unresolved plugin path
Maven85 pushed a commit to Maven85/kodi that referenced this pull request Aug 5, 2020
Don't try to generate a thumb for unresolved plugin path
Maven85 pushed a commit to Maven85/kodi that referenced this pull request Aug 6, 2020
Don't try to generate a thumb for unresolved plugin path
Maven85 pushed a commit to Maven85/kodi that referenced this pull request Aug 7, 2020
Don't try to generate a thumb for unresolved plugin path
Maven85 pushed a commit to Maven85/kodi that referenced this pull request Aug 7, 2020
Don't try to generate a thumb for unresolved plugin path
@rmrector rmrector deleted the no-extract-plugin-thumb branch July 17, 2024 22:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Video Type: Fix non-breaking change which fixes an issue v19 Matrix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants