Pieter Wuille

I have made a number of changes: * Addressed a number of typos * Moved connected-component logic to #30286 (where `MakeConnected` is now optional, and `FindConnectedComponent` has its own fuzz...

> At this point could we maybe not rebase on master unless necessary? :) I shall resist.

Ok, the mermaid diagrams should be fixed now, match the code, and I've verified (post- #30286) that the claims about difficulty of linearization hold for the code. > I shall...

It's unclear to me whether the `UniValue` type has a move constructor, actually. If not, then these `std::move`s have no effect. @ryanofsky If you theory is correct, then having a...

Oops, I clicked wrong. I just meant to Concept ACK.

I think the 86 kB option is a bit faster, and the change in binary size/memory is immaterial for Bitcoin Core, so from that perspective maybe we want to use...

If your hardware regularly corrupts the database, the system is just too unreliable, and there is little that can be done long term. Even with a regular snapshot feature, it...

Generally the "rolling forward" steps are slower than just normal block processing, because certain optimizations don't apply. It's working on an inherently inconsistent state, so some assumptions that are valid...

There isn't really a solution, besides an invasive redesign of the database and cache structure. If you set the dbcache high, and your node crashes in the middle of flushing...

utACK 8bd0a8b18d68bbacee731f3e0583397f81ca078e