Do not lint assertions_on_constants for const _: () = assert!(expr)

This commit is contained in:
StackOverflowExcept1on 2023-12-15 22:52:38 +03:00
parent 29bdc8b2bc
commit 058b74fce4
No known key found for this signature in database
GPG key ID: 878715DC338535BE
3 changed files with 28 additions and 4 deletions

View file

@ -1,7 +1,7 @@
use clippy_utils::consts::{constant, Constant};
use clippy_utils::consts::{constant_with_source, Constant, ConstantSource};
use clippy_utils::diagnostics::span_lint_and_help;
use clippy_utils::macros::{find_assert_args, root_macro_call_first_node, PanicExpn};
use rustc_hir::Expr;
use rustc_hir::{Expr, Item, ItemKind, Node};
use rustc_lint::{LateContext, LateLintPass};
use rustc_session::declare_lint_pass;
use rustc_span::sym;
@ -42,9 +42,18 @@ impl<'tcx> LateLintPass<'tcx> for AssertionsOnConstants {
let Some((condition, panic_expn)) = find_assert_args(cx, e, macro_call.expn) else {
return;
};
let Some(Constant::Bool(val)) = constant(cx, cx.typeck_results(), condition) else {
let Some((Constant::Bool(val), source)) = constant_with_source(cx, cx.typeck_results(), condition) else {
return;
};
if let ConstantSource::Constant = source
&& let Some(node) = cx.tcx.hir().find_parent(e.hir_id)
&& let Node::Item(Item {
kind: ItemKind::Const(.., _body_id),
..
}) = node
{
return;
}
if val {
span_lint_and_help(
cx,

View file

@ -45,4 +45,11 @@ fn main() {
const CFG_FLAG: &bool = &cfg!(feature = "hey");
assert!(!CFG_FLAG);
const _: () = assert!(true);
//~^ ERROR: `assert!(true)` will be optimized out by the compiler
// Don't lint if the value is dependent on a defined constant:
const N: usize = 1024;
const _: () = assert!(N.is_power_of_two());
}

View file

@ -72,5 +72,13 @@ LL | debug_assert!(true);
|
= help: remove it
error: aborting due to 9 previous errors
error: `assert!(true)` will be optimized out by the compiler
--> $DIR/assertions_on_constants.rs:49:19
|
LL | const _: () = assert!(true);
| ^^^^^^^^^^^^^
|
= help: remove it
error: aborting due to 10 previous errors