"Chip Davis" cdavis@codeweavers.com wrote:
+#ifdef HAVE_MACH_CONTINUOUS_TIME
- if (&mach_continuous_time != NULL)
- return mach_continuous_time() * timebase.numer / timebase.denom / 100;
+#endif
While the compiler will fix it for you it would still be better to use 'if (mach_continuous_time != NULL)' IMO, so that even if the type of the 'mach_continuous_time' is a real function pointer it still works. Same applies to the next patch. That's a pure cosmetics though.
In the case that mach_continuous_time is linked strong, which it will be if -mmacosx-version-min=10.12 or higher, the compiler will warn if I *don't* use '&'.
I see, thanks for the explanation.