Do not use bstr's "unicode" feature

Avoiding this feature eliminates the transitive regex-automata dependency.
The cost is that instead of trimming all unicode whitespace from &[u8], now
only ascii whitespace is trimmed. Given that the &[u8] slices represent
diffs, this seems like an okay compromise.

Author: jpgrayson

Cargo.lock

@@ -18,7 +18,7 @@ name = "stg"
 [dependencies]
 anyhow = "1.0"
 atty = "0.2"
-bstr = "0.2"
+bstr = { version = "0.2", default-features = false, features = ["std"] }
 chrono = { version = "0.4", default-features = false, features = ["clock"] }
 clap = { version = "3.1", default-features = false, features = ["color", "std", "suggestions", "wrap_help"] }
 encoding_rs = "0.8"

Cargo.toml

@@ -363,7 +363,7 @@ fn import_series(
             .find_char('#')
             .map(|pos| &line[..pos])
             .unwrap_or(line)
-            .trim();
+            .trim_with(|c| c.is_ascii_whitespace());
         if line.is_empty() {
             continue;
         }
@@ -617,7 +617,7 @@ fn create_patch<'repo>(
             .map(|s| s.parse::<usize>().expect("clap already validated"))
     });
 
-    let trimmed_diff = diff.trim_end();
+    let trimmed_diff = diff.trim_end_with(|c| c.is_ascii_whitespace());
 
     let tree_id = if trimmed_diff.is_empty() || trimmed_diff == b"---" {
         stack.branch_head.tree_id()
@@ -735,15 +735,18 @@ impl Headers {
         let mut split_message = Vec::with_capacity(message.len());
         let mut lines = message.lines_with_terminator();
 
-        while let Some(line) = lines.next().map(|line| line.trim()) {
+        while let Some(line) = lines
+            .next()
+            .map(|line| line.trim_with(|c| c.is_ascii_whitespace()))
+        {
             if line.is_empty() {
                 continue;
             }
 
             let parts: Vec<_> = line.splitn_str(2, b":").collect();
             if parts.len() == 2 {
                 let header = parts[0];
-                let value = parts[1].trim_start();
+                let value = parts[1].trim_start_with(|c| c.is_ascii_whitespace());
                 if header.eq_ignore_ascii_case(b"patch") && !value.is_empty() {
                     headers.patchname = Some(
                         value

src/cmd/import.rs

@@ -127,7 +127,7 @@ fn run(matches: &clap::ArgMatches) -> Result<()> {
                 .find_char('#')
                 .map(|pos| &line[..pos])
                 .unwrap_or(line)
-                .trim();
+                .trim_with(|c| c.is_ascii_whitespace());
             if line.is_empty() {
                 continue;
             }