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

Implement new EnumResolver.constructUsingMethod() via AnnotatedClass instead of Class<?> #4034

Merged

Conversation

JooHyukKim
Copy link
Member

parent issue : #3990

Motivation

Modification

  • Same as motivation

@cowtowncoder cowtowncoder merged commit 746adc6 into FasterXML:2.16 Jul 13, 2023
@cowtowncoder
Copy link
Member

Thank you @JooHyukKim !

The only (?) call to deprecated lookup now seems to be from EnumResolver:

    @Deprecated
    public static EnumResolver constructUsingIndex(DeserializationConfig config,
            Class<Enum<?>> enumCls0)

or rather, only thing called by non-deprecated code -- everything else is from deprecated methods.
With 2.x we need to keep deprecated methods around for a bit, but for master/3.0 we are almost done. :)

@JooHyukKim JooHyukKim deleted the retrofit_constructUsingMethod branch July 14, 2023 08:36
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