From 33744d25cfdd056552ab9a464ac250d56dfaaf2c Mon Sep 17 00:00:00 2001 From: Matt Valentine-House Date: Thu, 7 May 2026 15:19:34 +0100 Subject: [ruby/mmtk] Remove unnecessary null check. the only caller of this unconditionally constructs a binding options object now, So actually this is dead code https://github.com/ruby/mmtk/commit/d832004e89 --- gc/mmtk/src/api.rs | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/gc/mmtk/src/api.rs b/gc/mmtk/src/api.rs index 0c73cd74eb..1519d2b623 100644 --- a/gc/mmtk/src/api.rs +++ b/gc/mmtk/src/api.rs @@ -191,14 +191,7 @@ pub unsafe extern "C" fn mmtk_init_binding( crate::set_panic_hook(); let builder: Box = unsafe { Box::from_raw(builder) }; - let binding_options = if binding_options.is_null() { - RubyBindingOptions { - ractor_check_mode: false, - suffix_size: 0, - } - } else { - unsafe { (*binding_options).clone() } - }; + let binding_options = unsafe { (*binding_options).clone() }; let mmtk_boxed = mmtk_init(&builder); let mmtk_static = Box::leak(Box::new(mmtk_boxed)); -- cgit v1.2.3