acassis commented on code in PR #2131:
URL: https://github.com/apache/nuttx-apps/pull/2131#discussion_r1361076578
##########
crypto/mbedtls/Kconfig:
##########
@@ -17,7 +17,7 @@ config MBEDTLS_VERSION
config MBEDTLS_DEBUG_C
bool "This module provides debugging functions."
- default n
+ default y
Review Comment:
I think the debugging functions should be enabled by default because it
could increase the final firmware size if the user didn't see it is enabled.
Maybe you could use "default y if DEBUG_FEATURES"
--
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
