<blockquote>
<p>Does wrapping the shell in a while loop solve everyone's concerns with exiting on ^D?</p>
</blockquote>
<p>So this is how <code>shell_run()</code> currently looks like:</p>
<p><a href="https://github.com/RIOT-OS/RIOT/blob/711854562b77e2c9165942f9dc1a3ecef32dc8d0/sys/shell/shell.c#L293-L310">https://github.com/RIOT-OS/RIOT/blob/711854562b77e2c9165942f9dc1a3ecef32dc8d0/sys/shell/shell.c#L293-L310</a></p>
<p>Currenlty, ^D is just ignored (<code>handle_input_line()</code> sees <code>*pos <= ' '</code> so nothing happens). The functionality introduced here causes the function to go into the if condition and break that loop. Adding another while loop around the existing while loop however would have the same behavior, just with another while loop around: Hitting ^D would finish the inner while loop, the outer while loop would then cause another iteration of the inner while loop. So I don't really get the point of having this then except for the <code>handle_input_line()</code> not being entered.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/RIOT-OS/RIOT/pull/10788?email_source=notifications&email_token=ABE7WYHJ36RHGRWHE5RZM2TQKMPKRA5CNFSM4GQN74EKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7CN4QA#issuecomment-532995648">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABE7WYE5BWTWNESOR7ZCTOLQKMPKRANCNFSM4GQN74EA">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABE7WYHOSDPP4LJRVLWW3KDQKMPKRA5CNFSM4GQN74EKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7CN4QA.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/RIOT-OS/RIOT/pull/10788?email_source=notifications\u0026email_token=ABE7WYHJ36RHGRWHE5RZM2TQKMPKRA5CNFSM4GQN74EKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7CN4QA#issuecomment-532995648",
"url": "https://github.com/RIOT-OS/RIOT/pull/10788?email_source=notifications\u0026email_token=ABE7WYHJ36RHGRWHE5RZM2TQKMPKRA5CNFSM4GQN74EKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7CN4QA#issuecomment-532995648",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>