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

[BUGS#1247] fix: filters4: create event reader when entering event mode #945

Merged

Conversation

miurahr
Copy link
Member

@miurahr miurahr commented Feb 5, 2024

  • Complete solution to avoid StaX implementation difference.

Pull request type

  • Bug fix -> [bug]

Which ticket is resolved?

  • BUGS#1247

What does this PR change?

  • Create event reader when entering event mode

Other information

#937

- Complete solution to avoid StaX implementation difference.

Signed-off-by: Hiroshi Miura <miurahr@linux.com>
- AbstractXmlFilter: variable name, case-default
- OpenXmlFilter: variable name, suppression update

Signed-off-by: Hiroshi Miura <miurahr@linux.com>
@miurahr
Copy link
Member Author

miurahr commented Feb 8, 2024

#947 back-port the improvement here into 6.0 branch.

@omegat-org omegat-org deleted a comment from github-actions bot Feb 9, 2024
@miurahr miurahr merged commit f59958c into master Feb 10, 2024
9 checks passed
@miurahr miurahr deleted the topic/miurahr/filters4/fix-event-reader-difference-impl branch February 10, 2024 06:42
chelobaka pushed a commit to chelobaka/omegat that referenced this pull request Feb 10, 2024
…de (omegat-org#945)

* [BUGS#1247] fix: filters4: create event reader when entering event mode

- Complete solution to avoid StaX implementation difference.

Signed-off-by: Hiroshi Miura <miurahr@linux.com>

* style: improve filters4 styles

- AbstractXmlFilter: variable name, case-default
- OpenXmlFilter: variable name, suppression update

Signed-off-by: Hiroshi Miura <miurahr@linux.com>

---------

Signed-off-by: Hiroshi Miura <miurahr@linux.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
1 participant