Changeset ca940b for src/Helpers/Chronos.cpp

Timestamp:

Jul 14, 2014, 8:37:03 PM (11 years ago)

Author:

Frederik Heber <heber@…>

Children:

ef9dff6

Parents:

e93bfe

git-author:

Frederik Heber <heber@…> (06/18/14 08:14:25)

git-committer:

Frederik Heber <heber@…> (07/14/14 20:37:03)

Message:

FIX: Chronos is now working correctly, no more negative times.

• extracted difference calculator into static function, making it thus accessible to unit testing.
• kudos to Saskia Metzler for inspiring the test.

File:

• src/Helpers/Chronos.cpp (modified) (3 diffs)

src/Helpers/Chronos.cpp

@@ -102 +102 @@
 }
 
+double Chronos::calculateCorrectTimeDifference(
+    const sec_ncsec_t &_time1,
+    const sec_ncsec_t &_time2)
+{
+        double currenttime = 0.;
+  if (_time1.second < _time2.second)
+                currenttime = (_time1.first - _time2.first - 1)
+                        + (1e9 + _time1.second - _time2.second) * 1.e-9;
+  else
+                currenttime = (_time1.first - _time2.first)
+                        + (_time1.second - _time2.second) * 1.e-9;
+  return currenttime;
+}
+
 double Chronos::getCurrentTime() const
 {
@@ -108 +122 @@
   timespec time1;
   clock_gettime(CLOCK_PROCESS_CPUTIME_ID, &time1);
-  double currenttime;
-  if (time1.tv_nsec < basetime.tv_nsec)
-    currenttime = (time1.tv_sec - basetime.tv_sec - 1)
-        + (basetime.tv_nsec - time1.tv_nsec) * 1.e-9;
-  else
-    currenttime = (time1.tv_sec - basetime.tv_sec)
-        + (time1.tv_nsec - basetime.tv_nsec) * 1.e-9;
+  double currenttime = calculateCorrectTimeDifference(
+                  std::make_pair( time1.tv_sec, time1.tv_nsec),
+                  std::make_pair( basetime.tv_sec, basetime.tv_nsec)
+                  );
 #else
 #ifdef HAVE_SYS_TIME_H
@@ -121 +132 @@
   // gettimeofday gives microseconds accuracy
   gettimeofday(&time1, &timezone1);
-  double currenttime;
-  if (time1.tv_usec < basetime.tv_usec)
-    currenttime = (time1.tv_sec - basetime.tv_sec - 1)
-        + (basetime.tv_usec - time1.tv_usec) * 1.e-6;
-  else
-    currenttime = (time1.tv_sec - basetime.tv_sec)
-        + (time1.tv_usec - basetime.tv_usec) * 1.e-6;
+  double currenttime = calculateCorrectTimeDifference(
+      std::make_pair( time1.tv_sec, time1.tv_usec),
+                  std::make_pair( basetime.tv_sec, basetime.tv_usec)
+                  );
 #else
 #ifdef HAVE_SYS_TIMES_H
   // clock is only accurate up to milliseconds
   struct tms *buffer = new tms;
-  double currenttime;
   if (times(buffer) != (clock_t)(-1))
     currenttime =