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

Comparing files results in nullpointer in main #2

Closed
Siltyx opened this issue Jun 12, 2019 · 5 comments
Closed

Comparing files results in nullpointer in main #2

Siltyx opened this issue Jun 12, 2019 · 5 comments

Comments

@Siltyx
Copy link

Siltyx commented Jun 12, 2019

java.lang.NullPointerException
at io.github.valters.xsdiff.app.Main$App.run(Main.java:87)
at io.github.valters.xsdiff.app.Main.main(Main.java:52)

When comparing OpenDrive 1.4 H Schema to OpenDrive 1.5 M schema. Download here http://www.opendrive.org/download.html.

@valters
Copy link
Owner

valters commented Jun 12, 2019

Are you somehow comparing it in root folder? The code seems to rely on having file.parent field not null. Not sure what's going on there - it's been years since I had to use this tool, to be honest.

@Siltyx
Copy link
Author

Siltyx commented Jun 12, 2019

No not comparing in root folder. Another funny thing is if you change the command to java -jar xsdiff-app-1.1.0.jar a/OpenDRIVE_1.4H.xsd b/OpenDRIVE_1.5M.xsd the error changes. See error.txt
error.txt

@valters
Copy link
Owner

valters commented Jun 12, 2019

Ah, the XSD structure is just unexpected there. Something that code does not anticipate.

@Siltyx
Copy link
Author

Siltyx commented Jun 12, 2019

ok then this tool cant help me here, but thank you

@valters
Copy link
Owner

valters commented Aug 26, 2019

I corrected the issue regarding NPE, it was caused by incorrect handling of xsd:documentation node. @Siltyx I did try to comparte OpenDrive 1.4 vs 1.5 schemas, but they are so much different, the diff report is not useful at all. They have dropped so many types and introduced so many new ones, the tool simply gets confused. It would need new approach to be able to handle changes on that level.

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

No branches or pull requests

2 participants