[riot-notifications] [RIOT-OS/RIOT] sys/shell: Exit the shell on ctrl-D (#10788)

Martine Lenders notifications at github.com
Thu Sep 19 09:25:26 CEST 2019


Ok, got it, but to quote yourself:

> We just need to wrap existing consumers of the shell with a while loop to avoid functional changes in existing code which doesn't wish to have a shell capable of exiting.

would mean to avoid functional changes we need to touch _every_ call of `shell_run` then... Given that we use that a lot, this seems somewhat of a big effort to me. As a compromise: how about introducing `shell_run_once()` to the shell's with the current functionality of `shell_run()` (+ the one introduced in this PR) and moving `shell_run()` to a function that calls `shell_run_once()` in an infinite loop?

-- 
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/10788#issuecomment-533003772
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190919/73b82f72/attachment.htm>


More information about the notifications mailing list