perf: Avoid creating an unnecessary string cache entry

Signed-off-by: Dmitry Dygalo <[email protected]>

Author: Stranger6667

CHANGELOG.md

@@ -9,6 +9,7 @@
 ### Performance
 
 - Avoid unnecessary check for double quotes.
+- Avoid creating an unnecessary string cache entry.
 
 ## [0.16.0] - 2025-07-16
 

bindings/c/CHANGELOG.md

@@ -9,6 +9,7 @@
 ### Performance
 
 - Avoid unnecessary check for double quotes.
+- Avoid creating an unnecessary string cache entry.
 
 ## [0.16.0] - 2025-07-16
 

bindings/java/CHANGELOG.md

@@ -9,6 +9,7 @@
 ### Performance
 
 - Avoid unnecessary check for double quotes.
+- Avoid creating an unnecessary string cache entry.
 
 ## [0.16.0] - 2025-07-16
 

bindings/javascript/CHANGELOG.md

@@ -9,6 +9,7 @@
 ### Performance
 
 - Avoid unnecessary check for double quotes.
+- Avoid creating an unnecessary string cache entry.
 
 ## [0.16.0] - 2025-07-16
 

bindings/python/CHANGELOG.md

@@ -9,6 +9,7 @@
 ### Performance
 
 - Avoid unnecessary check for double quotes.
+- Avoid creating an unnecessary string cache entry.
 
 ## [0.16.0] - 2025-07-16
 

bindings/ruby/CHANGELOG.md

@@ -9,6 +9,7 @@
 ### Performance
 
 - Avoid unnecessary check for double quotes.
+- Avoid creating an unnecessary string cache entry.
 
 ## [0.16.0] - 2025-07-16
 

css-inline/src/html/attributes.rs

@@ -172,6 +172,16 @@ impl Attributes {
         let needle = QualName::new(None, ns!(), local);
         self.find(&needle)
     }
+
+    pub(crate) fn get_css_inline(&self) -> Option<&str> {
+        self.attributes.iter().find_map(|probe| {
+            if probe.name.local == *CSS_INLINE_ATTRIBUTE {
+                Some(&*probe.value)
+            } else {
+                None
+            }
+        })
+    }
 }
 
 #[cfg(test)]

css-inline/src/html/serializer.rs

@@ -76,8 +76,7 @@ impl<'a> Sink<'a> {
     #[inline]
     fn should_skip_element(&self, element: &ElementData) -> bool {
         if element.name.local == local_name!("style") {
-            !self.keep_style_tags
-                && element.attributes.get("data-css-inline".into()) != Some("keep")
+            !self.keep_style_tags && element.attributes.get_css_inline() != Some("keep")
         } else if element.name.local == local_name!("link")
             && element.attributes.get(local_name!("rel")) == Some("stylesheet")
         {
@@ -135,7 +134,7 @@ impl<'a> Sink<'a> {
             }
             NodeData::Document => self.serialize_children(serializer),
             NodeData::Doctype { name } => serializer.write_doctype(name),
-            NodeData::Text { text: content } => serializer.write_text(content),
+            NodeData::Text { text } => serializer.write_text(text),
             NodeData::Comment { text } => serializer.write_comment(text),
             NodeData::ProcessingInstruction { target, data } => {
                 serializer.write_processing_instruction(target, data)
@@ -299,7 +298,7 @@ impl<'a, W: Write> HtmlSerializer<'a, W> {
             }
             self.writer.write_all(b"\"")?;
         }
-        if let Some(styles) = &styles {
+        if let Some(styles) = styles {
             self.writer.write_all(b" style=\"")?;
             for (property, (_, value)) in styles {
                 write_declaration(&mut self.writer, property, value)?;