💬 TheCharlatan commented on pull request "Encapsulate warnings in generalized node::Warnings and remove globals":
(https://github.com/bitcoin/bitcoin/pull/30058#issuecomment-2100025531)
Concept ACK on removing the warnings globals.

💬 josibake commented on pull request "crypto: add `NUMS_H` const":
(https://github.com/bitcoin/bitcoin/pull/30048#discussion_r1593657407)
Oops! Should have ran it myself before including it. I have a slight preference for keeping the sage code, since its nice to be able to run that snippet and get the same value as a way of verifying. That being said, "be able to run that snippet" implies keeping the comment up to date, but I doubt that will be much (if any) of a maintenance burden.

Also think its more correct to output the x,y coordinate pair since in the comment we are referring to `NUMS_H` as a point, whereas the x-only valu
...

💬 josibake commented on pull request "crypto: add `NUMS_H` const":
(https://github.com/bitcoin/bitcoin/pull/30048#discussion_r1593668308)
Updated to fix the hex decoding. Went ahead and left the (x,y) print out, but can drop the y-value or drop the whole sage snippet if others feel strongly.

✅ maflcko closed a pull request: "build: Require libc++-16 or later"
(https://github.com/bitcoin/bitcoin/pull/29077)

📝 maflcko opened a pull request: "ci: Roll clang in test-each-commit task"
(https://github.com/bitcoin/bitcoin/pull/30060)
Needed for https://github.com/bitcoin/bitcoin/pull/29077#issuecomment-2099704210

💬 theStack commented on pull request "crypto: add `NUMS_H` const":
(https://github.com/bitcoin/bitcoin/pull/30048#discussion_r1593731959)
Agree that keeping the sage code and printing out the full point makes sense w.r.t. your point vs byte encoding argument. (By the way, at least in our test-framework implementation of GE, [`lift_x` always produces a point with even y coordinate](https://github.com/bitcoin/bitcoin/blob/43a66c55ec8770cf7c21112aac9b997f3f2fb704/test/functional/test_framework/crypto/secp256k1.py#L256), but no idea if this is also the case in the GE of sage.)

👍 theStack approved a pull request: "crypto: add `NUMS_H` const"
(https://github.com/bitcoin/bitcoin/pull/30048#pullrequestreview-2045153103)
Code-review ACK a5867a393cacbe4b2e0de51b842362754e98ab42

💬 maflcko commented on pull request "ci: add markdown link check job":
(https://github.com/bitcoin/bitcoin/pull/30034#discussion_r1593737753)
nit: It is already a `Vec`, so no need to iter+collect, no?

💬 maflcko commented on pull request "ci: add markdown link check job":
(https://github.com/bitcoin/bitcoin/pull/30034#discussion_r1593739133)
nit: Can also use `.args([...])` for less code?

💬 maflcko commented on pull request "ci: add markdown link check job":
(https://github.com/bitcoin/bitcoin/pull/30034#discussion_r1593737106)
nit: Could remove this, as the URL is below as well.

💬 maflcko commented on pull request "ci: add markdown link check job":
(https://github.com/bitcoin/bitcoin/pull/30034#discussion_r1593736433)
check_output is a helper that assumes the command exists, so I don't think it will work here, no?

Ref:

```
thread 'main' panicked at src/main.rs:23:28:
command error: Os { code: 2, kind: NotFound, message: "No such file or directory" }
stack backtrace:
0: rust_begin_unwind
...
```

I think in a previous version you had a check for `NotFound` directly, so you could restore that?

💬 laanwj commented on pull request "guix: build with glibc 2.31":
(https://github.com/bitcoin/bitcoin/pull/29987#issuecomment-2100177952)
New used symbols since 2.28:

81: 0000000000000000 0 FUNC GLOBAL DEFAULT UND fcntl64@GLIBC_2.28 (15)

`fcntl` function that can handle large file offsets on 32 bit platforms. Used in `unixFileLock`. `unixShmLock` and other posix/unix file functions.

195: 0000000000000000 0 FUNC GLOBAL DEFAULT UND statx@GLIBC_2.28 (15)

Extended file stat. Used in `QFileSystemEngine::fillMetaData`.

533: 0000000000000000 0 FUNC GLOBAL DEFAULT UND r
...

💬 willcl-ark commented on pull request "ci: add markdown link check job":
(https://github.com/bitcoin/bitcoin/pull/30034#discussion_r1593777495)
Reverted in 9d2a8a8b1d67af14e4a5fc8276a1069b6ef9fe4a

💬 willcl-ark commented on pull request "ci: add markdown link check job":
(https://github.com/bitcoin/bitcoin/pull/30034#discussion_r1593777653)
Removed in 9d2a8a8b1d67af14e4a5fc8276a1069b6ef9fe4a

💬 josibake commented on pull request "crypto, refactor: add method for applying the taptweak":
(https://github.com/bitcoin/bitcoin/pull/30051#issuecomment-2100225575)
Updated to use a utility function (instead of a method on `CKey`) per the @theuni 's feedback

💬 maflcko commented on issue "Memory leak with `rest/block` REST endpoint and `getblock` RPC when verbosity >=2":
(https://github.com/bitcoin/bitcoin/issues/30052#issuecomment-2100234965)
Did the same check with

```
valgrind --tool=massif ./src/bitcoind &
while sleep 3; do ./src/bitcoin-cli getblock 00000000000000000002a7c4c1e48d76c5a37902165a270156b7a8d72728a054 2 ; done

...

massif-visualizer ./massif.out.1234
```


✅ maflcko closed an issue: "Memory leak with `rest/block` REST endpoint and `getblock` RPC when verbosity >=2"
(https://github.com/bitcoin/bitcoin/issues/30052)

💬 maflcko commented on issue "Memory leak with `rest/block` REST endpoint and `getblock` RPC when verbosity >=2":
(https://github.com/bitcoin/bitcoin/issues/30052#issuecomment-2100235479)
Closing for now, as duplicate

💬 maflcko commented on issue "Slow memory leak in v22.0?":
(https://github.com/bitcoin/bitcoin/issues/24542#issuecomment-2100240679)
Not sure what can be done here. It would be good to use a tool to debug this, e.g. https://github.com/bitcoin/bitcoin/issues/24542#issuecomment-1067744040, if it still happens.

👍 fanquake approved a pull request: "ci: Roll clang in test-each-commit task"
(https://github.com/bitcoin/bitcoin/pull/30060#pullrequestreview-2045257886)
ACK 2094da680fe580ba1bc62e9bab1e196f5e1c5341