Further refactor macro internals

crates/objc2/CHANGELOG.md

@@ -19,6 +19,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 ### Fixed
 * Allow empty structs in `declare_class!` macro.
 * Allow using `extern_methods!` without the `ClassType` trait in scope.
+* Fixed a few small issues with `declare_class!`.
 
 
 ## 0.3.0-beta.4 - 2022-12-24

crates/objc2/src/declare/declare_class_tests.rs

@@ -297,3 +297,39 @@ declare_class!(
 fn test_all_the_bool() {
     let _ = DeclareClassAllTheBool::class();
 }
+
+declare_class!(
+    struct DeclareClassUnreachable;
+
+    unsafe impl ClassType for DeclareClassUnreachable {
+        type Super = NSObject;
+    }
+
+    // Ensure none of these warn
+    unsafe impl DeclareClassUnreachable {
+        #[method(unreachable)]
+        fn unreachable(&self) -> bool {
+            unreachable!()
+        }
+
+        #[method(unreachableClass)]
+        fn unreachable_class() -> bool {
+            unreachable!()
+        }
+
+        #[method(unreachableVoid)]
+        fn unreachable_void(&self) {
+            unreachable!()
+        }
+
+        #[method(unreachableClassVoid)]
+        fn unreachable_class_void() {
+            unreachable!()
+        }
+    }
+);
+
+#[test]
+fn test_unreachable() {
+    let _ = DeclareClassUnreachable::class();
+}