all: move mach.Timer, core Timer/Frequency to mach.time module

Signed-off-by: Stephen Gutekanst <stephen@hexops.com>

src/Core.zig

@@ -8,8 +8,6 @@ const log = std.log.scoped(.mach);
 const gamemode_log = std.log.scoped(.gamemode);
 
 pub const sysgpu = @import("../main.zig").sysgpu;
-pub const Timer = @import("core/Timer.zig");
-const Frequency = @import("core/Frequency.zig");
 
 pub const Platform = switch (build_options.core_platform) {
     .wasm => @panic("TODO: support mach.Core WASM platform"),
@@ -135,10 +133,10 @@ state: enum {
     exited,
 } = .running,
 linux_gamemode: ?bool = null,
-frame: Frequency,
+frame: mach.time.Frequency,
 
 // Might be accessed by Platform backend
-input: Frequency,
+input: mach.time.Frequency,
 swap_chain_update: std.Thread.ResetEvent = .{},
 
 // GPU
@@ -1095,12 +1093,6 @@ comptime {
     assertHasDecl(Platform, "mousePressed");
     assertHasDecl(Platform, "mouseReleased");
     assertHasDecl(Platform, "mousePosition");
-
-    // Timer
-    assertHasDecl(@This().Timer, "start");
-    assertHasDecl(@This().Timer, "read");
-    assertHasDecl(@This().Timer, "reset");
-    assertHasDecl(@This().Timer, "lap");
 }
 
 fn assertHasDecl(comptime T: anytype, comptime decl_name: []const u8) void {
@@ -1112,8 +1104,6 @@ fn assertHasField(comptime T: anytype, comptime field_name: []const u8) void {
 }
 
 test {
-    @import("std").testing.refAllDecls(Timer);
-    @import("std").testing.refAllDecls(Frequency);
     @import("std").testing.refAllDecls(Platform);
 
     @import("std").testing.refAllDeclsRecursive(InitOptions);

src/core/Darwin.zig

@@ -2,7 +2,6 @@ const std = @import("std");
 const mach = @import("../main.zig");
 const Core = @import("../Core.zig");
 const InputState = @import("InputState.zig");
-const Frequency = @import("Frequency.zig");
 const unicode = @import("unicode.zig");
 const detectBackendType = @import("common.zig").detectBackendType;
 const gpu = mach.gpu;

src/core/Null.zig

@@ -5,7 +5,6 @@ const std = @import("std");
 const mach = @import("../main.zig");
 const Core = @import("../Core.zig");
 const InputState = @import("InputState.zig");
-const Frequency = @import("Frequency.zig");
 const unicode = @import("unicode.zig");
 const detectBackendType = @import("common.zig").detectBackendType;
 const gpu = mach.gpu;

src/core/Timer.zig

@@ -1,57 +0,0 @@
-const std = @import("std");
-const builtin = @import("builtin");
-const PlatformTimer = if (builtin.cpu.arch == .wasm32) @panic("TODO: support WASM") else NativeTimer;
-
-const Timer = @This();
-
-platform: PlatformTimer,
-
-/// Initialize the timer.
-pub fn start() !Timer {
-    return .{ .platform = try PlatformTimer.start() };
-}
-
-/// Reads the timer value since start or the last reset in nanoseconds.
-pub inline fn readPrecise(timer: *Timer) u64 {
-    return timer.platform.read();
-}
-
-/// Reads the timer value since start or the last reset in seconds.
-pub inline fn read(timer: *Timer) f32 {
-    return @as(f32, @floatFromInt(timer.readPrecise())) / @as(f32, @floatFromInt(std.time.ns_per_s));
-}
-
-/// Resets the timer value to 0/now.
-pub inline fn reset(timer: *Timer) void {
-    timer.platform.reset();
-}
-
-/// Returns the current value of the timer in nanoseconds, then resets it.
-pub inline fn lapPrecise(timer: *Timer) u64 {
-    return timer.platform.lap();
-}
-
-/// Returns the current value of the timer in seconds, then resets it.
-pub inline fn lap(timer: *Timer) f32 {
-    return @as(f32, @floatFromInt(timer.lapPrecise())) / @as(f32, @floatFromInt(std.time.ns_per_s));
-}
-
-const NativeTimer = struct {
-    timer: std.time.Timer,
-
-    pub fn start() !NativeTimer {
-        return .{ .timer = try std.time.Timer.start() };
-    }
-
-    pub inline fn read(timer: *NativeTimer) u64 {
-        return timer.timer.read();
-    }
-
-    pub inline fn reset(timer: *NativeTimer) void {
-        timer.timer.reset();
-    }
-
-    pub inline fn lap(timer: *NativeTimer) u64 {
-        return timer.timer.lap();
-    }
-};

src/core/win32.zig

@@ -3,7 +3,6 @@ const w = @import("win32/win32.zig");
 const mach = @import("../main.zig");
 const Core = @import("../Core.zig");
 const InputState = @import("InputState.zig");
-const Frequency = @import("Frequency.zig");
 const unicode = @import("unicode.zig");
 
 const gpu = mach.gpu;

src/main.zig

@@ -4,7 +4,6 @@ const std = @import("std");
 
 // Core
 pub const Core = if (build_options.want_core) @import("Core.zig") else struct {};
-pub const Timer = if (build_options.want_core) Core.Timer else struct {};
 
 // Mach standard library
 // gamemode requires libc on linux
@@ -13,6 +12,7 @@ pub const gfx = if (build_options.want_mach) @import("gfx/main.zig") else struct
 pub const Audio = if (build_options.want_sysaudio) @import("Audio.zig") else struct {};
 pub const math = @import("math/main.zig");
 pub const testing = @import("testing.zig");
+pub const time = @import("time/main.zig");
 
 pub const sysaudio = if (build_options.want_sysaudio) @import("sysaudio/main.zig") else struct {};
 pub const sysgpu = if (build_options.want_sysgpu) @import("sysgpu/main.zig") else struct {};
@@ -57,6 +57,7 @@ test {
     _ = gfx;
     _ = math;
     _ = testing;
+    _ = time;
     std.testing.refAllDeclsRecursive(@import("module/Archetype.zig"));
     std.testing.refAllDeclsRecursive(@import("module/entities.zig"));
     // std.testing.refAllDeclsRecursive(@import("module/main.zig"));

src/time/Frequency.zig

@@ -1,4 +1,4 @@
-const std = @import("std");
+const mach = @import("../main.zig");
 const Timer = @import("Timer.zig");
 
 pub const Frequency = @This();
@@ -38,21 +38,21 @@ pub inline fn tick(f: *Frequency) void {
 
     if (f.delta_time) |delta_time| {
         f.delta_time_ns.* = current_time -| f.internal.last_time;
-        delta_time.* = @as(f32, @floatFromInt(f.delta_time_ns.*)) / @as(f32, @floatFromInt(std.time.ns_per_s));
+        delta_time.* = @as(f32, @floatFromInt(f.delta_time_ns.*)) / @as(f32, @floatFromInt(mach.time.ns_per_s));
     }
 
-    if (current_time >= std.time.ns_per_s) {
+    if (current_time >= mach.time.ns_per_s) {
         f.rate = f.internal.count;
         f.internal.count = 0;
         f.internal.timer.reset();
-        current_time -= std.time.ns_per_s;
+        current_time -= mach.time.ns_per_s;
     }
     f.internal.last_time = current_time;
     f.internal.count += 1;
 
     if (f.target != 0) {
         const limited_count = @min(f.target, f.internal.count);
-        const target_time_per_tick: u64 = (std.time.ns_per_s / f.target);
+        const target_time_per_tick: u64 = (mach.time.ns_per_s / f.target);
         const target_time = target_time_per_tick * limited_count;
         if (current_time > target_time) {
             f.delay_ns = 0;

src/time/Timer.zig

@@ -0,0 +1,38 @@
+const mach = @import("../main.zig");
+const std = @import("std");
+
+const Timer = @This();
+
+// TODO: support a WASM-based timer as well, which is the primary reason this abstraction exists.
+
+timer: std.time.Timer,
+
+/// Initialize the timer.
+pub fn start() !Timer {
+    return .{ .timer = try std.time.Timer.start() };
+}
+
+/// Reads the timer value since start or the last reset in nanoseconds.
+pub inline fn readPrecise(timer: *Timer) u64 {
+    return timer.timer.read();
+}
+
+/// Reads the timer value since start or the last reset in seconds.
+pub inline fn read(timer: *Timer) f32 {
+    return @as(f32, @floatFromInt(timer.readPrecise())) / @as(f32, @floatFromInt(mach.time.ns_per_s));
+}
+
+/// Resets the timer value to 0/now.
+pub inline fn reset(timer: *Timer) void {
+    timer.timer.reset();
+}
+
+/// Returns the current value of the timer in nanoseconds, then resets it.
+pub inline fn lapPrecise(timer: *Timer) u64 {
+    return timer.timer.lap();
+}
+
+/// Returns the current value of the timer in seconds, then resets it.
+pub inline fn lap(timer: *Timer) f32 {
+    return @as(f32, @floatFromInt(timer.lapPrecise())) / @as(f32, @floatFromInt(mach.time.ns_per_s));
+}

src/time/main.zig

@@ -0,0 +1,11 @@
+pub const std = @import("std");
+
+pub const ns_per_s = std.time.ns_per_s;
+
+pub const Timer = @import("Timer.zig");
+pub const Frequency = @import("Frequency.zig");
+
+test {
+    _ = Timer;
+    _ = Frequency;
+}