diff mbox series

time: drop cast from NOW()

Message ID e14bbf31-24cf-4b82-9b2e-a473ad16d008@suse.com (mailing list archive)
State New
Headers show
Series time: drop cast from NOW() | expand

Commit Message

Jan Beulich April 3, 2025, 11:25 a.m. UTC 
It gives the wrong impression of there being a type change, when
get_s_time() really returns s_time_t itself (kind of naturally given its
name).

Signed-off-by: Jan Beulich <jbeulich@suse.com>

Comments

Andrew Cooper April 3, 2025, 11:26 a.m. UTC | #1 
On 03/04/2025 12:25 pm, Jan Beulich wrote:
> It gives the wrong impression of there being a type change, when
> get_s_time() really returns s_time_t itself (kind of naturally given its
> name).
>
> Signed-off-by: Jan Beulich <jbeulich@suse.com>

Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>
diff mbox series

Patch

--- unstable.orig/xen/include/xen/time.h	2024-11-21 21:12:47.000000000 +0100
+++ unstable/xen/include/xen/time.h	2025-04-03 13:17:59.784804154 +0200
@@ -53,7 +53,7 @@  struct tm gmtime(unsigned long t);
 struct tm wallclock_time(uint64_t *ns);
 
 #define SYSTEM_TIME_HZ  1000000000ULL
-#define NOW()           ((s_time_t)get_s_time())
+#define NOW()           get_s_time()
 #define DAYS(_d)        SECONDS((_d) * 86400ULL)
 #define SECONDS(_s)     ((s_time_t)((_s)  * 1000000000ULL))
 #define MILLISECS(_ms)  ((s_time_t)((_ms) * 1000000ULL))