MacroFake da23320998
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
2022-08-01 11:19:55 +02:00
..
2022-07-20 15:34:36 +02:00
2022-07-20 15:34:36 +02:00
2022-06-29 10:33:13 +02:00
2022-06-20 12:22:05 +01:00
2021-11-12 11:46:34 +01:00
2022-04-20 14:29:29 +01:00
2022-07-20 15:34:36 +02:00
2022-06-20 12:22:05 +01:00
2022-07-15 12:25:51 -04:00
2022-07-20 15:34:36 +02:00
2022-07-20 15:34:36 +02:00
2022-07-20 10:34:46 +01:00
2022-07-20 10:34:46 +01:00
2021-12-30 19:36:57 +02:00
2022-07-20 10:34:46 +01:00
2022-07-20 10:34:46 +01:00
2022-07-25 18:45:57 +02:00
2022-07-20 15:34:36 +02:00
2022-07-26 11:05:04 +02:00
2021-12-30 19:36:57 +02:00
2021-12-30 19:36:57 +02:00
2022-07-19 14:12:33 +02:00
2022-01-02 11:40:31 +01:00