A giant pack of nits in validate.cpp
ClosedPublic
Actions

Authored by deadalnix on Jul 22 2017, 22:44.

Details

Reviewers

freetrader
sickpig
dagurval

Group Reviewers

Restricted Project

Commits

rSTAGINGb7d517811fb8: A giant pack of nits in validate.cpp
rABCb7d517811fb8: A giant pack of nits in validate.cpp

Summary

As per title. Mostly braces.

Test Plan

make check

Diff Detail

Repository

rABC Bitcoin ABC

Branch

valnits

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 572
Build 572: arc lint + arc unit

Event Timeline

deadalnix created this revision.Jul 22 2017, 22:44

Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 22 2017, 22:44

Harbormaster completed remote builds in B570: Diff 917.Jul 22 2017, 22:44

Rebase

Harbormaster completed remote builds in B572: Diff 920.Jul 23 2017, 12:58

CCulianu mentioned this in T73: Consider switching over to pre-increment rather than post-increment operator++ operator-- .Jul 23 2017, 15:01

deadalnix added a child revision: D371: Reject non replay protected transactions after the fork..Jul 24 2017, 00:53

Looks ok to me.

This revision is now accepted and ready to land.Jul 24 2017, 08:27

Closed by commit rABCb7d517811fb8: A giant pack of nits in validate.cpp (authored by deadalnix, committed by deadalnix). · Explain WhyJul 24 2017, 12:31

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

validation.h

9 lines

validation.cpp

305 lines

Diff 920

View Options

src/validation.h