leptris 1.9.222.2-x86_64-linux → 1.9.222.3-x86_64-linux

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1a2572ad1d071c50c78534599f7fd5085eb10440ee977adfcde86fdeb54124e5
-  data.tar.gz: 8de31d649e8716712d6b323d5424cb95963a8360fad615a401da1b607fa6d993
+  metadata.gz: c559bcf319459f78dc97a3d2c758e0465c2c0d21dc02b69a4f379661462b40d4
+  data.tar.gz: bce492a2a2d6de48566c781031826039fa897ac01ec689d55cee6ca63fb473c9
 SHA512:
-  metadata.gz: '0209e0b311b3811bea6948f8d36dfa0053b63efd117fe76b3eb471c3182dca8acead4de51a679a7273900ebd8ed3a0c81189abe3849c2896b9885ecba12c6aea'
-  data.tar.gz: 61ae26f2a6e027e7cb2eec1326a24b1f722e158ead86db7496f6e27f28cdb1e7083105906e418f358ba04a4c13e07b154b872c2ac7f1055cb7087e36c220cb25
+  metadata.gz: '019ca5f276d428d6ad5c043cc9fdcacf566cc7f67cd33a795af6f5d2045e45bf305badc2878833293475b164c7b7598a7206aa30cd41b65d4f8fabd5bd67979c'
+  data.tar.gz: 7191eb516af67747c5e2a9b634e4d625e8dfeec70ad264e02b2870d4d36beb0e64c2ecdb4773e16859eac488ded5cbcd23aff359208d206412c104616c8a5d1f

data/ext/leptris/native/native.c

@@ -135,6 +135,9 @@ static VALUE c_iteration_scope;
 static VALUE c_leptris_error;
 static VALUE c_b_document, c_b_freed, c_ffi_pointer;
 static ID id_ptr_new, id_freed_new, id_alive;
+static ID id_iv_c_address, id_iv_document, id_iv_parent;
+static ID id_iv_structure_memoizable, id_iv_native_fast;
+static ID id_iv_pub_document, id_iv_addr_reads_fast, id_iv_node_type;
 static void check_status_c(int st);
 
 #define NT_ELEMENT 0
@@ -839,8 +842,38 @@ struct trav_state {
     VALUE err;
     void *self_ptr;
     int for_visit;
+    /* Hoisted per-walk state (was re-resolved per node): the
+     * document's wrapper cache and the iteration-scope class
+     * check. #312 item 1. */
+    VALUE cache;
+    VALUE klass_memo_flag;
 };
 
