Page MenuHomePhabricator
Differential D422

Various nits in miner.cpp , mostly braces
ClosedPublic
Actions

Authored by deadalnix on Aug 5 2017, 11:41.

Details

Reviewers
freetrader
CCulianu
hanchon
Group Reviewers
Restricted Project
Commits
rSTAGINGbfe135d93259: Various nits in miner.cpp , mostly braces
rABCbfe135d93259: Various nits in miner.cpp , mostly braces
Summary

As per title.

Test Plan
make check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
minernits
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 656
Build 656: arc lint + arc unit

Event Timeline

deadalnix created this revision.Aug 5 2017, 11:41
Herald added a reviewer: Restricted Project. · View Herald TranscriptAug 5 2017, 11:41
Harbormaster completed remote builds in B656: Diff 1048.Aug 5 2017, 11:41
hanchon added a subscriber: hanchon.Aug 5 2017, 14:15

It looks good, make check is green.

hanchon accepted this revision.Aug 5 2017, 14:15
deadalnix retitled this revision from Variosu nits ine miner.cpp , mostly braces to Various nits in miner.cpp , mostly braces.Aug 5 2017, 17:04
CCulianu accepted this revision.Aug 5 2017, 20:56

Looks good, make check fine, accepted.

This revision is now accepted and ready to land.Aug 5 2017, 20:56
Closed by commit rABCbfe135d93259: Various nits in miner.cpp , mostly braces (authored by deadalnix, committed by deadalnix). · Explain WhyAug 6 2017, 02:01
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
27 lines

Diff 1048

View Options

src/miner.cpp

Loading...