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

fluent-builder: improve inheritance handling #82

Merged
merged 6 commits into from
Sep 13, 2024

Conversation

flaktack
Copy link
Contributor

Thank you for your work @mklemm! While updating from an older version we found two corner-cases related to fluent-builder inheritance handling which this PR improves:

  1. with*() methods were not generated for superclasses due to differing checks. This was corrected by using the same checks to determine if the methods should be generated. (6edaa7d, Inheritance is creating empty builder methods #79)

  2. when extending classes from previous runs of xjc using episodes the Builder inner classes were not found. This was corrected by modifying the isLocal() checks to also look at existing classes. (c5159fa)

Failing tests are added for both cases (ad02367, a5d93ef) using extracts from the SIRI and NeTEx standards, which pass after the relevant commits.

@mklemm mklemm merged commit 0cd5a49 into mklemm:master Sep 13, 2024
@flaktack
Copy link
Contributor Author

Thanks for the quick merge! Would it be possible to have a new release with the merged commits?

@flaktack flaktack deleted the feature/fluent-builder branch September 13, 2024 10:24
@mklemm
Copy link
Owner

mklemm commented Sep 13, 2024 via email

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