[riot-notifications] [RIOT-OS/RIOT] Add stdin pseudo module and set it as dependency for applications using getchar (#11598)

Alexandre Abadie notifications at github.com
Wed Jun 12 09:48:10 CEST 2019


I don't think `stdio_read` is supposed to return EOF directly, this should be done is `syscall.c` related function. For example, in newlib, one has `_read_r`: here the errno can be set to the `_reent` variable and EOF returned if `stdio_read` return value is < 0.

But then look at other getchar implementation, e.g. atmega, msp430 they should be changed to correctly return EOF when stdio_read fails.

IMHO, this should be done in follow-up PRs and it needs a tracking issue. What do you think ?

-- 
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/11598#issuecomment-501160046
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190612/34e91740/attachment.html>


More information about the notifications mailing list