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

add ClassInfo.memberClasses #185

Merged
merged 1 commit into from
Apr 14, 2022
Merged

Conversation

Ladicek
Copy link
Contributor

@Ladicek Ladicek commented Apr 14, 2022

This allows navigating from ClassInfo to its member classes.
The memberClasses() method returns a set of DotName objects,
because member classes don't necessarily have to be present in
the index. It is a responsibility of the caller to retrieve
a ClassInfo of each member class from the index and handle
possible absence.

This change requires persistent format version bump. That has already
happened on the smallrye branch, so this commit doesn't bump
again, but note that this commit can't be backported to Jandex 2.x.

Resolves #180

This allows navigating from `ClassInfo` to its member classes.
The `memberClasses()` method returns a set of `DotName` objects,
because member classes don't necessarily have to be present in
the index. It is a responsibility of the caller to retrieve
a `ClassInfo` of each member class from the index and handle
possible absence.

This change requires persistent format version bump. That has already
happened on the `smallrye` branch, so this commit doesn't bump
again, but note that this commit can't be backported to Jandex 2.x.
@Ladicek Ladicek added the serialization-format-change Changes that affect index serialization format label Apr 14, 2022
@Ladicek Ladicek added this to the 3.0.0 milestone Apr 14, 2022
@Ladicek Ladicek linked an issue Apr 14, 2022 that may be closed by this pull request
@Ladicek Ladicek merged commit ad74eb1 into smallrye:smallrye Apr 14, 2022
@Ladicek Ladicek deleted the member-classes branch April 14, 2022 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
serialization-format-change Changes that affect index serialization format
Projects
None yet
Development

Successfully merging this pull request may close these issues.

allow navigating from ClassInfo to its member classes
1 participant