From ced9d2ebdada96fe0ec28a9ed7209a91fe017061 Mon Sep 17 00:00:00 2001
From: Alex Saveau <saveau.alexandre@gmail.com>
Date: Tue, 17 May 2022 04:01:54 +0000
Subject: [PATCH] Fix frame count being a float (#4493)

Original reasoning: https://github.com/bevyengine/bevy/pull/678#issuecomment-930602773

That reasoning doesn't seem valid IMO since eventually +1 will do nothing. Using an integer is more intuitive and will wrap around which is probably better than getting stuck.
---
 .../bevy_diagnostic/src/frame_time_diagnostics_plugin.rs  | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/crates/bevy_diagnostic/src/frame_time_diagnostics_plugin.rs b/crates/bevy_diagnostic/src/frame_time_diagnostics_plugin.rs
index 2ff6f17af4f80..e34f431e00e4d 100644
--- a/crates/bevy_diagnostic/src/frame_time_diagnostics_plugin.rs
+++ b/crates/bevy_diagnostic/src/frame_time_diagnostics_plugin.rs
@@ -8,13 +8,13 @@ use bevy_ecs::system::{Res, ResMut};
 pub struct FrameTimeDiagnosticsPlugin;
 
 pub struct FrameTimeDiagnosticsState {
-    frame_count: f64,
+    frame_count: u64,
 }
 
 impl Plugin for FrameTimeDiagnosticsPlugin {
     fn build(&self, app: &mut bevy_app::App) {
         app.add_startup_system(Self::setup_system)
-            .insert_resource(FrameTimeDiagnosticsState { frame_count: 0.0 })
+            .insert_resource(FrameTimeDiagnosticsState { frame_count: 0 })
             .add_system(Self::diagnostic_system);
     }
 }
@@ -37,8 +37,8 @@ impl FrameTimeDiagnosticsPlugin {
         time: Res<Time>,
         mut state: ResMut<FrameTimeDiagnosticsState>,
     ) {
-        state.frame_count += 1.0;
-        diagnostics.add_measurement(Self::FRAME_COUNT, state.frame_count);
+        state.frame_count = state.frame_count.wrapping_add(1);
+        diagnostics.add_measurement(Self::FRAME_COUNT, state.frame_count as f64);
 
         if time.delta_seconds_f64() == 0.0 {
             return;