Skip to content

Bump MSRV of wgpu-hal and wgpu-core, then use error_in_core. #7232

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

Closed
wants to merge 2 commits into from

Conversation

kpreid
Copy link
Collaborator

@kpreid kpreid commented Feb 26, 2025

Connections
Part of #6826.

Description
Rplaces uses of std::error::Error with core::error::Error.

Also, with the MSRV bump, Clippy recommends using C-string literals, so I did that too.

Squash or Rebase?
Squash if you want zero warnings at any point; rebase otherwise.

Checklist

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

@kpreid kpreid requested a review from a team as a code owner February 26, 2025 23:27
This allows `wgpu-hal` to be used in `no_std` environments, except that
currently, only the `noop` backend supports `no_std`.
In the future, `cfg(all(gles, webgl))` should also be `no_std`, but that
requires further work.
@ErichDonGubler
Copy link
Member

This PR is almost a strict subset of #7043. I've rolled these changes into my latest push to that PR, with Co-Authored-By for @kpreid as appropriate. Closing here.

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