🚀 fanquake merged a pull request: "ci: Remove second user account"
(https://github.com/bitcoin/bitcoin/pull/27376)

⚠️ dergoegge opened an issue: "Intermittent failures in interface_usdt_mempool.py"
(https://github.com/bitcoin/bitcoin/issues/27380)
https://cirrus-ci.com/task/5779522121891840

```
[0;34m node0 2023-03-31T11:45:58.812384Z (mocktime: 2023-04-14T11:46:01Z) [http] [httpserver.cpp:257] [http_request_cb] [http] Received a POST request for / from 127.0.0.1:38238 [0m
[0;34m node0 2023-03-31T11:45:58.812543Z (mocktime: 2023-04-14T11:46:01Z) [httpworker.2] [rpc/request.cpp:179] [parse] [rpc] ThreadRPCServer method=getpeerinfo user=__cookie__ [0m
[0;36m test 2023-03-31T11:45:58.814000Z TestFramework (INFO): Hooking into mem
...

💬 dergoegge commented on issue "Intermittent failures in interface_usdt_mempool.py":
(https://github.com/bitcoin/bitcoin/issues/27380#issuecomment-1491844460)
Have seen this a couple times by now.

cc @0xB10C @virtu

💬 hebasto commented on pull request "net processing: #26140 follow-ups":
(https://github.com/bitcoin/bitcoin/pull/27379#discussion_r1154410052)
Mention this in the PR description?

⚠️ fanquake opened an issue: "miniscript: comparison of integers of different signs"
(https://github.com/bitcoin/bitcoin/issues/27381)
This is being emitted with [GCC 10.2.1](https://packages.debian.org/bullseye/g++-arm-linux-gnueabihf), in some CI jobs: i.e https://github.com/bitcoin/bitcoin/pull/25972/checks?check_run_id=12423693655 (where we actually turn on all our warning flags with depends builds).
```bash
In file included from test/miniscript_tests.cpp:19:
./script/miniscript.h:1197:71: error: comparison of integers of different signs: 'typename iterator_traits<int *>::difference_type' (aka 'int') and 'const uint32_t'
...

💬 fanquake commented on pull request "p2p: skip netgroup diversity of new connections for tor/i2p/cjdns":
(https://github.com/bitcoin/bitcoin/pull/27374#issuecomment-1491851596)
https://github.com/bitcoin/bitcoin/pull/27374/checks?check_run_id=12424651425
```bash
A new circular dependency in the form of "netaddress -> netbase -> netaddress" appears to have been introduced.

^---- failure generated from lint-circular-dependencies.py
```

💬 dergoegge commented on pull request "net processing: #26140 follow-ups":
(https://github.com/bitcoin/bitcoin/pull/27379#discussion_r1154414721)
Done

💬 MarcoFalke commented on issue "miniscript: comparison of integers of different signs":
(https://github.com/bitcoin/bitcoin/issues/27381#issuecomment-1491854885)
It should be fine to make the returned value of `std::count` unsigned.

💬 darosior commented on issue "miniscript: comparison of integers of different signs":
(https://github.com/bitcoin/bitcoin/issues/27381#issuecomment-1491856229)
I'm on it. I'll check if there are other occurrences too, had quite a lot of them in #27255.

💬 MarcoFalke commented on pull request "test: Remove python3.5 workaround":
(https://github.com/bitcoin/bitcoin/pull/27378#discussion_r1154418992)
thx, done

👍 Sjors approved a pull request: "script: add description for the functionality of each opcode"
(https://github.com/bitcoin/bitcoin/pull/27109)
ACK 40f36d0a3dac568bc9b05dea021d83986d8add84

It matches my (limited) understanding of what these opcodes do. I also walked through `EvalScript()` to compare.

💬 Sjors commented on pull request "script: add description for the functionality of each opcode":
(https://github.com/bitcoin/bitcoin/pull/27109#discussion_r1154337691)
Note to other reviewers: this is implemented in `CastToBool()` in `interpreter.cpp`, which is used by `OP_IF`, `OP_NOTIF`, `OP_VERIFY`, `OP_IFDUP`, `OP_NUMEQUALVERIFY` and after executing the `scriptPubKey`.

💬 Sjors commented on pull request "script: add description for the functionality of each opcode":
(https://github.com/bitcoin/bitcoin/pull/27109#discussion_r1154247024)
Can be added in a followup if you don't want to lose ACKs:

```cpp
OP_CHECKSIGADD = 0xba, // pop the public key, N and a signature, push N if signature is empty, fail if it's invalid, otherwise push N + 1 (see BIP 342)
```

💬 Sjors commented on pull request "script: add description for the functionality of each opcode":
(https://github.com/bitcoin/bitcoin/pull/27109#discussion_r1154350939)
Could add: `is true (1 for tapscript)`

Although we probably don't want to go into too much detail here, this is still useful to point out since it directly contradicts the `true` / `false` explanation at the top.

💬 Sjors commented on pull request "script: add description for the functionality of each opcode":
(https://github.com/bitcoin/bitcoin/pull/27109#discussion_r1154351337)
Could add: `is false ("" for tapscript)`

💬 Sjors commented on pull request "script: add description for the functionality of each opcode":
(https://github.com/bitcoin/bitcoin/pull/27109#discussion_r1154357244)
Note for other reviewers, the [Bitcoin Wiki](https://en.bitcoin.it/wiki/Script) provides a useful hint on how to read Satoshi's comments in `interpreter.cpp`:

> the inputs and outputs are both described by items as if they were pushed on the stack in that order. So for example, "x1 x2" indicates pushing value x1 on the stack, then x2, such that x1 is at the bottom of the stack, and x2 is at the top

This also explains why items from the stack are read "out of order":

```
valtype vch1 =
...

💬 Sjors commented on pull request "script: add description for the functionality of each opcode":
(https://github.com/bitcoin/bitcoin/pull/27109#discussion_r1154398410)
I still have no idea how `OP_CODESEPARATOR` works, but that will take more than one code comment to explain.

💬 dergoegge commented on pull request "addrman, refactor: improve stochastic test in `AddSingle`":
(https://github.com/bitcoin/bitcoin/pull/27319#issuecomment-1491872681)
This doesn't seem like a big win. Have you benchmarked this to see if it actually makes a difference?

(fwiw I think the bigger optimization you are making here is using `<<` over the loop, as opposed to what you mention in the description)

💬 dergoegge commented on pull request "net processing: #26140 follow-ups":
(https://github.com/bitcoin/bitcoin/pull/27379#issuecomment-1491873671)
CI failure is intermittent, see #27380

✅ fanquake closed a pull request: "util/system: Close non-std fds when execing slave processes"
(https://github.com/bitcoin/bitcoin/pull/22417)