From 3c7117381fcbf8aac95d8d234ccb1905a6c5857b Mon Sep 17 00:00:00 2001
From: Stephen Gutekanst <stephen@hexops.com>
Date: Sat, 16 Oct 2021 17:45:23 -0700
Subject: [PATCH] glfw: add glfw.getTime

Signed-off-by: Stephen Gutekanst <stephen@hexops.com>
---
 src/time.zig | 66 +++++++++++++++++++++++++++-------------------------
 1 file changed, 34 insertions(+), 32 deletions(-)

diff --git a/src/time.zig b/src/time.zig
index a0d5f69..b10c89b 100644
--- a/src/time.zig
+++ b/src/time.zig
@@ -10,8 +10,8 @@ const getError = @import("errors.zig").getError;
 /// has been set using @ref glfwSetTime it measures time elapsed since GLFW was
 /// initialized.
 ///
-/// This function and @ref glfwSetTime are helper functions on top of @ref
-/// glfwGetTimerFrequency and @ref glfwGetTimerValue.
+/// This function and @ref glfwSetTime are helper functions on top of glfw.getTimerFrequency
+/// and glfw.getTimerValue.
 ///
 /// The resolution of the timer is system dependent, but is usually on the order
 /// of a few micro- or nanoseconds. It uses the highest-resolution monotonic
@@ -27,9 +27,6 @@ const getError = @import("errors.zig").getError;
 /// externally synchronized with calls to @ref glfwSetTime.
 ///
 /// see also: time
-///
-///
-/// @ingroup input
 pub inline fn getTime() f64 {
     const time = c.glfwGetTime();
 
@@ -41,34 +38,31 @@ pub inline fn getTime() f64 {
     return time;
 }
 
-// TODO(time):
-// /// Sets the GLFW time.
-// ///
-// /// This function sets the current GLFW time, in seconds. The value must be
-// /// a positive finite number less than or equal to 18446744073.0, which is
-// /// approximately 584.5 years.
-// ///
-// /// This function and @ref glfwGetTime are helper functions on top of @ref
-// /// glfwGetTimerFrequency and @ref glfwGetTimerValue.
-// ///
-// /// @param[in] time The new value, in seconds.
-// ///
-// /// Possible errors include glfw.Error.NotInitialized and glfw.Error.InvalidValue.
-// ///
-// /// The upper limit of GLFW time is calculated as
-// /// floor((2<sup>64</sup> - 1) / 10<sup>9</sup>) and is due to implementations
-// /// storing nanoseconds in 64 bits. The limit may be increased in the future.
-// ///
-// /// @thread_safety This function may be called from any thread. Reading and
-// /// writing of the internal base time is not atomic, so it needs to be
-// /// externally synchronized with calls to @ref glfwGetTime.
-// ///
-// /// see also: time
-// ///
-// ///
-// /// @ingroup input
-// GLFWAPI void glfwSetTime(double time);
+/// Sets the GLFW time.
+///
+/// This function sets the current GLFW time, in seconds. The value must be a positive finite
+/// number less than or equal to 18446744073.0, which is approximately 584.5 years.
+///
+/// This function and @ref glfwGetTime are helper functions on top of glfw.getTimerFrequency and
+/// glfw.getTimerValue.
+///
+/// @param[in] time The new value, in seconds.
+///
+/// Possible errors include glfw.Error.NotInitialized and glfw.Error.InvalidValue.
+///
+/// The upper limit of GLFW time is calculated as `floor((2^64 - 1) / 10^9)` and is due to
+/// implementations storing nanoseconds in 64 bits. The limit may be increased in the future.
+///
+/// @thread_safety This function may be called from any thread. Reading and writing of the internal
+/// base time is not atomic, so it needs to be externally synchronized with calls to glfw.getTime.
+///
+/// see also: time
+pub inline fn setTime(time: f64) Error!void {
+    c.glfwSetTime(time);
+    try getError();
+}
 
+// TODO(time):
 // /// Returns the current value of the raw timer.
 // ///
 // /// This function returns the current value of the raw timer, measured in
@@ -112,3 +106,11 @@ test "getTime" {
 
     _ = getTime();
 }
+
+test "getTime" {
+    const glfw = @import("main.zig");
+    try glfw.init();
+    defer glfw.terminate();
+
+    _ = getTime();
+}