glozow
08b65df1bb
Merge bitcoin/bitcoin#26883: src/node/miner cleanups, follow-ups for #26695
...
6a5e88e5cf miner: don't re-apply default Options value if argument is unset (stickies-v)
ea72c3d9d5 refactor: avoid duplicating BlockAssembler::Options members (stickies-v)
cba749a9b7 refactor: rename local gArgs to args (stickies-v)
Pull request description:
Two follow-ups for #26695 , both refactoring and no observed (*) behaviour change:
- Rename `gArgs` to `args` because it's not actually a global
- Add `BlockAssembler::Options` as a (private) member to `BlockAssembler` to avoid having to assign all the options individually, essentially duplicating them
Reduces LoC and makes the code more readable, in my opinion.
---
(*) as [pointed out by ajtowns](https://github.com/bitcoin/bitcoin/pull/26883#discussion_r1068247937 ), this PR changes the interface of `ApplyArgsManOptions()`, making this not a pure refactoring PR. In practice, `ApplyArgsManOptions()` is never called in such a way that this leads to observed behaviour change. Regardless, I've carved out the potential behaviour change into a separate commit and would be okay with dropping it, should it turn out to be controversial.
ACKs for top commit:
glozow:
ACK 6a5e88e5cf
TheCharlatan:
Light code review ACK 6a5e88e5cf
Tree-SHA512: 15c30442ff0e070b1a58dc4c9615550d619ce35b4a2596b2c0a9d790259bbf987cab708f7cbb1057a8cf8b4c3226f3ad981282d3499ac442094806492a5f68ce
2023-02-20 11:32:43 +00:00
..
2023-02-17 12:47:52 -05:00
2023-01-26 10:44:05 +01:00
2023-01-04 12:00:25 +00:00
2023-02-06 13:56:51 -05:00
2022-08-13 13:55:38 +01:00
2023-02-15 14:58:47 +00:00
2023-02-10 04:39:11 -04:00
2022-12-24 23:49:50 +00:00
2023-02-07 11:02:01 +01:00
2022-12-24 23:49:50 +00:00
2023-02-10 04:39:11 -04:00
2022-09-29 16:43:30 +01:00
2023-01-31 18:48:50 +01:00
2022-10-23 15:03:04 +01:00
2023-02-20 11:32:43 +00:00
2023-01-31 11:50:10 +00:00
2022-12-24 23:49:50 +00:00
2023-02-17 13:34:40 -05:00
2023-02-17 13:34:40 -05:00
2023-02-16 10:01:33 +00:00
2022-12-12 23:40:36 -05:00
2023-01-31 11:50:10 +00:00
2023-02-17 17:28:14 -05:00
2023-01-31 11:50:10 +00:00
2023-02-13 15:31:50 +01:00
2023-02-17 17:28:14 -05:00
2023-01-31 11:50:10 +00:00
2023-02-01 11:33:35 +01:00
2023-01-31 16:08:44 -05:00
2023-01-26 18:11:13 -05:00
2023-02-17 13:34:40 -05:00
2023-02-01 10:18:08 -05:00
2022-12-24 23:49:50 +00:00
2023-02-01 11:33:35 +01:00
2022-12-24 23:49:50 +00:00
2022-12-24 23:49:50 +00:00
2022-12-24 23:49:50 +00:00
2022-12-24 23:49:50 +00:00
2022-12-24 23:49:50 +00:00
2023-02-10 04:39:11 -04:00
2023-02-02 13:14:48 -05:00
2023-01-13 09:40:57 +00:00
2023-01-26 10:44:05 +01:00
2023-01-13 09:40:57 +00:00
2022-12-24 23:49:50 +00:00
2023-01-26 10:44:05 +01:00
2023-02-15 14:42:28 -08:00
2023-01-31 19:34:35 +01:00
2022-12-24 23:49:50 +00:00
2022-12-24 23:49:50 +00:00
2023-01-05 09:05:14 +00:00
2022-12-24 23:49:50 +00:00
2022-09-09 10:07:39 +02:00
2022-12-24 23:49:50 +00:00
2022-12-24 23:49:50 +00:00
2022-12-24 23:49:50 +00:00
2023-02-02 09:00:15 -05:00
2023-02-02 09:00:15 -05:00
2023-02-03 18:15:42 +01:00
2023-01-26 10:44:05 +01:00
2023-02-03 18:15:42 +01:00
2023-01-31 11:50:10 +00:00
2023-02-10 04:39:11 -04:00
2023-02-10 04:39:11 -04:00
2022-12-24 23:49:50 +00:00
2022-12-24 23:49:50 +00:00
2023-02-01 11:33:35 +01:00
2023-02-01 11:33:35 +01:00
2022-12-24 23:49:50 +00:00
2023-02-16 12:33:26 +00:00
2023-01-18 15:47:06 +00:00
2022-12-24 23:49:50 +00:00
2023-01-30 15:57:12 +00:00
2022-08-30 12:09:04 -04:00
2022-08-29 08:10:35 -04:00
2023-02-17 13:34:40 -05:00
2023-02-17 13:34:40 -05:00
2022-12-24 23:49:50 +00:00
2023-02-17 13:34:40 -05:00
2022-12-24 23:49:50 +00:00
2023-02-17 16:54:55 -05:00
2022-12-24 23:49:50 +00:00
2023-01-31 19:34:35 +01:00
2023-01-31 11:50:10 +00:00
2022-12-24 23:49:50 +00:00
2022-12-24 23:49:50 +00:00
2023-02-10 04:39:11 -04:00
2022-11-15 19:23:39 -05:00
2022-07-21 10:42:09 +01:00
2022-09-13 17:15:17 +01:00
2022-09-13 17:15:17 +01:00
2022-11-17 14:52:45 +00:00
2023-02-17 17:28:14 -05:00
2023-02-15 14:58:47 +00:00
2023-02-17 13:34:40 -05:00
2023-01-28 15:24:13 +00:00
2022-12-24 23:49:50 +00:00
2022-12-24 23:49:50 +00:00
2023-02-17 13:34:40 -05:00
2022-12-24 23:49:50 +00:00
2022-12-24 23:49:50 +00:00
2023-02-17 14:21:06 -05:00
2023-01-26 18:11:13 -05:00
2023-02-17 13:34:40 -05:00
2023-02-17 13:34:40 -05:00
2023-02-17 13:34:40 -05:00
2022-12-24 23:49:50 +00:00
2022-12-24 23:49:50 +00:00
2022-12-24 23:49:50 +00:00
2022-12-24 23:49:50 +00:00
2022-12-24 23:49:50 +00:00
2022-12-24 23:49:50 +00:00
2022-12-24 23:49:50 +00:00
2022-12-24 23:49:50 +00:00
2022-12-24 23:49:50 +00:00
2022-12-24 23:49:50 +00:00
2023-02-16 10:01:33 +00:00
2023-02-16 09:39:50 +00:00
2023-01-13 09:40:57 +00:00
2023-01-13 09:40:57 +00:00
2023-02-17 15:01:50 +00:00
2023-02-15 14:58:47 +00:00
2023-01-23 16:36:27 +01:00
2023-01-26 10:44:05 +01:00
2023-01-12 13:42:44 +00:00
2022-12-24 23:49:50 +00:00
2022-12-24 23:49:50 +00:00
2023-01-31 11:50:10 +00:00
2022-12-24 23:49:50 +00:00
2023-01-31 11:50:10 +00:00
2023-02-02 10:47:37 +00:00
2023-01-25 09:33:26 +01:00
2022-12-24 23:49:50 +00:00
2022-12-24 23:49:50 +00:00
2023-01-31 11:50:10 +00:00
2023-02-17 13:34:40 -05:00
2023-01-31 11:50:10 +00:00
2023-02-10 04:39:11 -04:00
2023-02-10 04:39:11 -04:00
2023-02-01 11:33:35 +01:00
2023-02-01 11:33:35 +01:00
2023-01-26 10:36:18 +00:00
2023-01-26 10:36:18 +00:00
2022-12-10 14:34:44 -06:00
2023-02-06 13:56:51 -05:00
2023-02-10 04:39:11 -04:00
2023-02-10 04:39:11 -04:00
2023-02-01 11:33:35 +01:00
2023-01-05 09:05:14 +00:00
2023-01-03 19:31:29 -06:00
2022-12-24 23:49:50 +00:00
2022-12-24 23:49:50 +00:00
2022-12-24 23:49:50 +00:00