remove Windows-specific sleep code; no longer necessary as of Rust 1.75

Rust 1.75 changed thread::sleep to use high-resolution timers on Windows instead of Sleep()

see rust-lang/rust#116461

Author: jsgroth

Cargo.lock

@@ -47,7 +47,6 @@ toml = "0.8"
 wasm-bindgen = "0.2"
 wasm-bindgen-futures = "0.4"
 web-time = "1"
-windows = "0.58"
 winit = "0.30"
 wgpu = "23"
 xrandr = "0.2"

Cargo.toml

@@ -21,9 +21,6 @@ log = { workspace = true }
 serde = { workspace = true, optional = true }
 time = { workspace = true }
 
-[target.'cfg(target_os = "windows")'.dependencies]
-windows = { workspace = true, features = ["Win32_Media"] }
-
 [target.'cfg(target_arch = "wasm32")'.dependencies]
 js-sys = { workspace = true }
 

common/jgenesis-common/Cargo.toml

@@ -5,26 +5,6 @@ pub mod input;
 pub mod num;
 pub mod timeutils;
 
-use cfg_if::cfg_if;
-use std::thread;
-use std::time::Duration;
-
-#[inline]
-pub fn sleep(duration: Duration) {
-    cfg_if! {
-        if #[cfg(target_os = "windows")] {
-            // SAFETY: thread::sleep cannot panic, so timeEndPeriod will always be called after timeBeginPeriod.
-            unsafe {
-                windows::Win32::Media::timeBeginPeriod(1);
-                thread::sleep(duration);
-                windows::Win32::Media::timeEndPeriod(1);
-            }
-        } else {
-            thread::sleep(duration);
-        }
-    }
-}
-
 #[inline]
 #[must_use]
 pub fn is_appimage_build() -> bool {

common/jgenesis-common/src/lib.rs

@@ -693,7 +693,7 @@ fn collect_input(
             }
         }
 
-        jgenesis_common::sleep(Duration::from_millis(10));
+        thread::sleep(Duration::from_millis(10));
     }
 }
 

frontend/jgenesis-gui/src/emuthread.rs

@@ -48,11 +48,11 @@ use std::error::Error;
 use std::ffi::NulError;
 use std::fmt::Debug;
 use std::hash::Hash;
-use std::io;
 use std::path::{Path, PathBuf};
 use std::rc::Rc;
 use std::sync::LazyLock;
 use std::time::Duration;
+use std::{io, thread};
 use thiserror::Error;
 
 const MODAL_DURATION: Duration = Duration::from_secs(3);
@@ -570,7 +570,7 @@ where
 
         if !should_run_emulator {
             // Don't spin loop when the emulator is paused or rewinding
-            jgenesis_common::sleep(Duration::from_millis(1));
+            thread::sleep(Duration::from_millis(1));
         }
 
         Ok(None)

frontend/jgenesis-native-driver/src/mainloop.rs

@@ -3,6 +3,7 @@ use jgenesis_common::audio::DynamicResamplingRate;
 use jgenesis_common::frontend::AudioOutput;
 use sdl2::AudioSubsystem;
 use sdl2::audio::{AudioQueue, AudioSpecDesired};
+use std::thread;
 use std::time::Duration;
 use thiserror::Error;
 
@@ -167,7 +168,7 @@ impl AudioOutput for SdlAudioOutput {
             if self.audio_sync {
                 // Block until audio queue is not full
                 while self.audio_queue_len_samples() > audio_buffer_threshold {
-                    jgenesis_common::sleep(Duration::from_micros(250));
+                    thread::sleep(Duration::from_micros(250));
                 }
             } else if self.audio_queue_len_samples() > audio_buffer_threshold {
                 // Audio queue is full; drop samples

frontend/jgenesis-native-driver/src/mainloop/audio.rs

@@ -7,7 +7,7 @@ use std::collections::HashMap;
 use std::error::Error;
 use std::fmt::Debug;
 use std::time::Duration;
-use std::{cmp, iter};
+use std::{cmp, iter, thread};
 use thiserror::Error;
 use wgpu::util::DeviceExt;
 
@@ -960,7 +960,7 @@ impl FrameTimeTracker {
         let mut now = timeutils::current_time_nanos();
         let next_frame_time = self.last_frame_time_nanos + self.frame_interval_nanos;
         while now < next_frame_time {
-            jgenesis_common::sleep(Duration::from_micros(250));
+            thread::sleep(Duration::from_micros(250));
             now = timeutils::current_time_nanos();
         }