💬 ajtowns commented on issue "meta: Isolated fuzzing of net processing":
(https://github.com/bitcoin/bitcoin/issues/27502#issuecomment-1682349394)
I think even just having it as documentation of what parts of mempool/validation are used by net would be interesting -- I don't think the boundary there is entirely clean; eg the `CompareInvMempoolOrder` stuff could probably be moved to txmempool.

👋 brunoerg's pull request is ready for review: "fuzz: call `LookupSubNet` before calling `Ban` with a subnet"
(https://github.com/bitcoin/bitcoin/pull/27935)

💬 brunoerg commented on pull request "fuzz: call lookup functions before calling `Ban`":
(https://github.com/bitcoin/bitcoin/pull/27935#issuecomment-1682367492)
Thanks for review @dergoegge. I noted that we also need to call `LookupHost` when banning with `CNetAddr`. Just updated the PR description and title.

Also, this way we're closer to the reality (following what RPC does to set it).

✅ MarcoFalke closed an issue: "ci: Future of macOS and Windows MSVC CI tasks"
(https://github.com/bitcoin/bitcoin/issues/28098)

💬 MarcoFalke commented on issue "ci: Future of macOS and Windows MSVC CI tasks":
(https://github.com/bitcoin/bitcoin/issues/28098#issuecomment-1682367879)
Closing for now. macOS has been merged and Windows can be discussed in the already open pull: https://github.com/bitcoin/bitcoin/pull/28173

For other issues or new issues, a new issue can be opened.

👋 MarcoFalke's pull request is ready for review: "ci: Switch more tasks to self-hosted"
(https://github.com/bitcoin/bitcoin/pull/21652)

💬 MarcoFalke commented on pull request "test: Add unit & functional test coverage for blockstore":
(https://github.com/bitcoin/bitcoin/pull/27850#issuecomment-1682374092)
The final step is https://github.com/bitcoin/bitcoin/pull/21652. Feel free to rebase on top of that, to test your changes.

💬 TheCharlatan commented on pull request "refactor: Remove confusing static_cast in address types":
(https://github.com/bitcoin/bitcoin/pull/28284#issuecomment-1682379299)
Concept ACK

✅ MarcoFalke closed an issue: "What is depends BUILD_ID_SALT ?"
(https://github.com/bitcoin/bitcoin/issues/28276)

💬 sipa commented on pull request "Don't empty dbcache on prune flushes: >30% faster IBD":
(https://github.com/bitcoin/bitcoin/pull/28280#discussion_r1297300532)
Actually, it seems iterators may get invalidated when a map rehash occurs, but pointers to elements don't.

I think you can however store pointers to `std::pair<const COutPoint, CoinsCacheEntry>` (the items of the map). Would that help alleviate the need for `m_outpoint`?

💬 andrewtoth commented on pull request "Don't empty dbcache on prune flushes: >30% faster IBD":
(https://github.com/bitcoin/bitcoin/pull/28280#discussion_r1297307567)
Brilliant, thanks! Was just trying this last night, this looks like the way.

💬 MarcoFalke commented on issue "lint: MYPY_CACHE_DIR is unused":
(https://github.com/bitcoin/bitcoin/issues/28183#issuecomment-1682390715)
Let's move discussion to https://github.com/bitcoin/bitcoin/pull/28184

✅ MarcoFalke closed an issue: "lint: MYPY_CACHE_DIR is unused"
(https://github.com/bitcoin/bitcoin/issues/28183)

💬 MarcoFalke commented on issue "test: RPC coverage check doesn't work?":
(https://github.com/bitcoin/bitcoin/issues/27593#issuecomment-1682392919)
Looks like there are also other bugs with this: https://cirrus-ci.com/task/6126543575973888?logs=ci#L5061

💬 fanquake commented on pull request "crypto: more `Span<std::byte>` modernization & follow-ups":
(https://github.com/bitcoin/bitcoin/pull/28100#issuecomment-1682397002)
@sipa when you get a chance to respond to review comments here, we can decide to either merge as-is, and address in a followup, or touch up here.

💬 petertodd commented on pull request "policy: Enable full-rbf by default":
(https://github.com/bitcoin/bitcoin/pull/28132#issuecomment-1682399338)
> > I think you asking these questions inappropriately gives readers the impression
> > that there actually are non-trivial examples of real businesses who are relying
> > on unconfirmed transactions. The fact is in this entire thread no-one has been
> > able to provide clear, convincing, evidence that this is true.
>
> I think you have two layers of reasoning: a) browsing the matrix of alternatives solutions for unconfirmed transactions and b) the economics traffic of the use-cases which
...

💬 MarcoFalke commented on pull request "crypto: more `Span<std::byte>` modernization & follow-ups":
(https://github.com/bitcoin/bitcoin/pull/28100#issuecomment-1682414932)
I think they should either be fixed up here or not at all. I don't think it makes sense to create a follow-up to a follow-up.

💬 sipa commented on pull request "crypto: more `Span<std::byte>` modernization & follow-ups":
(https://github.com/bitcoin/bitcoin/pull/28100#issuecomment-1682416176)
I will address them.

💬 fanquake commented on pull request "util: Replace std::filesystem with util/fs.h":
(https://github.com/bitcoin/bitcoin/pull/28076#issuecomment-1682417462)
I don't think we are going to get the Rust linter over the line here. Obviously the filesystem commit is correct, and should be merged. Do you want to split that out, and continue the lint discussion, or drop the linter from this PR for now?

💬 MarcoFalke commented on pull request "util: Replace std::filesystem with util/fs.h":
(https://github.com/bitcoin/bitcoin/pull/28076#issuecomment-1682432081)
> Do you want to split that out, and continue the lint discussion

Seems better to keep the discussion in one place for now.