OpenE2K/rust
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Do not drop suffixes on decimal_literal_representation

Browse Source
This commit is contained in:
Yuki Okushi 2019-12-26 01:22:36 +09:00
parent a68ef55dbf
commit b68e65b4da
4 changed files with 18 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
clippy_lints/src/literal_representation.rs
View File

@ -560,7 +560,7 @@ impl DecimalLiteralRepresentation {
if val >= u128::from(self.threshold);
then {
let hex = format!("{:#X}", val);
let num_lit = NumericLiteral::new(&hex, None, false);
let num_lit = NumericLiteral::new(&hex, num_lit.suffix, false);
let _ = Self::do_lint(num_lit.integer).map_err(|warning_type| {
warning_type.display(num_lit.format(), cx, lit.span)
});

2
tests/ui/decimal_literal_representation.fixed
View File

@ -21,5 +21,7 @@ fn main() {
0x7FFF_FFFF, // 0x7FFF_FFFF
#[allow(overflowing_literals)]
0xF0F0_F0F0, // 0xF0F0_F0F0
0x8005_usize, // 0x8005_usize
0x7F0F_F00F_isize, // 0x7F0F_F00F_isize
);
}

2
tests/ui/decimal_literal_representation.rs
View File

@ -21,5 +21,7 @@ fn main() {
2_147_483_647, // 0x7FFF_FFFF
#[allow(overflowing_literals)]
4_042_322_160, // 0xF0F0_F0F0
32_773usize, // 0x8005_usize
2_131_750_927isize, // 0x7F0F_F00F_isize
);
}

14
tests/ui/decimal_literal_representation.stderr
View File

@ -30,5 +30,17 @@ error: integer literal has a better hexadecimal representation
LL | 4_042_322_160, // 0xF0F0_F0F0
| ^^^^^^^^^^^^^ help: consider: `0xF0F0_F0F0`
error: aborting due to 5 previous errors
error: integer literal has a better hexadecimal representation
--> $DIR/decimal_literal_representation.rs:24:9
|
LL | 32_773usize, // 0x8005_usize
| ^^^^^^^^^^^ help: consider: `0x8005_usize`
error: integer literal has a better hexadecimal representation
--> $DIR/decimal_literal_representation.rs:25:9
|
LL | 2_131_750_927isize, // 0x7F0F_F00F_isize
| ^^^^^^^^^^^^^^^^^^ help: consider: `0x7F0F_F00F_isize`
error: aborting due to 7 previous errors