profiler

.gitignore

@@ -4,9 +4,12 @@
 
 # These are backup files generated by rustfmt
 **/*.rs.bk
+
 .devenv*
 devenv.local.nix
 
+# generated by samply rust profiler
+profile.json
 
 # Added by cargo
 

client_cli/src/main.rs

@@ -37,9 +37,6 @@ fn main() -> Result<()> {
 
     // Create an application.
     let mut app = App::new(args);
-    if app.should_quit {
-        println!("{}", app.display());
-    }
 
     // Start the main loop.
     while !app.should_quit {
@@ -49,6 +46,9 @@ fn main() -> Result<()> {
         app.input(input.trim());
     }
 
+    // display app final state
+    println!("{}", app.display());
+
     Ok(())
 }
 

devenv.nix

@@ -5,6 +5,10 @@
   # env.GREET = "devenv";
 
   packages = [
+
+    # dev tools
+    pkgs.samply # code profiler
+
     # for bevy
     pkgs.alsaLib
     pkgs.udev

justfile

@@ -10,3 +10,7 @@ runcli:
 	RUST_LOG=info cargo run --bin=client_cli
 runclibots:
 	RUST_LOG=info cargo run --bin=client_cli -- --bot dummy,dummy
+profile:
+  echo '1' | sudo tee /proc/sys/kernel/perf_event_paranoid
+  cargo build --profile profiling
+  samply record ./target/profiling/client_cli --bot dummy,dummy

store/src/game.rs

@@ -582,8 +582,6 @@ impl GameState {
 /// The reasons why a game could end
 #[derive(Debug, Clone, Copy, Serialize, PartialEq, Deserialize)]
 pub enum EndGameReason {
-    // In tic tac toe it doesn't make sense to keep playing when one of the players disconnect.
-    // Note that it might make sense to keep playing in some other game (like Team Fight Tactics for instance).
     PlayerLeft { player_id: PlayerId },
     PlayerWon { winner: PlayerId },
 }