[v4,01/12] support: Add xclock_now helper function.
Commit Message
It's easier to read and write tests with:
const struct timespec ts = xclock_now(CLOCK_REALTIME);
than
struct timespec ts;
xclock_gettime(CLOCK_REALTIME, &ts);
* support/xtime.h: Add xclock_now() helper function.
---
ChangeLog | 4 ++++
support/xtime.h | 10 ++++++++++
2 files changed, 14 insertions(+)
Comments
On 18/06/2019 13:33, Mike Crowe wrote:
> It's easier to read and write tests with:
>
> const struct timespec ts = xclock_now(CLOCK_REALTIME);
>
> than
>
> struct timespec ts;
> xclock_gettime(CLOCK_REALTIME, &ts);
>
> * support/xtime.h: Add xclock_now() helper function.
LGTM, I will commit for you.
> ---
> ChangeLog | 4 ++++
> support/xtime.h | 10 ++++++++++
> 2 files changed, 14 insertions(+)
>
> diff --git a/ChangeLog b/ChangeLog
> index 7401f9f..17fd6ac 100644
> --- a/ChangeLog
> +++ b/ChangeLog
> @@ -1,3 +1,7 @@
> +2019-05-30 Mike Crowe <mac@mcrowe.com>
> +
> + * support/xtime.h: Add xclock_now() helper function.
> +
> 2019-06-17 Rafal Luzynski <digitalfreak@lingonborough.com>
>
> [BZ #24614]
> diff --git a/support/xtime.h b/support/xtime.h
> index 68af1a5..6e19ce1 100644
> --- a/support/xtime.h
> +++ b/support/xtime.h
> @@ -28,6 +28,16 @@ __BEGIN_DECLS
>
> void xclock_gettime (clockid_t clock, struct timespec *ts);
>
> +/* This helper can often simplify tests by avoiding an explicit
> + variable declaration or allowing that declaration to be const. */
> +
> +static inline struct timespec xclock_now (clockid_t clock)
> +{
> + struct timespec ts;
> + xclock_gettime (clock, &ts);
> + return ts;
> +}
> +
> __END_DECLS
>
> #endif /* SUPPORT_TIME_H */
>
@@ -1,3 +1,7 @@
+2019-05-30 Mike Crowe <mac@mcrowe.com>
+
+ * support/xtime.h: Add xclock_now() helper function.
+
2019-06-17 Rafal Luzynski <digitalfreak@lingonborough.com>
[BZ #24614]
@@ -28,6 +28,16 @@ __BEGIN_DECLS
void xclock_gettime (clockid_t clock, struct timespec *ts);
+/* This helper can often simplify tests by avoiding an explicit
+ variable declaration or allowing that declaration to be const. */
+
+static inline struct timespec xclock_now (clockid_t clock)
+{
+ struct timespec ts;
+ xclock_gettime (clock, &ts);
+ return ts;
+}
+
__END_DECLS
#endif /* SUPPORT_TIME_H */