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

Use available constants

Browse Source 
We don't need to intern "initialize" all the time because we already
have `idInitialize` available
This commit is contained in:
Aaron Patterson 2023-10-24 12:38:50 -07:00 committed by Aaron Patterson
parent afae8df373
commit 33bebee13a
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
vm_insnhelper.c
View File

@ -5493,7 +5493,7 @@ vm_define_method(const rb_execution_context_t *ec, VALUE obj, ID id, VALUE iseqv
rb_add_method_iseq(klass, id, (const rb_iseq_t *)iseqval, cref, visi); rb_add_method_iseq(klass, id, (const rb_iseq_t *)iseqval, cref, visi);
// Set max_iv_count on klasses based on number of ivar sets that are in the initialize method // Set max_iv_count on klasses based on number of ivar sets that are in the initialize method
if (id == rb_intern("initialize") && klass != rb_cObject && RB_TYPE_P(klass, T_CLASS) && (rb_get_alloc_func(klass) == rb_class_allocate_instance)) { if (id == idInitialize && klass != rb_cObject && RB_TYPE_P(klass, T_CLASS) && (rb_get_alloc_func(klass) == rb_class_allocate_instance)) {
RCLASS_EXT(klass)->max_iv_count = rb_estimate_iv_count(klass, (const rb_iseq_t *)iseqval); RCLASS_EXT(klass)->max_iv_count = rb_estimate_iv_count(klass, (const rb_iseq_t *)iseqval);
} }