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.

Sign up for GitHub

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

Jump to bottom

Fix writing events using Saxon #317

Merged
merged 3 commits into from
Feb 12, 2025
Merged

Fix writing events using Saxon #317

merged 3 commits into from
Feb 12, 2025

Conversation

marcphilipp
Copy link
Member

  • Add Saxon test task
  • Relax assertion because Saxon handles CDATA differently but correctly
  • Avoid closing the underlying XMLStreamWriter too early

Fixes #315.

@marcphilipp
Add Saxon test task
7aecc07
@marcphilipp
Relax assertion because Saxon handles CDATA differently but correctly
f6b1db1
@marcphilipp
Avoid closing the underlying XMLStreamWriter too early
e9b20e6 
Contrary to other implementations, Saxon closes the passed
`XMLStreamWriter` after transforming DOM to StAX which caused a
`NullPointerException` on the subsequent call to `flush()`.

Fixes #315.
@marcphilipp marcphilipp self-assigned this Feb 12, 2025
@marcphilipp marcphilipp merged commit aace613 into main Feb 12, 2025
5 checks passed
@marcphilipp marcphilipp deleted the marc/saxon branch February 12, 2025 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@marcphilipp marcphilipp

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

NullPointerException when using Saxon to write events XML
1 participant
@marcphilipp