Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

StackOverflowError when parsing generic signature #247

Closed
Ladicek opened this issue Aug 25, 2022 · 0 comments · Fixed by #248
Closed

StackOverflowError when parsing generic signature #247

Ladicek opened this issue Aug 25, 2022 · 0 comments · Fixed by #248
Assignees
Milestone

Comments

@Ladicek
Copy link
Contributor

Ladicek commented Aug 25, 2022

Byte Buddy's ByteCodeElement.TypeDependant type includes a generic signature that confuses Jandex's parser, which ends up in an infinite regress: public interface TypeDependant<T extends TypeDependant<?, S>, S extends Token<S>> { ... }. The T type parameter is defined recursively, but not recursively in T. When adding tests for recursive type variables, I forgot to add a test for this.

Fix actually simplifies the method that figures out if a type parameter is recursive, so this is a double win.

@Ladicek Ladicek added this to the 3.0.0 milestone Aug 25, 2022
@Ladicek Ladicek self-assigned this Aug 25, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant