xiaoxiang781216 commented on a change in pull request #4193:
URL: https://github.com/apache/incubator-nuttx/pull/4193#discussion_r675285099
##########
File path: include/nuttx/fs/hostfs.h
##########
@@ -106,8 +106,13 @@ typedef int16_t nuttx_uid_t;
typedef uint16_t nuttx_dev_t;
typedef uint16_t nuttx_ino_t;
typedef uint16_t nuttx_nlink_t;
+#ifdef CONFIG_FS_LARGEFILE
+typedef int64_t nuttx_off_t;
Review comment:
Yes, I found this issue before, but it's hard to let FS_LARGEFILE
depends on LONG_LONG since LONG_LONG isn't a Kconfig option, but auto define in
include/nuttx/compiler.h:
https://github.com/apache/incubator-nuttx/blob/master/include/nuttx/compiler.h#L169
Do you have some better method to resolve this issue?
--
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]