💬 jonatack commented on pull request "doc: i2p documentation updates":
(https://github.com/bitcoin/bitcoin/pull/27937#discussion_r1244030626)
Done.

💬 jonatack commented on issue "I2P: Creating SAM session with 127.0.0.1:7656":
(https://github.com/bitcoin/bitcoin/issues/22759#issuecomment-1609907115)
> > The "cannot decode" message above repeats every 0.002 seconds, so ends up flooding the log file.
>
> > "Cannot decode Base64: "STREAM STATUS RESULT=I2P_ERROR MESSAGE="Session was closed""
> > and repeat same message hundreds time per second!
>
> Indeed, I have a few instances in my mainnet debug log when this occurred, and like these reports, in high-frequency bursts (unsure, but they seem to have only happened when using the Java I2P router, not i2pd). Will look more into it.

I've
...

💬 carnhofdaki commented on issue "Add maxrelaytxfee":
(https://github.com/bitcoin/bitcoin/issues/27983#issuecomment-1609907306)
![Uploading mutinynet.com_tx_68be05aec97b7d114e978185f0df76e494196e2b160330c97870b284f444e1c4.png…]()

Adding the screenshot in case that explorer will need to switch to pruned mode.

This is the raw hex transaction:

```
0200000000010238662df9105edbfc3bc1afb921e3bd382098624f9b2f5b3efa3144fb6c0c95340000000000fdffffffbb5667fde0eb1f0e169b85377ec7b79c07adfd680f84b0bc5e36180912abdf000000000000fdffffff0100000000000000000f6a0d746561636820426974636f696e0247304402202a9db30eae0197c835093b2837aec99
...

💬 ryanofsky commented on pull request "refactor: Drop unsafe AsBytePtr function":
(https://github.com/bitcoin/bitcoin/pull/27978#issuecomment-1609910681)
Rebased 2109a147406247bb654ee5fb8421a8594919fbd5 -> 650ca0d937c2c90adeeac60adae090902618d771 ([`pr/noptr.2`](https://github.com/ryanofsky/bitcoin/commits/pr/noptr.2) -> [`pr/noptr.3`](https://github.com/ryanofsky/bitcoin/commits/pr/noptr.3), [compare](https://github.com/ryanofsky/bitcoin/compare/pr/noptr.2-rebase..pr/noptr.3)) due to conflict with #27479

💬 soheilwikey12 commented on pull request "bumpfee: ignore WALLET_INCREMENTAL_RELAY_FEE when user specifies fee_rate":
(https://github.com/bitcoin/bitcoin/pull/27969#issuecomment-1609942086)
14tDQpPnHqoOFvjBQzKoNM6bgwHL1oklVNLPRQr4Kvauq8oHICNDFvHv

💬 theStack commented on pull request "test: add python implementation of Elligator swift":
(https://github.com/bitcoin/bitcoin/pull/24005#discussion_r1244157426)
Thanks for clarifying, that makes sense.

💬 hebasto commented on pull request "build: produce a .zip for macOS distribution":
(https://github.com/bitcoin/bitcoin/pull/27099#discussion_r1244194688)
Unfortunately, using the `-r` option still introduces non-reproducibilty. Diffoscope points at some metadata related to directories. Not sure, but it looks like they are about creation timestamps.

Suggesting to use the same approach as we are using for Windows builds:
```suggestion
find $(APP_DIST_DIR) | sort | zip -X@ $@
```

Two subsequent runs on my Ubuntu 22.04 returned the same hashes.

💬 hebasto commented on pull request "build: produce a .zip for macOS distribution":
(https://github.com/bitcoin/bitcoin/pull/27099#issuecomment-1610057916)
> As evidence, Visual Studio Code has recently started distributing their macOS builds as ZIP files instead of DMGs, as you can see on their download page: [code.visualstudio.com/Download](https://code.visualstudio.com/Download).

In comparison to UX of downloading `VSCode-darwin.zip`, in the current PR branch, there is a top directory named `dist`, which seems redundant:


💬 0xB10C commented on pull request "test: various USDT functional test cleanups (27831 follow-ups)":
(https://github.com/bitcoin/bitcoin/pull/27944#issuecomment-1610093695)
Thanks for picking this up! Concept ACK. I'll have a closer look.

💬 achow101 commented on pull request "util: implement `noexcept` move assignment & move ctor for `prevector`":
(https://github.com/bitcoin/bitcoin/pull/27334#issuecomment-1610102318)
ACK bfb9291a8661fe5b26c14ed755cfa89d27c37110

🚀 achow101 merged a pull request: "util: implement `noexcept` move assignment & move ctor for `prevector`"
(https://github.com/bitcoin/bitcoin/pull/27334)

🤔 jonatack reviewed a pull request: "refactor: Drop unsafe AsBytePtr function"
(https://github.com/bitcoin/bitcoin/pull/27978#pullrequestreview-1501709807)
ACK 650ca0d937c2c90adeeac60adae090902618d771

💬 jonatack commented on pull request "refactor: Drop unsafe AsBytePtr function":
(https://github.com/bitcoin/bitcoin/pull/27978#discussion_r1244251001)
Would it make sense for `SpanFromDbt()` to be a static method in `src/wallet/bdb.cpp` instead, which contains its only callers?

💬 fjahr commented on pull request "Rework validation logic for assumeutxo":
(https://github.com/bitcoin/bitcoin/pull/27746#discussion_r1244328552)
This is what I had in mind: https://github.com/fjahr/bitcoin/commit/e26c89251b9fc2ae24c2c7ff90b8fd5490335ee5

💬 furszy commented on pull request "index: improve initialization and pruning violation check":
(https://github.com/bitcoin/bitcoin/pull/27607#discussion_r1244338868)
Sure. Added the changes and documented the preconditions.

💬 jonatack commented on pull request "util: implement `noexcept` move assignment & move ctor for `prevector`":
(https://github.com/bitcoin/bitcoin/pull/27334#issuecomment-1610203248)
The merge script picked my quoted github snippet rather than the ACK; maybe it should ignore lines that begin with `> `.

```
ACKs for top commit:
achow101:
ACK bfb9291a8661fe5b26c14ed755cfa89d27c37110
jonatack:
> Tested Approach ACK [bfb9291](https://github.com/bitcoin/bitcoin/commit/bfb9291a8661fe5b26c14ed755cfa89d27c37110), ~ACK modulo re-verifying the implementation of the last commit.
john-moffett:
Approach ACK bfb9291a8661fe5b26c14ed75
...

🤔 furszy reviewed a pull request: "index: make startup more efficient"
(https://github.com/bitcoin/bitcoin/pull/27607#pullrequestreview-1501863590)
Updated per feedback, thanks ryanofsky!.

* Reordered code so `g_indexes_ready_to_sync` removal occurs at the last commit.
* Documented the `CheckBlockDataAvailability` preconditions.
* And fixed a little bug that could have arose when the user erases the blocks directory without erasing the indexes directory. In this scenario, the process need to verify that `CheckBlockDataAvailability` return value is the genesis block.

📝 sipa opened a pull request: "Add support for RFC8439 variant of ChaCha20"
(https://github.com/bitcoin/bitcoin/pull/27985)
Based on and replaces part of #25361, part of the BIP324 project (#27634).

There are two variants of ChaCha20 in use. The original one uses a 64-bit nonce and a 64-bit block counter, while the one used in RFC8439 uses a 96-bit nonce and 32-bit block counter. This adds support for the 96/32 variant to our existing ChaCha20 classes.

👍 furszy approved a pull request: "wallet: Load database records in a particular order"
(https://github.com/bitcoin/bitcoin/pull/24914#pullrequestreview-1501871959)
reACK 3c83b1d8

💬 mzumsande commented on pull request "index: make startup more efficient":
(https://github.com/bitcoin/bitcoin/pull/27607#discussion_r1243985565)
commit 68b13a2b6c96fd061a31add58b8626c5bf1cdf3d
nit: this is not a struct