mirror of
https://github.com/nushell/nushell.git
synced 2025-01-12 01:09:05 +01:00
Use '?' for error propagation, remove match
(#2194)
This commit is contained in:
parent
a04dfca63a
commit
0f688d7da7
@ -67,17 +67,10 @@ async fn operate(
|
|||||||
let mut ret = v;
|
let mut ret = v;
|
||||||
|
|
||||||
for path in &column_paths {
|
for path in &column_paths {
|
||||||
let swapping = ret.swap_data_by_column_path(
|
ret = ret.swap_data_by_column_path(
|
||||||
path,
|
path,
|
||||||
Box::new(move |old| action(old, old.tag())),
|
Box::new(move |old| action(old, old.tag())),
|
||||||
);
|
)?;
|
||||||
|
|
||||||
match swapping {
|
|
||||||
Ok(new_value) => {
|
|
||||||
ret = new_value;
|
|
||||||
}
|
|
||||||
Err(err) => return Err(err),
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
ReturnSuccess::value(ret)
|
ReturnSuccess::value(ret)
|
||||||
|
@ -67,19 +67,10 @@ async fn operate(
|
|||||||
let mut ret = v;
|
let mut ret = v;
|
||||||
|
|
||||||
for path in &column_paths {
|
for path in &column_paths {
|
||||||
let swapping = ret.swap_data_by_column_path(
|
ret = ret.swap_data_by_column_path(
|
||||||
path,
|
path,
|
||||||
Box::new(move |old| action(old, old.tag())),
|
Box::new(move |old| action(old, old.tag())),
|
||||||
);
|
)?;
|
||||||
|
|
||||||
match swapping {
|
|
||||||
Ok(new_value) => {
|
|
||||||
ret = new_value;
|
|
||||||
}
|
|
||||||
Err(err) => {
|
|
||||||
return Err(err);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
ReturnSuccess::value(ret)
|
ReturnSuccess::value(ret)
|
||||||
|
@ -87,19 +87,10 @@ async fn operate(
|
|||||||
for path in &column_paths {
|
for path in &column_paths {
|
||||||
let options = options.clone();
|
let options = options.clone();
|
||||||
|
|
||||||
let swapping = ret.swap_data_by_column_path(
|
ret = ret.swap_data_by_column_path(
|
||||||
path,
|
path,
|
||||||
Box::new(move |old| action(old, &options, old.tag())),
|
Box::new(move |old| action(old, &options, old.tag())),
|
||||||
);
|
)?;
|
||||||
|
|
||||||
match swapping {
|
|
||||||
Ok(new_value) => {
|
|
||||||
ret = new_value;
|
|
||||||
}
|
|
||||||
Err(err) => {
|
|
||||||
return Err(err);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
ReturnSuccess::value(ret)
|
ReturnSuccess::value(ret)
|
||||||
|
@ -100,18 +100,10 @@ async fn operate(
|
|||||||
} else {
|
} else {
|
||||||
let mut ret = v;
|
let mut ret = v;
|
||||||
for path in &column_paths {
|
for path in &column_paths {
|
||||||
let swapping = ret.swap_data_by_column_path(
|
ret = ret.swap_data_by_column_path(
|
||||||
path,
|
path,
|
||||||
Box::new(move |old| action(old, old.tag(), digits, group_digits)),
|
Box::new(move |old| action(old, old.tag(), digits, group_digits)),
|
||||||
);
|
)?;
|
||||||
match swapping {
|
|
||||||
Ok(new_value) => {
|
|
||||||
ret = new_value;
|
|
||||||
}
|
|
||||||
Err(err) => {
|
|
||||||
return Err(err);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
ReturnSuccess::value(ret)
|
ReturnSuccess::value(ret)
|
||||||
|
@ -80,19 +80,10 @@ async fn operate(
|
|||||||
for path in &column_paths {
|
for path in &column_paths {
|
||||||
let options = options.clone();
|
let options = options.clone();
|
||||||
|
|
||||||
let swapping = ret.swap_data_by_column_path(
|
ret = ret.swap_data_by_column_path(
|
||||||
path,
|
path,
|
||||||
Box::new(move |old| action(old, &options, old.tag())),
|
Box::new(move |old| action(old, &options, old.tag())),
|
||||||
);
|
)?;
|
||||||
|
|
||||||
match swapping {
|
|
||||||
Ok(new_value) => {
|
|
||||||
ret = new_value;
|
|
||||||
}
|
|
||||||
Err(err) => {
|
|
||||||
return Err(err);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
ReturnSuccess::value(ret)
|
ReturnSuccess::value(ret)
|
||||||
|
@ -115,19 +115,10 @@ async fn operate(
|
|||||||
for path in &column_paths {
|
for path in &column_paths {
|
||||||
let options = options.clone();
|
let options = options.clone();
|
||||||
|
|
||||||
let swapping = ret.swap_data_by_column_path(
|
ret = ret.swap_data_by_column_path(
|
||||||
path,
|
path,
|
||||||
Box::new(move |old| action(old, &options, old.tag())),
|
Box::new(move |old| action(old, &options, old.tag())),
|
||||||
);
|
)?;
|
||||||
|
|
||||||
match swapping {
|
|
||||||
Ok(new_value) => {
|
|
||||||
ret = new_value;
|
|
||||||
}
|
|
||||||
Err(err) => {
|
|
||||||
return Err(err);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
ReturnSuccess::value(ret)
|
ReturnSuccess::value(ret)
|
||||||
|
@ -87,19 +87,11 @@ async fn operate(
|
|||||||
|
|
||||||
for path in &column_paths {
|
for path in &column_paths {
|
||||||
let options = options.clone();
|
let options = options.clone();
|
||||||
let swapping = ret.swap_data_by_column_path(
|
|
||||||
|
ret = ret.swap_data_by_column_path(
|
||||||
path,
|
path,
|
||||||
Box::new(move |old| action(old, &options, old.tag())),
|
Box::new(move |old| action(old, &options, old.tag())),
|
||||||
);
|
)?;
|
||||||
|
|
||||||
match swapping {
|
|
||||||
Ok(new_value) => {
|
|
||||||
ret = new_value;
|
|
||||||
}
|
|
||||||
Err(err) => {
|
|
||||||
return Err(err);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
ReturnSuccess::value(ret)
|
ReturnSuccess::value(ret)
|
||||||
|
@ -70,19 +70,10 @@ async fn operate(
|
|||||||
let mut ret = v;
|
let mut ret = v;
|
||||||
|
|
||||||
for path in &column_paths {
|
for path in &column_paths {
|
||||||
let swapping = ret.swap_data_by_column_path(
|
ret = ret.swap_data_by_column_path(
|
||||||
path,
|
path,
|
||||||
Box::new(move |old| action(old, old.tag())),
|
Box::new(move |old| action(old, old.tag())),
|
||||||
);
|
)?;
|
||||||
|
|
||||||
match swapping {
|
|
||||||
Ok(new_value) => {
|
|
||||||
ret = new_value;
|
|
||||||
}
|
|
||||||
Err(err) => {
|
|
||||||
return Err(err);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
ReturnSuccess::value(ret)
|
ReturnSuccess::value(ret)
|
||||||
|
@ -70,19 +70,10 @@ async fn operate(
|
|||||||
let mut ret = v;
|
let mut ret = v;
|
||||||
|
|
||||||
for path in &column_paths {
|
for path in &column_paths {
|
||||||
let swapping = ret.swap_data_by_column_path(
|
ret = ret.swap_data_by_column_path(
|
||||||
path,
|
path,
|
||||||
Box::new(move |old| action(old, old.tag())),
|
Box::new(move |old| action(old, old.tag())),
|
||||||
);
|
)?;
|
||||||
|
|
||||||
match swapping {
|
|
||||||
Ok(new_value) => {
|
|
||||||
ret = new_value;
|
|
||||||
}
|
|
||||||
Err(err) => {
|
|
||||||
return Err(err);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
ReturnSuccess::value(ret)
|
ReturnSuccess::value(ret)
|
||||||
|
@ -84,19 +84,10 @@ async fn operate(
|
|||||||
let mut ret = v;
|
let mut ret = v;
|
||||||
|
|
||||||
for path in &column_paths {
|
for path in &column_paths {
|
||||||
let swapping = ret.swap_data_by_column_path(
|
ret = ret.swap_data_by_column_path(
|
||||||
path,
|
path,
|
||||||
Box::new(move |old| action(old, old.tag(), to_trim)),
|
Box::new(move |old| action(old, old.tag(), to_trim)),
|
||||||
);
|
)?;
|
||||||
|
|
||||||
match swapping {
|
|
||||||
Ok(new_value) => {
|
|
||||||
ret = new_value;
|
|
||||||
}
|
|
||||||
Err(err) => {
|
|
||||||
return Err(err);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
ReturnSuccess::value(ret)
|
ReturnSuccess::value(ret)
|
||||||
|
@ -67,19 +67,10 @@ async fn operate(
|
|||||||
let mut ret = v;
|
let mut ret = v;
|
||||||
|
|
||||||
for path in &column_paths {
|
for path in &column_paths {
|
||||||
let swapping = ret.swap_data_by_column_path(
|
ret = ret.swap_data_by_column_path(
|
||||||
path,
|
path,
|
||||||
Box::new(move |old| action(old, old.tag())),
|
Box::new(move |old| action(old, old.tag())),
|
||||||
);
|
)?;
|
||||||
|
|
||||||
match swapping {
|
|
||||||
Ok(new_value) => {
|
|
||||||
ret = new_value;
|
|
||||||
}
|
|
||||||
Err(err) => {
|
|
||||||
return Err(err);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
ReturnSuccess::value(ret)
|
ReturnSuccess::value(ret)
|
||||||
|
Loading…
Reference in New Issue
Block a user