From 01af2199da19e927f68dec6eea685e922fae4f90 Mon Sep 17 00:00:00 2001
From: Mikhail Atuchin <matuchin@brave.com>
Date: Fri, 31 Oct 2025 02:49:23 +0400
Subject: [PATCH] Optimize vec[] creation

---
 src/filters/network.rs | 11 +++--------
 src/utils.rs           | 31 ++++++++++++++++++++++---------
 2 files changed, 25 insertions(+), 17 deletions(-)

diff --git a/src/filters/network.rs b/src/filters/network.rs
index 5c267a8d..cefb7576 100644
--- a/src/filters/network.rs
+++ b/src/filters/network.rs
@@ -900,10 +900,7 @@ impl NetworkFilter {
                         (self.is_plain() || self.is_regex()) && !self.is_right_anchor();
                     let skip_first_token = self.is_right_anchor();
 
-                    let filter_tokens =
-                        utils::tokenize_filter(f, skip_first_token, skip_last_token);
-
-                    tokens.extend(filter_tokens);
+                    utils::tokenize_filter_to(f, skip_first_token, skip_last_token, &mut tokens);
                 }
             }
             FilterPart::AnyOf(_) => (), // across AnyOf set of filters no single token is guaranteed to match to a request
@@ -913,16 +910,14 @@ impl NetworkFilter {
         // Append tokens from hostname, if any
         if !self.mask.contains(NetworkFilterMask::IS_HOSTNAME_REGEX) {
             if let Some(hostname) = self.hostname.as_ref() {
-                let hostname_tokens = utils::tokenize(hostname);
-                tokens.extend(hostname_tokens);
+                utils::tokenize_to(hostname, &mut tokens);
             }
         }
 
         if tokens.is_empty() && self.mask.contains(NetworkFilterMask::IS_REMOVEPARAM) {
             if let Some(removeparam) = &self.modifier_option {
                 if VALID_PARAM.is_match(removeparam) {
-                    let param_tokens = utils::tokenize(&removeparam.to_ascii_lowercase());
-                    tokens.extend(param_tokens);
+                    utils::tokenize_to(&removeparam.to_ascii_lowercase(), &mut tokens);
                 }
             }
         }
diff --git a/src/utils.rs b/src/utils.rs
index 4714e6b5..b9cbe995 100644
--- a/src/utils.rs
+++ b/src/utils.rs
@@ -81,25 +81,23 @@ pub(crate) fn tokenize_pooled(pattern: &str, tokens_buffer: &mut Vec<Hash>) {
 
 pub fn tokenize(pattern: &str) -> Vec<Hash> {
     let mut tokens_buffer: Vec<Hash> = Vec::with_capacity(TOKENS_BUFFER_SIZE);
-    fast_tokenizer_no_regex(
-        pattern,
-        &is_allowed_filter,
-        false,
-        false,
-        &mut tokens_buffer,
-    );
+    tokenize_to(pattern, &mut tokens_buffer);
     tokens_buffer
 }
 
+pub(crate) fn tokenize_to(pattern: &str, tokens_buffer: &mut Vec<Hash>) {
+    fast_tokenizer_no_regex(pattern, &is_allowed_filter, false, false, tokens_buffer);
+}
+
+#[cfg(test)]
 pub(crate) fn tokenize_filter(
     pattern: &str,
     skip_first_token: bool,
     skip_last_token: bool,
 ) -> Vec<Hash> {
     let mut tokens_buffer: Vec<Hash> = Vec::with_capacity(TOKENS_BUFFER_SIZE);
-    fast_tokenizer_no_regex(
+    tokenize_filter_to(
         pattern,
-        &is_allowed_filter,
         skip_first_token,
         skip_last_token,
         &mut tokens_buffer,
@@ -107,6 +105,21 @@ pub(crate) fn tokenize_filter(
     tokens_buffer
 }
 
+pub(crate) fn tokenize_filter_to(
+    pattern: &str,
+    skip_first_token: bool,
+    skip_last_token: bool,
+    tokens_buffer: &mut Vec<Hash>,
+) {
+    fast_tokenizer_no_regex(
+        pattern,
+        &is_allowed_filter,
+        skip_first_token,
+        skip_last_token,
+        tokens_buffer,
+    );
+}
+
 pub(crate) fn bin_lookup<T: Ord>(arr: &[T], elt: T) -> bool {
     arr.binary_search(&elt).is_ok()
 }