Skip to content

pipes.rs: don't mix raw write and buffered write#12195

Merged
cakebaker merged 1 commit intouutils:mainfrom
oech3:splice-no-mix
May 10, 2026
Merged

pipes.rs: don't mix raw write and buffered write#12195
cakebaker merged 1 commit intouutils:mainfrom
oech3:splice-no-mix

Conversation

@oech3
Copy link
Copy Markdown
Contributor

@oech3 oech3 commented May 8, 2026

This fallback code pass is rarely used, but line-buffered which should be removed as same as 055a66b .

@oech3 oech3 force-pushed the splice-no-mix branch from 6bbc2dc to 6a647f5 Compare May 8, 2026 10:26
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 8, 2026

GNU testsuite comparison:

Skip an intermittent issue tests/date/date-locale-hour (fails in this run but passes in the 'main' branch)
Skip an intermittent issue tests/tail/tail-n0f (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/cut/bounded-memory (passes in this run but fails in the 'main' branch)
Congrats! The gnu test tests/misc/io-errors is no longer failing!
Note: The gnu test tests/printf/printf-surprise is now being skipped but was previously passing.
Congrats! The gnu test tests/dd/no-allocate is now passing!

@oech3 oech3 marked this pull request as ready for review May 8, 2026 11:10
@oech3 oech3 force-pushed the splice-no-mix branch 2 times, most recently from da75145 to ed48045 Compare May 8, 2026 12:02
@oech3 oech3 marked this pull request as draft May 9, 2026 08:10
@oech3 oech3 force-pushed the splice-no-mix branch from ed48045 to 8773526 Compare May 9, 2026 08:15
@oech3 oech3 marked this pull request as ready for review May 9, 2026 08:17
@oech3 oech3 marked this pull request as draft May 10, 2026 08:01
@oech3 oech3 marked this pull request as ready for review May 10, 2026 08:16
@cakebaker cakebaker merged commit 94b06d9 into uutils:main May 10, 2026
175 of 233 checks passed
@cakebaker
Copy link
Copy Markdown
Contributor

Thanks!

@oech3 oech3 deleted the splice-no-mix branch May 10, 2026 14:46
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