Skip to content
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

Copy bytes of the stream to use it correctly more than once. #6

Merged
merged 1 commit into from
Jun 26, 2018

Conversation

DavidLuptak
Copy link
Member

This was probably the cause of the issue reported in MathMLCan repo.

The inputStream was "empty" when used for the second time and therefore some odd exception was raised.

@Witiko
Copy link
Member

Witiko commented Jun 23, 2018

Nice catch.

@DavidLuptak DavidLuptak merged commit 3dbaa02 into MIR-MU:master Jun 26, 2018
@Witiko
Copy link
Member

Witiko commented Jun 26, 2018

Future pull requests should now trigger continuous integration (see CircleCI → MIR-MU → MIaS → Settings → Advanced Settings → Build forked pull requests).

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

Successfully merging this pull request may close these issues.

2 participants