[v4,01/12] support: Add xclock_now helper function.

Message ID f55dc4a10b299db1dea46dfebe77bad8ebc3d546.1560875584.git-series.mac@mcrowe.com
State Committed
Headers

Commit Message

Mike Crowe June 18, 2019, 4:33 p.m. UTC
  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

Adhemerval Zanella Netto June 19, 2019, 8:01 p.m. UTC | #1
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 */
>
  

Patch

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 */