aboutsummaryrefslogtreecommitdiffhomepage
path: root/crates/shirabe/src/util
diff options
context:
space:
mode:
Diffstat (limited to 'crates/shirabe/src/util')
-rw-r--r--crates/shirabe/src/util/hg.rs2
-rw-r--r--crates/shirabe/src/util/platform.rs4
2 files changed, 2 insertions, 4 deletions
diff --git a/crates/shirabe/src/util/hg.rs b/crates/shirabe/src/util/hg.rs
index 8570af6d..d9e58623 100644
--- a/crates/shirabe/src/util/hg.rs
+++ b/crates/shirabe/src/util/hg.rs
@@ -158,7 +158,7 @@ impl Hg {
&output,
)
{
- return matches.into_iter().nth(1);
+ return matches.into_iter().nth(1).flatten();
}
None
})
diff --git a/crates/shirabe/src/util/platform.rs b/crates/shirabe/src/util/platform.rs
index 21ad869c..629854b6 100644
--- a/crates/shirabe/src/util/platform.rs
+++ b/crates/shirabe/src/util/platform.rs
@@ -95,14 +95,12 @@ impl Platform {
// Regex pattern compatibility:
// The original pattern uses a conditional subpattern to make the trailing `%` required
// only for the `%VAR%` form. The Rust regex crate does not support conditionals, so the
- // two forms are written as an explicit alternation: `$VAR` or `%VAR%`. The branch that did
- // not participate is reported as an empty string, which `\w+` can never capture.
+ // two forms are written as an explicit alternation: `$VAR` or `%VAR%`.
Preg::replace_callback(
php_regex!(r"#^(?:\$(?P<dvar>\w+)|%(?P<pvar>\w+)%)(?P<path>.*)#"),
|matches: &PregMatchedGroups| -> String {
let var = matches
.get(&CaptureKey::ByName("dvar".to_string()))
- .filter(|dvar| !dvar.is_empty())
.or_else(|| matches.get(&CaptureKey::ByName("pvar".to_string())))
.map(|s| s.as_str())
.unwrap_or("");