Fixed windows-specific problem in subprocess_repl.read_bytes()

[simonzack]

Fixed windows-specific problem in subprocess_repl.read_bytes(), so that more than 1 byte is returned on every read, by converting the pipe to an asynchronous one.

This issue comes up as a bug at some places, for example open an ipython repl prompt, then enter some non-existent variable name to generate an error. Then the color escape sequences are not stripped even when the option is set to True in SublimeREPL, as the color stripping function only has a single byte as input instead of all the output bytes.

This fixes the aforementioned problem.

[wuub]

@simonzack thanks :)

Are you using Sublime Text 2 or 3? Did you test both and are all repls working as before?
I'm trying to decide how much testing should I do before releasing this ;)

[simonzack]

I think it should work fine after the new commits.
On sublime text 3 build 3059, the repls I've tested are: python, ipython, haskell (ghc), node.js, windows shell, and powershell.
I also tested sublime text 2's windows shell.
They are all the shells I have, hope it works for all the other ones :)

[simonzack]

Any updates on this? Thanks.

[gacelita]

Please solve the conflicts in repl.js, since I lack the knowledge to do so, and then we'll merge this in merges.