Skip to content

Commit f41eec1

Browse files
authored
Merge pull request rust-lang#4528 from rust-lang/rustup-2025-08-18
Automatic Rustup
2 parents dfa807b + b4b17ac commit f41eec1

File tree

6 files changed

+17
-18
lines changed

6 files changed

+17
-18
lines changed

clippy_lints/src/missing_asserts_for_indexing.rs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ use rustc_ast::{BinOpKind, LitKind, RangeLimits};
1111
use rustc_data_structures::packed::Pu128;
1212
use rustc_data_structures::unhash::UnindexMap;
1313
use rustc_errors::{Applicability, Diag};
14-
use rustc_hir::{Block, Body, Expr, ExprKind, UnOp};
14+
use rustc_hir::{Body, Expr, ExprKind};
1515
use rustc_lint::{LateContext, LateLintPass};
1616
use rustc_session::declare_lint_pass;
1717
use rustc_span::source_map::Spanned;
@@ -135,12 +135,12 @@ fn assert_len_expr<'hir>(
135135
cx: &LateContext<'_>,
136136
expr: &'hir Expr<'hir>,
137137
) -> Option<(LengthComparison, usize, &'hir Expr<'hir>)> {
138-
let (cmp, asserted_len, slice_len) = if let Some(higher::If { cond, then, .. }) = higher::If::hir(expr)
139-
&& let ExprKind::Unary(UnOp::Not, condition) = &cond.kind
140-
&& let ExprKind::Binary(bin_op, left, right) = &condition.kind
138+
let (cmp, asserted_len, slice_len) = if let Some(
139+
higher::IfLetOrMatch::Match(cond, [_, then], _)
140+
) = higher::IfLetOrMatch::parse(cx, expr)
141+
&& let ExprKind::Binary(bin_op, left, right) = &cond.kind
141142
// check if `then` block has a never type expression
142-
&& let ExprKind::Block(Block { expr: Some(then_expr), .. }, _) = then.kind
143-
&& cx.typeck_results().expr_ty(then_expr).is_never()
143+
&& cx.typeck_results().expr_ty(then.body).is_never()
144144
{
145145
len_comparison(bin_op.node, left, right)?
146146
} else if let Some((macro_call, bin_op)) = first_node_macro_backtrace(cx, expr).find_map(|macro_call| {

clippy_utils/src/macros.rs

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -250,18 +250,13 @@ impl<'a> PanicExpn<'a> {
250250
};
251251
let name = path.segments.last().unwrap().ident.name;
252252

253-
// This has no argument
254-
if name == sym::panic_cold_explicit {
255-
return Some(Self::Empty);
256-
}
257-
258253
let [arg, rest @ ..] = args else {
259254
return None;
260255
};
261256
let result = match name {
262257
sym::panic if arg.span.eq_ctxt(expr.span) => Self::Empty,
263258
sym::panic | sym::panic_str => Self::Str(arg),
264-
sym::panic_display | sym::panic_cold_display => {
259+
sym::panic_display => {
265260
let ExprKind::AddrOf(_, _, e) = &arg.kind else {
266261
return None;
267262
};

clippy_utils/src/sym.rs

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -241,9 +241,6 @@ generate! {
241241
or_insert,
242242
or_insert_with,
243243
outer_expn,
244-
panic_cold_display,
245-
panic_cold_explicit,
246-
panic_display,
247244
panic_str,
248245
parse,
249246
partition,

tests/ui/const_is_empty.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -196,6 +196,7 @@ fn issue_13106() {
196196

197197
const {
198198
assert!(EMPTY_STR.is_empty());
199+
//~^ const_is_empty
199200
}
200201

201202
const {

tests/ui/const_is_empty.stderr

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -158,10 +158,16 @@ LL | let _ = val.is_empty();
158158
| ^^^^^^^^^^^^^^
159159

160160
error: this expression always evaluates to true
161-
--> tests/ui/const_is_empty.rs:202:9
161+
--> tests/ui/const_is_empty.rs:198:17
162+
|
163+
LL | assert!(EMPTY_STR.is_empty());
164+
| ^^^^^^^^^^^^^^^^^^^^
165+
166+
error: this expression always evaluates to true
167+
--> tests/ui/const_is_empty.rs:203:9
162168
|
163169
LL | EMPTY_STR.is_empty();
164170
| ^^^^^^^^^^^^^^^^^^^^
165171

166-
error: aborting due to 27 previous errors
172+
error: aborting due to 28 previous errors
167173

tests/ui/incompatible_msrv.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
#![warn(clippy::incompatible_msrv)]
22
#![feature(custom_inner_attributes)]
3-
#![allow(stable_features)]
3+
#![allow(stable_features, clippy::diverging_sub_expression)]
44
#![feature(strict_provenance)] // For use in test
55
#![clippy::msrv = "1.3.0"]
66

0 commit comments

Comments
 (0)