diff oct-conf-post.in.h @ 30826:243b51ec9ff0 stable

Remove `time_t` from API (bug #61821). The bit-width of `time_t` might change depending on compile time switches. That can cause a mismatch between class declarations in public headers and the actual alignment of the compiled classes. Avoid that problem by not using `time_t` in (public) headers. * build-aux/mk-octave-config-h.sh, oct-conf-post.in.h: Define OCTAVE_TIME_T to `int64_t`. The type of `time_t` is not defined in the C/C++ standards. However in all(?) implementations, it's either a signed 32-bit or a signed 64-bit integer. Use a signed 64-bit integer for storing those values in Octave classes. * other files: Use OCTAVE_TIME_T instead of `time_t`. Don't include <ctime> in files where it's not needed.
author Markus Mützel <markus.muetzel@gmx.de>
date Sat, 26 Feb 2022 13:46:48 +0100
parents 796f54d4ddbf
children
line wrap: on
line diff
--- a/oct-conf-post.in.h	Thu Mar 10 18:58:29 2022 +0100
+++ b/oct-conf-post.in.h	Sat Feb 26 13:46:48 2022 +0100
@@ -279,6 +279,9 @@
 
 #define OCTAVE_HAVE_F77_INT_TYPE 1
 
+/* time type in API is always 64 bits wide */
+#define OCTAVE_TIME_T int64_t
+
 #if defined (__cplusplus) && ! defined (OCTAVE_THREAD_LOCAL)
 #  define OCTAVE_THREAD_LOCAL
 #endif