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