Replace it with proper initializer use.
Details
Details
- Reviewers
majcosta - Group Reviewers
Restricted Project - Commits
- rABC89778bcdb086: Remove CBlockIndex::SetNull
ninja all check-all
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
src/chain.h | ||
---|---|---|
55 ↗ | (On Diff #20698) | although for these PODs it won't matter, consider using direct list initialization, since in-class assignment might make copies |
113 ↗ | (On Diff #20698) | no need to delegate to CBlockIndex() since it doesn't do anything. Rather you could also use an initializer list instead of assignment in the constructor body. |
src/chain.h | ||
---|---|---|
55 ↗ | (On Diff #20698) | TIL initializer lists aren't movable :/ Is it possible to statically assert that CBlockIndex IS movable? Otherwise regressing on this to match upstream appears imminent. |
Comment Actions
This is very similar to PR17162. Might be useful to add the reference to the revision description.