✅ hebasto closed an issue: "Weird focus rect displayed on inital sync"
(https://github.com/bitcoin-core/gui/issues/783)

🚀 hebasto merged a pull request: "Keep focus on "Hide" while ModalOverlay is visible"
(https://github.com/bitcoin-core/gui/pull/795)

👍 dergoegge approved a pull request: "kernel: De-globalize static validation variables"
(https://github.com/bitcoin/bitcoin/pull/30425#pullrequestreview-2177300151)
Code review ACK 51fa26239af9bbfd44029aaf595cb4c6a8d4a75d

💬 hebasto commented on issue "Welcome screen uses GiB values as GB, risking running out of space":
(https://github.com/bitcoin-core/gui/issues/821#issuecomment-2228117027)
Also see https://github.com/bitcoin/bitcoin/pull/29678.

👍 dergoegge approved a pull request: "fuzz: bound some miniscript operations to avoid fuzz timeouts"
(https://github.com/bitcoin/bitcoin/pull/30197#pullrequestreview-2177308717)
utACK bc34bc288824978ef4b98e8802b47cb863c8a8c2

👍 dergoegge approved a pull request: "fuzz: fix key size in `crypter`"
(https://github.com/bitcoin/bitcoin/pull/30373#pullrequestreview-2177316284)
utACK 4383dc90bac1b5def73352fe222f99807d8ca4dd

💬 glozow commented on pull request "Ephemeral Dust":
(https://github.com/bitcoin/bitcoin/pull/30239#issuecomment-2228142149)
concept ACK

👍 dergoegge approved a pull request: "Add fuzz test for FSChaCha20Poly1305, AEADChacha20Poly1305"
(https://github.com/bitcoin/bitcoin/pull/28263#pullrequestreview-2177332281)
tACK 39805638663bdd567a50ecac962631b45ce8d73a

💬 dergoegge commented on pull request "Add fuzz test for FSChaCha20Poly1305, AEADChacha20Poly1305":
(https://github.com/bitcoin/bitcoin/pull/28263#discussion_r1677597969)
nit:

```suggestion
auto aad{rng.randbytes<std::byte>(aad_length)};
```

💬 Sjors commented on pull request "Have createNewBlock() return a BlockTemplate interface":
(https://github.com/bitcoin/bitcoin/pull/30440#issuecomment-2228153853)
Also added `getHashPrevBlock()`.

💬 hebasto commented on pull request "OptionsDialog: Allow Maximize of window":
(https://github.com/bitcoin-core/gui/pull/826#issuecomment-2228155212)
The PR description is empty, so I have to ask: why is this needed?

🤔 hebasto reviewed a pull request: "OptionsDialog: Prefer to stretch actual options area rather than waste space"
(https://github.com/bitcoin-core/gui/pull/827#pullrequestreview-2177355412)
Tested b71bfd9eef8b0017cef3c05361c02ddbd837e6ac on Ubuntu 24.04 + Qt 5.15.13:

- the master branch @ 6ae903e24afa95214e59a9dc87dc74db3535a5b5:

- this PR:

@GBKS What do you think?

💬 GBKS commented on pull request "OptionsDialog: Prefer to stretch actual options area rather than waste space":
(https://github.com/bitcoin-core/gui/pull/827#issuecomment-2228175727)
That is a nice improvement.

💬 1440000bytes commented on issue "scriptPubKey no address":
(https://github.com/bitcoin/bitcoin/issues/30450#issuecomment-2228177423)
https://github.com/bitcoin/bitcoin/issues/22524#issuecomment-884663685

💬 fjahr commented on pull request "assumeutxo: Add dumptxoutset height param, remove shell scripts":
(https://github.com/bitcoin/bitcoin/pull/29553#issuecomment-2228190409)
Fixed some outdated information in the remainin design doc and added that to the last commit.

✅ fanquake closed an issue: "fuzz: crypter: Abrt in __cxxabiv1::failed_throw"
(https://github.com/bitcoin/bitcoin/issues/30251)

🚀 fanquake merged a pull request: "fuzz: fix key size in `crypter`"
(https://github.com/bitcoin/bitcoin/pull/30373)

💬 Sjors commented on pull request "Have createNewBlock() return a BlockTemplate interface":
(https://github.com/bitcoin/bitcoin/pull/30440#issuecomment-2228203041)
Also added `getBlockHeader()` which the Template Provider needs for the `NewTemplate` message and a few other places.

📝 fanquake opened a pull request: "depends: remove Darwin ENV unsetting"
(https://github.com/bitcoin/bitcoin/pull/30451)
Now that we use the native compiler, and have fixed Qt, and these vars
are unset it Guix, we can remove the unsetting from our compiler command
here.

Couldn't manage to make a darwin-clang-cross only exclusion of `-lm` work properly,
so opted for just removing the explicit link entirely. I do not think this should have any
other unwanted side-effects.

Fixes #21552.

💬 maflcko commented on pull request "kernel: De-globalize static validation variables":
(https://github.com/bitcoin/bitcoin/pull/30425#discussion_r1677633601)
39f9b80fba85d9818222c4d76e99ea1a804f5dda: Not sure about making this public mutable. Would it not be better to make this a private field (along with making `NotifyHeaderTip` a private method)?