Skip to content

yes: fix confusing 1.. and comment#12424

Open
oech3 wants to merge 1 commit into
uutils:mainfrom
oech3:yes-1
Open

yes: fix confusing 1.. and comment#12424
oech3 wants to merge 1 commit into
uutils:mainfrom
oech3:yes-1

Conversation

@oech3
Copy link
Copy Markdown
Contributor

@oech3 oech3 commented May 21, 2026

yes never reach to end of file (ENOSPC is not EOF).
Since we don't use busy loop, let Ok(1..) = does not reduce CPU workload. We should continue zero-copy when tee was temporarily blocked by stdout by some reason.

@github-actions
Copy link
Copy Markdown

GNU testsuite comparison:

Skipping an intermittent issue tests/cut/bounded-memory (passes in this run but fails in the 'main' branch)
Skipping an intermittent issue tests/tail/tail-n0f (passes in this run but fails in the 'main' branch)

@oech3 oech3 marked this pull request as ready for review May 21, 2026 15:31
Comment thread src/uu/yes/src/yes.rs
Comment thread src/uu/yes/src/yes.rs
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