Skip to content

Commit 631faa1

Browse files
committed
Auto merge of #11146 - hydro-project:tuple-array-conversions, r=Centri3,xFrednet
[`tuple_array_conversions`]: move from `complexity` to `nursery` The lint suggestion is arguably often less readable and more complex than the original code. For example, which of the following is the most readable: ```rust let _vertices = edges.flat_map(|(src, dst)| [src, dst]); let _vertices = edges.flat_map(<_ as Into<[i32; 2]>>::into); let _vertices = edges.flat_map(<[i32; 2]>::from); ``` The lint can be useful, but really only applies if the tuple is either long enough that naming the fields is silly (maybe at least 4 entries long), or if the author intends the fields to be homogenous, which is author intent and can't be determined by the lint. Therefore I think the lint should be marked as pedantic. Currently, there are also a lot of false positives with the lint: * #11082 * #11085 * #11100 (#11105) * #11124 * #11144 Should fix those issues before enabling it for everyone. --- changelog: Move [`tuple_array_conversions`] to `nursery` (Now allow-by-default) <!-- FIY: Ignore this change, if the commit gets backported --> [#11146](#11146)
2 parents b10a0aa + 4102a30 commit 631faa1

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

clippy_lints/src/tuple_array_conversions.rs

+3-3
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,8 @@ declare_clippy_lint! {
1616
/// Checks for tuple<=>array conversions that are not done with `.into()`.
1717
///
1818
/// ### Why is this bad?
19-
/// It's unnecessary complexity. `.into()` works for tuples<=>arrays at or below 12 elements and
20-
/// conveys the intent a lot better, while also leaving less room for hard to spot bugs!
19+
/// It may be unnecessary complexity. `.into()` works for converting tuples
20+
/// <=> arrays of up to 12 elements and may convey intent more clearly.
2121
///
2222
/// ### Example
2323
/// ```rust,ignore
@@ -31,7 +31,7 @@ declare_clippy_lint! {
3131
/// ```
3232
#[clippy::version = "1.72.0"]
3333
pub TUPLE_ARRAY_CONVERSIONS,
34-
complexity,
34+
pedantic,
3535
"checks for tuple<=>array conversions that are not done with `.into()`"
3636
}
3737
impl_lint_pass!(TupleArrayConversions => [TUPLE_ARRAY_CONVERSIONS]);

0 commit comments

Comments
 (0)