xiaoxiang781216 commented on PR #17409: URL: https://github.com/apache/nuttx/pull/17409#issuecomment-3595927807
> these are not standard macros, not defined in POSIX or any other OS. I don't think something like this should be merged without much thinking. At least it should be clearly stated that these macros are non-standard.`inttype.h` is standard defined header and I'm not sure if we should place nuttx-specific code here. 32-bit `time_t` will be removed in the future (not compatible with POSIX), so these custom macros will become useless. this macro is specific designed for 32bit/64 time_t/clock_t, once 32bit version is removed, all these macros can be removed too. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
