Remove jemalloc dependencies from factor utility

The jemalloc allocator and related dependencies (jemalloc-ctl, jemalloc-sys, paste) were removed from the factor utility's benchmark code. This change simplifies the dependency tree and removes platform-specific allocator logic that was only used for memory profiling in benchmarks. The benchmark functionality remains intact, just without jemalloc-specific memory statistics collection.

Additionally, a new regression test was added to verify correct factorization of a very large number (15111234931751377131713914373267893176342349831) to ensure the utility handles large inputs properly.

Author: mattsu2020

Cargo.lock

@@ -33,10 +33,6 @@ path = "src/main.rs"
 divan = { workspace = true }
 uucore = { workspace = true, features = ["benchmark"] }
 
-[target.'cfg(any(target_os = "linux", target_os = "macos", target_os = "freebsd"))'.dev-dependencies]
-jemalloc-ctl = { workspace = true }
-jemallocator = { workspace = true }
-
 [lib]
 path = "src/factor.rs"
 

src/uu/factor/Cargo.toml

@@ -9,39 +9,16 @@ use divan::{Bencher, black_box};
 use uu_factor::uumain;
 use uucore::benchmark::run_util_function;
 
-#[cfg(any(target_os = "linux", target_os = "macos", target_os = "freebsd"))]
-use jemallocator::Jemalloc;
-
-#[cfg(any(target_os = "linux", target_os = "macos", target_os = "freebsd"))]
-#[global_allocator]
-static ALLOC: Jemalloc = Jemalloc;
-
-#[cfg(any(target_os = "linux", target_os = "macos", target_os = "freebsd"))]
-fn log_jemalloc_stats(label: &str) {
-    use jemalloc_ctl::{epoch, stats};
-
-    epoch::advance().unwrap();
-    let allocated = stats::allocated::read().unwrap();
-    let resident = stats::resident::read().unwrap();
-
-    println!("jemalloc {label}: allocated={allocated} bytes, resident={resident} bytes");
-}
-
-#[cfg(not(any(target_os = "linux", target_os = "macos", target_os = "freebsd")))]
-fn log_jemalloc_stats(_label: &str) {}
-
 /// Benchmark multiple u64 digits
 #[divan::bench(args = [(2)])]
 fn factor_multiple_u64s(bencher: Bencher, start_num: u64) {
     bencher
         // this is a range of 5000 different u128 integers
         .with_inputs(|| (start_num, start_num + 2500))
         .bench_values(|(start_u64, end_u64)| {
-            log_jemalloc_stats("before factor_multiple_u64s");
             for u64_digit in start_u64..=end_u64 {
                 black_box(run_util_function(uumain, &[&u64_digit.to_string()]));
             }
-            log_jemalloc_stats("after factor_multiple_u64s");
         });
 }
 

src/uu/factor/benches/factor_bench.rs

@@ -169,6 +169,18 @@ fn test_cli_args() {
         .stdout_contains("9: 3 3");
 }
 
+#[test]
+fn test_large_regression_number() {
+    let n = "15111234931751377131713914373267893176342349831";
+    new_ucmd!()
+        .arg(n)
+        .timeout(Duration::from_secs(240))
+        .succeeds()
+        .stdout_is(format!(
+            "{n}: 29 127 115319 1971799 82465494029827 218807630881735711\n"
+        ));
+}
+
 #[test]
 fn test_random() {
     let log_num_primes = f64::from(u32::try_from(NUM_PRIMES).unwrap()).log2().ceil();