Skip to content

Conversation

@andrewliebenow
Copy link
Contributor

Multiple stdin ("-") inputs were causing a deadlock, since stdin was locked when processing the first occurrence.

Also: fix a couple other bugs in paste, and add tests for them.

Multiple stdin ("-") inputs were causing a deadlock, since stdin
was locked when processing the first occurrence.

Also: fix a couple other bugs in paste, and add tests for them.
@jgarzik jgarzik merged commit 6940edb into rustcoreutils:main Sep 21, 2024
2 checks passed
@jgarzik
Copy link
Contributor

jgarzik commented Sep 21, 2024

Well done splitting multiple logical changes into separate commits.

@andrewliebenow
Copy link
Contributor Author

👍🏻

Rc<Cell<Option<Box<dyn BufRead>>>> is not great, but I don't know Rust well enough to know the usual solution in these situations.

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