From 1af8ed5f0a2c381c5dee4a5bcffff61f270c30d9 Mon Sep 17 00:00:00 2001 From: Jean Boussier Date: Tue, 16 Nov 2021 11:33:39 +0100 Subject: `Primitive.mandatory_only?` consider splat args `vm_ci_argc` gives the number of arguments, but `*[1, 2, 3]` only counts for one. --- vm_insnhelper.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'vm_insnhelper.c') diff --git a/vm_insnhelper.c b/vm_insnhelper.c index 8e0793e070..9974a8a4f8 100644 --- a/vm_insnhelper.c +++ b/vm_insnhelper.c @@ -1848,8 +1848,10 @@ vm_search_cc(const VALUE klass, const struct rb_callinfo * const ci) } } - if ((cme->def->iseq_overload && - (int)vm_ci_argc(ci) == method_entry_iseqptr(cme)->body->param.lead_num)) { + if (cme->def->iseq_overload && + (vm_ci_flag(ci) & (VM_CALL_ARGS_SIMPLE)) && + (int)vm_ci_argc(ci) == method_entry_iseqptr(cme)->body->param.lead_num + ) { // use alternative cme = overloaded_cme(cme); METHOD_ENTRY_CACHED_SET((struct rb_callable_method_entry_struct *)cme); -- cgit v1.2.3