partly disable unneeded_wildcard_pattern when rest_pattern_accessible_field is enabled

Author: Erk-

clippy_lints/src/misc_early/unneeded_wildcard_pattern.rs

@@ -1,7 +1,7 @@
 use clippy_utils::diagnostics::span_lint_and_sugg;
 use rustc_ast::ast::{Pat, PatFieldsRest, PatKind};
 use rustc_errors::Applicability;
-use rustc_lint::EarlyContext;
+use rustc_lint::{EarlyContext, LintContext as _};
 use rustc_span::Span;
 
 use super::UNNEEDED_WILDCARD_PATTERN;
@@ -40,6 +40,8 @@ pub(super) fn check(cx: &EarlyContext<'_>, pat: &Pat) {
             .take_while(|patfield| matches!(patfield.pat.kind, PatKind::Wild))
             .enumerate()
             .last()
+        // Only run if `rest_pattern_accessible_field` is Allow, as they otherwise will contradict each other.
+        && cx.get_lint_level_spec(crate::rest_when_destructuring_struct::REST_PATTERN_ACCESSIBLE_FIELD).is_allow()
     {
         // Unlike the tuples above, structs have patfields rather than patterns, and separate out the
         // `..` into a separate parameter. Also, the `..` can only be at the end of the pattern.

tests/ui/rest_pattern_accessible_field.fixed

@@ -1,7 +1,7 @@
 //@aux-build:proc_macros.rs
 //@aux-build:non-exhaustive-struct.rs
 #![warn(clippy::rest_pattern_accessible_field)]
-#![allow(clippy::unneeded_wildcard_pattern)]
+#![warn(clippy::unneeded_wildcard_pattern)]
 
 use non_exhaustive_struct::{NonExhaustiveStruct, NonExhaustiveStructNoPrivateFields};
 

tests/ui/rest_pattern_accessible_field.rs

@@ -1,7 +1,7 @@
 //@aux-build:proc_macros.rs
 //@aux-build:non-exhaustive-struct.rs
 #![warn(clippy::rest_pattern_accessible_field)]
-#![allow(clippy::unneeded_wildcard_pattern)]
+#![warn(clippy::unneeded_wildcard_pattern)]
 
 use non_exhaustive_struct::{NonExhaustiveStruct, NonExhaustiveStructNoPrivateFields};