mirror of
https://github.com/bitcoin/bitcoin.git
synced 2026-05-24 04:44:42 +02:00
8caf60dbbemove-only: Group and re-order CAddrMan members by access type (Hennadii Stepanov)5cd7f8abe3refactor: Do not expose CAddrMan members as protected without need (Hennadii Stepanov) Pull request description: This PR is split from #19238 as all of its commits are trivial to review. The last commit is easy to review with `git diff --color-moved=dimmed-zebra`. Addressed the following comments from #19238: -130b82ff35 (r550865131)> Can you consolidate all the private members and protected members to be next to each other? Multiple private and protected access specifiers make this harder to read than is necessary. -130b82ff35 (r557271783)> Yeah, class declaration is easier to read if there is just one instance of `public:`, `protected:` and `private:` (in that order). ACKs for top commit: jnewbery: ACK8caf60dbbelaanwj: Code review ACK8caf60dbbejarolrod: ACK8caf60dbbevasild: ACK8caf60dbbeTree-SHA512: e6127fc658da7876e36f22e2fae162dc19502ed7f8e931fdebc827dabd627e5346c6fbe6f6d0cd27fd3e5c96690ff35022ff6b48f2747b748ebd66a45c851c2b