-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Fix the Android OpenGL About tab. #60
Fix the Android OpenGL About tab. #60
Conversation
@lioncash care to review? |
Will review it when I get home. Phone is being weird and I can't view
|
@Sonicadvance1 Does logcat say anything about context creation failing on rotation? Not saying this code causes it, just being extra finicky. If it doesn't, then it looks fine to me. |
It does complain, this probably does cause it. May have to look in to it. |
@@ -262,6 +268,7 @@ private boolean detect() | |||
private void create(int width, int height, int renderableType) | |||
{ | |||
int[] attribs = { | |||
EGL10.EGL_RENDERABLE_TYPE, renderableType, |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
Looks good to me. |
Move EGLHelper to be local to the creation of the about GL/GLES tabs so we don't have 3 EGL contexts running at a time. Fix issues with OpenGL context creation here so we show the correct information. This requires adding an EGL function to the NativeLibrary since Android's JAVA bindings don't expose eglBindAPI.
Fix the Android OpenGL About tab.
Replace Backslash with forward-slash
brings us up to 22642cc0c360af9b10c53592249927f04ebb6832 (for the most part) still needs playback fixed, adapter polling rate, and good config defaults
Move EGLHelper to be local to the creation of the about GL/GLES tabs so we don't have 3 EGL contexts running at a time.
Fix issues with OpenGL context creation here so we show the correct information.
This requires adding an EGL function to the NativeLibrary since Android's JAVA bindings don't expose eglBindAPI.