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

gles: Return the version as driver_info #5753

Merged
merged 2 commits into from
May 29, 2024

Conversation

valaphee
Copy link
Contributor

@valaphee valaphee commented May 29, 2024

Connections

Description
While working on ruffle I noticed that on Chromium with WebGL

Adapter Driver Name: "WebGL 2.0 (OpenGL ES"
Adapter Driver Info: "3.0 Chromium)"

is split incorrectly, and theoretically there shouldn't be a driver name. It was more of a workaround to check if the adapter is OpenGL or OpenGL ES in tests (which I never used though).

This PR simplifies that by just returning the version in the driver info.

Don't know if its worth back porting.

Testing

Checklist

  • Run cargo fmt.
  • Run cargo clippy. If applicable, add:
    • --target wasm32-unknown-unknown
    • --target wasm32-unknown-emscripten
  • Run cargo xtask test to run tests.
  • Add change to CHANGELOG.md. See simple instructions inside file.

@valaphee valaphee changed the title gles: Just return the version as driver_info gles: Return the version as driver_info May 29, 2024
@valaphee valaphee marked this pull request as ready for review May 29, 2024 10:49
@valaphee valaphee requested a review from a team as a code owner May 29, 2024 10:49
@cwfitzgerald cwfitzgerald merged commit 23307e1 into gfx-rs:trunk May 29, 2024
25 checks passed
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.

2 participants