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

perf: use parking_lot::Mutex #172

Closed
wants to merge 1 commit into from
Closed

perf: use parking_lot::Mutex #172

wants to merge 1 commit into from

Conversation

eareimu
Copy link
Contributor

@eareimu eareimu commented Aug 28, 2024

No description provided.

@huster-zhangpeng
Copy link
Contributor

没必要用它,它有兼容性更差,况且std Mutex已经有过一次升级,吸收了parking_lot的思路,性能已不差,且兼容最好

@huster-zhangpeng huster-zhangpeng deleted the perf/async_deque branch August 29, 2024 03:01
@huster-zhangpeng
Copy link
Contributor

huster-zhangpeng commented Aug 29, 2024

对外部库的依赖当然越少越好。要说parking_lot足够好,std早吸收了,下面是两者的讨论,大部分时候parking_lot已比不上std的

FYI.
https://users.rust-lang.org/t/which-mutex-to-use-parking-lot-or-std-sync/85060
rust-lang/rust#56410
Amanieu/parking_lot#248

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants