[unnecessary_cast
] Do not lint negative hexadecimal literals when cast as float
Floats cannot be expressed as hexadecimal literals
This commit is contained in:
parent
2c8e473ffe
commit
6f4546a4be
@ -59,9 +59,6 @@ pub(super) fn check<'tcx>(
|
||||
lint_unnecessary_cast(cx, expr, literal_str, cast_from, cast_to);
|
||||
return false;
|
||||
},
|
||||
LitKind::Int(_, LitIntType::Unsuffixed) | LitKind::Float(_, LitFloatType::Unsuffixed) => {
|
||||
return false;
|
||||
},
|
||||
LitKind::Int(_, LitIntType::Signed(_) | LitIntType::Unsigned(_))
|
||||
| LitKind::Float(_, LitFloatType::Suffixed(_))
|
||||
if cast_from.kind() == cast_to.kind() =>
|
||||
|
@ -69,12 +69,13 @@ pub fn from_lit_kind(src: &'a str, lit_kind: &LitKind) -> Option<NumericLiteral<
|
||||
|
||||
#[must_use]
|
||||
pub fn new(lit: &'a str, suffix: Option<&'a str>, float: bool) -> Self {
|
||||
let unsigned_lit = lit.trim_start_matches('-');
|
||||
// Determine delimiter for radix prefix, if present, and radix.
|
||||
let radix = if lit.starts_with("0x") {
|
||||
let radix = if unsigned_lit.starts_with("0x") {
|
||||
Radix::Hexadecimal
|
||||
} else if lit.starts_with("0b") {
|
||||
} else if unsigned_lit.starts_with("0b") {
|
||||
Radix::Binary
|
||||
} else if lit.starts_with("0o") {
|
||||
} else if unsigned_lit.starts_with("0o") {
|
||||
Radix::Octal
|
||||
} else {
|
||||
Radix::Decimal
|
||||
|
@ -111,4 +111,8 @@ mod fixable {
|
||||
|
||||
let _num = foo();
|
||||
}
|
||||
|
||||
fn issue_9603() {
|
||||
let _: f32 = -0x400 as f32;
|
||||
}
|
||||
}
|
||||
|
@ -111,4 +111,8 @@ fn foo() -> f32 {
|
||||
|
||||
let _num = foo() as f32;
|
||||
}
|
||||
|
||||
fn issue_9603() {
|
||||
let _: f32 = -0x400 as f32;
|
||||
}
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user