Modernized for loops (range based for loops were introduced in C++11). Range based for loops were used in this file earlier, not sure why they weren't used for the entire file.
consensus refactor: range based for loops #17814
pull sanjaykdragon wants to merge 2 commits into bitcoin:master from sanjaykdragon:master changing 1 files +24 −24-
sanjaykdragon commented at 4:45 PM on December 28, 2019: contributor
-
caf91fac90
consensus refactor: changed for loops to range based for loops (modern cpp).
Should not change behavior of consensus itself.
-
fixed some stuff, im an idiot 99d84fcd6a
-
fanquake commented at 6:40 PM on December 28, 2019: member
Please see the section on refactoring in CONTRIBUTING.md. In general it's rare for the project to refactor just for the sake of it. Particularly in consensus related code. If you're interested in contributing, checkout out the good first issue list.
- fanquake closed this on Dec 28, 2019
- DrahtBot locked this on Dec 16, 2021
Contributors