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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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 ↗	(On Diff #3886)	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 3889

View Options

src/validation.h