Skip to content

build: replace send/recv with swrite/sread where missing#20459

Closed
vszakats wants to merge 11 commits intocurl:masterfrom
vszakats:swrite-vs-send
Closed

build: replace send/recv with swrite/sread where missing#20459
vszakats wants to merge 11 commits intocurl:masterfrom
vszakats:swrite-vs-send

Conversation

@vszakats
Copy link
Member

@vszakats vszakats commented Jan 28, 2026 •
edited
Loading

Also:

  • checksrc: ban send and recv, as documented in CODE_STYLE.md.

Follow-up to 9a26633 #17572
Ref: a585cc3 #20097
Ref: #20441

@vszakats vszakats marked this pull request as draft January 28, 2026 12:08
@vszakats vszakats marked this pull request as ready for review January 28, 2026 12:17
@vszakats vszakats closed this in 483f429 Jan 28, 2026
@vszakats vszakats deleted the swrite-vs-send branch January 28, 2026 13:40
@vszakats vszakats mentioned this pull request Jan 28, 2026
Closed
vszakats added a commit that referenced this pull request Jan 28, 2026
@vszakats
tests/server: replace 4 remaining send() calls with swrite()
2e5cc08 
They were masked by `()`, which was also not necessary anymore.

Follow-up to 483f429 #20459
Follow-up to a585cc3 #20097
Follow-up to bf7375e #18503

Closes #20460
@vszakats vszakats mentioned this pull request Jan 28, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

cmdline tool tests

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@vszakats