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

Jump to bottom

Add getCurrentLocation()/getTokenLocation() to replace currentLocation()/currentTokenLocation() #671

Closed
cowtowncoder opened this issue Jan 27, 2021 · 1 comment
Closed

Add getCurrentLocation()/getTokenLocation() to replace currentLocation()/currentTokenLocation() #671

cowtowncoder opened this issue Jan 27, 2021 · 1 comment
Labels
3.x
Milestone
2.13.0

Comments

@cowtowncoder
Copy link
Member

cowtowncoder commented Jan 27, 2021 •
edited

As part of on-going naming clean up for Jackson 3.x, let's change these 2 location accessor as per title.
And with some more thought, add these also as aliases in 2.13 (but not yet deprecate old variants; may do that in 2.14 or later).
@cowtowncoder cowtowncoder added this to the 3.0.0 milestone Jan 27, 2021
@cowtowncoder cowtowncoder reopened this Jan 28, 2021
@cowtowncoder
Copy link
Member Author

Actually I think it'd make sense to add these aliases in 2.13 for forwards-compatibility as well, reopening.

@cowtowncoder cowtowncoder changed the title Replace getCurrentLocation()/getTokenLocation() with currentLocation()/currentTokenLocation() Add getCurrentLocation()/getTokenLocation() to replace currentLocation()/currentTokenLocation() Jan 28, 2021
@cowtowncoder cowtowncoder modified the milestones: 3.0.0, 2.13.0 Jan 28, 2021
cowtowncoder added a commit that referenced this issue Jan 28, 2021
@cowtowncoder
Backport #671
dfa3b11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
3.x
Projects
None yet
Milestone
2.13.0
Development

No branches or pull requests
1 participant
@cowtowncoder