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

Move minimum supported rust version (MSRV) from 1.65 back to 1.64. #3231

Merged
merged 3 commits into from
Nov 23, 2022

Conversation

jimblandy
Copy link
Member

Checklist

  • Run cargo clippy.
  • Run RUSTFLAGS=--cfg=web_sys_unstable_apis cargo clippy --target wasm32-unknown-unknown if applicable.
  • Add change to CHANGELOG.md. See simple instructions inside file.

Connections
Link to the issues addressed by this PR, or dependent PRs in other repositories

Description
Describe what problem this is solving, and how it's solved.

Testing
Explain how this change is tested.

The following error variants were large enough to trip Clippy's
`result_large_err` lint:

- RenderPassCompatibilityError::IncompatibleColorAttachment
- CreateShaderModuleError::Validation

Large error types are a problem, because they make the `Result` type
large, requiring the program to copy around large numbers of bytes
even in the success case. Since the failure case is rare, it's usually
not a problem to just heap-allocate the contents to keep the `Ok` case
small.
@codecov-commenter
Copy link

Codecov Report

Merging #3231 (4e8abe7) into master (04d12ba) will decrease coverage by 0.08%.
The diff coverage is 62.03%.

❗ Current head 4e8abe7 differs from pull request most recent head c7297cf. Consider uploading reports for the commit c7297cf to get more accurate results

@@            Coverage Diff             @@
##           master    #3231      +/-   ##
==========================================
- Coverage   64.70%   64.62%   -0.09%     
==========================================
  Files          81       81              
  Lines       38819    39479     +660     
==========================================
+ Hits        25118    25512     +394     
- Misses      13701    13967     +266     
Impacted Files Coverage Δ
player/src/lib.rs 53.31% <ø> (ø)
wgpu-core/src/id.rs 81.25% <0.00%> (-2.63%) ⬇️
wgpu-core/src/lib.rs 96.55% <ø> (ø)
wgpu-core/src/pipeline.rs 21.11% <ø> (ø)
wgpu-core/src/present.rs 0.00% <0.00%> (ø)
wgpu-core/src/resource.rs 25.13% <ø> (ø)
wgpu-hal/src/dx11/device.rs 0.00% <0.00%> (ø)
wgpu-hal/src/empty.rs 0.00% <ø> (ø)
wgpu-hal/src/lib.rs 26.21% <ø> (ø)
wgpu/src/backend/direct.rs 54.80% <0.00%> (-0.30%) ⬇️
... and 46 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

wgpu-hal/Cargo.toml Show resolved Hide resolved
@cwfitzgerald cwfitzgerald enabled auto-merge (squash) November 23, 2022 21:16
@cwfitzgerald cwfitzgerald merged commit 73b23f1 into gfx-rs:master Nov 23, 2022
@jimblandy jimblandy deleted the revert-to-rust-1.64 branch November 23, 2022 23:58
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.

3 participants