Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-08-07 06:25:32 +01:00
Code Issues Projects Releases Packages Wiki Activity

simplifying min and max calculation for profiler get_statistics function

Browse source
This commit is contained in:
Malisa Smith 2016-07-07 13:08:20 -07:00
parent 7dccf09ff6
commit e1092b8b1d
2 changed files with 4 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

5
components/profile/time.rs
View file

@ -334,11 +334,12 @@ impl Profiler {
}); });
let data_len = data.len(); let data_len = data.len();
debug_assert!(data_len > 0);
let (mean, median, min, max) = let (mean, median, min, max) =
(data.iter().sum::<f64>() / (data_len as f64), (data.iter().sum::<f64>() / (data_len as f64),
data[data_len / 2], data[data_len / 2],
data.iter().fold(f64::INFINITY, |a, &b| a.min(b)), data[0],
data.iter().fold(-f64::INFINITY, |a, &b| a.max(b))); data[data_len-1]);
(mean, median, min, max) (mean, median, min, max)
} }

2
tests/unit/profile/time.rs
View file

@ -37,7 +37,7 @@ fn time_profiler_stats_test() {
assert_eq!(13.2599, odd_max); assert_eq!(13.2599, odd_max);
} }
#[cfg(debug)] #[cfg(debug_assertions)]
#[test] #[test]
#[should_panic] #[should_panic]
fn time_profiler_unsorted_stats_test() { fn time_profiler_unsorted_stats_test() {