mikros/rust
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Add a Lint for Pointer to Integer Transmutes in Consts

Browse Source
This commit is contained in:
Veera 2024-09-22 19:32:56 -04:00
parent c8f5136fe8
commit 98b59a1cf6
2 changed files with 12 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
tests/ui/transmutes_expressible_as_ptr_casts.fixed
View File

@ -84,8 +84,11 @@ fn issue_10449() {
} }
// Pointers cannot be cast to integers in const contexts // Pointers cannot be cast to integers in const contexts
#[allow(ptr_to_integer_transmute_in_consts, reason = "This is tested in the compiler test suite")]
const fn issue_12402<P>(ptr: *const P) { const fn issue_12402<P>(ptr: *const P) {
unsafe { transmute::<*const i32, usize>(&42i32) }; // This test exists even though the compiler lints against it
unsafe { transmute::<fn(*const P), usize>(issue_12402) }; // to test that clippy's transmute lints do not trigger on this.
let _ = unsafe { transmute::<_, usize>(ptr) }; unsafe { std::mem::transmute::<*const i32, usize>(&42i32) };
unsafe { std::mem::transmute::<fn(*const P), usize>(issue_12402) };
let _ = unsafe { std::mem::transmute::<_, usize>(ptr) };
} }

9
tests/ui/transmutes_expressible_as_ptr_casts.rs
View File

@ -84,8 +84,11 @@ fn f() {}
} }
// Pointers cannot be cast to integers in const contexts // Pointers cannot be cast to integers in const contexts
#[allow(ptr_to_integer_transmute_in_consts, reason = "This is tested in the compiler test suite")]
const fn issue_12402<P>(ptr: *const P) { const fn issue_12402<P>(ptr: *const P) {
unsafe { transmute::<*const i32, usize>(&42i32) }; // This test exists even though the compiler lints against it
unsafe { transmute::<fn(*const P), usize>(issue_12402) }; // to test that clippy's transmute lints do not trigger on this.
let _ = unsafe { transmute::<_, usize>(ptr) }; unsafe { std::mem::transmute::<*const i32, usize>(&42i32) };
unsafe { std::mem::transmute::<fn(*const P), usize>(issue_12402) };
let _ = unsafe { std::mem::transmute::<_, usize>(ptr) };
} }