leptris 1.9.311.2 → 1.9.311.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9b6ce086c202566aa39cb833bb1033abd8f734c187fced742f7fb5f0f35bbadb
-  data.tar.gz: 3ef07e258734ab68cb1468be51c990bd882290dd7708aaf6b868bb065c67bad1
+  metadata.gz: a004384f7927ff6e71fd0277c1920ca6890a69c9dee52f36f267c85807cf8b0b
+  data.tar.gz: a02220a1a36d2016d947f35065d204a92811fc99559b8e5d596f0c5278d4089a
 SHA512:
-  metadata.gz: 0af5534cc38131cec871844ad89d1f57c387e7257a5e3bd4aa688fe77e3f2e10521a8a3f0461d01b27b098c0f4b55e2c81ecfd7a232380b9392032caaa6d4197
-  data.tar.gz: c6f47da5b6332b3b518a98afc1856a449d1443a72cb96a0eb02adb8deac348f4b79467f0300ab5171f5d9a25727da217e1a22e7b05c89476a9ec92cce689a2a8
+  metadata.gz: 7f599ca78c88649f4778c81644947dc939fcbfe2db06179f4b7785d62bb46ce3ea4fc86ccf6b589d5f83b06ea1ab3e0686a1427fba31ad27ca7b67c39b6993a2
+  data.tar.gz: d1030e40b0903052901e5a245b17531f44ad7ee718daefc303644e7bebc630debea0594bd930273fdcaabe0c9745951263965706e54fdf5ffb583f2bc58bd4e1

data/CHANGELOG.md

@@ -1,3 +1,21 @@
+## [1.9.311.3] - 2026-10-06
+
+### Performance
+
+- **Same-document splices stop paying the cross-document sweep
+  (#374, the fresh-doc 0.80x follow-up)**: every fast splice path
+  is reached only after `foreign_to?` said no — the child's
+  document IS this one — so the post-splice
+  `invalidate_cross_document!` (a method call plus two ivar reads
+  that always returned at the equal check) was dead weight on the
+  hottest construction loop. Removed from the add_child /
+  prepend_child / sibling-insert fast paths and their FFI tails
+  (the foreign and batch paths keep their sweeps); `add_child` no
+  longer runs the Ruby-side readonly/version gate twice when the
+  C face self-gates. Measured under load with interleaved
+  CPU-time medians (3 rounds, 15k appends): add_child 1.69 →
+  1.41 µs/call (~16%), programmatic build ~14%.
+
 ## [1.9.311.2] - 2026-10-06
 
 ### Fixed

data/lib/leptris/version.rb

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

data/lib/leptris/xml/element.rb

@@ -446,7 +446,6 @@ class Leptris::XML::Element < Leptris::XML::Node
     if native_fast_children?
       unless Leptris::XML::Native.insert_binding_child(
         @document, @c_address, node.c_ptr.address, 1).nil?
-        Leptris::XML::Node.invalidate_cross_document!(node, @document)
         return node
       end
     end
@@ -456,7 +455,6 @@ class Leptris::XML::Element < Leptris::XML::Node
     end
     Leptris::XML::FFI.check_status(
       Leptris::XML::FFI.leptris_element_prepend_child(c_ptr, node.c_ptr))
-    Leptris::XML::Node.invalidate_cross_document!(node, @document)
     node
   end
 
@@ -481,7 +479,6 @@ class Leptris::XML::Element < Leptris::XML::Node
     if native_fast_children?
       unless Leptris::XML::Native.insert_binding_child(
         @document, @c_address, node.c_ptr.address, 2).nil?
-        Leptris::XML::Node.invalidate_cross_document!(node, @document)
         return node
       end
     end
@@ -491,7 +488,6 @@ class Leptris::XML::Element < Leptris::XML::Node
     end
     Leptris::XML::FFI.check_status(
       Leptris::XML::FFI.leptris_element_insert_after(c_ptr, node.c_ptr))
-    Leptris::XML::Node.invalidate_cross_document!(node, @document)
     node
   end
 
@@ -516,7 +512,6 @@ class Leptris::XML::Element < Leptris::XML::Node
     if native_fast_children?
       unless Leptris::XML::Native.insert_binding_child(
         @document, @c_address, node.c_ptr.address, 3).nil?
-        Leptris::XML::Node.invalidate_cross_document!(node, @document)
         return node
       end
     end
@@ -526,7 +521,6 @@ class Leptris::XML::Element < Leptris::XML::Node
     end
     Leptris::XML::FFI.check_status(
       Leptris::XML::FFI.leptris_element_insert_before(c_ptr, node.c_ptr))
-    Leptris::XML::Node.invalidate_cross_document!(node, @document)
     node
   end
 
@@ -672,7 +666,6 @@ class Leptris::XML::Element < Leptris::XML::Node
   end
 
   def add_child(node_or_markup)
-    ensure_writable!
     case node_or_markup
     when Leptris::XML::Node
       # Foreign-child adoption (1.9.304, #1528): the engine
@@ -686,22 +679,24 @@ class Leptris::XML::Element < Leptris::XML::Node
       # readonly/liveness gates, the provable no-op lift
       # predicate, the version bump, and the engine append. Qnil
       # means the child needs the namespace lift — fall through
-      # to the full path.
+      # to the full path. Same-document children cannot reach
+      # here from another document (foreign ones routed above),
+      # so there is no cross-document sweep to run (#374): the
+      # C-side version bump invalidates this document's memos.
       if native_fast_children?
         # The face raises on failure (TODO.perf/36); Qnil = the
         # child needs the lift — the full path handles it.
         unless Leptris::XML::Native.append_binding_child(
           @document, @c_address, node_or_markup.c_ptr.address).nil?
-          Leptris::XML::Node.invalidate_cross_document!(node_or_markup, @document)
           return node_or_markup
         end
       end
+      ensure_writable!
       unless Leptris::XML::Element.skip_adoption_lift?(node_or_markup)
         Leptris::XML::Element.lift_namespaces_for_adoption(node_or_markup, namespaces)
       end
       Leptris::XML::FFI.check_status(
         Leptris::XML::FFI.leptris_element_append_child(c_ptr, node_or_markup.c_ptr))
-      Leptris::XML::Node.invalidate_cross_document!(node_or_markup, @document)
       node_or_markup
     when String
       # TODO.perf/34: one dispatch parses the markup and appends
@@ -721,6 +716,7 @@ class Leptris::XML::Element < Leptris::XML::Node
         # path (it re-fails fast)
         Leptris::XML::DocumentFragment.parse(node_or_markup, @document)
       end
+      ensure_writable!
       frag = Leptris::XML::DocumentFragment.parse(node_or_markup, @document)
       added = []
       frag.children.each do |n|
@@ -741,6 +737,7 @@ class Leptris::XML::Element < Leptris::XML::Node
   # original is removed after (the source loses the node, as
   # every pre-304 caller observed).
   def append_installed(node)
+    ensure_writable!
     unless Leptris::XML::Element.skip_adoption_lift?(node)
       Leptris::XML::Element.lift_namespaces_for_adoption(node, namespaces)
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: leptris
 version: !ruby/object:Gem::Version
-  version: 1.9.311.2
+  version: 1.9.311.3
 platform: ruby
 authors:
 - Ribose Inc.