Skip to content

Update kernel.py #37

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update kernel.py #37

wants to merge 1 commit into from

Conversation

kl2792
Copy link

@kl2792 kl2792 commented Oct 9, 2017

Fix #36 on MacOS, have not tested on Linux, though I don't see why it wouldn't (I use only built-ins).

@brendan-rius
Copy link
Owner

brendan-rius commented Oct 9, 2017

I would accept it, but you actually removed the use of master.c which is needed to disable stdout buffering :/

@kl2792
Copy link
Author

kl2792 commented Oct 12, 2017

Hm, I'm just curious -- why should stdout buffering be disabled? Isn't it the default for C programs?

@brendan-rius
Copy link
Owner

There are some cases where there is a long loop and something being printed in it.
For example:

for (int i = 0; i < 1000000; i++) {
  printf("%d\n", i);
  sleep_10_ms();
}

we want the result to be printed as the loop loops, not after it has ended

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants