The Wayback Machine - https://web.archive.org/web/20201017054512/https://github.com/json-iterator/go/pull/447
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

Move to sync.Map #447

Open
wants to merge 1 commit into
base: master
from
Open

Move to sync.Map #447

wants to merge 1 commit into from

Conversation

@mythi
Copy link

@mythi mythi commented Mar 3, 2020

Go 1.9 (released in August 2017) added sync.Map. The release has been
around some time (and the release is not even supported by the Go
developers anymore) so it's safe to make a claim sync.Map is available for
everyone.

This commit keeps ./test.sh passing.

Fixes: #446

Signed-off-by: Mikko Ylinen mikko.ylinen@intel.com

Go 1.9 (released in August 2017) added sync.Map. The release has been
around some time (and the release is not even supported by the Go
developers anymore) so it's safe to make a claim sync.Map is available for
everyone.

This commit keeps ./test.sh passing.

Fixes: #446

Signed-off-by: Mikko Ylinen <mikko.ylinen@intel.com>
@mythi mythi force-pushed the mythi:syncmap branch from 722c5ab to de6d9c1 Mar 19, 2020
@mythi
Copy link
Author

@mythi mythi commented Mar 19, 2020

any comments?

@AllenX2018 AllenX2018 requested a review from taowen Mar 26, 2020
@mythi
Copy link
Author

@mythi mythi commented Sep 2, 2020

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
1 participant
You can’t perform that action at this time.