ACK fa2247a.
In my understanding, these changes disable copy-initialization of CTransaction to avoid potentially hashing twice.

On a related note, should we make all single-argument constructors explicit ?
https://github.com/isocpp/CppCoreGuidelines/blob/6f27719b2b994d1304f78194dc7824e4ddeea5f5/CppCoreGuidelines.md#c46-by-default-declare-single-argument-constructors-explicit

In my understanding, these changes disable copy-initialization of CTransaction to avoid potentially hashing twice.

Not sure what you mean, but this should not change any constructor or code logic. This simply forces devs to write CTransaction at least once if they want to construct one.

On a related note, should we make all single-argument constructors explicit ?
https://github.com/isocpp/CppCoreGuidelines/blob/6f27719b2b994d1304f78194dc7824e4ddeea5f5/CppCoreGuidelines.md#c46-by-default-declare-single-argument-constructors-explicit

Probably not, as the url you link to advises against that.

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Conflicts

Reviewers, this pull request conflicts with the following ones:

• #17529 (rpc: Faster getblock using PureBlock by MarcoFalke)

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.