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

Better XdmNode to InputSource conversion #86

Closed
wants to merge 2 commits into
base: saxon94
from

Conversation

Projects
None yet
2 participants
@rdeltour
Contributor

rdeltour commented Mar 7, 2013

The proposed change serializes the XdmNode to a temporary ByteArray instead of a String. It's more efficient and prevents some OutOfMemoryErrors.

@rdeltour

This comment has been minimized.

Show comment
Hide comment
@rdeltour

rdeltour Mar 8, 2013

Contributor

Please do not apply this one yet, I just found out there's an issue with it (it breaks some of my pipelines), but not sure where yet.
Sorry for the noise.

Contributor

rdeltour commented Mar 8, 2013

Please do not apply this one yet, I just found out there's an issue with it (it breaks some of my pipelines), but not sure where yet.
Sorry for the noise.

@rdeltour

This comment has been minimized.

Show comment
Hide comment
@rdeltour

rdeltour Mar 8, 2013

Contributor

Should be OK now!

Contributor

rdeltour commented Mar 8, 2013

Should be OK now!

ndw added a commit that referenced this pull request Mar 8, 2013

@ndw ndw closed this Mar 8, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment