emilk opened a new pull request, #10673: URL: https://github.com/apache/arrow-rs/pull/10673
# Which issue does this PR close? No issue in particular - Follow-up to #10558 - Follow-up to #10552 - Follow-up to #10551 - Follow-up to #10533 # Rationale for this change More lints hand-picked from [egui's `Cargo.toml`](https://github.com/emilk/egui/blob/main/Cargo.toml), to simplify the code, catch bugs, and write more efficient code. ## What changes are included in this PR? One commit per new lint (maybe easiest to review commit by commit!), except for ten lints with zero violations, which share one commit. Let me know if you disagree with any of them. I hope to add even more lints in later PRs. ## Are these changes tested? Covered by existing tests plus the clippy CI job. I also ran clippy with default features, not just `--all-features`: `needless_pass_by_ref_mut` fires on eleven more sites when a feature is off. ## Are there any user-facing changes? Two additive ones: `IntoIterator` is now implemented for references to `FixedSizeListArray`, `GenericListArray`, `GenericListViewArray`, `MapArray`, `BitChunks`, `UnalignedBitChunk` and `VariantArray`, and the `UnalignedBitChunkIterator` alias now names `Copied` instead of `Cloned`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
