Define RUBY_DEBUG macro if undefined (#532)

* Define RUBY_DEBUG macro if undefined

Add a check and default definition for RUBY_DEBUG to ensure compatibility when RUBY_DEBUG is not already defined.

* Fix TYPED_DATA_EMBEDDED flag for 32/64-bit targets

Ensure correct flag type is used for pointer width, preventing mismatched types on 32-bit and 64-bit architectures.

* Refine ruby debug and assert header inclusion

Replace RUBY_DEBUG guard with HAVE_RUBY_ASSERT_H, include ruby/assert.h when available, and remove manual RUBY_DEBUG definition.

* Update Docker workflow and fix RUBY_DEBUG reference

- Trigger Docker workflow on docker branches
- Fix RUBY_DEBUG path in debug_ruby_assert_type macro

* Move ruby/assert.h include after other headers

Relocate ruby/assert.h inclusion to the end and add RUBY_DEBUG fallback if not present. Ensures correct header order and macro definition.

* Remove ruby/assert.h include and update debug macro

Remove conditional inclusion of ruby/assert.h from wrapper.h.
Update debug_ruby_assert_type! macro to use cfg(ruby_ruby_debug).

* Prevent unused variable warnings in debug macro

Add code to use macro arguments when debug is disabled to avoid compiler warnings.

Author: ianks

.github/workflows/docker.yml

@@ -6,6 +6,7 @@ on:
   push:
     branches:
       - main
+      - "*docker*"
     tags:
       - v*
 

crates/rb-sys/src/stable_api/ruby_3_3.rs

@@ -292,8 +292,12 @@ impl StableApiDefinition for Definition {
 
         let rdata = obj as *const crate::internal::RTypedData;
         let typed_flag = (*rdata).typed_flag;
-        // TYPED_DATA_EMBEDDED (2) flag indicates embedded data
-        (typed_flag & (crate::TYPED_DATA_EMBEDDED as u64)) != 0
+        #[cfg(target_pointer_width = "64")]
+        const FLAG: u64 = crate::TYPED_DATA_EMBEDDED as u64;
+        #[cfg(target_pointer_width = "32")]
+        const FLAG: u32 = crate::TYPED_DATA_EMBEDDED as u32;
+
+        (typed_flag & FLAG) != 0
     }
 
     #[inline]

crates/rb-sys/src/stable_api/ruby_3_4.rs

@@ -3,7 +3,7 @@ use crate::{
     debug_ruby_assert_type,
     internal::{RArray, RString, RTypedData},
     ruby_value_type::RUBY_T_DATA,
-    value_type, TYPED_DATA_EMBEDDED, VALUE,
+    value_type, VALUE,
 };
 use std::{
     ffi::c_void,
@@ -293,8 +293,12 @@ impl StableApiDefinition for Definition {
 
         let rdata = obj as *const RTypedData;
         let typed_flag = (*rdata).typed_flag;
-        // TYPED_DATA_EMBEDDED (2) flag indicates embedded data
-        (typed_flag & (TYPED_DATA_EMBEDDED as u64)) != 0
+        #[cfg(target_pointer_width = "64")]
+        const FLAG: u64 = crate::TYPED_DATA_EMBEDDED as u64;
+        #[cfg(target_pointer_width = "32")]
+        const FLAG: u32 = crate::TYPED_DATA_EMBEDDED as u32;
+
+        (typed_flag & FLAG) != 0
     }
 
     #[inline]

crates/rb-sys/src/utils.rs

@@ -43,7 +43,8 @@ pub(crate) unsafe fn is_ruby_vm_started() -> bool {
 #[macro_export]
 macro_rules! debug_ruby_assert_type {
     ($obj:expr, $type:expr, $message:expr) => {
-        if $crate::RUBY_DEBUG != 0 {
+        #[cfg(ruby_ruby_debug = "true")]
+        {
             #[allow(clippy::macro_metavars_in_unsafe)]
             unsafe {
                 assert!(
@@ -52,6 +53,10 @@ macro_rules! debug_ruby_assert_type {
                 );
             }
         }
+        #[cfg(not(ruby_ruby_debug = "true"))]
+        {
+            let _ = ($obj, $type, $message); // Prevent unused variable warnings
+        }
     };
 }