Fix comments in instr_time.h and remove an unneeded cast to int64
authorHeikki Linnakangas <[email protected]>
Tue, 6 Aug 2024 11:28:02 +0000 (14:28 +0300)
committerHeikki Linnakangas <[email protected]>
Tue, 6 Aug 2024 11:28:02 +0000 (14:28 +0300)
03023a2664 represented time as an int64 on all platforms but forgot to
update the comment related to INSTR_TIME_GET_MICROSEC() and provided
an incorrect comment for INSTR_TIME_GET_NANOSEC().

In passing remove an unneeded cast to int64.

Author: Bertrand Drouvot
Discussion: https://www.postgresql.org/message-id/[email protected]

src/include/portability/instr_time.h

index a6fc1922f2080d5d536bd391d21c06626f31a398..8f9ba2f151f11d45aba82d4f8add43a6e2fcb266 100644 (file)
@@ -32,9 +32,9 @@
  *
  * INSTR_TIME_GET_MILLISEC(t)          convert t to double (in milliseconds)
  *
- * INSTR_TIME_GET_MICROSEC(t)          convert t to uint64 (in microseconds)
+ * INSTR_TIME_GET_MICROSEC(t)          get t in microseconds
  *
- * INSTR_TIME_GET_NANOSEC(t)           convert t to uint64 (in nanoseconds)
+ * INSTR_TIME_GET_NANOSEC(t)           get t in nanoseconds
  *
  * Note that INSTR_TIME_SUBTRACT and INSTR_TIME_ACCUM_DIFF convert
  * absolute times to intervals.  The INSTR_TIME_GET_xxx operations are
@@ -123,7 +123,7 @@ pg_clock_gettime_ns(void)
        ((t) = pg_clock_gettime_ns())
 
 #define INSTR_TIME_GET_NANOSEC(t) \
-       ((int64) (t).ticks)
+       ((t).ticks)
 
 
 #else                                                  /* WIN32 */