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

feat: ✨ add mutex map #4

Merged
merged 1 commit into from
Feb 27, 2023
Merged

feat: ✨ add mutex map #4

merged 1 commit into from
Feb 27, 2023

Conversation

thanhpp
Copy link
Member

@thanhpp thanhpp commented Feb 22, 2023

No description provided.

@thanhpp thanhpp force-pushed the ds/mutexmap branch 2 times, most recently from e6b3b00 to a39f8c9 Compare February 22, 2023 09:53
@thanhpp
feat: ✨ add mutex map
d5bc0e5 
Signed-off-by: thanhpp <thanhphanphu18@gmail.com>
@thanhpp thanhpp merged commit f022430 into main Feb 27, 2023
@thanhpp thanhpp deleted the ds/mutexmap branch February 27, 2023 07:10
@ngocthanh1389
Copy link
Member

@thanhpp should name "lockedmap" is better, mutex map mean a map of mutex

@thanhpp
Copy link
Member Author

thanhpp commented Feb 28, 2023

@thanhpp should name "lockedmap" is better, mutex map mean a map of mutex

lockedMap means a map is locked :| lockMap means a map of lock. I think the name is not so important 😕

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@onlylove97 onlylove97 onlylove97 left review comments

@secmask secmask secmask left review comments

@ngocthanh1389 ngocthanh1389 ngocthanh1389 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.
4 participants
@thanhpp @ngocthanh1389 @secmask @onlylove97