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

Fixed EOF Error #332

Merged
merged 1 commit into from
Mar 19, 2019
Merged

Fixed EOF Error #332

merged 1 commit into from
Mar 19, 2019

Conversation

NikolasKomonen
Copy link
Contributor

@NikolasKomonen NikolasKomonen commented Mar 18, 2019

Fixes #157

The most common ocurrences of this error are
when the document is empty or there is just the
xml prolog (<?xml ... ?>). All errors that give negative values
will be ignored for now.

Signed-off-by: Nikolas [email protected]

@fbricon
Copy link
Contributor

fbricon commented Mar 18, 2019

unit test?

@fbricon
Copy link
Contributor

fbricon commented Mar 18, 2019

I'd expect an error to be raised to the user. When only the xml prolog is present in a file, Intellij shows: "Valid XML document should have a root tag".

Fixes eclipse#157

The most common ocurrences of this error are
when the document is empty or there is just the
xml prolog (<?xml ... ?>). Errors that give negative values
will be ignored for now.

Signed-off-by: Nikolas <[email protected]>
@NikolasKomonen
Copy link
Contributor Author

@fbricon fixed the unit test names

@fbricon fbricon merged commit 6194bdb into eclipse:master Mar 19, 2019
@NikolasKomonen NikolasKomonen deleted the prematureEOFError branch April 5, 2019 14:03
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