💬 sipa commented on pull request "feefrac: avoid explicitly computing diagram; compare based on chunks":
(https://github.com/bitcoin/bitcoin/pull/29757#discussion_r1552553259)
The `FeeFrac::size` field is an `int32_t`, which allows using the `FeeFrac{replacement_fees, replacement_vsize}` constructor below.

💬 sipa commented on pull request "feefrac: avoid explicitly computing diagram; compare based on chunks":
(https://github.com/bitcoin/bitcoin/pull/29757#discussion_r1552556360)
Good point, this isn't testing anymore at all. Gone.

💬 theStack commented on pull request "test: refactor: introduce and use `calculate_input_weight` helper":
(https://github.com/bitcoin/bitcoin/pull/29777#discussion_r1552562226)
removed

💬 theStack commented on pull request "test: refactor: introduce and use `calculate_input_weight` helper":
(https://github.com/bitcoin/bitcoin/pull/29777#discussion_r1552562328)
removed

💬 theStack commented on pull request "test: refactor: introduce and use `calculate_input_weight` helper":
(https://github.com/bitcoin/bitcoin/pull/29777#discussion_r1552562536)
done

💬 theStack commented on pull request "test: refactor: introduce and use `calculate_input_weight` helper":
(https://github.com/bitcoin/bitcoin/pull/29777#discussion_r1552565066)
removed the duplicated `scriptSig_large` and added the assertion (small scriptSig, empty witness stack) as suggested

💬 theStack commented on pull request "test: refactor: introduce and use `calculate_input_weight` helper":
(https://github.com/bitcoin/bitcoin/pull/29777#issuecomment-2038425304)
Thanks for your reviews @kevkevinpal, @naiyoma, I force-pushed with all of your suggestions taken!

✅ achow101 closed an issue: "changed username"
(https://github.com/bitcoin/bitcoin/issues/29808)

:lock: achow101 locked an issue: "."
(https://github.com/bitcoin/bitcoin/issues/29808)

⚠️ tuttheking81 opened an issue: "."
(https://github.com/bitcoin/bitcoin/issues/29810)
.

_Originally posted by @tuttheking81 in https://github.com/bitcoin/bitcoin/issues/29808_

✅ pinheadmz closed an issue: "."
(https://github.com/bitcoin/bitcoin/issues/29810)

:lock: achow101 locked an issue: "."
(https://github.com/bitcoin/bitcoin/issues/29810)

⚠️ Rojolee opened an issue: "Should this heed --noshutdown?"
(https://github.com/bitcoin/bitcoin/issues/29811)
Should this heed --noshutdown?

_Originally posted by @laanwj in https://github.com/bitcoin/bitcoin/issues/12904#issuecomment-379314241_

✅ achow101 closed an issue: "Should this heed --noshutdown?"
(https://github.com/bitcoin/bitcoin/issues/29811)

:lock: achow101 locked an issue: "Should this heed --noshutdown?"
(https://github.com/bitcoin/bitcoin/issues/29811)

⚠️ Richlilyjoy100 opened an issue: "H"
(https://github.com/bitcoin/bitcoin/issues/29812)

🤔 stratospher reviewed a pull request: "net: update comment for service bit support info for seed.bitcoin.sipa.be"
(https://github.com/bitcoin/bitcoin/pull/29809#pullrequestreview-1981717481)
Concept ACK. thanks for opening this! it's easy to forget to update support for service bit filtering here and the list just keeps getting longer.

another possibility could be just mentioning that the seeder supports service bit filtering and linking to the seeder repo to check the exact list of service bits supported. curious to know what other people think.

💬 stratospher commented on pull request "net: update comment for service bit support info for seed.bitcoin.sipa.be":
(https://github.com/bitcoin/bitcoin/pull/29809#discussion_r1552802169)
48afa53: there's also x49.

💬 instagibbs commented on pull request "feefrac: avoid explicitly computing diagram; compare based on chunks":
(https://github.com/bitcoin/bitcoin/pull/29757#issuecomment-2039001407)
ACK https://github.com/bitcoin/bitcoin/pull/29757/commits/c2fdcf42e9c4ca0157115e465cca298df70fc700

💬 instagibbs commented on pull request "AcceptMultipleTransactions: Fix workspace not being set as client_maxfeerate failure":
(https://github.com/bitcoin/bitcoin/pull/29735#discussion_r1552969962)
that's the error string as it's reported unfortunately!