Skip to content

Put CVariant into own namespace#320

Merged
phunkyfish merged 3 commits intoxbmc:Piersfrom
David-EIPI:cvariant_collision
Apr 1, 2025
Merged

Put CVariant into own namespace#320
phunkyfish merged 3 commits intoxbmc:Piersfrom
David-EIPI:cvariant_collision

Conversation

@David-EIPI
Copy link
Copy Markdown
Contributor

Put CVariant into ffmpegdirect namespace to avoid collisions with CVariant in XBMC tree.
This fixes Kodi crashes as described in #319

Copy link
Copy Markdown
Collaborator

@phunkyfish phunkyfish left a comment

Choose a reason for hiding this comment

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

Just one minor. Otherwise looks good.

Would you like this to be released? If so you would need to add a commit that updates the addon.xml.in (increment the micro version as its fix) and add a changelog entry.

Comment thread src/stream/url/Variant.h Outdated
@David-EIPI
Copy link
Copy Markdown
Contributor Author

Yes, I think this is an important fix that needs to go to release. I did the requested edits.

@phunkyfish phunkyfish merged commit 973d8d5 into xbmc:Piers Apr 1, 2025
1 of 8 checks passed
@phunkyfish
Copy link
Copy Markdown
Collaborator

phunkyfish commented Apr 1, 2025

Released. Should appear on binary addon servers over the next while.

Thanks for the contribution!

@basilgello
Copy link
Copy Markdown
Contributor

@phunkyfish This also hits Omega. Debian's approaching full freeze now so I upgraded and was immediately struck. I will make a targeted bugfix release into Debian but I'd maake one more upstream release of 21.3.8 just in case.

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.

Kodi crashes when a PVR stream is stopped via remote app (unhandled bad_variant_access exception)

3 participants