all: enable ciippy::collapsible_if

Author: oech3

Cargo.toml

@@ -720,7 +720,6 @@ unexpected_cfgs = { level = "warn", check-cfg = [
 unused_qualifications = "warn"
 
 [workspace.lints.clippy]
-collapsible_if = { level = "allow", priority = 127 } # remove me
 # The counts were generated with this command:
 #   cargo clippy --all-targets --workspace --message-format=json --quiet \
 #     | jq -r '.message.code.code | select(. != null and startswith("clippy::"))' \

src/uu/date/src/format_modifiers.rs

@@ -316,6 +316,7 @@ fn strip_default_padding(value: &str) -> String {
         if stripped.is_empty() {
             return "0".to_string();
         }
+        #[allow(clippy::collapsible_if)]
         if let Some(first_char) = stripped.chars().next() {
             if first_char.is_ascii_digit() {
                 return stripped.to_string();
@@ -443,6 +444,7 @@ fn apply_modifiers(value: &str, parsed: &ParsedSpec<'_>) -> Result<String, Forma
     // GNU behavior: + only adds sign if:
     // 1. An explicit width is provided, OR
     // 2. The value exceeds the default width for that specifier (e.g., year > 4 digits)
+    #[allow(clippy::collapsible_if)]
     if force_sign && !result.starts_with('+') && !result.starts_with('-') {
         if result.chars().next().is_some_and(|c| c.is_ascii_digit()) {
             let default_w = get_default_width(specifier);

src/uu/df/src/df.rs

@@ -156,7 +156,7 @@ impl Options {
         let exclude: Option<Vec<_>> = matches
             .get_many::<OsString>(OPT_EXCLUDE_TYPE)
             .map(|v| v.map(|s| s.to_string_lossy().to_string()).collect());
-
+        #[allow(clippy::collapsible_if)]
         if let (Some(include), Some(exclude)) = (&include, &exclude) {
             if let Some(types) = Self::get_intersected_types(include, exclude) {
                 return Err(OptionsError::FilesystemTypeBothSelectedAndExcluded(types));
@@ -235,11 +235,13 @@ fn is_included(mi: &MountInfo, opt: &Options) -> bool {
     }
 
     // Don't show filesystems if they have been explicitly excluded.
+    #[allow(clippy::collapsible_if)]
     if let Some(ref excludes) = opt.exclude {
         if excludes.contains(&mi.fs_type) {
             return false;
         }
     }
+    #[allow(clippy::collapsible_if)]
     if let Some(ref includes) = opt.include {
         if !includes.contains(&mi.fs_type) {
             return false;
@@ -311,6 +313,7 @@ fn get_all_filesystems(opt: &Options) -> UResult<Vec<Filesystem>> {
         // this loop quadratic in the length of `vmi`. This could be
         // improved by a more efficient implementation of `is_best()`,
         // but `vmi` is probably not very long in practice.
+        #[allow(clippy::collapsible_if)]
         if is_included(&mi, opt) && is_best(&mounts, &mi) {
             let dev_path: &Path = Path::new(&mi.dev_name);
             // Only check is_symlink() for absolute paths. For non-absolute paths

src/uu/du/src/du.rs

@@ -517,6 +517,7 @@ fn safe_du(
         }
 
         // Process directories recursively
+        #[allow(clippy::collapsible_if)]
         if is_dir {
             if options.one_file_system {
                 if let (Some(this_inode), Some(my_inode)) = (this_stat.inode, my_stat.inode) {
@@ -637,6 +638,7 @@ fn du_regular(
                                 && options.dereference == Deref::All
                                 && this_stat.metadata.is_dir()
                             {
+                                #[allow(clippy::collapsible_if)]
                                 if let Some(inode) = this_stat.inode {
                                     if ancestors.contains(&inode) {
                                         // This symlink points to an ancestor directory - skip to avoid cycle
@@ -674,7 +676,7 @@ fn du_regular(
                                 // Mark this inode as seen
                                 seen_inodes.insert(inode);
                             }
-
+                            #[allow(clippy::collapsible_if)]
                             if this_stat.metadata.is_dir() {
                                 if options.one_file_system {
                                     if let (Some(this_inode), Some(my_inode)) =
@@ -1143,6 +1145,7 @@ pub fn uumain(args: impl uucore::Args) -> UResult<()> {
 
         // Pre-populate seen_inodes with the starting directory to detect cycles
         let stat = Stat::new(&path, None, &traversal_options);
+        #[allow(clippy::collapsible_if)]
         if let Ok(stat) = stat.as_ref() {
             if let Some(inode) = stat.inode {
                 if !traversal_options.count_links && seen_inodes.contains(&inode) {
@@ -1172,6 +1175,7 @@ pub fn uumain(args: impl uucore::Args) -> UResult<()> {
                     }
                     Err(e) => {
                         // Check if this is our "already handled" error
+                        #[allow(clippy::collapsible_if)]
                         if let mpsc::SendError(Err(simple_error)) = e.as_ref() {
                             if simple_error.code() == 0 {
                                 // Error already handled, continue to next file

src/uu/env/src/env.rs

@@ -697,6 +697,7 @@ impl EnvAppData {
         self.do_input_debug_printing = self
             .do_input_debug_printing
             .or(Some(matches.get_count("debug") >= 2));
+        #[allow(clippy::collapsible_if)]
         if let Some(value) = self.do_input_debug_printing {
             if value {
                 debug_print_args(&original_args);

src/uu/env/src/variable_parser.rs

@@ -18,6 +18,7 @@ impl<'a> VariableParser<'a, '_> {
     }
 
     fn check_variable_name_start(&self) -> Result<(), EnvError> {
+        #[allow(clippy::collapsible_if)]
         if let Some(c) = self.get_current_char() {
             if c.is_ascii_digit() {
                 return Err(EnvError::EnvParsingOfVariableUnexpectedNumber(

src/uu/expr/src/syntax_tree.rs

@@ -518,6 +518,7 @@ fn evaluate_match_expression(left_bytes: Vec<u8>, right_bytes: Vec<u8>) -> ExprR
         let left_encoded = onig::EncodedBytes::ascii(&left_bytes);
         let pos = regex_search(&regex, left_encoded, left_bytes.len(), &mut region);
 
+        #[allow(clippy::collapsible_if)]
         if pos.is_some() {
             if let Some((start, end)) = region.pos(1) {
                 let capture_bytes = &left_bytes[start..end];

src/uu/ln/src/ln.rs

@@ -298,6 +298,7 @@ fn link_files_in_dir(files: &[PathBuf], target_dir: &Path, settings: &Settings)
             let remove_target = || {
                 // In that case, we don't want to do link resolution
                 // We need to clean the target
+                #[allow(clippy::collapsible_if)]
                 if target_dir.is_file() {
                     if let Err(e) = fs::remove_file(target_dir) {
                         show_error!(
@@ -374,6 +375,7 @@ fn link_files_in_dir(files: &[PathBuf], target_dir: &Path, settings: &Settings)
 }
 
 fn relative_path<'a>(src: &'a Path, dst: &Path) -> Cow<'a, Path> {
+    #[allow(clippy::collapsible_if)]
     if let Ok(src_abs) = canonicalize(src, MissingHandling::Missing, ResolveMode::Physical) {
         if let Ok(dst_abs) = canonicalize(
             dst.parent().unwrap(),

src/uu/ls/src/colors.rs

@@ -73,6 +73,7 @@ impl<'a> StyleManager<'a> {
         let mut force_suffix_reset: bool = false;
         let mut applied_raw_code = false;
 
+        #[allow(clippy::collapsible_if)]
         if self.is_reset() {
             if let Some(norm_sty) = self.get_normal_style().copied() {
                 style_code.push_str(&self.get_style_code(&norm_sty));
@@ -324,6 +325,7 @@ impl<'a> StyleManager<'a> {
             return None;
         }
         let mut target = path.path().read_link().ok()?;
+        #[allow(clippy::collapsible_if)]
         if target.is_relative() {
             if let Some(parent) = path.path().parent() {
                 target = parent.join(target);
@@ -404,6 +406,7 @@ impl<'a> StyleManager<'a> {
 
     #[cfg(unix)]
     fn indicator_for_file(&self, path: &PathData) -> Option<Indicator> {
+        #[allow(clippy::collapsible_if)]
         if self.needs_file_metadata() {
             if let Some(metadata) = path.metadata() {
                 let mode = metadata.mode();
@@ -442,6 +445,7 @@ impl<'a> StyleManager<'a> {
 
     #[cfg(unix)]
     fn indicator_for_directory(&self, path: &PathData) -> Option<Indicator> {
+        #[allow(clippy::collapsible_if)]
         if self.needs_dir_metadata() {
             if let Some(metadata) = path.metadata() {
                 let mode = metadata.mode();
@@ -541,6 +545,7 @@ pub(crate) fn color_name(
         }
     }
 
+    #[allow(clippy::collapsible_if)]
     if target_symlink.is_none() && path.file_type().is_some_and(fs::FileType::is_symlink) {
         if let Some(colored) = style_manager.color_symlink_name(path, name.clone(), wrap) {
             return colored;

src/uu/ls/src/config.rs

@@ -719,6 +719,7 @@ impl Config {
             {
                 format = Format::Long;
             } else if let Some(mut indices) = options.indices_of(options::format::ONE_LINE) {
+                #[allow(clippy::collapsible_if)]
                 if options.value_source(options::format::ONE_LINE)
                     == Some(clap::parser::ValueSource::CommandLine)
                     && indices.any(|i| i > idx)
@@ -918,6 +919,7 @@ impl Config {
             locale_quoting = None;
         }
 
+        #[allow(clippy::collapsible_if)]
         if needs_color {
             if let Err(err) = validate_ls_colors_env() {
                 if let LsColorsParseError::UnrecognizedPrefix(prefix) = &err {