Trim glob input

Kirill Bulatov created 2 years ago

Change summary

crates/project/src/search.rs        | 12 ++++++++----
crates/search/src/project_search.rs |  6 ++++--
2 files changed, 12 insertions(+), 6 deletions(-)

Detailed changes

crates/project/src/search.rs

@@ -98,13 +98,15 @@ impl SearchQuery {
                 message
                     .files_to_include
                     .split(',')
-                    .filter(|glob_str| !glob_str.trim().is_empty())
+                    .map(str::trim)
+                    .filter(|glob_str| !glob_str.is_empty())
                     .map(|glob_str| glob::Pattern::new(glob_str))
                     .collect::<Result<_, _>>()?,
                 message
                     .files_to_exclude
                     .split(',')
-                    .filter(|glob_str| !glob_str.trim().is_empty())
+                    .map(str::trim)
+                    .filter(|glob_str| !glob_str.is_empty())
                     .map(|glob_str| glob::Pattern::new(glob_str))
                     .collect::<Result<_, _>>()?,
             )
@@ -116,13 +118,15 @@ impl SearchQuery {
                 message
                     .files_to_include
                     .split(',')
-                    .filter(|glob_str| !glob_str.trim().is_empty())
+                    .map(str::trim)
+                    .filter(|glob_str| !glob_str.is_empty())
                     .map(|glob_str| glob::Pattern::new(glob_str))
                     .collect::<Result<_, _>>()?,
                 message
                     .files_to_exclude
                     .split(',')
-                    .filter(|glob_str| !glob_str.trim().is_empty())
+                    .map(str::trim)
+                    .filter(|glob_str| !glob_str.is_empty())
                     .map(|glob_str| glob::Pattern::new(glob_str))
                     .collect::<Result<_, _>>()?,
             ))

crates/search/src/project_search.rs

@@ -562,7 +562,8 @@ impl ProjectSearchView {
             .read(cx)
             .text(cx)
             .split(',')
-            .filter(|glob_str| !glob_str.trim().is_empty())
+            .map(str::trim)
+            .filter(|glob_str| !glob_str.is_empty())
             .map(|glob_str| glob::Pattern::new(glob_str))
             .collect::<Result<_, _>>() else {
                 self.query_contains_error = true;
@@ -574,7 +575,8 @@ impl ProjectSearchView {
             .read(cx)
             .text(cx)
             .split(',')
-            .filter(|glob_str| !glob_str.trim().is_empty())
+            .map(str::trim)
+            .filter(|glob_str| !glob_str.is_empty())
             .map(|glob_str| glob::Pattern::new(glob_str))
             .collect::<Result<_, _>>() else {
                 self.query_contains_error = true;