💬 maflcko commented on pull request "miner: always treat SegWit as active":
(https://github.com/bitcoin/bitcoin/pull/31625#issuecomment-2582320880)
Not sure if it was ever mentioned publicly where a link exists.

Obviously, if you want to reproduce the inconsistency, it would be easier in regtest, and the change may theoretically break someone's test deployment.

💬 ismaelsadeeq commented on pull request "mining: bugfix: Fix duplicate coinbase tx weight reservation":
(https://github.com/bitcoin/bitcoin/pull/31384#discussion_r1910175994)
Taken, thanks.

💬 ismaelsadeeq commented on pull request "mining: bugfix: Fix duplicate coinbase tx weight reservation":
(https://github.com/bitcoin/bitcoin/pull/31384#discussion_r1910176196)
fixed

💬 ismaelsadeeq commented on pull request "mining: bugfix: Fix duplicate coinbase tx weight reservation":
(https://github.com/bitcoin/bitcoin/pull/31384#discussion_r1910177236)
fixed

💬 0xB10C commented on pull request "tracing: Rename the `MIN` macro to `_TRACEPOINT_TEST_MIN` in log_raw_p2p_msgs":
(https://github.com/bitcoin/bitcoin/pull/31623#issuecomment-2582351026)
tested ACK f93f0c93961bbce413101c2a92300a7a29277506

📝 hodlinator opened a pull request: "net: Disconnect message follow-ups to #28521"
(https://github.com/bitcoin/bitcoin/pull/31633)
- Add missing calls to `DisconnectMsg()` - https://github.com/bitcoin/bitcoin/pull/28521#discussion_r1890824361
- Specify context when stopping nodes - https://github.com/bitcoin/bitcoin/pull/28521#discussion_r1890780754
- Bring back log message when resetting socket in case new entrypoints are added - https://github.com/bitcoin/bitcoin/pull/28521#discussion_r1890795074
- Use `DisconnectMsg()` in `CConnman` as well - https://github.com/bitcoin/bitcoin/pull/28521#discussion_r1791797716

💬 Sjors commented on pull request "mining: bugfix: Fix duplicate coinbase tx weight reservation":
(https://github.com/bitcoin/bitcoin/pull/31384#issuecomment-2582401336)
re-ACK dba211a9ceafc57a953efc757adfe09c0f3fdb14

👍 danielabrozzoni approved a pull request: "init,log: Unify block index log line"
(https://github.com/bitcoin/bitcoin/pull/31616#pullrequestreview-2542141582)
tACK e04be3731f4921cd51d25b1d6210eace7600fea4

🚀 fanquake merged a pull request: "doc: upgrade license to 2025."
(https://github.com/bitcoin/bitcoin/pull/31611)

💬 maflcko commented on pull request "net: Disconnect message follow-ups to #28521":
(https://github.com/bitcoin/bitcoin/pull/31633#issuecomment-2582444004)
review ACK 286de9749612565fd8d42f08f66c8426faf8a1f7 📋

<details><summary>Show signature</summary>

Signature:

```
untrusted comment: signature from minisign secret key on empty file; verify via: minisign -Vm "${path_to_any_empty_file}" -P RWTRmVTMeKV5noAMqVlsMugDDCyyTSbA3Re5AkUrhvLVln0tSaFWglOw -x "${path_to_this_whole_four_line_signature_blob}"
RUTRmVTMeKV5npGrKx1nqXCw5zeVHdtdYURB/KlyA/LMFgpNCs+SkW9a8N95d+U4AP1RJMi+krxU1A3Yux4bpwZNLvVBKy0wLgM=
trusted comment: review ACK 286de9749612
...

👍 danielabrozzoni approved a pull request: "test: importdescriptors is not available for non-descriptor wallets"
(https://github.com/bitcoin/bitcoin/pull/31609#pullrequestreview-2542174643)
My understanding of the legacy removal PRs is limited, but it appears that the importdescriptor function will still include the check for legacy wallets.

utACK f9dc45680fa958bc89335fa50f46052e6bb29b37

💬 stickies-v commented on pull request "kernel: Move kernel-related cache constants to kernel cache":
(https://github.com/bitcoin/bitcoin/pull/31483#discussion_r1910229692)
> Are there places where this would be re-used?

I can't quite find one - it's also a bit difficult to search for given how much `<<` is used for streams too. I agree there's no need to generalize too early, but given how similar the functionality is I still have a preference for the more generic `CheckedLeftShift()` and/or `SaturatingLeftShift()` as suggested by @ryanofsky. But, I'm okay with either approach.

> I think I would drop the MiBToBytes function and just write the constants as by
...

🚀 fanquake merged a pull request: "depends: Fix spacing issue"
(https://github.com/bitcoin/bitcoin/pull/31627)

👍 BrandonOdiwuor approved a pull request: "init,log: Unify block index log line"
(https://github.com/bitcoin/bitcoin/pull/31616#pullrequestreview-2542199335)
Code Review ACK e04be3731f4921cd51d25b1d6210eace7600fea4

🚀 fanquake merged a pull request: "tracing: Rename the `MIN` macro to `_TRACEPOINT_TEST_MIN` in log_raw_p2p_msgs"
(https://github.com/bitcoin/bitcoin/pull/31623)

🚀 fanquake merged a pull request: "init,log: Unify block index log line"
(https://github.com/bitcoin/bitcoin/pull/31616)

👍 l0rinc approved a pull request: "net: Disconnect message follow-ups to #28521"
(https://github.com/bitcoin/bitcoin/pull/31633#pullrequestreview-2542183397)
utACK 286de9749612565fd8d42f08f66c8426faf8a1f7

💬 l0rinc commented on pull request "net: Disconnect message follow-ups to #28521":
(https://github.com/bitcoin/bitcoin/pull/31633#discussion_r1910228378)
I found the `%d%s` format weird at first, but I see it is an existing format, e.g. https://github.com/bitcoin/bitcoin/blob/master/src/net.cpp#L705

💬 l0rinc commented on pull request "net: Disconnect message follow-ups to #28521":
(https://github.com/bitcoin/bitcoin/pull/31633#discussion_r1910230992)
super-nit: above we're using a simple `,` before the `DisconnectMsg`, not a `;`.

💬 l0rinc commented on pull request "net: Disconnect message follow-ups to #28521":
(https://github.com/bitcoin/bitcoin/pull/31633#discussion_r1910231692)
nit: many other similar logs here were capitalized