+/* Construct-or-fetch the binding wrapper for node_ptr against the
+ * per-walk hoisted cache (#312 item 1): static-ID ivar writes (no
+ * per-call string interning), the scope class check precomputed. */
+static VALUE trav_wrapper_for(struct trav_state *st, void *node_ptr,
+                              int kind)
+{
+    VALUE key = ULL2NUM((uint64_t)(uintptr_t)node_ptr);
+    VALUE node = rb_hash_aref(st->cache, key);
+    if (!NIL_P(node)) return node;
+
+    node = rb_obj_alloc(binding_klass_for(kind));
+    rb_ivar_set(node, id_iv_c_address, key);
+    rb_ivar_set(node, id_iv_document, st->document);
+    rb_ivar_set(node, id_iv_parent, Qnil);
+    VALUE fast = st->klass_memo_flag;
+    rb_ivar_set(node, id_iv_structure_memoizable, fast);
+    rb_ivar_set(node, id_iv_native_fast, fast);
+    rb_ivar_set(node, id_iv_pub_document,
+                fast == Qtrue ? st->document : Qnil);
+    rb_ivar_set(node, id_iv_addr_reads_fast, Qtrue);
+    rb_ivar_set(node, id_iv_node_type, INT2FIX(kind));
+    rb_hash_aset(st->cache, key, node);
+    return node;
+}
+
 static VALUE trav_yield_one(VALUE arg)
 {
     return rb_yield(arg);
@@ -849,33 +882,13 @@ static VALUE trav_yield_one(VALUE arg)
 static int trav_cb(void *node_ptr, void *user)
 {
     struct trav_state *st = user;
-    VALUE cache, key, node;
+    VALUE node;
     int kind, state;
 
     if (st->err != Qnil)
         return 1;
     kind = f_node_type(node_ptr);
-    cache = binding_cache_of(st->document);
-    key = ULL2NUM((uint64_t)(uintptr_t)node_ptr);
-    node = rb_hash_aref(cache, key);
-    if (NIL_P(node)) {
-        node = rb_obj_alloc(binding_klass_for(kind));
-        rb_iv_set(node, "@c_address", key);
-        rb_iv_set(node, "@document", st->document);
-        rb_iv_set(node, "@parent", Qnil);
-        if (rb_obj_class(st->document) == c_iteration_scope) {
-            rb_iv_set(node, "@structure_memoizable", Qfalse);
-            rb_iv_set(node, "@native_fast", Qfalse);
-            rb_iv_set(node, "@pub_document", Qnil);
-        } else {
-            rb_iv_set(node, "@structure_memoizable", Qtrue);
-            rb_iv_set(node, "@native_fast", Qtrue);
-            rb_iv_set(node, "@pub_document", st->document);
-        }
-        rb_iv_set(node, "@addr_reads_fast", Qtrue);
-        rb_iv_set(node, "@node_type", INT2FIX(kind));
-        rb_hash_aset(cache, key, node);
-    }
+    node = trav_wrapper_for(st, node_ptr, kind);
     rb_protect(trav_yield_one, node, &state);
     if (state) {
         st->err = rb_errinfo();
@@ -894,6 +907,9 @@ static VALUE nf_traverse_binding(VALUE self, VALUE document,
     st.document = document;
     st.err = Qnil;
     st.self_ptr = (void *)(uintptr_t)NUM2ULL(addr);
+    st.cache = binding_cache_of(document);
+    st.klass_memo_flag =
+        rb_obj_class(document) == c_iteration_scope ? Qfalse : Qtrue;
     st.for_visit = 0;
     f_node_traverse(st.self_ptr, 1 /* TRAVERSE_POST_ORDER */,
                     trav_cb, &st);
@@ -922,33 +938,13 @@ static void visit_cb(void *user, void *node_ptr, int entering,
      * happens after the walk, same observable outcome. */
     struct trav_state *st = user;
     struct visit_yield_args va;
-    VALUE cache, key, node;
+    VALUE node;
     int kind, state;
 
     if (st->err != Qnil)
         return;
     kind = f_node_type(node_ptr);
-    cache = binding_cache_of(st->document);
-    key = ULL2NUM((uint64_t)(uintptr_t)node_ptr);
-    node = rb_hash_aref(cache, key);
-    if (NIL_P(node)) {
-        node = rb_obj_alloc(binding_klass_for(kind));
-        rb_iv_set(node, "@c_address", key);
-        rb_iv_set(node, "@document", st->document);
-        rb_iv_set(node, "@parent", Qnil);
-        if (rb_obj_class(st->document) == c_iteration_scope) {
-            rb_iv_set(node, "@structure_memoizable", Qfalse);
-            rb_iv_set(node, "@native_fast", Qfalse);
-            rb_iv_set(node, "@pub_document", Qnil);
-        } else {
-            rb_iv_set(node, "@structure_memoizable", Qtrue);
-            rb_iv_set(node, "@native_fast", Qtrue);
-            rb_iv_set(node, "@pub_document", st->document);
-        }
-        rb_iv_set(node, "@addr_reads_fast", Qtrue);
-        rb_iv_set(node, "@node_type", INT2FIX(kind));
-        rb_hash_aset(cache, key, node);
-    }
+    node = trav_wrapper_for(st, node_ptr, kind);
     va.node = node;
     va.entering = entering ? Qtrue : Qfalse;
     va.depth = INT2NUM(depth);
@@ -966,6 +962,9 @@ static VALUE nf_visit_binding(VALUE self, VALUE document, VALUE addr)
     st.document = document;
     st.err = Qnil;
     st.self_ptr = (void *)(uintptr_t)NUM2ULL(addr);
+    st.cache = binding_cache_of(document);
+    st.klass_memo_flag =
+        rb_obj_class(document) == c_iteration_scope ? Qfalse : Qtrue;
     st.for_visit = 1;
     f_node_visit(st.self_ptr, visit_cb, &st);
     if (st.err != Qnil)
@@ -2648,6 +2647,14 @@ LEPTRIS_INIT_EXPORT void Init_native(void)
     id_iv_c_address = rb_intern("@c_address");
     id_iv_native_cache = rb_intern("@native_cache");
     id_iv_binding_cache = rb_intern("@wrapper_cache");
+    id_iv_c_address = rb_intern("@c_address");
+    id_iv_document = rb_intern("@document");
+    id_iv_parent = rb_intern("@parent");
+    id_iv_structure_memoizable = rb_intern("@structure_memoizable");
+    id_iv_native_fast = rb_intern("@native_fast");
+    id_iv_pub_document = rb_intern("@pub_document");
+    id_iv_addr_reads_fast = rb_intern("@addr_reads_fast");
+    id_iv_node_type = rb_intern("@node_type");
     id_iv_version = rb_intern("@version");
     id_iv_readonly = rb_intern("@readonly");
     id_address = rb_intern("address");

data/lib/leptris/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Leptris
-  VERSION = "1.9.222.2"
+  VERSION = "1.9.222.3"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: leptris
 version: !ruby/object:Gem::Version
-  version: 1.9.222.2
+  version: 1.9.222.3
 platform: x86_64-linux
 authors:
 - Ribose Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2026-09-22 00:00:00.000000000 Z
+date: 2026-09-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ffi