RGBCube/uutils-coreutils
1
Fork 0
mirror of https://github.com/RGBCube/uutils-coreutils synced 2025-07-28 11:37:44 +00:00
Code Issues Activity

tr: use "sets_len == 1" instead of "sets_len < 2"

Browse source 
because sets_len can never be 0
This commit is contained in:
Daniel Hofstetter 2025-06-09 15:41:22 +02:00
parent ed70d7bf16
commit 4cb7490246
1 changed files with 5 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

10
src/uu/tr/src/tr.rs
View file

@ -47,13 +47,13 @@ pub fn uumain(args: impl uucore::Args) -> UResult<()> {
.map(ToOwned::to_owned) .map(ToOwned::to_owned)
.collect(); .collect();
let sets_len = sets.len();
if sets.is_empty() { if sets.is_empty() {
return Err(UUsageError::new(1, get_message("tr-error-missing-operand"))); return Err(UUsageError::new(1, get_message("tr-error-missing-operand")));
} }
if !(delete_flag || squeeze_flag) && sets_len < 2 { let sets_len = sets.len();
if !(delete_flag || squeeze_flag) && sets_len == 1 {
return Err(UUsageError::new( return Err(UUsageError::new(
1, 1,
get_message_with_args( get_message_with_args(
@ -63,7 +63,7 @@ pub fn uumain(args: impl uucore::Args) -> UResult<()> {
)); ));
} }
if delete_flag & squeeze_flag && sets_len < 2 { if delete_flag & squeeze_flag && sets_len == 1 {
return Err(UUsageError::new( return Err(UUsageError::new(
1, 1,
get_message_with_args( get_message_with_args(
@ -143,7 +143,7 @@ pub fn uumain(args: impl uucore::Args) -> UResult<()> {
translate_input(&mut locked_stdin, &mut buffered_stdout, op)?; translate_input(&mut locked_stdin, &mut buffered_stdout, op)?;
} }
} else if squeeze_flag { } else if squeeze_flag {
if sets_len < 2 { if sets_len == 1 {
let op = SqueezeOperation::new(set1); let op = SqueezeOperation::new(set1);
translate_input(&mut locked_stdin, &mut buffered_stdout, op)?; translate_input(&mut locked_stdin, &mut buffered_stdout, op)?;
} else { } else {