win: more reliable uv__hrtime precision

Fishrock123 · Fishrock123 · commit 8efe97d8abc8 · 2015-04-10T12:44:20.000-04:00
Reduces floating-point operations that can cause precision loss. Thanks to @Arnavion for suggesting the fix: nodejs/node#1272 (comment)
diff --git a/src/win/util.c b/src/win/util.c
@@ -58,8 +58,8 @@
 static char *process_title;
 static CRITICAL_SECTION process_title_lock;
 
-/* Interval (in seconds) of the high-resolution clock. */
-static double hrtime_interval_ = 0;
+/* Interval of the high-resolution clock. */
+static uint64_t hrtime_interval_ = 0;
 
 
 /*
@@ -71,11 +71,9 @@ void uv__util_init() {
   /* Initialize process title access mutex. */
   InitializeCriticalSection(&process_title_lock);
 
-  /* Retrieve high-resolution timer frequency
-   * and precompute its reciprocal. 
-   */
+  /* Retrieve high-resolution timer frequency */
   if (QueryPerformanceFrequency(&perf_frequency)) {
-    hrtime_interval_ = 1.0 / perf_frequency.QuadPart;
+    hrtime_interval_ = perf_frequency.QuadPart;
   } else {
     hrtime_interval_= 0;
   }
@@ -484,7 +482,7 @@ uint64_t uv__hrtime(double scale) {
    * performance counter interval, integer math could cause this computation
    * to overflow. Therefore we resort to floating point math.
    */
-  return (uint64_t) ((double) counter.QuadPart * hrtime_interval_ * scale);
+  return (uint64_t) ((counter.QuadPart / hrtime_interval_) * scale);
 }
 
 
