moxml 0.5.20 → 0.5.21

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 59c8df5982f2b80280386a94385ceed9a65b1b80524772d914e19052a564aedd
-  data.tar.gz: 3e3432a24b65e60bddf345955d4454a5a459f4898693fa9037b71e8a052a6115
+  metadata.gz: efd15ef624c00bcb6215f7a120272833f45f4c0f297d2a98b19507540878eec8
+  data.tar.gz: 640713084bd178087975387abca9cec4a681c2e94ce905ac16becb5514e25ca2
 SHA512:
-  metadata.gz: 3c041e77c6230d320e5a2779f08ffe29505780d4eac520ce1c12c83d5532102153787b592d999067daef2aabfedf08c5a1830d374baa224a9d795e20816faf69
-  data.tar.gz: 54367812683befdea88202d68a7c15c14b91184e3d350cef9510d94a8e6c769d307f311282041d2816626de181697a8f78b429cb7e71ae2b2d11638d34d0a1f9
+  metadata.gz: 2d418bb84689e70721a4e9b16359136e94d781d7da0bc2c9b7374f58ff911d08fff0eb874a8ed31a6434b9d953381714ad8d1dad5911a6b457b8ae6143a53491
+  data.tar.gz: 6aca55287d55514bd1dfc52c4286909760fb2b9230c33989c21bda4b78d873977ca07ef30df165413d959a42cd7ffa052677aaae672dfe1884f6e84af3b270c5

data/lib/moxml/entity.rb

@@ -43,6 +43,12 @@ module Moxml
     SERIALIZED_MARKER_RE = /(#{NAME_PATTERN});/
     STANDARD_ENTITIES = %w[amp lt gt quot apos].freeze
 
+    # One regex pass, no allocations: true when a NON-standard named
+    # entity exists somewhere. Documents carrying only the five
+    # predefined entities (the overwhelmingly common case) skip the
+    # marker gsub's full-buffer copy entirely.
+    NON_STANDARD_ENTITY_RE = /&(?!amp;|lt;|gt;|quot;|apos;)(#{NAME_PATTERN});/
+
     NAMED_DECODE_MAP = {
       "amp" => "&", "lt" => "<", "gt" => ">",
       "quot" => '"', "apos" => "'"
@@ -82,6 +88,10 @@ module Moxml
       # the regex scan and string allocation entirely. The vast
       # majority of XML payloads contain no entity references.
       return [str, false] unless str.include?("&")
+      # Second fast path: only predefined entities — the gsub would
+      # pass every match through unchanged, so skip its full-buffer
+      # copy too.
+      return [str, false] unless str.match?(NON_STANDARD_ENTITY_RE)
 
       marked = false
       processed = str.gsub(NAME_RE) do |match|

data/lib/moxml/node.rb

@@ -339,10 +339,18 @@ module Moxml
 
     TYPES.each do |node_type|
       define_method "#{node_type}?" do
-        adapter.node_type(native) == node_type
+        node_type_cached == node_type
       end
     end
 
+    # The adapter's type probe is an FFI call per invocation; the
+    # type is fixed for a node's lifetime, so the first answer is
+    # memoized on the wrapper.
+    def node_type_cached
+      @node_type_cached ||= adapter.node_type(@native)
+    end
+    private :node_type_cached
+
     # Returns the primary identifier for this node type
     # For Element: the tag name
     # For Attribute: the attribute name
@@ -392,7 +400,9 @@ module Moxml
     protected
 
     def adapter
-      context.config.adapter
+      # A context's adapter object is fixed for its lifetime; the
+      # chain deref ran on every node access.
+      @adapter ||= context.config.adapter
     end
 
     def self.adapter(context)

data/lib/moxml/node_set.rb

@@ -23,9 +23,16 @@ module Moxml
     def each
       return to_enum(:each) unless block_given?
 
-      @nodes.each_with_index do |node, i|
-        @wrapped[i] ||= wrap_with_parent(node)
-        yield @wrapped[i]
+      wrapped = @wrapped
+      index = 0
+      @nodes.each do |node|
+        wrapper = wrapped[index]
+        unless wrapper
+          wrapper = wrap_with_parent(node)
+          wrapped[index] = wrapper
+        end
+        yield wrapper
+        index += 1
       end
       self
     end

data/lib/moxml/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Moxml
-  VERSION = "0.5.20"
+  VERSION = "0.5.21"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: moxml
 version: !ruby/object:Gem::Version
-  version: 0.5.20
+  version: 0.5.21
 platform: ruby
 authors:
 - Ribose Inc.