Fixes remaining issues and turns back on dead code and unused (#6584)

ruby · Oct 18, 2022 · 12c30f2 · 12c30f2
1 parent 6bd72a6
commit 12c30f2
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git 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
@@ -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;