diff options
| author | glozow <gloriajzhao@gmail.com> | 2024-04-24 16:39:54 +0100 |
|---|---|---|
| committer | glozow <gloriajzhao@gmail.com> | 2024-04-24 16:51:56 +0100 |
| commit | 2a07c4662d7266158d47f79fa2433ab22e22c907 (patch) | |
| tree | dabdd7341c79685465a8b42cc1a3fe8e36fb8671 /depends | |
| parent | 50729c060907b69543755a8e1ed6b0d32d071a8a (diff) | |
| parent | b22901dfa9cc3af94bf13163a28300eb1ab25b22 (diff) | |
| download | bitcoin-2a07c4662d7266158d47f79fa2433ab22e22c907.tar.xz | |
Merge bitcoin/bitcoin#29757: feefrac: avoid explicitly computing diagram; compare based on chunks
b22901dfa9cc3af94bf13163a28300eb1ab25b22 Avoid explicitly computing diagram; compare based on chunks (Pieter Wuille)
Pull request description:
This merges the `BuildDiagramFromChunks` and `CompareFeeRateDiagram` introduced in #29242 into a single `CompareChunks` function, which operates on sorted chunk data rather than diagrams, instead computing the diagram on the fly.
This avoids the need for the construction of an intermediary diagram object, and removes the slightly arbitrary "all diagrams must start at (0, 0)" requirement.
Not a big deal, but I think the result is a bit cleaner and not really more complicated.
ACKs for top commit:
glozow:
reACK b22901d
instagibbs:
reACK https://github.com/bitcoin/bitcoin/pull/29757/commits/b22901dfa9cc3af94bf13163a28300eb1ab25b22
Tree-SHA512: ca37bdf61d9a9cb5435f4da73e97ead33bf65828ad9af49b87336b1ece70db8ced1c21f517fc6eb6d616311c91f3da75ecae6b9bd42547133e3a3c5320b7816d
Diffstat (limited to 'depends')
0 files changed, 0 insertions, 0 deletions