acassis commented on code in PR #19179: URL: https://github.com/apache/nuttx/pull/19179#discussion_r3481147330
########## include/nuttx/fs/hostfs.h: ########## @@ -110,27 +110,29 @@ /* These must match the definitions in include/sys/types.h */ typedef int16_t nuttx_blksize_t; + # ifdef CONFIG_SMALL_MEMORY -typedef int16_t nuttx_gid_t; -typedef int16_t nuttx_uid_t; Review Comment: @xiaoxiang781216 I think instead of removing support to multi-user from SMALL_MEMORY, it is better just to keep it disabled by default. Even for a MCU with feel KBs of RAM could be useful to have option to users login, for minimal security. -- 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]
