davids5 commented on code in PR #9499:
URL: https://github.com/apache/nuttx/pull/9499#discussion_r1230540219


##########
Kconfig:
##########
@@ -1904,7 +1904,7 @@ config HEAP_COLORATION
 
 config DEBUG_SYMBOLS
        bool "Generate Debug Symbols"
-       default n
+       default y

Review Comment:
   Please do not couple optimization setting and symbols. 
   
   This is easier, but not the way the release code will really run. Chaining 
it may make an image not fit in Flash or change the target of a crash. 
   
   Please do no make is easier for the user by turning on symbols 
automatically. 
   
   It is a trivial change a user can make and does not expose them to 
unconscious security issues without their own actions.  
   
   



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