💬 tdb3 commented on pull request "rpc: add getorphantxs":
(https://github.com/bitcoin/bitcoin/pull/30793#discussion_r1783657306)
Updated

💬 tdb3 commented on pull request "rpc: add getorphantxs":
(https://github.com/bitcoin/bitcoin/pull/30793#discussion_r1783657604)
Updated

💬 tdb3 commented on pull request "rpc: add getorphantxs":
(https://github.com/bitcoin/bitcoin/pull/30793#discussion_r1783657742)
Updated

💬 tdb3 commented on pull request "rpc: add getorphantxs":
(https://github.com/bitcoin/bitcoin/pull/30793#issuecomment-2387291364)
Updated to e6853592361341c27103ed74b25470ac1e098d6d to address comments from @hodlinator and @itornaza

💬 mzumsande commented on pull request "test: add missing sync to feature_fee_estimation.py":
(https://github.com/bitcoin/bitcoin/pull/31016#issuecomment-2387293853)
> Should also fix #30640?

Yes! I didn't analyze that in detail but I think that's very likely. I'll add it to the OP.

👍 tdb3 approved a pull request: "test: add missing sync to feature_fee_estimation.py"
(https://github.com/bitcoin/bitcoin/pull/31016#pullrequestreview-2341658200)
code review ACK a1576edab356053c4c736691e4950b58e9a14f76

👍 tdb3 approved a pull request: "doc: add testnet4 section header for config file"
(https://github.com/bitcoin/bitcoin/pull/31007#pullrequestreview-2341683240)
ACK 61cdb1c9d83778b95f4f9596f34617b7a191d0a5

Good find. Thanks.

💬 RandyMcMillan commented on pull request "doc/build-osx.md:brew relinking note":
(https://github.com/bitcoin/bitcoin/pull/30993#discussion_r1783713469)
This PR isn't in response to and issue or comment. I just noticed it may be useful to have a note about relinking.

👍 tdb3 approved a pull request: "refactor: move util/pcp and util/netif to common/"
(https://github.com/bitcoin/bitcoin/pull/31011#pullrequestreview-2341666390)
ACK fd38711217cafbd62e8abd22d2b43f85fede8cde

cherry-pick'ed fd38711217cafbd62e8abd22d2b43f85fede8cde into master (intentionally without d51edecddcb7fa52349c8aa5ef88b01f72be44c7), ran `check-deps.sh`, and verified (with `echo $?`) that the script exited with 1 (vs 0 on master without the cherry-pick'ed commit).

💬 tdb3 commented on pull request "refactor: move util/pcp and util/netif to common/":
(https://github.com/bitcoin/bitcoin/pull/31011#discussion_r1783673649)
femto nit. Ignore if not already updating the file for something else.

```diff
- #include <common/pcp.h>
- #include <common/netif.h>
+ #include <common/netif.h>
+ #include <common/pcp.h>
```

💬 glozow commented on pull request "refactor: TxDownloadManager + fuzzing":
(https://github.com/bitcoin/bitcoin/pull/30110#discussion_r1783599080)
Added the `Assume`. I agree it makes more sense.

💬 glozow commented on pull request "refactor: TxDownloadManager + fuzzing":
(https://github.com/bitcoin/bitcoin/pull/30110#discussion_r1783611446)
Ah, sorry for missing that. I'm not really convinced this is more efficient overall?

💬 glozow commented on pull request "refactor: TxDownloadManager + fuzzing":
(https://github.com/bitcoin/bitcoin/pull/30110#discussion_r1783578259)
edited somewhat. I like the annotations there but only did first row.

💬 glozow commented on pull request "refactor: TxDownloadManager + fuzzing":
(https://github.com/bitcoin/bitcoin/pull/30110#discussion_r1783602138)
Thanks, done

💬 glozow commented on pull request "refactor: TxDownloadManager + fuzzing":
(https://github.com/bitcoin/bitcoin/pull/30110#discussion_r1783604279)
More efficient to shuffle ints than objects. I've changed to use the ctor with the size.

💬 glozow commented on pull request "refactor: TxDownloadManager + fuzzing":
(https://github.com/bitcoin/bitcoin/pull/30110#discussion_r1783572311)
edited to no longer be for a chain

💬 glozow commented on pull request "refactor: TxDownloadManager + fuzzing":
(https://github.com/bitcoin/bitcoin/pull/30110#discussion_r1783604666)
done

💬 glozow commented on pull request "refactor: TxDownloadManager + fuzzing":
(https://github.com/bitcoin/bitcoin/pull/30110#discussion_r1783602611)
Unsure, it's just a habit I guess.

💬 glozow commented on pull request "refactor: TxDownloadManager + fuzzing":
(https://github.com/bitcoin/bitcoin/pull/30110#discussion_r1783609907)
This is part of a block of code that was moved from `PeerManagerImpl`. The members still have the same name, so I don't think this is incorrect.

💬 glozow commented on pull request "refactor: TxDownloadManager + fuzzing":
(https://github.com/bitcoin/bitcoin/pull/30110#discussion_r1783569179)
added