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

Hide unit function return types #3858

Merged
merged 1 commit into from
Apr 5, 2020
Merged

Conversation

lnicola
Copy link
Member

@lnicola lnicola commented Apr 5, 2020

r? @flodiebold

This might be a bit heavy-handed (e.g. || -> () to ||), what do you think?

@lnicola lnicola changed the title Hide unit types on Hide unit function return types Apr 5, 2020
@flodiebold
Copy link
Member

Hm, for fn I think this is good, but for || it seems a bit weird...

@lnicola
Copy link
Member Author

lnicola commented Apr 5, 2020

Updated to revert the closure change.

@flodiebold
Copy link
Member

bors r+

@bors
Copy link
Contributor

bors bot commented Apr 5, 2020

@bors bors bot merged commit bf2d91b into rust-lang:master Apr 5, 2020
@lnicola lnicola deleted the fn-unit-ret branch April 5, 2020 23:02
lnicola pushed a commit to lnicola/rust-analyzer that referenced this pull request Jan 7, 2025
Fix tokio test ICE

Fixes rust-lang#3858

It turned out that the issue mentioned [here](rust-lang/miri#3858 (comment)) is the exact cause of ICE.
So in this PR, I changed the type of ``EpollEventInterest::epfd`` from ``i32`` to ``WeakFileDescriptionRef``.
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