slorquet commented on code in PR #8867:
URL: https://github.com/apache/nuttx/pull/8867#discussion_r1144809842


##########
libs/libc/string/Kconfig:
##########
@@ -90,3 +90,29 @@ config MEMSET_64BIT
                efficiently.
 
 endmenu # memcpy/memset Options
+
+menu "signal Decode Support"
+
+config LIBC_STRSIGNAL
+       bool "Enable strsignal"
+       default y

Review Comment:
   Can we warn the user that strsignal is not enabled if they use it and this 
config is not set?
   
   What
   #ifndef CONFIG_LIBC_STRSIGNAL
   #define strsignal(val) static_assert(false,"strsignal needs to be enabled in 
config")
   #endif
   
   This is probably not possible as-is but maybe there is another practical way 
to inform the user that CONFIG_LIBC_STRSIGNAL is required at build time?



-- 
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

Reply via email to