leptris 1.9.16 → 1.9.17

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 85d05772181a6d3c913f331b48f900f0781596428308759b12ac07d6cdc392d5
-  data.tar.gz: 2e267b1225848fff3d265d0b810df576d3114e84e2a1627aacb143e650340a59
+  metadata.gz: 790fde1b932902a17438b2abed525393ebb61464a61ff35081384409277a42cc
+  data.tar.gz: 356b2a9a9426c03a25c42577952f3634127162e449e3540816b99a8bf6cd354f
 SHA512:
-  metadata.gz: 959165b993b24b641b64246799168ef0d53d2180866596996069c28adabbf7305d1d78600af0eb25bf7bccac8a8053f6a0714d97315df61fb032fd4fcb2ef4ac
-  data.tar.gz: 2a41355f33fcfefff1ee6b8aa2274a1d5464a8f567f611c7035df328eae3620f5b74018cea500f9bae4240593d4e8f9b9e263b17b32573e13cfcd81f4d2cd847
+  metadata.gz: 8fdfd7ed098ae1501040d5f82808c57813f5f2f8dec606d6ac966bd916cff635dd510ebf68e235a4a3e9c0da6a5a10cc9765ffe20eb06a476876dc2cf6dbc2fc
+  data.tar.gz: 26142e64a808758e72332fa8aaef029b7fee19c82c3f634f6f0ec9ba7afe7990261f958173a124f2f89ced5270c253f53256f4b12eafc99235a066e38c5b7f3a

data/CHANGELOG.md

@@ -5,6 +5,22 @@ All notable changes to Leptris will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.9.17] - 2026-08-27
+
+### Changed
+
+- **Element-hinted batch materialization**: the XPath result-set
+  batch (`leptris_xpath_result_get_nodes_ex`) fills an out_kinds
+  array the binding discarded; `Node.wrap` now accepts a known
+  `node_type:` so element entries skip the per-node get_type
+  dispatch. ELEMENT is the only hintable value — XPath's data
+  model reports CDATA as TEXT, so a TEXT hint cannot distinguish
+  Text from CDATA (different content getters); comments/CDATA/PIs
+  keep the get_type fallback. Cold 500-element materialization:
+  0.131 s -> 0.110 s (~15%). Upstream note: a full-type out_kinds
+  (or kinds on `leptris_node_children`) belongs to the #560/#562
+  ask family.
+
 ## [1.9.16] - 2026-08-27
 
 ### Changed

data/lib/leptris/version.rb

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

data/lib/leptris/xml/ffi.rb

@@ -747,14 +747,27 @@ module Leptris
       end
 
       # XPath result-set batch (leptris_xpath_result_get_nodes_ex):
-      # copies all node kinds into a caller buffer and hands back
-      # the pointer array.
+      # the call fills out_kinds in the 4-value XPATH_NODE space
+      # (ELEMENT/ATTRIBUTE/TEXT/OTHER). ELEMENT is the only value
+      # that maps unambiguously — XPath's data model reports CDATA
+      # as TEXT, so a TEXT hint cannot distinguish Text from CDATA
+      # (different content getters) — everything else falls back to
+      # get_type in the wrapper. Returns [pointers, element_hints].
+      XPATH_KIND_HINT = { XPATH_NODE_ELEMENT => NODE_ELEMENT }.freeze
+
       def self.fetch_result_nodes(result_ptr, count)
-        return [] if count.zero?
+        return [[], nil] if count.zero?
         with_pointer_buffer(count) do |buffer|
-          copied = leptris_xpath_result_get_nodes_ex(
-            result_ptr, buffer, nil, count)
-          buffer.get_array_of_pointer(0, copied)
+          kinds = ::FFI::MemoryPointer.new(:int, count)
+          begin
+            copied = leptris_xpath_result_get_nodes_ex(
+              result_ptr, buffer, kinds, count)
+            hints = kinds.get_array_of_int(0, copied)
+              .map { |k| XPATH_KIND_HINT[k] }
+            [buffer.get_array_of_pointer(0, copied), hints]
+          ensure
+            kinds.free
+          end
         end
       end
 

data/lib/leptris/xml/node.rb

@@ -12,7 +12,10 @@ class Leptris::XML::Node
     @node_type = node_type
   end
 
-  def self.wrap(c_ptr, document, parent: nil)
+  # node_type: callers holding a batch-fetched kind (the XPath
+  # result-set batch fills out_kinds) pass it so the wrap skips the
+  # get_type dispatch; nil (the default) dispatches as before.
+  def self.wrap(c_ptr, document, parent: nil, node_type: nil)
     # Per-document weak-ref cache. Returns the existing wrapper when the
     # same c_ptr is wrapped twice (common in children/sibling walks,
     # repeated xpath queries, traverse-then-access patterns). The cache
@@ -21,7 +24,7 @@ class Leptris::XML::Node
       return cached
     end
 
-    node_type = Leptris::XML::FFI.leptris_node_get_type(c_ptr)
+    node_type ||= Leptris::XML::FFI.leptris_node_get_type(c_ptr)
     node =
       case node_type
       when Leptris::XML::FFI::NODE_ELEMENT

data/lib/leptris/xml/node_set.rb

@@ -81,11 +81,11 @@ class Leptris::XML::NodeSet
         # not just elements); the batch accessor under-copies
         # mixed-kind results (upstream leptris#477), so per-index
         # fetch covers the remainder.
-        pointers = Leptris::XML::FFI.fetch_result_nodes(@result_ptr, n)
+        pointers, kinds = Leptris::XML::FFI.fetch_result_nodes(@result_ptr, n)
         nodes = []
-        pointers.each do |ptr|
+        pointers.each_with_index do |ptr, i|
           next if ptr.null?
-          nodes << Leptris::XML::Node.wrap(ptr, @document)
+          nodes << Leptris::XML::Node.wrap(ptr, @document, node_type: kinds[i])
         end
         (pointers.length...n).each do |i|
           ptr = Leptris::XML::FFI.leptris_xpath_result_get_node(@result_ptr, i)

metadata

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