Skip to content

Many clippy errors in Rust 1.72 #7398

@alamb

Description

@alamb

Describe the bug

Rust 1.72 was just released and Clippy has gotten pickier: https://github.com/rust-lang/rust/blob/master/RELEASES.md#version-1720-2023-08-24

To Reproduce

update rust

Run

cargo clippy --all-targets --workspace --features avro,pyarrow -- -D warnings

You'll see errors like this:

https://github.com/alamb/arrow-datafusion/actions/runs/5965191830/job/16182337693
https://github.com/apache/arrow-datafusion/actions/runs/5964521711/job/16180238082

...
error: usage of `Iterator::fold` on a type that implements `Try`
  --> datafusion/expr/src/type_coercion/other.rs:31:10
   |
31 |           .fold(Some(expr_type.clone()), |left, right_type| match left {
   |  __________^
32 | |             None => None,
33 | |             Some(left_type) => comparison_coercion(&left_type, right_type),
34 | |         })
   | |__________^ help: use `try_fold` instead: `try_fold(expr_type.clone(), |left, right_type| ...)`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_try_fold
   = note: `-D clippy::manual-try-fold` implied by `-D warnings`
...

Expected behavior

Clean run

Additional context

No response

Metadata

Metadata

Assignees

Labels

bugSomething isn't working

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions