mab122
/
radicle-heartwood-lfs
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

remote-helper: Refactor option handling 

Group `["option", "progress", ..]`, and `["option", ..]` cases together during
match.

Make the key/value matching cleaner using `ok_or_else` and then matching on the
key value.
This commit is contained in:
Lorenz Leutgeb 2025-09-11 21:35:43 +02:00 committed by Fintan Halpenny
parent 876d22b072
commit 4e67467966
1 changed files with 15 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
crates/radicle-remote-helper/src/main.rs
View File

@ -236,10 +236,7 @@ pub fn run(profile: radicle::Profile) -> Result<(), Error> {
push_option(args, &mut opts)?;
println!("ok");
}
["option", "progress", ..] => {
println!("unsupported");
}
["option", ..] => {
["option", "progress", ..] | ["option", ..] => {
println!("unsupported");
}
["fetch", oid, refstr] => {
@ -302,22 +299,21 @@ fn push_option(args: &[&str], opts: &mut Options) -> Result<(), Error> {
_ => {
let args = args.join(" ");
if let Some((key, val)) = args.split_once('=') {
match key {
"patch.message" => {
opts.message.append(val);
}
"patch.base" => {
let base =
cli::args::rev(&val.into()).map_err(|e| Error::Base(e.into()))?;
opts.base = Some(base);
}
other => {
return Err(Error::UnsupportedPushOption(other.to_owned()));
}
let (key, val) = args
.split_once('=')
.ok_or_else(|| Error::UnsupportedPushOption(args.to_owned()))?;
match key {
"patch.message" => {
opts.message.append(val);
}
"patch.base" => {
let base = cli::args::rev(&val.into()).map_err(|e| Error::Base(e.into()))?;
opts.base = Some(base);
}
other => {
return Err(Error::UnsupportedPushOption(other.to_owned()));
}
} else {
return Err(Error::UnsupportedPushOption(args.to_owned()));
}
}
}