clippy: fix inline_always lint

https://rust-lang.github.io/rust-clippy/master/index.html#inline_always

Author: xtqqczze

Cargo.toml

@@ -663,7 +663,6 @@ float_cmp = "allow"                          # 12
 items_after_statements = "allow"             # 11
 return_self_not_must_use = "allow"           # 8
 needless_continue = "allow"                  # 6
-inline_always = "allow"                      # 6
 fn_params_excessive_bools = "allow"          # 6
 used_underscore_items = "allow"              # 2
 should_panic_without_expect = "allow"        # 2

src/uu/mknod/src/mknod.rs

@@ -26,7 +26,6 @@ mod options {
     pub const CONTEXT: &str = "context";
 }
 
-#[inline(always)]
 fn makedev(maj: u64, min: u64) -> dev_t {
     // pick up from <sys/sysmacros.h>
     ((min & 0xff) | ((maj & 0xfff) << 8) | ((min & !0xff) << 12) | ((maj & !0xfff) << 32)) as dev_t

src/uu/more/src/more.rs

@@ -380,7 +380,6 @@ fn setup_term() -> UResult<OutputType> {
 }
 
 #[cfg(target_os = "fuchsia")]
-#[inline(always)]
 fn setup_term() -> UResult<OutputType> {
     // no real stdout/tty on Fuchsia, just write into a pipe
     Ok(OutputType::Pipe(Box::new(stdout())))
@@ -400,7 +399,6 @@ fn reset_term() -> UResult<()> {
 }
 
 #[cfg(target_os = "fuchsia")]
-#[inline(always)]
 fn reset_term() -> UResult<()> {
     Ok(())
 }

src/uu/sort/src/numeric_str_cmp.rs

@@ -223,7 +223,6 @@ pub fn human_numeric_str_cmp(
 
 /// Compare two numbers as strings without parsing them as a number first. This should be more performant and can handle numbers more precisely.
 /// [`NumInfo`] is needed to provide a fast path for most numbers.
-#[inline(always)]
 pub fn numeric_str_cmp((a, a_info): (&[u8], &NumInfo), (b, b_info): (&[u8], &NumInfo)) -> Ordering {
     // check for a difference in the sign
     if a_info.sign != b_info.sign {

src/uu/sort/src/sort.rs

@@ -2100,7 +2100,6 @@ pub enum GeneralBigDecimalParseResult {
 
 /// Parse the beginning string into a [`GeneralBigDecimalParseResult`].
 /// Using a [`GeneralBigDecimalParseResult`] instead of [`ExtendedBigDecimal`] is necessary to correctly order floats.
-#[inline(always)]
 fn general_bd_parse(a: &[u8]) -> GeneralBigDecimalParseResult {
     // The string should be valid ASCII to be parsed.
     let Ok(a) = std::str::from_utf8(a) else {

src/uucore/src/lib/features/parser/num_parser.rs

@@ -433,7 +433,7 @@ fn pow_with_context(bd: &BigDecimal, exp: i64, ctx: &Context) -> BigDecimal {
     // in exp, and the number of times we can divide exp by 2.
     let mut n = exp.unsigned_abs();
     // Note: 63 - n.leading_zeros() == n.ilog2, but that's only available in recent Rust versions.
-    let muls = (n.count_ones() + (63 - n.leading_zeros()) - 1) as u64;
+    let muls = (n.count_ones() + n.ilog2() - 1) as u64;
     // Note: div_ceil would be nice to use here, but only available in recent Rust versions.
     // (see note above about minimum Rust version in use)
     let margin_extra = (muls + MUL_PER_MARGIN_EXTRA / 2) / MUL_PER_MARGIN_EXTRA;

src/uucore/src/lib/features/uptime.rs

@@ -501,8 +501,7 @@ mod tests {
         // (This is just a sanity check)
         assert!(
             uptime < 365 * 86400,
-            "Uptime seems unreasonably high: {} seconds",
-            uptime
+            "Uptime seems unreasonably high: {uptime} seconds"
         );
     }
 
@@ -518,9 +517,7 @@ mod tests {
         let diff = (uptime1 - uptime2).abs();
         assert!(
             diff <= 1,
-            "Consecutive uptime calls should be consistent, got {} and {}",
-            uptime1,
-            uptime2
+            "Consecutive uptime calls should be consistent, got {uptime1} and {uptime2}"
         );
     }
 }