Don't assume T_FLOAT for flonums

XrXr · XrXr · commit a9e377d6bb1e · 2024-10-31T15:45:01.000-04:00
For debugging.
diff --git a/include/ruby/internal/globals.h b/include/ruby/internal/globals.h
@@ -193,11 +193,7 @@ rb_class_of(VALUE obj)
         return rb_cFloat;
     }
 
-#if !RUBY_DEBUG
-    RBIMPL_UNREACHABLE_RETURN(Qfalse);
-#else
     RUBY_ASSERT_FAIL("unexpected type");
-#endif
 }
 
 #define CLASS_OF rb_class_of /**< @old{rb_class_of} */
diff --git a/include/ruby/internal/value_type.h b/include/ruby/internal/value_type.h
@@ -246,7 +246,7 @@ rb_type(VALUE obj)
         return RUBY_T_SYMBOL;
     }
     else {
-        RBIMPL_ASSUME(RB_FLONUM_P(obj));
+        RUBY_ASSERT_ALWAYS(RB_FLONUM_P(obj));
         return RUBY_T_FLOAT;
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -193,11 +193,7 @@ rb_class_of(VALUE obj)`
`193`	`193`	`return rb_cFloat;`
`194`	`194`	`}`
`195`	`195`
`196`		`-#if !RUBY_DEBUG`
`197`		`- RBIMPL_UNREACHABLE_RETURN(Qfalse);`
`198`		`-#else`
`199`	`196`	`RUBY_ASSERT_FAIL("unexpected type");`
`200`		`-#endif`
`201`	`197`	`}`
`202`	`198`
`203`	`199`	`#define CLASS_OF rb_class_of /*< @old{rb_class_of} /`
Original file line number	Diff line number	Diff line change
`@@ -246,7 +246,7 @@ rb_type(VALUE obj)`
`246`	`246`	`return RUBY_T_SYMBOL;`
`247`	`247`	`}`
`248`	`248`	`else {`
`249`		`- RBIMPL_ASSUME(RB_FLONUM_P(obj));`
	`249`	`+ RUBY_ASSERT_ALWAYS(RB_FLONUM_P(obj));`
`250`	`250`	`return RUBY_T_FLOAT;`
`251`	`251`	`}`
`252`	`252`	`}`