8000 fuzz: Move-only net utils by maflcko · Pull Request #26561 · bitcoin/bitcoin · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fuzz: Move-only net utils #26561

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

Merged
merged 1 commit into from
Nov 25, 2022

Conversation

maflcko
Copy link
Member
@maflcko maflcko commented Nov 23, 2022

This should speed up fuzz builds when src/test/fuzz/util.h is modified. Also, it makes sense on its own.

@DrahtBot
Copy link
Contributor
DrahtBot commented Nov 23, 2022

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Reviews

See the guideline for information on the review process.

Type Reviewers
ACK dergoegge

Conflicts

Reviewers, this pull request conflicts with the following ones:

  • #26312 (Remove Sock::Get() and Sock::Sock() by vasild)
  • #24545 (BIP324: Enable v2 P2P encrypted transport by dhruv)
  • #21878 (Make all networking code mockable by vasild)

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

@DrahtBot DrahtBot added the Tests label Nov 23, 2022
@maflcko maflcko force-pushed the 2211-fuzz-move-only-net-utils- branch 3 times, most recently from ffff1c1 to fadcd8c Compare November 23, 2022 16:19
@dergoegge
Copy link
Member

ACK fa3b2cf

@fanquake fanquake merged commit 93cae70 into bitcoin:master Nov 25, 2022
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Nov 25, 2022
@maflcko maflcko deleted the 2211-fuzz-move-only-net-utils-📮 branch November 28, 2022 09:48
@bitcoin bitcoin locked and limited conversation to collaborators Nov 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0