Re: [PATCH] struct.c: speedup for big structs

[Eric Wong <e@80x24.org>]

Eric Wong <e@80x24.org> wrote:
> use simple custom open-addressing hash for big structs.
> 
> Thanks to Sokolov Yura aka funny_falcon <funny.falcon@gmail.com>
> in https://bugs.ruby-lang.org/issues/10585

Will squash the following when I commit.
Freezing __members__ is necessary, but
RB_GC_GUARD isn't needed since __members__ already becomes an ivar.

diff --git a/struct.c b/struct.c
index fbde80d..eda5673 100644
--- a/struct.c
+++ b/struct.c
@@ -84,12 +84,13 @@ struct_member_pos_probe(long prev, long mask)
     return (prev * 5 + 2) & mask;
 }
 
-static void
+static VALUE
 struct_set_members(VALUE klass, VALUE members)
 {
     VALUE back;
 
     members = rb_ary_dup(members);
+    rb_obj_freeze(members);
 
     if (RARRAY_LEN(members) <= AREF_HASH_THRESHOLD) {
 	back = members;
@@ -121,7 +122,8 @@ struct_set_members(VALUE klass, VALUE members)
     rb_obj_freeze(back);
     rb_ivar_set(klass, id_members, members);
     rb_ivar_set(klass, id_back_members, back);
-    RB_GC_GUARD(members);
+
+    return members;
 }
 
 static inline int
@@ -305,8 +307,7 @@ setup_struct(VALUE nstr, VALUE members)
     const VALUE *ptr_members;
     long i, len;
 
-    OBJ_FREEZE(members);
-    struct_set_members(nstr, members);
+    members = struct_set_members(nstr, members);
 
     rb_define_alloc_func(nstr, struct_alloc);
     rb_define_singleton_method(nstr, "new", rb_class_new_instance, -1);