✅ furszy closed a pull request: "wallet: remove unused DummySignTx and CKeyPool from GetReservedDestination"
(https://github.com/bitcoin/bitcoin/pull/25881)

💬 furszy commented on pull request "wallet: remove unused DummySignTx and CKeyPool from GetReservedDestination":
(https://github.com/bitcoin/bitcoin/pull/25881#issuecomment-1837520765)
Closing it for now. Have other PRs with more priority than this one.

💬 furszy commented on pull request "p2p: adaptive connections services flags":
(https://github.com/bitcoin/bitcoin/pull/28170#discussion_r1413143266)
> This flag is only used to connect to limited peers or not. In that case, why not remove the indirection about syncing and just call it `allow_limited_peers`?

Because of what I wrote above, this flag can (and most likely will) be used in other places as well, fulfilling two goals: (1) minimizing `cs_main` locks, which improves the overall node response, and (2) minimizing the usage of the chainstate IBD flag, which lacks back-and-forth functionality

💬 hebasto commented on pull request "test: Fix test by checking the actual exception instance":
(https://github.com/bitcoin/bitcoin/pull/28989#issuecomment-1837526938)
Updated 24d978a36d1b405f968161a9d046beffffc0b249 -> 55e3dc3e03510e97caba1547a82e3e022b0bbd42 ([pr28989.01](https://github.com/hebasto/bitcoin/commits/pr28989.01) -> [pr28989.02](https://github.com/hebasto/bitcoin/commits/pr28989.02), [diff](https://github.com/hebasto/bitcoin/compare/pr28989.01..pr28989.02)):

- addressed @furszy's [comment](https://github.com/bitcoin/bitcoin/pull/28989#discussion_r1413137963)

💬 hebasto commented on pull request "test: Fix test by checking the actual exception instance":
(https://github.com/bitcoin/bitcoin/pull/28989#discussion_r1413145308)
Thanks! [Updated](https://github.com/bitcoin/bitcoin/pull/28989#issuecomment-1837526938).

🤔 furszy reviewed a pull request: "test: Fix test by checking the actual exception instance"
(https://github.com/bitcoin/bitcoin/pull/28989#pullrequestreview-1761301944)
Non-Windows code ACK 55e3dc3e

💬 andrewtoth commented on pull request "p2p: adaptive connections services flags":
(https://github.com/bitcoin/bitcoin/pull/28170#discussion_r1413149420)
Right, then I think `m_is_close_to_tip` would make the most sense, since it's set initially after calling `IsBlockCloseToTip`.

👍 andrewtoth approved a pull request: "cli: improve error message on multiwallet and add validation to cli-side commands"
(https://github.com/bitcoin/bitcoin/pull/26990#pullrequestreview-1761314443)
ACK 755320f75f2141909e84b62f420462c1c5b193e6

💬 andrewtoth commented on pull request "cli: improve error message on multiwallet and add validation to cli-side commands":
(https://github.com/bitcoin/bitcoin/pull/26990#discussion_r1413160578)
nit: `3. unrecognized cli-commands or other arguments starting with a slash`

💬 andrewtoth commented on pull request "cli: improve error message on multiwallet and add validation to cli-side commands":
(https://github.com/bitcoin/bitcoin/pull/26990#discussion_r1413159532)
nit:
```
* @returns true if cli_command is included in the set of the exclusively_commands.
```

💬 andrewtoth commented on pull request "cli: improve error message on multiwallet and add validation to cli-side commands":
(https://github.com/bitcoin/bitcoin/pull/26990#discussion_r1413159552)
nit: could be `static`.

💬 andrewtoth commented on pull request "cli: improve error message on multiwallet and add validation to cli-side commands":
(https://github.com/bitcoin/bitcoin/pull/26990#discussion_r1413159420)
nit: This parameter is not a reference. It is being passed by value. This comment could be more descriptive eg:
```
@param[in] cli_command The string to check if it is a cli-command that needs to run exclusively.
```

💬 andrewtoth commented on pull request "cli: improve error message on multiwallet and add validation to cli-side commands":
(https://github.com/bitcoin/bitcoin/pull/26990#discussion_r1413160472)
nit: `2. multiple bitcoin-cli commands that need to run exclusively`

👍 andrewtoth approved a pull request: "p2p: make block download logic aware of limited peers threshold"
(https://github.com/bitcoin/bitcoin/pull/28120#pullrequestreview-1761328425)
ACK a0687ffd568b0984a8b1e51fd0061f87f25e95d7

💬 andrewtoth commented on pull request "p2p: make block download logic aware of limited peers threshold":
(https://github.com/bitcoin/bitcoin/pull/28120#discussion_r1413170635)
nit: Could add a comment on why number 12 is chosen here?

💬 andrewtoth commented on pull request "Don't empty dbcache on prune flushes: >30% faster IBD":
(https://github.com/bitcoin/bitcoin/pull/28280#issuecomment-1837561632)
Rebased, ran the following benchmark using hyperfine. Sync to block 800k 3 times on master commit and the last commit of this branch (I updated the commit recently, but no code changes only commit order changes to satisfy CI), dbcache=4000 and prune=550:

```
hyperfine \
--export-markdown ~/bench.md \
--show-output --parameter-list commit 498994b6f55d04a7940f832e7fbd17e5acdaff15,216bd93138a46cdfa66523878a36d0e634476079 \
--prepare 'git checkout {commit} && make -j$(nproc) src/bitcoind; syn
...

💬 hebasto commented on pull request "POC: Replace Boost.Process with cpp-subprocess":
(https://github.com/bitcoin/bitcoin/pull/28981#issuecomment-1837570291)
Rebased on top of the #28989.

💬 mzumsande commented on issue "getrawtransaction xxxxxx.... 2 causes a segfault":
(https://github.com/bitcoin/bitcoin/issues/28986#issuecomment-1837588656)
Are you using txindex or was the transaction in your mempool at the time of querying?

📝 NoviX-Labs opened a pull request: "Novix Plus"
(https://github.com/bitcoin/bitcoin/pull/28990)
What is Novix-Plus Core?
---------------------

Novix-Plus Core connects to the Novix-Plus peer-to-peer network to download and fully
validate blocks and transactions. It also includes a wallet and graphical user
interface, which can be optionally built.

Further information about Novix-Plus Core is available in the [doc folder](/doc).

License
-------

Novix-Plus Core is released under the terms of the MIT license. See [COPYING](COPYING) for more
information or see https://opensour
...

✅ hebasto closed a pull request: "Novix Plus"
(https://github.com/bitcoin/bitcoin/pull/28990)

📝 hebasto locked a pull request: "."
(https://github.com/bitcoin/bitcoin/pull/28990)
What is Novix-Plus Core?
---------------------

Novix-Plus Core connects to the Novix-Plus peer-to-peer network to download and fully
validate blocks and transactions. It also includes a wallet and graphical user
interface, which can be optionally built.

Further information about Novix-Plus Core is available in the [doc folder](/doc).

License
-------

Novix-Plus Core is released under the terms of the MIT license. See [COPYING](COPYING) for more
information or see https://opensour
...