Page MenuHomePhabricator

One more C++11 loop in validation.cpp
ClosedPublic

Authored by deadalnix on May 29 2018, 18:20.

Details

Summary

As per title.

Test Plan
make check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
validcpp11
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 2673
Build 3458: Bitcoin ABC Buildbot (legacy)
Build 3457: arc lint + arc unit

Event Timeline

jasonbcox added inline comments.
src/validation.cpp
5064

Cool. First time I've seen this function.

5273–5274

Trying to understand this part... Is this a line we'd like to add in, but won't for performance reasons?

This revision is now accepted and ready to land.May 31 2018, 01:35
This revision was automatically updated to reflect the committed changes.