diff options
author | Ryan Geary <rtgnj42@gmail.com> | 2019-10-09 18:23:02 -0400 |
---|---|---|
committer | Ryan Geary <rtgnj42@gmail.com> | 2019-10-09 18:23:02 -0400 |
commit | f722cbcbbb61ba46c5d6e63d543a834c65a51751 (patch) | |
tree | 86e0fb28d058cd36f9f8d960ac6ee810eae9b7bc /src/choice.rs | |
parent | 09ddbb4f659076e08b2e85fb65fea97623178eb7 (diff) |
Remove repeated Regex compilation
Diffstat (limited to 'src/choice.rs')
-rw-r--r-- | src/choice.rs | 15 |
1 files changed, 3 insertions, 12 deletions
diff --git a/src/choice.rs b/src/choice.rs index eb20dc4..65e1acd 100644 --- a/src/choice.rs +++ b/src/choice.rs @@ -219,22 +219,13 @@ impl Choice { &self, line: &String, opt: &Opt, + re: &Regex, handle: &mut BufWriter<std::io::StdoutLock>, ) { - write!(handle, "{}", self.get_choice_slice(line, opt).join(" ")); + write!(handle, "{}", self.get_choice_slice(line, opt, re).join(" ")); } - fn get_choice_slice<'a>(&self, line: &'a String, opt: &Opt) -> Vec<&'a str> { - let re = Regex::new(match &opt.field_separator { - Some(s) => s, - None => "[[:space:]]", - }) - .unwrap_or_else(|e| { - eprintln!("Failed to compile regular expression: {}", e); - // Exit code of 1 means failed to compile field_separator regex - process::exit(1); - }); - + fn get_choice_slice<'a>(&self, line: &'a String, opt: &Opt, re: &Regex) -> Vec<&'a str> { let words = re .split(line) .into_iter() |