canon 0.3.20 → 0.3.21

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 581a483d81eba304f20fefc4e14777efced86bd7b829d1181209bee00c3ac7df
-  data.tar.gz: ea8929a3686f268e5c99376a0aded4999714b298fe7a558c2ac713929fe3d6ac
+  metadata.gz: fd899c17d1512c815d189076fe08721dbe66b7d964f8bac69abde35f207b8c76
+  data.tar.gz: 205ad0b365288bf883f2ce57f68b612217151e30ae90fb0f959b29f252063826
 SHA512:
-  metadata.gz: 19d37bd2c6fec3949791787ca2a64e24afce11588bf17d335efa81fba225d945d77e578a2eb8dae0b8df48904f0f824830f43018cd78830448e05da127498f99
-  data.tar.gz: d3c3f990c6a133cb9cada5dbfae4bb752f524c972f3b75253c24c71226e7ad86138467559e533611b20beb95520f4441bfb80fafd4648bacf5b63e0717a69e70
+  metadata.gz: 5aa4d552592cda5a55417415efb225151eb0d39d899fe0471eb26f10ceeca64ae56de786d8e807e0085264ba800cfd5937a6ec15e182e71957381414f3036105
+  data.tar.gz: b52ceb55e3ca6c6affcca3c7091a0e3f6e22d1c077b65732e7073915c3f6914cb86a61e675b11636baeb1ea09e94c223c933e961c0b366fb3e621c592a50170b

data/lib/canon/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Canon
-  VERSION = "0.3.20"
+  VERSION = "0.3.21"
 end

data/lib/canon/xml/character_encoder.rb

@@ -5,10 +5,18 @@ module Canon
     # Character encoder for C14N 1.1
     # Handles UTF-8 encoding and character reference encoding per spec
     class CharacterEncoder
+      # Most text and attribute values contain nothing to escape —
+      # the zero-allocation guard keeps those from paying the gsub
+      # copy on every render.
+      TEXT_ESCAPABLE = /[&<>\r]/
+      ATTRIBUTE_ESCAPABLE = /[&<"\t\n\r]/
+
       # Encode text node content
       # Replace: & → &amp;, < → &lt;, > → &gt;, #xD → &#xD;
       def encode_text(text)
-        text.gsub(/[&<>\r]/) do |char|
+        return text unless text.match?(TEXT_ESCAPABLE)
+
+        text.gsub(TEXT_ESCAPABLE) do |char|
           case char
           when "&" then "&amp;"
           when "<" then "&lt;"
@@ -22,7 +30,9 @@ module Canon
       # Replace: & → &amp;, < → &lt;, " → &quot;,
       #          #x9 → &#x9;, #xA → &#xA;, #xD → &#xD;
       def encode_attribute(value)
-        value.gsub(/[&<"\t\n\r]/) do |char|
+        return value unless value.match?(ATTRIBUTE_ESCAPABLE)
+
+        value.gsub(ATTRIBUTE_ESCAPABLE) do |char|
           case char
           when "&" then "&amp;"
           when "<" then "&lt;"

data/lib/canon/xml/namespace_handler.rb

@@ -14,7 +14,7 @@ module Canon
       def process_namespaces(element, output, parent_element = nil)
         return unless element.in_node_set?
 
-        namespaces = element.sorted_namespace_nodes.select(&:in_node_set?)
+        namespaces = renderable_namespaces(element)
 
         # Check if we need to emit xmlns="" for empty default namespace
         if should_emit_empty_default_namespace?(element, namespaces,
@@ -61,26 +61,45 @@ module Canon
       end
 
       # Check if a namespace node should be skipped
-      def should_skip_namespace?(ns, _element, parent_element)
+      def should_skip_namespace?(ns, element, parent_element)
         # Skip xml namespace with standard URI
         return true if ns.xml_namespace?
 
         # Skip if an ancestor already declared this namespace
-        return true if namespace_declared_by_ancestor?(ns, parent_element)
+        return true if namespace_declared_by_ancestor?(ns, element,
+                                                       parent_element)
 
         false
       end
 
       # Check if a namespace is already declared by an ancestor
-      def namespace_declared_by_ancestor?(ns, parent_element)
+      def namespace_declared_by_ancestor?(ns, element, parent_element)
         return false unless parent_element
 
-        parent_ns = parent_element.namespace_nodes.find do |parent_ns|
+        parent_nodes = parent_element.namespace_nodes
+        # Elements that declared nothing share their parent's scope
+        # object (TreeBuilder scope sharing), so identical arrays mean
+        # identical bindings — every namespace of this element is
+        # already declared above and will be skipped without a scan.
+        return true if parent_nodes.equal?(element.namespace_nodes)
+
+        parent_ns = parent_nodes.find do |parent_ns|
           parent_ns.prefix == ns.prefix && parent_ns.in_node_set?
         end
 
         parent_ns && parent_ns.uri == ns.uri
       end
+
+      # Sorted, in-node-set namespaces for one element. Undeclaring
+      # elements share one frozen array object, and node-set marking is
+      # fixed before processing runs, so the rendered form is cached
+      # per array identity for the lifetime of this handler (one
+      # canonicalization run).
+      def renderable_namespaces(element)
+        cache = (@renderable_namespaces ||= {}.compare_by_identity)
+        nodes = element.namespace_nodes
+        cache[nodes] ||= nodes.sort_by(&:local_name).select(&:in_node_set?)
+      end
     end
   end
 end

data/lib/canon/xml/nodes/element_node.rb

@@ -53,15 +53,26 @@ module Canon
           (@attribute_nodes ||= []) << attribute_node
         end
 
-        # Get namespace nodes in sorted order (lexicographically by local name)
+        # Get namespace nodes in sorted order (lexicographically by local name).
         def sorted_namespace_nodes
-          namespace_nodes.sort_by(&:local_name)
+          nodes = namespace_nodes
+          return nodes if nodes.empty?
+
+          nodes.sort_by(&:local_name)
         end
 
-        # Get attribute nodes in sorted order (by namespace URI then local name)
+        # Get attribute nodes in sorted order (by namespace URI then local
+        # name). A comparator block instead of sort_by keys allocates no
+        # per-attribute key arrays; (uri, name) pairs are unique per
+        # element (duplicates are resolved at build), so the order is
+        # identical.
         def sorted_attribute_nodes
-          attribute_nodes.sort_by do |attr|
-            [attr.namespace_uri.to_s, attr.local_name]
+          attrs = attribute_nodes
+          return attrs if attrs.empty?
+
+          attrs.sort do |a, b|
+            (a.namespace_uri.to_s <=> b.namespace_uri.to_s).nonzero? ||
+              (a.local_name <=> b.local_name)
           end
         end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: canon
 version: !ruby/object:Gem::Version
-  version: 0.3.20
+  version: 0.3.21
 platform: ruby
 authors:
 - Ribose Inc.