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
use TextIOWrapper when communicating with pandoc subprocess #4205
Conversation
uses 'universal-newline' mode, to avoid confusion with CRLF. closes ipython#4122
The alternative would be to |
That sounds like a job for |
TextIOWrapper is about 2-5x as fast as split/rejoin (not that this is ever an expensive operation), so I'll stick with the current implementation. |
Great, my tests showed a smaller difference in the same direction. |
Adding a test before merging. |
test added and passed on OS X and Windows 7, just waiting on Travis. |
Travis is happy as well. Merging. |
use TextIOWrapper when communicating with pandoc subprocess
use TextIOWrapper when communicating with pandoc subprocess
uses 'universal-newline' mode, to avoid any confusion with CRLF.
closes #4122
closes #3819
alternative to #4202