💬 knst commented on pull request "fix: rendering an amp characters in the wallet name for QMenu":
(https://github.com/bitcoin/bitcoin/pull/30446#issuecomment-2226884796)
closed as moved to bitcoin-core/gui: https://github.com/bitcoin-core/gui/pull/828

👍 hodlinator approved a pull request: "rpc: Use CHECK_NONFATAL over Assert"
(https://github.com/bitcoin/bitcoin/pull/30429#pullrequestreview-2176246261)
ACK fa6270737eb9655bfb4e29b7070ecb6cd2087b7f

Ran linter without the C++ change:
```console
$ ./test/lint/lint-assertions.py
CHECK_NONFATAL(condition) or NONFATAL_UNREACHABLE should be used instead of assert for RPC code.
src/rpc/blockchain.cpp:804: const auto& first_unpruned{*Assert(blockman.GetFirstBlock(*chain_tip, /*status_mask=*/BLOCK_HAVE_MASK, first_block))};
src/rpc/blockchain.cpp:811: return Assert(first_unpruned.pprev)->nHeight;
```
Including the C++ change silences the
...

👍 hodlinator approved a pull request: "refactor: Remove Span operator==, Use std::ranges::equal"
(https://github.com/bitcoin/bitcoin/pull/29071#pullrequestreview-2176250245)
ACK fabcd0c4fe44e3bc2d6a59f2839f459fd5c81171

💬 hodlinator commented on pull request "refactor: Remove Span operator==, Use std::ranges::equal":
(https://github.com/bitcoin/bitcoin/pull/29071#discussion_r1676823487)
nit: Double newline

💬 hodlinator commented on pull request "refactor: Remove Span operator==, Use std::ranges::equal":
(https://github.com/bitcoin/bitcoin/pull/29071#discussion_r1676825369)
(Attempted switching around the parameters to `lexicographical_compare()` and confirmed that it broke `test_bitcoin -t db_tests`).

💬 andrewtoth commented on pull request "Don't empty dbcache on prune flushes: >30% faster IBD":
(https://github.com/bitcoin/bitcoin/pull/28280#issuecomment-2226936005)
> @andrewtoth, I'm running the benchmark now on a cosy Hetzner dedicated server (without a local Bitcoin node, let's see if that makes it too unreliable).
>
> Does this seem like a realistic measurement?
>
> ```shell
> hyperfine \
> --warmup 1 --runs 2 \
> --show-output \
> --export-markdown bench.md \
> --parameter-list COMMIT 0383de,21090d \
> --prepare 'git checkout {COMMIT} && make -j10 && rm -rf /mnt/hdd1/BitcoinData/*' \
> 'echo {COMMIT} && ./src/bitcoind -datadir=/mnt/hdd1/Bi
...

💬 andrewtoth commented on pull request "Don't empty dbcache on prune flushes: >30% faster IBD":
(https://github.com/bitcoin/bitcoin/pull/28280#discussion_r1676837143)
Yes!

💬 andrewtoth commented on pull request "Don't empty dbcache on prune flushes: >30% faster IBD":
(https://github.com/bitcoin/bitcoin/pull/28280#discussion_r1676838221)
There's a long thread [here](https://github.com/bitcoin/bitcoin/pull/17487#discussion_r402037193) that is where this comment and `if/else` block arose from. I think you might have to click expand a few times to see it.

From that thread, I think using the ternary operator will result in an extra move instead of branching for the different cases.

💬 andrewtoth commented on pull request "Don't empty dbcache on prune flushes: >30% faster IBD":
(https://github.com/bitcoin/bitcoin/pull/28280#issuecomment-2226949831)
Also, you should probably wrap hyperfine with `nohup ... &` so you can exit the ssh session.

💬 fjahr commented on pull request "index: Check all necessary block data is available before starting to sync":
(https://github.com/bitcoin/bitcoin/pull/29770#issuecomment-2226993556)
Rebased with updates that resulted from the changes in https://github.com/bitcoin/bitcoin/pull/29668 before merge plus an additional commit that addresses left-over comments in https://github.com/bitcoin/bitcoin/pull/29668.

💬 cbergqvist commented on pull request "net: Favor peers from addrman over fetching seednodes":
(https://github.com/bitcoin/bitcoin/pull/29605#issuecomment-2227100893)
CI failure would be fixed by.
```diff
diff --git a/src/net.cpp b/src/net.cpp
index 01522a6286..36889a6da6 100644
--- a/src/net.cpp
+++ b/src/net.cpp
@@ -3269,9 +3269,7 @@ bool CConnman::Start(CScheduler& scheduler, const Options& connOptions)

// Randomize the order in which we may query seednode to potentially prevent connecting to the same one every restart (and signal that we have restarted)
std::vector<std::string> seed_nodes = connOptions.vSeedNodes;
- if (!seed_node
...

💬 sutt0n commented on issue "contrib/signet/getcoins.py no longer useable":
(https://github.com/bitcoin/bitcoin/issues/29679#issuecomment-2227104524)
@willcl-ark @kallewoof It seems that this issue is still occurring, or at least there's another issue related to this one with the Signet faucet -- is the faucet drained, or non-functioning at the moment? I've failed to get _any_ transactions to go through whenever it says "A pay-out has been queued"

💬 hodlinator commented on pull request "rest: Reject negative outpoint index early in getutxos parsing":
(https://github.com/bitcoin/bitcoin/pull/30444#issuecomment-2227109971)
Have you considered migrating away from the other `ParseInt32()` call in the same file? Gets rid of the Valgrind comment.
This version keeps this one signed since it used in 2 places expecting signed:
```diff
diff --git a/src/rest.cpp b/src/rest.cpp
index 185508d2c9..e3e2dd96e6 100644
--- a/src/rest.cpp
+++ b/src/rest.cpp
@@ -954,8 +954,8 @@ static bool rest_blockhash_by_height(const std::any& context, HTTPRequest* req,
std::string height_str;
const RESTResponseFormat rf = Par
...

👍 tdb3 approved a pull request: "init: change shutdown order of load block thread and scheduler"
(https://github.com/bitcoin/bitcoin/pull/30435#pullrequestreview-2176555055)
ACK 5fd48360198d2ac49e43b24cc1469557b03567b8
Nice work.
Had to use a higher sleep than 50ms to reproduce the error on my local machine (used 150ms).

💬 sr-gi commented on pull request "net: Favor peers from addrman over fetching seednodes":
(https://github.com/bitcoin/bitcoin/pull/29605#issuecomment-2227150267)
@cbergqvist I'll squash for simplicity if you're fine with that

⚠️ lifesizebox opened an issue: "Block size evaluation requested"
(https://github.com/bitcoin/bitcoin/issues/30448)
### Please describe the feature you'd like to see added.

I recommend a block size preference evaluation. Let the users decide outside of default.

### Is your feature related to a problem, if so please describe it.

Increase transaction times, decrease costs.

### Describe the solution you'd like

Bigger blocks

### Describe any alternatives you've considered

BCH

### Please leave any additional context

A store of value that has no value besides its deemed value may not have the longevity fi
...

✅ hebasto closed an issue: "Block size evaluation requested"
(https://github.com/bitcoin/bitcoin/issues/30448)

:lock: hebasto locked an issue: "Block size evaluation requested"
(https://github.com/bitcoin/bitcoin/issues/30448)

💬 hebasto commented on pull request "Cluster size 2 package rbf":
(https://github.com/bitcoin/bitcoin/pull/28984#issuecomment-2227216176)
Ported to the CMake-based build system in https://github.com/hebasto/bitcoin/pull/264.

💬 hebasto commented on pull request "kernel: Streamline util library":
(https://github.com/bitcoin/bitcoin/pull/29015#issuecomment-2227216319)
Ported to the CMake-based build system in https://github.com/hebasto/bitcoin/pull/264.