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

RetroPlayer: Fix black screen when playing games #12507

Merged
merged 5 commits into from
Jul 18, 2017

Conversation

garbear
Copy link
Member

@garbear garbear commented Jul 15, 2017

This contains two fixes for two separate bugs causing a black screen during gameplay. It also contains some minor improvements that shouldn't affect playback.

The use of unique_ptr after VideoBuffers switched to shared_ptr caused an
exception to be thrown on every frame.
The RenderManager (or somewhere in the rendering pipeline) no longer checks
for DVD_NOPTS_VALUE, so passing a pts with this value caused RenderManager
to think the frame is far in the future.

By setting it to zero, RenderManager should never add any extra delays
waiting for pts to arrive.
@garbear garbear added Type: Fix non-breaking change which fixes an issue Component: Games v18 Leia labels Jul 15, 2017
@garbear garbear added this to the L 18.0-alpha1 milestone Jul 15, 2017
@garbear
Copy link
Member Author

garbear commented Jul 17, 2017

jenkins build this please

@garbear
Copy link
Member Author

garbear commented Jul 17, 2017

jenkins build and merge

@garbear
Copy link
Member Author

garbear commented Jul 18, 2017

Windows failed due to an IO error.

@garbear garbear merged commit ed251ff into xbmc:master Jul 18, 2017
@garbear garbear deleted the fix-retroplayer branch July 18, 2017 01:43
@thejims
Copy link

thejims commented Jul 21, 2017

After this was added whenever I launch games via IARL I only get a blank/black screen. Works fine in prior releases.
Crash logs from when this occurs:
http://sprunge.us/NTjX

@garbear
Copy link
Member Author

garbear commented Jul 21, 2017

Fixed by #12546

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Games Type: Fix non-breaking change which fixes an issue v18 Leia
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants