Page MenuHomePhabricator
Differential D635

Comment, cast and assert cleanup in pow.cpp
ClosedPublic
Actions

Authored by freetrader on Nov 4 2017, 20:19.

Details

Reviewers
deadalnix
kyuupichan
Group Reviewers
Restricted Project
Commits
rABC2bc42bd0ca41: Comment, cast and assert cleanup in pow.cpp
rSTAGING2bc42bd0ca41: Comment, cast and assert cleanup in pow.cpp
Summary

Comments improved and a superfluous assert which is
covered by subsequent code is removed

Test Plan

make check
../qa/pull-tester/rpc-tests.py -extended

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

freetrader created this revision.Nov 4 2017, 20:19
Herald added 1 blocking reviewer(s): Restricted Project. · View Herald TranscriptNov 4 2017, 20:19
freetrader updated this revision to Diff 1641.Nov 4 2017, 20:22

Fix up the commenting that the lint auto-adjusted

freetrader updated this revision to Diff 1642.Nov 4 2017, 20:25

Formatting was still bad, fix

freetrader updated this revision to Diff 1643.Nov 4 2017, 20:26

Another grammar fix

freetrader updated this revision to Diff 1644.Nov 4 2017, 22:30

Explicit conversion to int64_t

freetrader retitled this revision from Comment and assert cleanup in pow.cpp to Comment, cast and assert cleanup in pow.cpp.Nov 4 2017, 22:53
freetrader edited the test plan for this revision. (Show Details)
kyuupichan accepted this revision.Nov 5 2017, 03:10

Looks good to me.

This revision is now accepted and ready to land.Nov 5 2017, 03:10
Closed by commit rABC2bc42bd0ca41: Comment, cast and assert cleanup in pow.cpp (authored by freetrader). · Explain WhyNov 5 2017, 10:14
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msrc/pow.cpp (27 lines)

Diff 1645

View Options

src/pow.cpp

Loading...