try more limited escape check

Signed-off-by: Luke Steensen <[email protected]>
vectordotdev · Nov 15, 2023 · 222ee68 · 222ee68
1 parent 2d73ac7
commit 222ee68
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 5 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/lib/vector-core/Cargo.toml b/lib/vector-core/Cargo.toml
@@ -63,7 +63,7 @@ vector-config = { path = "../vector-config" }
 vector-config-common = { path = "../vector-config-common" }
 vector-config-macros = { path = "../vector-config-macros" }
 vrl.workspace = true
-memchr = "2.6.4"
+bytecount = "0.6.7"
 
 [target.'cfg(target_os = "macos")'.dependencies]
 security-framework = "2.9.2"

diff --git a/lib/vector-core/src/event/estimated_json_encoded_size_of.rs b/lib/vector-core/src/event/estimated_json_encoded_size_of.rs
@@ -97,8 +97,7 @@ impl EstimatedJsonEncodedSizeOf for str {
 // serializing to JSON. The goal is improve our overall accuracy in the case of relatively common
 // scenarios like strings of nested JSON, etc.
 fn count_escapes(input: &[u8]) -> usize {
-    memchr::memchr3_iter(b'"', b'\\', b'\t', input).count()
-        + memchr::memchr2_iter(b'\n', b'\r', input).count()
+    bytecount::count(input, b'"')
 }
 
 impl EstimatedJsonEncodedSizeOf for String {