07:01 < bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/2c0c3f8e8ca6...c7007babb7c2

07:01 < bitcoin-git> bitcoin/master d0ea9ba Andrew Chow: walletdb: Don't remove database transaction logs and instead error

07:01 < bitcoin-git> bitcoin/master c7007ba MarcoFalke: Merge #18907: walletdb: Don't remove database transaction logs and instead...

07:02 < bitcoin-git> [bitcoin] MarcoFalke merged pull request #18907: walletdb: Don't remove database transaction logs and instead error (master...dont-retry-bdbenv) https://github.com/bitcoin/bitcoin/pull/18907

08:16 < mutatrum> phantomcircuit: I'll give it a try. I'm curious to see what it would bring.

09:19 < bitcoin-git> [bitcoin] theStack opened pull request #19564: test: p2p_feefilter improvements (logging, refactoring, speedup) (master...20200722-test-p2p_feefilter_improvements) https://github.com/bitcoin/bitcoin/pull/19564

09:33 < bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/c7007babb7c2...597d2f905eca

09:33 < bitcoin-git> bitcoin/master c0f09c2 Jon Atack: fuzz: add missing overrides to signature_checker

09:33 < bitcoin-git> bitcoin/master 597d2f9 fanquake: Merge #19548: fuzz: add missing overrides to signature_checker

09:33 < bitcoin-git> [bitcoin] fanquake merged pull request #19548: fuzz: add missing overrides to signature_checker (master...signature-checker-missing-overrides) https://github.com/bitcoin/bitcoin/pull/19548

10:04 < bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/597d2f905eca...2031aa92a32d

10:04 < bitcoin-git> bitcoin/master c8992e8 freenancial: test: Fix fuzzer compilation on macOS

10:04 < bitcoin-git> bitcoin/master 2031aa9 fanquake: Merge #19562: test: Fix fuzzer compilation on macOS

10:05 < bitcoin-git> [bitcoin] fanquake merged pull request #19562: test: Fix fuzzer compilation on macOS (master...fix-fuzzer-macos) https://github.com/bitcoin/bitcoin/pull/19562

10:35 < bitcoin-git> [bitcoin] fanquake opened pull request #19565: build: call AC_PATH_TOOL for dsymutil in macOS cross-compile (master...cant_find_dsymutil) https://github.com/bitcoin/bitcoin/pull/19565

11:51 < sorcus> Hi :-)

11:52 < sorcus> Can i ask a question here about libsecp256k1?

11:58 < bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/2031aa92a32d...c7b4968552c7

11:58 < bitcoin-git> bitcoin/master 2a701a1 fanquake: build: pass -fcommon when building genisoimage

11:58 < bitcoin-git> bitcoin/master c7b4968 Wladimir J. van der Laan: Merge #19553: build: pass -fcommon when building genisoimage

11:58 < bitcoin-git> [bitcoin] laanwj merged pull request #19553: build: pass -fcommon when building genisoimage (master...retain_fno_common_genisoimage) https://github.com/bitcoin/bitcoin/pull/19553

12:06 < sorcus> Can i generate points bit-by-bit like here https://github.com/q9f/secp256k1.cr/blob/master/src/core.cr#L104-L120 with libsecp256k1?

12:07 < sorcus> If bit is zero, call `add` function, else call `double`.

13:00 < bitcoin-git> [bitcoin] hebasto opened pull request #19567: p2p, refactor: Do not over-reserve vAddr capacity (master...200722-addr) https://github.com/bitcoin/bitcoin/pull/19567

13:47 < instagibbs> sorcus, try #secp256k1

13:48 < instagibbs> that's where libsecp devs hang

13:55 < bitcoin-git> [bitcoin] fjahr opened pull request #19568: Wallet should not override signing errors (master...err_override) https://github.com/bitcoin/bitcoin/pull/19568

13:56 < sorcus> instagibbs: Oh, thank you :-)

17:25 < bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/c7b4968552c7...93decbc7a486

17:25 < bitcoin-git> bitcoin/master 1554b54 Daniel Kraft: Static asserts for consistency of fee defaults.

17:25 < bitcoin-git> bitcoin/master 93decbc Wladimir J. van der Laan: Merge #19370: Static asserts for consistency of fee defaults

17:26 < bitcoin-git> [bitcoin] laanwj merged pull request #19370: Static asserts for consistency of fee defaults (master...fee-asserts) https://github.com/bitcoin/bitcoin/pull/19370

17:49 < bitcoin-git> [bitcoin] laanwj pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/93decbc7a486...1397afc5ec03

17:49 < bitcoin-git> bitcoin/master fa49289 MarcoFalke: refactor: Switch ValidationState mode to C++11 enum class

17:49 < bitcoin-git> bitcoin/master fa56eda MarcoFalke: log: Avoid treating remote misbehvior as local system error

17:49 < bitcoin-git> bitcoin/master 1397afc Wladimir J. van der Laan: Merge #19526: log: Avoid treating remote misbehvior as local system error

17:49 < bitcoin-git> [bitcoin] laanwj merged pull request #19526: log: Avoid treating remote misbehvior as local system error (master...2007-logErrorVal) https://github.com/bitcoin/bitcoin/pull/19526

18:59 < bitcoin-git> [bitcoin] laanwj pushed 19 commits to master: https://github.com/bitcoin/bitcoin/compare/1397afc5ec03...ccef10261efc

18:59 < bitcoin-git> bitcoin/master 2b4b90a Suhas Daftuar: Add a wtxid-index to the mempool

18:59 < bitcoin-git> bitcoin/master c7eb6b4 Amiti Uttarwar: Add wtxid to mempool unbroadcast tracking

18:59 < bitcoin-git> bitcoin/master 60f0acd Suhas Daftuar: Just pass a hash to AddInventoryKnown

19:00 < bitcoin-git> [bitcoin] laanwj merged pull request #18044: Use wtxid for transaction relay (master...2020-01-wtxid-inv) https://github.com/bitcoin/bitcoin/pull/18044

19:05 < instagibbs> \o/

19:08 < sdaftuar> thanks wumpus!

19:45 < sipa> [A

20:02 < wumpus> sdaftuar: sorry for all the rebase hell you had to go through

20:25 < jeremyrubin> sipa: Does it rely on it both ways or just one way? e.g. wtxid is txid, or txid is wtxid?

20:26 < jeremyrubin> because if one way you can make wtxid downcastable to txid through a "is_a_txid_i_swear" cast method

20:28 < sipa> jeremyrubin: sure, it's of course possible

20:28 < sipa> i mean it's just not a particularly clean abstraction

20:29 < sipa> if we go that direction, perhaps we should start with having txids and blockhashes and merkle roots be distinct types

20:29 < sipa> instead of starting with this distinction

20:44 < jeremyrubin> didn't jb55 try to do this?

20:45 < jb55> hrm?

20:45 < jb55> I don't think so

20:47 < sipa> we recently had a chance that sounds vaguely related, where the different TxDestionation types were made more independent types

20:47 < sipa> https://github.com/bitcoin/bitcoin/pull/17938

20:49 < jeremyrubin> ah was not jb55 that's what I was thinking of.

21:05 < sipa> sdaftuar: do you have thoughts on how to integrate the txid 2s delay into #19184 ?

21:05 < gribble> https://github.com/bitcoin/bitcoin/issues/19184 | Overhaul transaction request logic by sipa · Pull Request #19184 · bitcoin/bitcoin · GitHub