-
-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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
VideoPlayer: rewrite yuv to rgb conversion for OpenGL #13428
Conversation
FernetMenta
commented
Jan 24, 2018
•
edited
Loading
edited
- rewrite of YUV - RGB conversion for OpenGL
- move renderflags into renderer
54d80b5
to
b91d586
Compare
@popcornmix please adapt omxplayer |
@afedchin you may want to rework Windows too. The old code is wrong, i.e. it does not consider transformations between colour primaries. Only HD videos had correct colours because bt.709 is identical with sRGB. |
jenkins build this please |
@@ -199,7 +199,7 @@ CDVDVideoCodec::VCReturn CAddonVideoCodec::GetPicture(VideoPicture* pVideoPictur | |||
pVideoPicture->pts = static_cast<double>(picture.pts); | |||
pVideoPicture->dts = DVD_NOPTS_VALUE; | |||
pVideoPicture->color_range = 0; | |||
pVideoPicture->color_matrix = 4; | |||
pVideoPicture->color_space = 4; |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
@FernetMenta which is main change between old YUVMatrix and new implementation? |
|
e54ce2d
to
8cfc0eb
Compare
[videoplayer] Fix vdpau crash after #13428.
Could this be related? forum |
possible |
Yep: |