p_css 0.1.7 → 0.1.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 99fe89569a8cb686b5fa018e7403b55ba6603a46ccad4a8df4821135579b6ada
-  data.tar.gz: 8474073401ef004e365a6e294dcf35c92338ad6c4688168e999fccecac6a95e5
+  metadata.gz: 7c7c770667eefbfabcfd7543684970c971588c7d31ef7eaa8dfde8dfc71382ec
+  data.tar.gz: fe5c819bb338d0417fed4e15fbf85f9c8a158038a0fd9d17e78e308c2e1c949f
 SHA512:
-  metadata.gz: 7c79fcc55da6b57149893332f9d491912f66d04d8a2f6d6ffef9f57fb30872ecf22b00b4184730088fbcc92639bf7d55204c075147b78174f2f5e48d057f9ba8
-  data.tar.gz: efb733700697de31e950e90b35f10597d22975c3e0ae64ed8575474c0da175bd80bed92c39eedd3a3154560a0b7a604c79eeb3cb0ccec3b785429d7705918346
+  metadata.gz: cf33855007bc4c7691d7a9c726eb51f85215492643ec54c2510438a4d10da5afefa176cc062c8eab5774b4a86a23049c5b7b5f64f121682581f34aceb826870d
+  data.tar.gz: c72b82737b2c24a7585022da646135f743dfc9a59dc918267482f57ae2e68cd79dcfd6da106a14b6962ac2fd325ab09032a40e59148191d4146b08cc057ef37f

data/lib/css/cascade.rb

@@ -35,8 +35,15 @@ module CSS
     # `state:` opts into stateful-pseudo matching — see
     # `Selectors::Matcher#matches?` for the shape. Defaults to the
     # stateless behavior (`:hover`, `:focus`, etc. never match).
-    def resolve(element, inline_style: nil, state: nil)
-      cache       = {}
+    #
+    # `cache:` lets callers share a per-element context cache across many
+    # resolves. The default `{}` is local to one call. Pass a persistent
+    # Hash when the DOM is stable across many resolves — Context (tag, id,
+    # classes) computation runs once per element instead of per resolve.
+    # The caller is responsible for clearing/replacing the cache on DOM
+    # mutation.
+    def resolve(element, inline_style: nil, state: nil, cache: nil)
+      cache     ||= {}
       candidates  = collect_candidate_indexes(element, cache)
       order       = 0
       matches     = []

data/lib/css/selectors/matcher.rb

@@ -43,7 +43,7 @@ module CSS
       # the duration of a single matcher invocation.
       Context = Data.define(:tag, :id, :classes)
 
-      EMPTY_CLASS_SET = Set.new.freeze
+      EMPTY_CLASSES = [].freeze
 
       def matches?(element, selector, cache: nil, state: nil)
         sel = selector.is_a?(String) ? Parser.parse_selector_list(selector) : selector
@@ -148,11 +148,15 @@ module CSS
         )
       end
 
+      # Returns an Array of class names. We deliberately don't wrap in a Set:
+      # construction allocates two objects (Array + Set), and on the typical
+      # 1–5 classes per element, Array#include? is fast enough that the
+      # construction win dominates the lookup penalty.
       def build_class_set(element)
         v = attr(element, 'class')
-        return EMPTY_CLASS_SET if v.nil? || v.empty?
+        return EMPTY_CLASSES if v.nil? || v.empty?
 
-        v.to_s.split(/\s+/).to_set
+        v.to_s.split(' ')
       end
 
       # Attribute matching ----------------------------------------------
@@ -453,9 +457,15 @@ module CSS
 
       # Element protocol helpers ---------------------------------------
 
+      # Callers (LINK_TAGS.include?, case statements) compare against
+      # lowercase literals, so the result must be lowercase. But Nokogiri's
+      # HTML parsers already emit lowercase names — the .downcase only fires
+      # in XML / uppercase-tag cases. Skip the allocation when there's
+      # nothing to lower.
       def tag(element)
         name = element.respond_to?(:tag_name) ? element.tag_name : element.name
-        name.to_s.downcase
+        name = name.to_s
+        name.match?(/[A-Z]/) ? name.downcase : name
       end
 
       def attr(element, name)

data/lib/css/version.rb

@@ -1,3 +1,3 @@
 module CSS
-  VERSION = '0.1.7'
+  VERSION = '0.1.9'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: p_css
 version: !ruby/object:Gem::Version
-  version: 0.1.7
+  version: 0.1.9
 platform: ruby
 authors:
 - Keita Urashima