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

Don't raise EndOfFile when stdin::read reaches EOF. #11126

Closed
wants to merge 1 commit into from

Conversation

spaolacci
Copy link
Contributor

Just return None' as stipulated by theReader' trait. Fixes #11104.

Just return `None' as stipulated by the `Reader' trait. Fixes rust-lang#11104.
@alexcrichton
Copy link
Member

I didn't get around to this before, but this is all working as intended. Without this condition raise, the read_to_end function would not be able to complete.

Closing for now. It's still a little hazy on what we're going to move to from conditions because as you've seen conditions aren't exactly the easiest things to use. That being said, you can always drop some io::ignore_io_error guards in order to ignore the condition and handle the return value itself.

flip1995 pushed a commit to flip1995/rust that referenced this pull request Sep 25, 2023
…=dswij

Ignore span's parents in `collect_ast_format_args`/`find_format_args`

Fixes rust-lang#11470, covers some cases missed by rust-lang#10980

Can't have a test yet because of rust-lang#11126 but it works locally

changelog: none

r? `@dswij`
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.

Reader::read raises EndOfFile condition when reading after EOF reached
2 participants