diff options
author | Jimmy Miller <jimmy.miller@shopify.com> | 2022-10-18 16:03:55 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-10-18 16:03:55 -0400 |
commit | 12c30f215cfadb8feb678c659941d4cfe5310785 (patch) | |
tree | 947395fcfb9d958914c19f90266ae18a66c8af5c | |
parent | 6bd72a6406bedce6267b9737b38b9f1b89cf6a51 (diff) | |
download | ruby-12c30f215cfadb8feb678c659941d4cfe5310785.tar.gz |
Fixes remaining issues and turns back on dead code and unused (#6584)
-rw-r--r-- | yjit/src/core.rs | 4 | ||||
-rw-r--r-- | yjit/src/lib.rs | 3 |
2 files changed, 2 insertions, 5 deletions
diff --git a/yjit/src/core.rs b/yjit/src/core.rs index ea7eb56405..38c567d100 100644 --- a/yjit/src/core.rs +++ b/yjit/src/core.rs @@ -530,9 +530,9 @@ fn get_or_create_iseq_payload(iseq: IseqPtr) -> &'static mut IseqPayload { /// Iterate over all existing ISEQs pub fn for_each_iseq<F: FnMut(IseqPtr)>(mut callback: F) { unsafe extern "C" fn callback_wrapper(iseq: IseqPtr, data: *mut c_void) { - let callback: &mut &mut dyn FnMut(IseqPtr) -> bool = unsafe { std::mem::transmute(data) }; + let callback: &mut &mut dyn FnMut(IseqPtr) -> bool = std::mem::transmute(&mut *data); callback(iseq); - }; + } let mut data: &mut dyn FnMut(IseqPtr) = &mut callback; unsafe { rb_yjit_for_each_iseq(Some(callback_wrapper), (&mut data) as *mut _ as *mut c_void) }; } diff --git a/yjit/src/lib.rs b/yjit/src/lib.rs index 9b19c7d6a0..ce87cc250a 100644 --- a/yjit/src/lib.rs +++ b/yjit/src/lib.rs @@ -3,9 +3,6 @@ #![allow(clippy::too_many_arguments)] // :shrug: #![allow(clippy::identity_op)] // Sometimes we do it for style -// Temporary while switching to the new backend -#![allow(dead_code)] -#![allow(unused)] mod asm; mod backend; |