Rollup merge of #129907 - saethlin:solid-io-error, r=WaffleLapkin · patricklam/verify-rust-std@9b3c3fe (original) (raw)
Navigation Menu
- GitHub Copilot Write better code with AI
- GitHub Models New Manage and compare prompts
- GitHub Advanced Security Find and fix vulnerabilities
- Actions Automate any workflow
- Codespaces Instant dev environments
- Issues Plan and track work
- Code Review Manage code changes
- Discussions Collaborate outside of code
- Code Search Find more, search less
- Explore
- Pricing
Provide feedback
Saved searches
Use saved searches to filter your results more quickly
Appearance settings
Commit 9b3c3fe
Rollup merge of rust-lang#129907 - saethlin:solid-io-error, r=WaffleLapkin
Fix compile error in solid's remove_dir_all Before this PR, `x check library/std --target=aarch64-kmc-solid_asp3` will fail with: ``` error[E0382]: use of partially moved value: `result` --> std/src/sys/pal/solid/fs.rs:544:20 | 541 | if let Err(err) = result | --- value partially moved here ... 544 | return result; | ^^^^^^ value used here after partial move | = note: partial move occurs because value has type `io::error::Error`, which does not implement the `Copy` trait help: borrow this binding in the pattern to avoid moving the value | 541 | if let Err(ref err) = result | +++ ``` cc `@kawadakk` I think this will clear up https://solid-rs.github.io/toolstate/ :)
File tree
1 file changed
lines changed
1 file changed
lines changed
Lines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -538,7 +538,7 @@ pub fn remove_dir_all(path: &Path) -> io::Result<()> { | ||
538 | 538 | } |
539 | 539 | }; |
540 | 540 | // ignore internal NotFound errors |
541 | -if let Err(err) = result | |
541 | +if let Err(err) = &result | |
542 | 542 | && err.kind() != io::ErrorKind::NotFound |
543 | 543 | { |
544 | 544 | return result; |