[riot-notifications] [RIOT-OS/RIOT] sys/shell: add heap command (#10953)

Martine Lenders notifications at github.com
Wed Apr 24 23:18:51 CEST 2019


> @smlng What could be the right place for the definition of the data structure heap_info_t? Do you have any suggestion? Shell command modules usually don't have header files.

How about introducing a `heap_stats` module that comes with its own header? This might also provide the possibility to remove the need for the awkward `HAVE_HEAP_STATS` define and could be replaced with `MODULE_HEAP_STATS`. However, as the `heap_stats()` function is not introduced in this PR I disagree with @smlng, that the refactoring should be done in this PR. Rather, this should be done in a follow-up IMHO.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/10953#issuecomment-486430666
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190424/accb5cea/attachment.html>


More information about the notifications mailing list