You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2021/10/19 05:31:14 UTC

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #4693: time:follow POSIX use int64_t to define time_t

xiaoxiang781216 commented on a change in pull request #4693:
URL: https://github.com/apache/incubator-nuttx/pull/4693#discussion_r731506736



##########
File path: include/sys/types.h
##########
@@ -230,8 +230,10 @@ typedef uint16_t     sa_family_t;
 
 #ifdef CONFIG_SYSTEM_TIME64
 typedef uint64_t     clock_t;
+typedef int64_t      time_t;
 #else
 typedef uint32_t     clock_t;
+typedef int32_t      time_t;
 #endif
 

Review comment:
       time.h include sys/types.h at the begin, so there is no real difference from the user perspective. putting here is to make all CONFIG_SYSTEM_TIME64 related public definition in the same place.




-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org