macos and windows as 240

Author: raphamorim

frontends/rioterm/src/application.rs

@@ -1022,6 +1022,7 @@ impl ApplicationHandler<EventPayload> for Application<'_> {
 
             WindowEvent::RedrawRequested => {
                 route.window.winit_window.pre_present_notify();
+                let start = std::time::Instant::now();
                 match route.path {
                     RoutePath::Assistant => {
                         route.window.screen.render_assistant(&route.assistant);
@@ -1030,8 +1031,7 @@ impl ApplicationHandler<EventPayload> for Application<'_> {
                         route.window.screen.render_welcome();
                     }
                     RoutePath::Terminal => {
-                        let duration = route.window.screen.render();
-                        route.window.compute_timestamp(duration);
+                        route.window.screen.render();
                     }
                     RoutePath::ConfirmQuit => {
                         route
@@ -1040,8 +1040,10 @@ impl ApplicationHandler<EventPayload> for Application<'_> {
                             .render_dialog("Do you want to leave Rio?");
                     }
                 }
+                let duration = start.elapsed();
                 // println!("Time elapsed in render() is: {:?}", duration);
                 // }
+                route.window.compute_timestamp(duration);
 
                 event_loop.set_control_flow(ControlFlow::Wait);
             }

frontends/rioterm/src/screen/mod.rs

@@ -1904,8 +1904,7 @@ impl Screen<'_> {
         self.sugarloaf.render();
     }
 
-    #[inline]
-    pub fn render(&mut self) -> std::time::Duration {
+    pub fn render(&mut self) {
         // let start_total = std::time::Instant::now();
         // println!("_____________________________\nrender time elapsed");
         let is_search_active = self.search_active();
@@ -1941,7 +1940,6 @@ impl Screen<'_> {
             data
         };
         self.renderer.set_ime(self.ime.preedit());
-        let duration = std::time::Instant::now();
         self.renderer.prepare_term(
             &rows,
             cursor,
@@ -1953,7 +1951,6 @@ impl Screen<'_> {
             &self.search_state.focused_match,
         );
         self.sugarloaf.render();
-        let duration = duration.elapsed();
         // In this case the configuration of blinking cursor is enabled
         // and the terminal also have instructions of blinking enabled
         // TODO: enable blinking for selection after adding debounce (https://github.com/raphamorim/rio/issues/437)
@@ -1962,7 +1959,6 @@ impl Screen<'_> {
                 .blink_cursor(self.renderer.config_blinking_interval);
         }
 
-        duration
         // let duration = start_total.elapsed();
         // println!("Total whole render function is: {:?}\n", duration);
     }

rio-backend/src/config/defaults.rs

@@ -17,10 +17,8 @@ pub fn default_cursor_interval() -> u64 {
 
 #[inline]
 pub fn default_max_fps() -> u64 {
-    if cfg!(target_os = "macos") {
+    if cfg!(target_os = "macos") || cfg!(target_os = "windows") {
         240
-    } else if cfg!(target_os = "windows") {
-        120
     } else {
         90
     }