[Openocd-development] [PATCH 02/10] time_support: improve use of types

[Zachary T Welch]

Update timeval_add_time to use long int; implement timeval_add with it.
Update timeval_ms to check gettimeofday return value, return int64_t.

Signed-off-by: Zachary T Welch <z...@superlucidity.net>
---
 src/helper/time_support.c |   37 +++++++++++++------------------------
 src/helper/time_support.h |    4 ++--
 2 files changed, 15 insertions(+), 26 deletions(-)

diff --git a/src/helper/time_support.c b/src/helper/time_support.c
index 9b86e52..693528f 100644
--- a/src/helper/time_support.c
+++ b/src/helper/time_support.c
@@ -55,20 +55,11 @@ int timeval_subtract(struct timeval *result, struct timeval 
*x, struct timeval *
 /* add two struct timeval values */
 int timeval_add(struct timeval *result, struct timeval *x, struct timeval *y)
 {
-       result->tv_sec = x->tv_sec + y->tv_sec;
-
-       result->tv_usec = x->tv_usec + y->tv_usec;
-
-       while (result->tv_usec > 1000000)
-       {
-               result->tv_usec -= 1000000;
-               result->tv_sec++;
-       }
-
-       return 0;
+       memcpy(result, x, sizeof(struct timeval));
+       return timeval_add_time(result, y->tv_sec, y->tv_usec);
 }
 
-int timeval_add_time(struct timeval *result, int sec, int usec)
+int timeval_add_time(struct timeval *result, long sec, long usec)
 {
        result->tv_sec += sec;
        result->tv_usec += usec;
@@ -82,6 +73,16 @@ int timeval_add_time(struct timeval *result, int sec, int 
usec)
        return 0;
 }
 
+int64_t timeval_ms()
+{
+       struct timeval now;
+       int retval = gettimeofday(&now, NULL);
+       if (retval < 0)
+               return retval;
+       return (int64_t)now.tv_sec * 1000 + now.tv_usec / 1000;
+}
+
+
 int duration_start(struct duration *duration)
 {
        return gettimeofday(&duration->start, NULL);
@@ -107,15 +108,3 @@ float duration_kbps(struct duration *duration, size_t 
count)
 {
        return count / (1024.0 * duration_elapsed(duration));
 }
-
-long long timeval_ms()
-{
-       struct timeval now;
-       long long t = 0;
-       gettimeofday(&now, NULL);
-
-       t += now.tv_usec/1000;
-       t += now.tv_sec*1000;
-
-       return t;
-}
diff --git a/src/helper/time_support.h b/src/helper/time_support.h
index 9e8e61f..a2d180a 100644
--- a/src/helper/time_support.h
+++ b/src/helper/time_support.h
@@ -39,10 +39,10 @@
 
 int timeval_subtract(struct timeval *result, struct timeval *x, struct timeval 
*y);
 int timeval_add(struct timeval *result, struct timeval *x, struct timeval *y);
-int timeval_add_time(struct timeval *result, int sec, int usec);
+int timeval_add_time(struct timeval *result, long sec, long usec);
 
 /// @returns gettimeofday() timeval as 64-bit in ms
-long long timeval_ms(void);
+int64_t timeval_ms(void);
 
 struct duration
 {
-- 
1.6.4.4

_______________________________________________
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development