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

DelegatingDeserializer missing override of getAbsentValue() (and couple of other methods) #3748

Closed
cowtowncoder opened this issue Jan 22, 2023 · 0 comments
Closed

DelegatingDeserializer missing override of getAbsentValue() (and couple of other methods) #3748

cowtowncoder opened this issue Jan 22, 2023 · 0 comments
Milestone
2.15.0

Comments

@cowtowncoder
Copy link
Member

Looks like some of the methods added in later 2.x versions are missing overrides in DelegatingDeserializer:

  • unwrappingDeserializer() (added in )
  • getAbsentValue() (added in 2.13)
  • getEmptyAccessPattern() (added in 2.9)
  • deserialize(JsonParser p, DeserializationContext ctxt, T intoValue) (added in 2.10)

So let's add them.
@cowtowncoder cowtowncoder added to-evaluate Issue that has been received but not yet evaluated 2.15 and removed to-evaluate Issue that has been received but not yet evaluated labels Jan 22, 2023
cowtowncoder added a commit that referenced this issue Jan 22, 2023
@cowtowncoder
Fix #3748: add missing overrides in DelegatingDeserializer
444f150
@cowtowncoder cowtowncoder added this to the 2.15.0 milestone Jan 22, 2023
@fdutton fdutton mentioned this issue Feb 13, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.15.0
Development

No branches or pull requests
1 participant
@cowtowncoder