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

join! and try_join! deadlock with empty invocation #5502

Closed
adrianheine opened this issue Feb 23, 2023 · 0 comments · Fixed by #5504
Closed

join! and try_join! deadlock with empty invocation #5502

adrianheine opened this issue Feb 23, 2023 · 0 comments · Fixed by #5504
Labels
A-tokio Area: The main tokio crate C-bug Category: This is a bug. M-macros Module: macros in the main Tokio crate

Comments

@adrianheine
Copy link
Contributor

Version
Starting with 1.19.0 (i. e. everything including 1872a42 / #4624).

Platform
any

Description
join!() and try_join!() emit an empty loop {}, see playground. I would expect this to either work (preferred) or be rejected at compile time (acceptable).

@adrianheine adrianheine added A-tokio Area: The main tokio crate C-bug Category: This is a bug. labels Feb 23, 2023
@Darksonn Darksonn added the M-macros Module: macros in the main Tokio crate label Feb 23, 2023
adrianheine added a commit to adrianheine/tokio that referenced this issue Feb 25, 2023
adrianheine added a commit to adrianheine/tokio that referenced this issue Feb 25, 2023
adrianheine added a commit to adrianheine/tokio that referenced this issue Feb 25, 2023
adrianheine added a commit to adrianheine/tokio that referenced this issue Feb 25, 2023
Darksonn pushed a commit that referenced this issue Feb 25, 2023
amab8901 pushed a commit to amab8901/tokio that referenced this issue Feb 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-tokio Area: The main tokio crate C-bug Category: This is a bug. M-macros Module: macros in the main Tokio crate
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants