leptris 1.9.10 → 1.9.11

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4e7dca7e58492747ba0b5843937a7a8a914afa52f4f7271e2b808d1b895a2ded
-  data.tar.gz: bf399063035b7cacd20893813e89f85582594ef7644ab8b2ef076772cc40c094
+  metadata.gz: 4e218d64aa7c30174d4fe7a22e47cc78c35f07cbfe3d0f22430f3ba411651143
+  data.tar.gz: 030eed3a6b5a7387da757ae20406591490a1f55893425d63a443c77631903a6f
 SHA512:
-  metadata.gz: df766bafa6dc7d590909f4f6d2a475446115b5d9cd1d57605558654959e699188f1a36329248c653b501e5e432a9a604533e01dd476e9b350a5510be27fa7ca3
-  data.tar.gz: c0c8f0f164ceadd0c24d476ca8c4948582e091379f7f976a0fdbfc073d482a97c8a80bf29ad6601f2ad474e0fd4464c5e903cb382b56350be2a6c07a1ef0ecf6
+  metadata.gz: 45c707da19fc4114c5f35641b799e104130a8d7c93d8fd365fe72cafec0b894f80e06dba7a73d0109b0fde77868d5db3836035411a3d9951d2f5c3db3c7c8a0f
+  data.tar.gz: '06388d993c6331e3089ad5c9f87056d978096f8bbd56598a1e8abdd37db03575e7d79989f3ed9ef5ba836d58804daaa1039d1b22e2c30c6b1bc4fb59d88fbd85'

data/CHANGELOG.md

@@ -5,6 +5,34 @@ 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.11] - 2026-08-26
+
+### Changed
+
+- **Readonly `Element#[]` serves from the memoized attributes
+  hash** (materializing on demand): repeated reads become hash
+  lookups — no FFI dispatch, no lifetime guard. Completes the
+  readonly contract on its hottest member. Writable documents keep
+  the direct path (values can change).
+- **Materialized NodeSets stop re-batching**: `each`, `[]`, and
+  `length` consult the materialized array first; iterating or
+  indexing after `to_a` no longer re-pays the count + fetch + wrap
+  pass. Repeated iteration measures 2.4x faster.
+- **NodeSet negative indexes are consistent**: `ns[-1]` answers the
+  last element for lazy and eager sets alike (Ruby-Array slice
+  semantics, Nokogiri parity); previously only eager sets
+  supported them.
+- **SAX start_element attributes build pairs in one pass** — no
+  intermediate flat string array or each_slice enumerator per
+  event (~13% on the SAX parse loop).
+
+### Measured and rejected
+
+- A compiled-expression cache for ad-hoc xpath (the round's main
+  hypothesis): compiled vs ad-hoc eval measured within 2.5% on
+  500-member loops — the engine's expression parse is effectively
+  free. Killed before implementation.
+
 ## [1.9.10] - 2026-08-26
 
 ### Changed

data/lib/leptris/version.rb

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

data/lib/leptris/xml/element.rb

@@ -32,6 +32,14 @@ class Leptris::XML::Element < Leptris::XML::Node
   end
 
   def [](key)
+    # Readonly: the attributes hash cannot go stale (ADR 0003) and
+    # materializes on demand — repeated reads become hash lookups,
+    # with no dispatch and no lifetime guard (a hash read cannot
+    # use-after-free).
+    if @document&.readonly?
+      cached = attributes[key.to_s]
+      return cached.nil? ? nil : cached.value
+    end
     ensure_alive!
     Leptris::XML::FFI.leptris_element_attribute(@c_ptr, key.to_s)
   end

data/lib/leptris/xml/node_set.rb

@@ -41,7 +41,8 @@ class Leptris::XML::NodeSet
   end
 
   def length
-    @result_ptr ? Leptris::XML::FFI.leptris_xpath_result_count(@result_ptr) : @array.length
+    return @array.length if @array
+    @result_ptr ? Leptris::XML::FFI.leptris_xpath_result_count(@result_ptr) : 0
   end
   alias_method :size, :length
 
@@ -49,19 +50,26 @@ class Leptris::XML::NodeSet
     length == 0
   end
 
+  # The materialized array (after to_a) is authoritative: every
+  # reader consults it first so a materialized set stops paying the
+  # batch fetch on each iteration or index.
   def [](idx)
+    return @array[idx] if @array
+    # Negative indexes are Ruby-Array semantics (Nokogiri's NodeSet
+    # is slice-like); materialize rather than answer nil
+    # inconsistently with an eager set.
+    return to_a[idx] if idx.negative?
     if @result_ptr
       return nil if idx < 0 || idx >= length
       ptr = Leptris::XML::FFI.leptris_xpath_result_get_node(@result_ptr, idx)
       return nil if ptr.null?
       Leptris::XML::Node.wrap(ptr, @document)
-    else
-      @array[idx]
     end
   end
 
   def each
     return enum_for(:each) unless block_given?
+    return @array.each { |n| yield n } if @array
     if @result_ptr
       # Batch-fetch all node pointers through the FFI seam
       # (leptris_xpath_result_get_nodes_ex copies every node kind,

data/lib/leptris/xml/sax/parser.rb

@@ -148,8 +148,19 @@ class Leptris::XML::SAX::Parser
   end
 
   # The C `const char** attrs` is a NULL-terminated flat array of
-  # name/value pairs; CStringArray owns the format, we slice into pairs.
+  # name/value pairs. Build the [name, value] pairs in one pass —
+  # no intermediate flat string array, no each_slice enumerator.
   def walk_attr_array(attrs_ptr)
-    Leptris::XML::CStringArray.to_ruby(attrs_ptr).each_slice(2).to_a
+    ptr_size = ::FFI.type_size(:pointer)
+    pairs = []
+    i = 0
+    loop do
+      name_ptr = attrs_ptr.get_pointer(i * ptr_size)
+      break if name_ptr.null?
+      value_ptr = attrs_ptr.get_pointer((i + 1) * ptr_size)
+      pairs << [utf8(name_ptr.read_string), utf8(value_ptr.read_string)]
+      i += 2
+    end
+    pairs
   end
 end

metadata

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