1berry/ruby
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Replace assert with RUBY_ASSERT in complex.c

Browse Source 
assert does not print the bug report, only the file and line number of
the assertion that failed. RUBY_ASSERT prints the full bug report, which
makes it much easier to debug.
This commit is contained in:
Peter Zhu 2024-02-08 09:42:16 -05:00
parent de7a29ef8d
commit 7256e38f86
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
complex.c
View File

@ -411,15 +411,15 @@ nucomp_s_alloc(VALUE klass)
inline static VALUE inline static VALUE
f_complex_new_bang1(VALUE klass, VALUE x) f_complex_new_bang1(VALUE klass, VALUE x)
{ {
assert(!RB_TYPE_P(x, T_COMPLEX)); RUBY_ASSERT(!RB_TYPE_P(x, T_COMPLEX));
return nucomp_s_new_internal(klass, x, ZERO); return nucomp_s_new_internal(klass, x, ZERO);
} }
inline static VALUE inline static VALUE
f_complex_new_bang2(VALUE klass, VALUE x, VALUE y) f_complex_new_bang2(VALUE klass, VALUE x, VALUE y)
{ {
assert(!RB_TYPE_P(x, T_COMPLEX)); RUBY_ASSERT(!RB_TYPE_P(x, T_COMPLEX));
assert(!RB_TYPE_P(y, T_COMPLEX)); RUBY_ASSERT(!RB_TYPE_P(y, T_COMPLEX));
return nucomp_s_new_internal(klass, x, y); return nucomp_s_new_internal(klass, x, y);
} }
@ -432,7 +432,7 @@ nucomp_real_check(VALUE num)
!RB_TYPE_P(num, T_RATIONAL)) { !RB_TYPE_P(num, T_RATIONAL)) {
if (RB_TYPE_P(num, T_COMPLEX) && nucomp_real_p(num)) { if (RB_TYPE_P(num, T_COMPLEX) && nucomp_real_p(num)) {
VALUE real = RCOMPLEX(num)->real; VALUE real = RCOMPLEX(num)->real;
assert(!RB_TYPE_P(real, T_COMPLEX)); RUBY_ASSERT(!RB_TYPE_P(real, T_COMPLEX));
return real; return real;
} }
if (!k_numeric_p(num) || !f_real_p(num)) if (!k_numeric_p(num) || !f_real_p(num))