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

renderer: Use a structure for information passed from renderer to codec #7087

Merged
merged 1 commit into from
May 9, 2015

Conversation

popcornmix
Copy link
Member

I need an additional opaque pointer to be passed from renderer to codec
(to allow a pool owned by renderer to be used by decoder).

It was decided rather than adding an extra function/parameter that
entending the information to use a structure would be more maintainable.

@popcornmix
Copy link
Member Author

This is required by a forthcoming MMAL bug fix, so I'd like this to be considered for Isenguard.

jenkins build this please

I need an additional opaque pointer to be passed from renderer to codec
(to allow a pool owned by renderer to be used by decoder).

It was decided rather than adding an extra function/parameter that
entending the information to use a structure would be more maintainable.
@popcornmix
Copy link
Member Author

Updated.
jenkins build this please

@popcornmix
Copy link
Member Author

Windows failure doesn't look to be related:
./buildffmpeg.sh: line 101: 2260 Segmentation fault (core dumped) make $MAKEFLAGS
@FernetMenta @fritsch is this okay?

@FernetMenta
Copy link
Contributor

looks ok, would be great if this could be runtime tested on some platforms.

@popcornmix
Copy link
Member Author

Tested a linux build with and without hardware acceleration and videos played fine.

@FernetMenta
Copy link
Contributor

ok, merge at you will

popcornmix added a commit that referenced this pull request May 9, 2015
renderer: Use a structure for information passed from renderer to codec
@popcornmix popcornmix merged commit 72da898 into xbmc:master May 9, 2015
@popcornmix popcornmix deleted the renderer_decoder branch May 9, 2015 12:06
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

3 participants