diff options
| author | MarcoFalke <falke.marco@gmail.com> | 2019-11-04 08:32:09 -0500 |
|---|---|---|
| committer | MarcoFalke <falke.marco@gmail.com> | 2019-11-04 08:32:22 -0500 |
| commit | fba574c908bb61eff1a0e83c935f3526ba9035f2 (patch) | |
| tree | dfc8c14d1d42f661e9fd492982573730a289a920 /depends | |
| parent | def69e5a8632987765b409d5fdd5044250a631f1 (diff) | |
| parent | fa8919889f3c1bd3e2700ecbb56493e3cd1e25ad (diff) | |
| download | bitcoin-fba574c908bb61eff1a0e83c935f3526ba9035f2.tar.xz | |
Merge #17349: Remove redundant copy constructors
fa8919889f3c1bd3e2700ecbb56493e3cd1e25ad bench: Remove redundant copy constructor in mempool_stress (MarcoFalke)
29f84343681831baf02a17d3af566c5c57ecf3c2 refactor: Remove redundant PSBT copy constructor (Hennadii Stepanov)
Pull request description:
I fail to see why people add these copy constructors manually without explanation, when the compiler can generate them at least as good automatically with less code.
ACKs for top commit:
promag:
ACK fa8919889f3c1bd3e2700ecbb56493e3cd1e25ad.
hebasto:
ACK fa8919889f3c1bd3e2700ecbb56493e3cd1e25ad, nit s/constructor/operator/ in commit fa8919889f3c1bd3e2700ecbb56493e3cd1e25ad message, as @promag [mentioned](https://github.com/bitcoin/bitcoin/pull/17349#discussion_r341776389) above.
jonatack:
ACK fa8919889f3c1bd3e2700ecbb56493e3cd1e25ad
Tree-SHA512: ce024fdb894328f41037420b881169b8b1b48c87fbae5f432edf371a35c82e77e21468ef97cda6f54d34f1cf9bb010235d62904bb0669793457ed1c3b2a89723
Diffstat (limited to 'depends')
0 files changed, 0 insertions, 0 deletions