From 0939d6d7bc15f29c1f755a5fc4a1db57ca93d73f Mon Sep 17 00:00:00 2001
From: Rekai Musuka <rekai@musuka.dev>
Date: Fri, 21 Oct 2022 05:12:53 -0300
Subject: [PATCH] chore: move audio sync, video sync variables

---
 src/emu.zig  | 17 +++++++++--------
 src/main.zig |  2 +-
 2 files changed, 10 insertions(+), 9 deletions(-)

diff --git a/src/emu.zig b/src/emu.zig
index ec0c092..815d5c8 100644
--- a/src/emu.zig
+++ b/src/emu.zig
@@ -10,7 +10,8 @@ const Timer = std.time.Timer;
 const Thread = std.Thread;
 const Atomic = std.atomic.Atomic;
 
-const audio_sync = true;
+const sync_audio = false;
+const sync_video: RunKind = .UnlimitedFPS;
 
 // 228 Lines which consist of 308 dots (which are 4 cycles long)
 const cycles_per_frame: u64 = 228 * (308 * 4); //280896
@@ -35,10 +36,10 @@ const RunKind = enum {
     LimitedBusy,
 };
 
-pub fn run(kind: RunKind, quit: *Atomic(bool), fps: *FpsTracker, sched: *Scheduler, cpu: *Arm7tdmi) void {
-    if (audio_sync) log.info("Audio sync enabled", .{});
+pub fn run(quit: *Atomic(bool), fps: *FpsTracker, sched: *Scheduler, cpu: *Arm7tdmi) void {
+    if (sync_audio) log.info("Audio sync enabled", .{});
 
-    switch (kind) {
+    switch (sync_video) {
         .Unlimited => runUnsynchronized(quit, sched, cpu, null),
         .Limited => runSynchronized(quit, sched, cpu, null),
         .UnlimitedFPS => runUnsynchronized(quit, sched, cpu, fps),
@@ -77,14 +78,14 @@ pub fn runUnsynchronized(quit: *Atomic(bool), sched: *Scheduler, cpu: *Arm7tdmi,
 
         while (!quit.load(.SeqCst)) {
             runFrame(sched, cpu);
-            if (audio_sync) syncToAudio(cpu);
+            if (sync_audio) syncToAudio(cpu);
 
             tracker.tick();
         }
     } else {
         while (!quit.load(.SeqCst)) {
             runFrame(sched, cpu);
-            if (audio_sync) syncToAudio(cpu);
+            if (sync_audio) syncToAudio(cpu);
         }
     }
 }
@@ -105,7 +106,7 @@ pub fn runSynchronized(quit: *Atomic(bool), sched: *Scheduler, cpu: *Arm7tdmi, f
             // If we happen to also be syncing to audio, we choose to spin on
             // the amount of time needed for audio to catch up rather than
             // our expected wake-up time
-            if (audio_sync) syncToAudio(cpu) else spinLoop(&timer, wake_time);
+            if (sync_audio) syncToAudio(cpu) else spinLoop(&timer, wake_time);
             wake_time = new_wake_time;
 
             tracker.tick();
@@ -115,7 +116,7 @@ pub fn runSynchronized(quit: *Atomic(bool), sched: *Scheduler, cpu: *Arm7tdmi, f
             runFrame(sched, cpu);
             const new_wake_time = syncToVideo(&timer, wake_time);
             // see above comment
-            if (audio_sync) syncToAudio(cpu) else spinLoop(&timer, wake_time);
+            if (sync_audio) syncToAudio(cpu) else spinLoop(&timer, wake_time);
 
             wake_time = new_wake_time;
         }
diff --git a/src/main.zig b/src/main.zig
index e617c6b..27ff719 100644
--- a/src/main.zig
+++ b/src/main.zig
@@ -99,7 +99,7 @@ pub fn main() anyerror!void {
     // Run Emulator in it's separate thread
     // From this point on, interacting with Arm7tdmi or Scheduler
     // be justified, as it will require to be thread-afe
-    const emu_thread = try Thread.spawn(.{}, emu.run, .{ .LimitedFPS, &quit, &emu_rate, &scheduler, &cpu });
+    const emu_thread = try Thread.spawn(.{}, emu.run, .{ &quit, &emu_rate, &scheduler, &cpu });
     defer emu_thread.join();
 
     var title_buf: [0x20]u8 = std.mem.zeroes([0x20]u8);