pkarashchenko commented on code in PR #8867: URL: https://github.com/apache/nuttx/pull/8867#discussion_r1143853597
########## libs/libc/string/lib_strsignal.c: ########## @@ -96,80 +69,51 @@ FAR char *strsignal(int signum) switch (signum) { +#ifdef CONFIG_LIBC_STRSIGNAL Review Comment: > CONFIG_LIBC_STRSIGNAL=y/CONFIG_LIBC_STRSIGNAL_SHORT=n and CONFIG_LIBC_STRSIGNAL=n `CONFIG_LIBC_STRSIGNAL=y` enables `strsignal()` to produce strings different from `"Signal X"` for well known signal numbers. `CONFIG_LIBC_STRSIGNAL_SHORT=n` will enable more human friendly strings compared to `y` when the string will be just a string alias `SIGHUP` will get `"SIGHUP"` string. Please take a look into the Kconfig description. -- 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