perf: Remove unnecessary contains call (#512)

* perf: Remove unnecessary `contains` call

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

* chore: Update changelog

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

---------

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

Author: Stranger6667

CHANGELOG.md

@@ -6,6 +6,10 @@
 
 - `keep_at_rules` option to keep all "at-rules" (ones starting with `@`) inside a "style" element. [#265](https://github.com/Stranger6667/css-inline/issues/265)
 
+### Performance
+
+- Avoid unnecessary check for double quotes.
+
 ## [0.16.0] - 2025-07-16
 
 ### Added

bindings/c/CHANGELOG.md

@@ -6,6 +6,10 @@
 
 - `keep_at_rules` option to keep all "at-rules" (ones starting with `@`) inside a "style" element [#265](https://github.com/Stranger6667/css-inline/issues/265)
 
+### Performance
+
+- Avoid unnecessary check for double quotes.
+
 ## [0.16.0] - 2025-07-16
 
 ### Changed

bindings/java/CHANGELOG.md

@@ -6,6 +6,10 @@
 
 - `keep_at_rules` option to keep all "at-rules" (ones starting with `@`) inside a "style" element [#265](https://github.com/Stranger6667/css-inline/issues/265)
 
+### Performance
+
+- Avoid unnecessary check for double quotes.
+
 ## [0.16.0] - 2025-07-16
 
 ### Changed

bindings/javascript/CHANGELOG.md

@@ -6,6 +6,10 @@
 
 - `keep_at_rules` option to keep all "at-rules" (ones starting with `@`) inside a "style" element [#265](https://github.com/Stranger6667/css-inline/issues/265)
 
+### Performance
+
+- Avoid unnecessary check for double quotes.
+
 ## [0.16.0] - 2025-07-16
 
 ### Changed

bindings/python/CHANGELOG.md

@@ -6,6 +6,10 @@
 
 - `keep_at_rules` option to keep all "at-rules" (ones starting with `@`) inside a "style" element [#265](https://github.com/Stranger6667/css-inline/issues/265)
 
+### Performance
+
+- Avoid unnecessary check for double quotes.
+
 ## [0.16.0] - 2025-07-16
 
 ### Changed

bindings/ruby/CHANGELOG.md

@@ -6,6 +6,10 @@
 
 - `keep_at_rules` option to keep all "at-rules" (ones starting with `@`) inside a "style" element [#265](https://github.com/Stranger6667/css-inline/issues/265)
 
+### Performance
+
+- Avoid unnecessary check for double quotes.
+
 ## [0.16.0] - 2025-07-16
 
 ### Changed

css-inline/src/html/serializer.rs

@@ -428,28 +428,24 @@ fn write_declaration<Wr: Write>(
 #[inline]
 fn write_declaration_value<Wr: Write>(writer: &mut Wr, value: &str) -> Result<(), InlineError> {
     let value = value.trim();
-    if value.as_bytes().contains(&b'"') {
-        // Roughly based on `str::replace`
-        let mut last_end = 0;
-        for (start, part) in value.match_indices('"') {
-            writer.write_all(
-                value
-                    .get(last_end..start)
-                    .expect("Invalid substring")
-                    .as_bytes(),
-            )?;
-            writer.write_all(b"'")?;
-            last_end = start.checked_add(part.len()).expect("Size overflow");
-        }
+    // Roughly based on `str::replace`
+    let mut last_end = 0;
+    for (start, part) in value.match_indices('"') {
         writer.write_all(
             value
-                .get(last_end..value.len())
+                .get(last_end..start)
                 .expect("Invalid substring")
                 .as_bytes(),
         )?;
-    } else {
-        writer.write_all(value.as_bytes())?;
+        writer.write_all(b"'")?;
+        last_end = start.checked_add(part.len()).expect("Size overflow");
     }
+    writer.write_all(
+        value
+            .get(last_end..value.len())
+            .expect("Invalid substring")
+            .as_bytes(),
+    )?;
     Ok(())
 }