xevion/Pac-Man
1
0
Fork 0
mirror of https://github.com/Xevion/Pac-Man.git synced 2025-12-06 13:15:47 -06:00
Code Issues Packages Projects Releases Wiki Activity

fix: use const MAX_SYSTEMS to ensure micromap maps are aligned in size

Browse Source
This commit is contained in:
Xevion
2025-08-15 18:40:24 -05:00
parent 0133dd5329
commit 80930ddd35
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/systems/profiling.rs
View File

@@ -6,6 +6,9 @@ use parking_lot::{Mutex, RwLock};
use std::time::Duration; use std::time::Duration;
use thousands::Separable; use thousands::Separable;
/// The maximum number of systems that can be profiled. Must not be exceeded, or it will panic.
const MAX_SYSTEMS: usize = 11;
/// The number of durations to keep in the circular buffer.
const TIMING_WINDOW_SIZE: usize = 30; const TIMING_WINDOW_SIZE: usize = 30;
#[derive(Resource, Default, Debug)] #[derive(Resource, Default, Debug)]
@@ -17,7 +20,7 @@ pub struct SystemTimings {
/// ///
/// Also, we use a micromap::Map as the number of systems is generally quite small. /// Also, we use a micromap::Map as the number of systems is generally quite small.
/// Just make sure to set the capacity appropriately, or it will panic. /// Just make sure to set the capacity appropriately, or it will panic.
pub timings: RwLock<Map<&'static str, Mutex<CircularBuffer<TIMING_WINDOW_SIZE, Duration>>, 10>>, pub timings: RwLock<Map<&'static str, Mutex<CircularBuffer<TIMING_WINDOW_SIZE, Duration>>, MAX_SYSTEMS>>,
} }
impl SystemTimings { impl SystemTimings {
@@ -43,7 +46,7 @@ impl SystemTimings {
}); });
} }
pub fn get_stats(&self) -> Map<&'static str, (Duration, Duration), 10> { pub fn get_stats(&self) -> Map<&'static str, (Duration, Duration), MAX_SYSTEMS> {
let timings = self.timings.read(); let timings = self.timings.read();
let mut stats = Map::new(); let mut stats = Map::new();