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.

Sign up for GitHub

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

Jump to bottom

fix: Stack overflow on multi maps (and other non-trivial generic classes) #4413

Closed
wants to merge 1 commit into from
Closed

fix: Stack overflow on multi maps (and other non-trivial generic classes) #4413

wants to merge 1 commit into from

Commits on Sep 13, 2022

  1. fix: Stack overflow on multi maps (and other non-trivial generic clas… 

    …ses)

Logic in sundrio that is responsible for replacing generic type arguments on methods and properties with their concrete instantiations from subclasses contained a bug, in which it looped infinitely if the same type argument name was used at multiple classes in the hierarchy. A common occurence were multimaps.

A very similar code was also present in crd-generator, also suffering from the same bug.

Sundrio has been updated to account for this situation, and updated utilities from sundrio have been introduced to crd-generator, which replace the previous implementation of teh same algorithm.

Fixes fabric8io#4357
    @xRodney
    xRodney committed Sep 13, 2022
    Configuration menu
    Copy the full SHA
    1a3960b View commit details
    Browse the repository at this point in the history