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

Return stdbool from recursive_check()

Browse Source 
The return value is used as a boolean value in C. Since it's not used as
a Ruby object, it just seems confusing that it returns a VALUE.
This commit is contained in:
Takashi Kokubun 2024-03-26 11:32:09 -07:00
parent e16086b7f2
commit 696b2716e0
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
thread.c
View File

@ -5098,12 +5098,12 @@ recursive_list_access(VALUE sym)
} }
/* /*
* Returns Qtrue if and only if obj (or the pair <obj, paired_obj>) is already * Returns true if and only if obj (or the pair <obj, paired_obj>) is already
* in the recursion list. * in the recursion list.
* Assumes the recursion list is valid. * Assumes the recursion list is valid.
*/ */
static VALUE static bool
recursive_check(VALUE list, VALUE obj, VALUE paired_obj_id) recursive_check(VALUE list, VALUE obj, VALUE paired_obj_id)
{ {
#if SIZEOF_LONG == SIZEOF_VOIDP #if SIZEOF_LONG == SIZEOF_VOIDP
@ -5115,18 +5115,18 @@ recursive_check(VALUE list, VALUE obj, VALUE paired_obj_id)
VALUE pair_list = rb_hash_lookup2(list, obj, Qundef); VALUE pair_list = rb_hash_lookup2(list, obj, Qundef);
if (UNDEF_P(pair_list)) if (UNDEF_P(pair_list))
return Qfalse; return false;
if (paired_obj_id) { if (paired_obj_id) {
if (!RB_TYPE_P(pair_list, T_HASH)) { if (!RB_TYPE_P(pair_list, T_HASH)) {
if (!OBJ_ID_EQL(paired_obj_id, pair_list)) if (!OBJ_ID_EQL(paired_obj_id, pair_list))
return Qfalse; return false;
} }
else { else {
if (NIL_P(rb_hash_lookup(pair_list, paired_obj_id))) if (NIL_P(rb_hash_lookup(pair_list, paired_obj_id)))
return Qfalse; return false;
} }
} }
return Qtrue; return true;
} }
/* /*