[GitHub] [incubator-nuttx] fjpanag commented on pull request #6685: mm/mm_heap: update MM_BACKTRACE and malloc failed output

[GitBox]

fjpanag commented on PR #6685:
URL: https://github.com/apache/incubator-nuttx/pull/6685#issuecomment-1193962303

   > If we use another option to enable the pid track, the code will become 
more complex, because user can enable backtrace, but disable pid.
   
   I don't think this is a real problem.
   
   You can do it like this:
   
   ```
   
   config MM_OWNERSHIP
        bool "Owner tracking"
        default n
        ---help---
                Enables ownership tracking in heap allocations.
   
   config MM_BACKTRACE
        bool "Backtrace"
        default n
        depends on MM_OWNERSHIP && DEBUG_MM
        ---help---
                Enables backtrace printing.
   
   ```
   
   The dependencies are resolved by Kconfig.  
   The code is much cleaner, I believe, and the features are enabled in 
*compile* 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