Improve GNUStep message sending a bit

Author: madsmtm

objc2/src/message/gnustep.rs

@@ -1,10 +1,26 @@
+use core::hint;
 use core::mem;
 
 use super::conditional_try;
 use crate::ffi;
-use crate::runtime::{Class, Object, Sel};
+use crate::runtime::{Class, Imp, Object, Sel};
 use crate::{Encode, MessageArguments};
 
+#[inline]
+fn unwrap_msg_send_fn(msg_send_fn: Option<Imp>) -> Imp {
+    match msg_send_fn {
+        Some(msg_send_fn) => msg_send_fn,
+        None => {
+            if cfg!(debug_assertions) {
+                unreachable!("got unexpected null IMP")
+            } else {
+                // This shouldn't ever happen
+                unsafe { hint::unreachable_unchecked() }
+            }
+        }
+    }
+}
+
 #[track_caller]
 pub(crate) unsafe fn send_unverified<A, R>(receiver: *mut Object, sel: Sel, args: A) -> R
 where
@@ -22,7 +38,7 @@ where
     }
 
     let msg_send_fn = unsafe { ffi::objc_msg_lookup(receiver.cast(), sel.as_ptr()) };
-    let msg_send_fn = msg_send_fn.expect("Null IMP");
+    let msg_send_fn = unwrap_msg_send_fn(msg_send_fn);
     unsafe { conditional_try(|| A::__invoke(msg_send_fn, receiver, sel, args)) }
 }
 
@@ -48,6 +64,6 @@ where
         super_class: superclass.cast(),
     };
     let msg_send_fn = unsafe { ffi::objc_msg_lookup_super(&sup, sel.as_ptr()) };
-    let msg_send_fn = msg_send_fn.expect("Null IMP");
+    let msg_send_fn = unwrap_msg_send_fn(msg_send_fn);
     unsafe { conditional_try(|| A::__invoke(msg_send_fn, receiver, sel, args)) }
 }