pkarashchenko commented on issue #8869: URL: https://github.com/apache/nuttx/issues/8869#issuecomment-1478716260
We already have ``` #ifdef CONFIG_SYSTEM_TIME64 typedef uint64_t time_t; /* Holds time in seconds */ #else typedef uint32_t time_t; /* Holds time in seconds */ #endif ``` so I was thinking about use cases for systems that do not have `CONFIG_HAVE_LONG_LONG` (about if 4 realtime signals would be enough). Using `uint64_t` for sigset will hide a lot of work behind the scene (done by the compiler) and modification to existing code would be minimal plus compiler can insert assembler if ISA can deal natively with 64 bit values, so I was considering this only from such perspective. Going with `uint32_t [2]` array is obviously the most portable solution. -- 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: commits-unsubscr...@nuttx.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
