mirror of
https://github.com/rust-lang/rust-clippy
synced 2024-11-27 07:00:55 +00:00
262 lines
6.5 KiB
Rust
262 lines
6.5 KiB
Rust
#![feature(repr128)]
|
|
#![allow(incomplete_features)]
|
|
#![warn(
|
|
clippy::cast_precision_loss,
|
|
clippy::cast_possible_truncation,
|
|
clippy::cast_sign_loss,
|
|
clippy::cast_possible_wrap
|
|
)]
|
|
#![allow(clippy::cast_abs_to_unsigned, clippy::no_effect, clippy::unnecessary_operation)]
|
|
|
|
fn main() {
|
|
// Test clippy::cast_precision_loss
|
|
let x0 = 1i32;
|
|
x0 as f32;
|
|
let x1 = 1i64;
|
|
x1 as f32;
|
|
x1 as f64;
|
|
let x2 = 1u32;
|
|
x2 as f32;
|
|
let x3 = 1u64;
|
|
x3 as f32;
|
|
x3 as f64;
|
|
// Test clippy::cast_possible_truncation
|
|
1f32 as i32;
|
|
1f32 as u32;
|
|
1f64 as f32;
|
|
1i32 as i8;
|
|
1i32 as u8;
|
|
1f64 as isize;
|
|
1f64 as usize;
|
|
// Test clippy::cast_possible_wrap
|
|
1u8 as i8;
|
|
1u16 as i16;
|
|
1u32 as i32;
|
|
1u64 as i64;
|
|
1usize as isize;
|
|
// Test clippy::cast_sign_loss
|
|
1i32 as u32;
|
|
-1i32 as u32;
|
|
1isize as usize;
|
|
-1isize as usize;
|
|
0i8 as u8;
|
|
i8::MAX as u8;
|
|
i16::MAX as u16;
|
|
i32::MAX as u32;
|
|
i64::MAX as u64;
|
|
i128::MAX as u128;
|
|
|
|
(-1i8).abs() as u8;
|
|
(-1i16).abs() as u16;
|
|
(-1i32).abs() as u32;
|
|
(-1i64).abs() as u64;
|
|
(-1isize).abs() as usize;
|
|
|
|
(-1i8).checked_abs().unwrap() as u8;
|
|
(-1i16).checked_abs().unwrap() as u16;
|
|
(-1i32).checked_abs().unwrap() as u32;
|
|
(-1i64).checked_abs().unwrap() as u64;
|
|
(-1isize).checked_abs().unwrap() as usize;
|
|
|
|
(-1i8).rem_euclid(1i8) as u8;
|
|
(-1i8).rem_euclid(1i8) as u16;
|
|
(-1i16).rem_euclid(1i16) as u16;
|
|
(-1i16).rem_euclid(1i16) as u32;
|
|
(-1i32).rem_euclid(1i32) as u32;
|
|
(-1i32).rem_euclid(1i32) as u64;
|
|
(-1i64).rem_euclid(1i64) as u64;
|
|
(-1i64).rem_euclid(1i64) as u128;
|
|
(-1isize).rem_euclid(1isize) as usize;
|
|
(1i8).rem_euclid(-1i8) as u8;
|
|
(1i8).rem_euclid(-1i8) as u16;
|
|
(1i16).rem_euclid(-1i16) as u16;
|
|
(1i16).rem_euclid(-1i16) as u32;
|
|
(1i32).rem_euclid(-1i32) as u32;
|
|
(1i32).rem_euclid(-1i32) as u64;
|
|
(1i64).rem_euclid(-1i64) as u64;
|
|
(1i64).rem_euclid(-1i64) as u128;
|
|
(1isize).rem_euclid(-1isize) as usize;
|
|
|
|
(-1i8).checked_rem_euclid(1i8).unwrap() as u8;
|
|
(-1i8).checked_rem_euclid(1i8).unwrap() as u16;
|
|
(-1i16).checked_rem_euclid(1i16).unwrap() as u16;
|
|
(-1i16).checked_rem_euclid(1i16).unwrap() as u32;
|
|
(-1i32).checked_rem_euclid(1i32).unwrap() as u32;
|
|
(-1i32).checked_rem_euclid(1i32).unwrap() as u64;
|
|
(-1i64).checked_rem_euclid(1i64).unwrap() as u64;
|
|
(-1i64).checked_rem_euclid(1i64).unwrap() as u128;
|
|
(-1isize).checked_rem_euclid(1isize).unwrap() as usize;
|
|
(1i8).checked_rem_euclid(-1i8).unwrap() as u8;
|
|
(1i8).checked_rem_euclid(-1i8).unwrap() as u16;
|
|
(1i16).checked_rem_euclid(-1i16).unwrap() as u16;
|
|
(1i16).checked_rem_euclid(-1i16).unwrap() as u32;
|
|
(1i32).checked_rem_euclid(-1i32).unwrap() as u32;
|
|
(1i32).checked_rem_euclid(-1i32).unwrap() as u64;
|
|
(1i64).checked_rem_euclid(-1i64).unwrap() as u64;
|
|
(1i64).checked_rem_euclid(-1i64).unwrap() as u128;
|
|
(1isize).checked_rem_euclid(-1isize).unwrap() as usize;
|
|
|
|
// no lint for `cast_possible_truncation`
|
|
// with `signum` method call (see issue #5395)
|
|
let x: i64 = 5;
|
|
let _ = x.signum() as i32;
|
|
|
|
let s = x.signum();
|
|
let _ = s as i32;
|
|
|
|
// Test for signed min
|
|
(-99999999999i64).min(1) as i8; // should be linted because signed
|
|
|
|
// Test for various operations that remove enough bits for the result to fit
|
|
(999999u64 & 1) as u8;
|
|
(999999u64 % 15) as u8;
|
|
(999999u64 / 0x1_0000_0000_0000) as u16;
|
|
({ 999999u64 >> 56 }) as u8;
|
|
({
|
|
let x = 999999u64;
|
|
x.min(1)
|
|
}) as u8;
|
|
999999u64.clamp(0, 255) as u8;
|
|
999999u64.clamp(0, 256) as u8; // should still be linted
|
|
|
|
#[derive(Clone, Copy)]
|
|
enum E1 {
|
|
A,
|
|
B,
|
|
C,
|
|
}
|
|
impl E1 {
|
|
fn test(self) {
|
|
let _ = self as u8; // Don't lint. `0..=2` fits in u8
|
|
}
|
|
}
|
|
|
|
#[derive(Clone, Copy)]
|
|
enum E2 {
|
|
A = 255,
|
|
B,
|
|
}
|
|
impl E2 {
|
|
fn test(self) {
|
|
let _ = self as u8;
|
|
let _ = Self::B as u8;
|
|
let _ = self as i16; // Don't lint. `255..=256` fits in i16
|
|
let _ = Self::A as u8; // Don't lint.
|
|
}
|
|
}
|
|
|
|
#[derive(Clone, Copy)]
|
|
enum E3 {
|
|
A = -1,
|
|
B,
|
|
C = 50,
|
|
}
|
|
impl E3 {
|
|
fn test(self) {
|
|
let _ = self as i8; // Don't lint. `-1..=50` fits in i8
|
|
}
|
|
}
|
|
|
|
#[derive(Clone, Copy)]
|
|
enum E4 {
|
|
A = -128,
|
|
B,
|
|
}
|
|
impl E4 {
|
|
fn test(self) {
|
|
let _ = self as i8; // Don't lint. `-128..=-127` fits in i8
|
|
}
|
|
}
|
|
|
|
#[derive(Clone, Copy)]
|
|
enum E5 {
|
|
A = -129,
|
|
B = 127,
|
|
}
|
|
impl E5 {
|
|
fn test(self) {
|
|
let _ = self as i8;
|
|
let _ = Self::A as i8;
|
|
let _ = self as i16; // Don't lint. `-129..=127` fits in i16
|
|
let _ = Self::B as u8; // Don't lint.
|
|
}
|
|
}
|
|
|
|
#[derive(Clone, Copy)]
|
|
#[repr(u32)]
|
|
enum E6 {
|
|
A = u16::MAX as u32,
|
|
B,
|
|
}
|
|
impl E6 {
|
|
fn test(self) {
|
|
let _ = self as i16;
|
|
let _ = Self::A as u16; // Don't lint. `2^16-1` fits in u16
|
|
let _ = self as u32; // Don't lint. `2^16-1..=2^16` fits in u32
|
|
let _ = Self::A as u16; // Don't lint.
|
|
}
|
|
}
|
|
|
|
#[derive(Clone, Copy)]
|
|
#[repr(u64)]
|
|
enum E7 {
|
|
A = u32::MAX as u64,
|
|
B,
|
|
}
|
|
impl E7 {
|
|
fn test(self) {
|
|
let _ = self as usize;
|
|
let _ = Self::A as usize; // Don't lint.
|
|
let _ = self as u64; // Don't lint. `2^32-1..=2^32` fits in u64
|
|
}
|
|
}
|
|
|
|
#[derive(Clone, Copy)]
|
|
#[repr(i128)]
|
|
enum E8 {
|
|
A = i128::MIN,
|
|
B,
|
|
C = 0,
|
|
D = i128::MAX,
|
|
}
|
|
impl E8 {
|
|
fn test(self) {
|
|
let _ = self as i128; // Don't lint. `-(2^127)..=2^127-1` fits it i128
|
|
}
|
|
}
|
|
|
|
#[derive(Clone, Copy)]
|
|
#[repr(u128)]
|
|
enum E9 {
|
|
A,
|
|
B = u128::MAX,
|
|
}
|
|
impl E9 {
|
|
fn test(self) {
|
|
let _ = Self::A as u8; // Don't lint.
|
|
let _ = self as u128; // Don't lint. `0..=2^128-1` fits in u128
|
|
}
|
|
}
|
|
|
|
#[derive(Clone, Copy)]
|
|
#[repr(usize)]
|
|
enum E10 {
|
|
A,
|
|
B = u32::MAX as usize,
|
|
}
|
|
impl E10 {
|
|
fn test(self) {
|
|
let _ = self as u16;
|
|
let _ = Self::B as u32; // Don't lint.
|
|
let _ = self as u64; // Don't lint.
|
|
}
|
|
}
|
|
}
|
|
|
|
fn avoid_subtract_overflow(q: u32) {
|
|
let c = (q >> 16) as u8;
|
|
c as usize;
|
|
|
|
let c = (q / 1000) as u8;
|
|
c as usize;
|
|
}
|