Skip to content

Add an ObjectMapper shorthand #620

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
Merged
merged 2 commits into from
Jan 15, 2022
Merged

Add an ObjectMapper shorthand #620

merged 2 commits into from
Jan 15, 2022

Conversation

@cyberdelia
Copy link
Contributor

@cyberdelia cyberdelia commented Jan 14, 2022

Add an ObjectMapper shorthand, allowing a less verbose initialization:

ObjectMapper objectMapper = new MessagePackMapper();

Rather than:

ObjectMapper objectMapper = new ObjectMapper(new MessagePackFactory());

It's similar to other dataformat implementation as well, see: https://github.com/FasterXML/jackson-dataformats-binary/tree/master/protobuf#creating-objectmapper

komamitsu
komamitsu approved these changes Jan 15, 2022
View reviewed changes
Copy link
Member

@komamitsu komamitsu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Thanks! 🍷

@komamitsu komamitsu merged commit 6086904 into msgpack:develop Jan 15, 2022
@cyberdelia cyberdelia deleted the shorthand branch January 15, 2022 03:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@komamitsu komamitsu komamitsu approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@cyberdelia @komamitsu