🤔 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
...

💬 pablomartin4btc 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_r1413211901)
Thanks, there was an extra space as well.

💬 pablomartin4btc 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_r1413211918)
Right, thanks!

💬 pablomartin4btc 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_r1413211927)
I'd prefer to keep the definition as is since the function is called only once, thanks for the suggestion.

💬 pablomartin4btc 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_r1413211937)
Yeah, your version is clearer. Thanks!