admin/mruby-mruby
1
0
Fork 0
mirror of https://github.com/mruby/mruby synced 2026-06-08 16:11:16 +00:00
Code Issues Packages Projects Releases Wiki Activity

array.c: no need to assert if blk is a Proc; ref #6344

Browse Source 
Since `blk` always comes from `mrb_get_args`, it should always be either
`nil` or a Proc.
This commit is contained in:
Yukihiro "Matz" Matsumoto
2024-09-05 15:26:09 +09:00
parent 3ca3849f54
commit 1f9fa06119
1 changed files with 0 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/array.c
-5
View File
@@ -320,7 +320,6 @@ mrb_ary_init(mrb_state *mrb, mrb_value ary)
val = obj;
}
else {
mrb_assert(mrb_proc_p(blk));
val = mrb_yield(mrb, blk, mrb_fixnum_value(i));
}
mrb_ary_set(mrb, ary, i, val);
@@ -1169,7 +1168,6 @@ mrb_ary_index_m(mrb_state *mrb, mrb_value self)
}
else {
for (mrb_int i = 0; i < RARRAY_LEN(self); i++) {
mrb_assert(mrb_proc_p(blk));
mrb_value eq = mrb_yield(mrb, blk, RARRAY_PTR(self)[i]);
if (mrb_test(eq)) {
return mrb_int_value(mrb, i);
@@ -1210,7 +1208,6 @@ mrb_ary_rindex_m(mrb_state *mrb, mrb_value self)
}
}
else {
mrb_assert(mrb_proc_p(blk));
mrb_value eq = mrb_yield(mrb, blk, RARRAY_PTR(self)[i]);
if (mrb_test(eq)) return mrb_int_value(mrb, i);
}
@@ -1584,7 +1581,6 @@ mrb_ary_delete(mrb_state *mrb, mrb_value self)
if (i == j) {
if (mrb_nil_p(blk)) return mrb_nil_value();
mrb_assert(mrb_proc_p(blk));
return mrb_yield(mrb, blk, obj);
}
@@ -1608,7 +1604,6 @@ sort_cmp(mrb_state *mrb, mrb_value ary, mrb_value *p, mrb_int a, mrb_int b, mrb_
if (cmp == -2) cmp_failed(mrb, a, b);
}
else {
mrb_assert(mrb_proc_p(blk));
mrb_value args[2] = {p[a], p[b]};
mrb_value c = mrb_yield_argv(mrb, blk, 2, args);
if (mrb_nil_p(c) || !mrb_fixnum_p(c)) {