Merge pull request #8775 from cakebaker/clippy_re_enable_semicolon_if_nothing_returned

clippy: re-enable `semicolon_if_nothing_returned`

Cargo.toml

@@ -697,7 +697,6 @@ bool_to_int_with_if = "allow"
 needless_raw_string_hashes = "allow"
 unreadable_literal = "allow"
 unnested_or_patterns = "allow"
-semicolon_if_nothing_returned = "allow"
 implicit_hasher = "allow"
 struct_field_names = "allow"
 doc_link_with_quotes = "allow"

src/uucore/src/lib/features/checksum.rs

@@ -1645,7 +1645,7 @@ mod tests {
         for (filename, result, prefix, expected) in cases {
             let mut buffer: Vec<u8> = vec![];
             print_file_report(&mut buffer, filename, *result, prefix, opts.verbose);
-            assert_eq!(&buffer, expected)
+            assert_eq!(&buffer, expected);
         }
     }
 }

src/uucore/src/lib/features/fast_inc.rs

@@ -62,7 +62,7 @@ pub fn fast_inc(val: &mut [u8], start: &mut usize, end: usize, inc: &[u8]) {
         return;
     }
 
-    fast_inc_one(val, start, pos)
+    fast_inc_one(val, start, pos);
 }
 
 /// Fast increment by one function, operating on ASCII strings.

src/uucore/src/lib/features/perms.rs

@@ -742,13 +742,14 @@ pub fn configure_symlink_and_recursion(
         None // Default behavior
     };
 
-    let mut traverse_symlinks = default_traverse_symlinks;
-    if matches.get_flag("L") {
-        traverse_symlinks = TraverseSymlinks::All
+    let mut traverse_symlinks = if matches.get_flag("L") {
+        TraverseSymlinks::All
     } else if matches.get_flag("H") {
-        traverse_symlinks = TraverseSymlinks::First
+        TraverseSymlinks::First
     } else if matches.get_flag("P") {
-        traverse_symlinks = TraverseSymlinks::None
+        TraverseSymlinks::None
+    } else {
+        default_traverse_symlinks
     };
 
     let recursive = matches.get_flag(options::RECURSIVE);