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

Use the EGL API with Android #417

Merged
merged 1 commit into from
May 1, 2015
Merged

Use the EGL API with Android #417

merged 1 commit into from
May 1, 2015

Conversation

tomaka
Copy link
Contributor

@tomaka tomaka commented May 1, 2015

No description provided.

@ozkriff
Copy link
Contributor

ozkriff commented May 1, 2015

Color picker is ZoC is broken with this commit - https://github.com/ozkriff/zoc/blob/ea6370/src/visualizer/src/picker.rs#L130. Hmmm...

@tomaka
Copy link
Contributor Author

tomaka commented May 1, 2015

@ozkriff Ah, thanks for testing!

The new EGL module should be much cleaner than the old one. However what this changes is that the format of the color buffer, size of the depth buffer, etc. that are chosen by default by glutin are different than with the old implementation.

Maybe there are some rounding errors with your code? You could try adding with_pixel_format(24, 8) when creating the window?

@ozkriff
Copy link
Contributor

ozkriff commented May 1, 2015

yes, with_pixel_format(24, 8) solves the problem :)

@tomaka
Copy link
Contributor Author

tomaka commented May 1, 2015

Nice, let's merge then.

tomaka added a commit that referenced this pull request May 1, 2015
Use the EGL API with Android
@tomaka tomaka merged commit cb9a804 into master May 1, 2015
@tomaka tomaka deleted the android-egl branch May 1, 2015 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants