-
-
Notifications
You must be signed in to change notification settings - Fork 295
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor Child.main to handle EOF #631
Conversation
self._unpacker.feed(sys.stdin.buffer.read(1)) | ||
return self._unpacker | ||
while True: | ||
feed = sys.stdin.buffer.raw.read(102400) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This also reads up to 100kb (I am seeing ~10kb at minimum here).
Reading this byte per byte seems to be very wasteful.
It is very big change. I will test it later. |
More testers are needed. |
I've seen child processes being left over after quitting Neovim, looping in `read()`. This appears to fix it. Maybe this is what Shougo#625 tried to address also?!
I haven't test yet but code looks quite simple and efficient. LGTM |
I will test it later. |
It seems works. I will merge it later. |
Merged. |
I've seen child processes being left over after quitting Neovim, looping
in
read()
.This appears to fix it.
Maybe this is what #625
tried to address also?!