Added and fixed comments in validation.h/cpp
ClosedPublic
Actions

Authored by jasonbcox on May 14 2018, 18:48.

Details

Reviewers

deadalnix
schancel

Group Reviewers

Restricted Project

Commits

rSTAGINGde65064cefbb: Added and fixed comments in validation.h/cpp
rABCde65064cefbb: Added and fixed comments in validation.h/cpp

Summary

These comments haven't been keep up-to-date. Improving these will make the code easier to navigate.

Test Plan

Reading

Diff Detail

Repository

rABC Bitcoin ABC

Branch

val

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 2582
Build 3279: Bitcoin ABC Buildbot (legacy)
Build 3278: arc lint + arc unit

Event Timeline

jasonbcox created this revision.May 14 2018, 18:48

Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 14 2018, 18:48

Build Bitcoin-ABC :: Staging :: Bitcoin-ABC Staging started.

Build Bitcoin-ABC :: Staging :: Bitcoin-ABC Staging passed.

Harbormaster completed remote builds in B2582: Diff 3886.May 14 2018, 19:24

likestamp

schancel accepted this revision.May 14 2018, 22:56

schancel added inline comments.

src/validation.cpp
3822	This smacks of the function doing something retarded.

This revision is now accepted and ready to land.May 14 2018, 22:56

Closed by commit rABCde65064cefbb: Added and fixed comments in validation.h/cpp (authored by jasonbcox). · Explain WhyMay 15 2018, 00:43

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

validation.h

16 lines

validation.cpp

13 lines

Diff 3886

View Options

src/validation.h