diff options
| author | MacroFake <falke.marco@gmail.com> | 2022-08-01 11:18:26 +0200 |
|---|---|---|
| committer | MacroFake <falke.marco@gmail.com> | 2022-08-01 11:19:55 +0200 |
| commit | da2332099873977873402227c6e69239cbb2235a (patch) | |
| tree | 18891ebf671a7d97e2a53e6907d5892a5e6521e5 /test/functional/test_framework/test_shell.py | |
| parent | f5eadcb148713add9a183a31b011ce86ecf7fa20 (diff) | |
| parent | 4bedfd702ad878645c51bea6ee8ce40d8c0bd3da (diff) | |
Merge bitcoin/bitcoin#25651: refactor: make all NodeImpl/ChainImpl/ExternalSignerImpl members public, rm temporaries, simplify
4bedfd702ad878645c51bea6ee8ce40d8c0bd3da refactor: remove unneeded temporaries in node/interfaces, simplify code (Jon Atack)
b27ba169ebd4a8e4ec29be590f03a4d0da61a0cc refactor: make all NodeImpl/ChainImpl/ExternalSignerImpl members public (Jon Atack)
Pull request description:
- Make all `NodeImpl`, `ChainImpl` and `ExternalSignerImpl` class members `public` (and document why), to be consistent in all the `*Impl` classes in `src/node/interfaces.cpp` and `src/wallet/interfaces.cpp` and to help future reviewers and contributors.
- Remove unneeded temporaries in `NodeImpl` and `ChainImpl` methods in `src/node/interfaces.cpp` and simplify, to make the code easier to read and understand and to improve performance by avoiding unnecessary move operations.
ACKs for top commit:
ryanofsky:
Code review ACK 4bedfd702ad878645c51bea6ee8ce40d8c0bd3da. Changes since last review, applying suggested style & simplifiying first commit. Also avoiding another lock in second commit.
Tree-SHA512: 112f7cad5e2838c94c5b79d61328f42fe75fdb97f401ab49eccf696fc2c6a8a0c0ee55ec974c0602acf7423f78bb82e90eb8a0cc531e1d3347f73b7c83685504
Diffstat (limited to 'test/functional/test_framework/test_shell.py')
0 files changed, 0 insertions, 0 deletions