Comments: More comments on functions/globals in standard.h.
ClosedPublic
Actions

Authored by deadalnix on May 25 2019, 15:00.

Details

Reviewers

Mengerian
markblundeberg

Group Reviewers

Restricted Project

Commits

rSTAGING1f2a6d15575b: Comments: More comments on functions/globals in standard.h.
rABC1f2a6d15575b: Comments: More comments on functions/globals in standard.h.

Summary

Backport of Core PR11058

Test Plan

make check

Diff Detail

Repository

rABC Bitcoin ABC

Branch

stddoc

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 5985
Build 10029: Bitcoin ABC Buildbot (legacy)
Build 10028: arc lint + arc unit

Event Timeline

deadalnix created this revision.May 25 2019, 15:00

Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 25 2019, 15:00

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing passed.

Harbormaster completed remote builds in B5985: Diff 8877.May 25 2019, 15:15

Looks good to me. I checked it against the core PR, and current Core code.

This revision is now accepted and ready to land.May 25 2019, 16:22

markblundeberg requested changes to this revision.May 25 2019, 16:57

markblundeberg added a subscriber: markblundeberg.

markblundeberg added inline comments.

src/script/standard.h
85	this doesn't match PR

This revision now requires changes to proceed.May 25 2019, 16:57

markblundeberg added inline comments.May 25 2019, 16:58

src/script/standard.h
75	intentional?
75	oh nvm

ah changed my mind, it's fine :)

This revision is now accepted and ready to land.May 25 2019, 17:00

Mengerian added inline comments.May 25 2019, 17:40

src/script/standard.h
85	Looks like it was missed in this previous backport: https://reviews.bitcoinabc.org/D540 https://github.com/bitcoin/bitcoin/pull/11117/files

markblundeberg added inline comments.May 25 2019, 18:06

src/script/standard.h
85	Yep, good find. :)

Fabien added a subscriber: Fabien.May 25 2019, 18:17

Fabien added inline comments.

src/script/standard.h
119	The last line can be removed

Closed by commit rABC1f2a6d15575b: Comments: More comments on functions/globals in standard.h. (authored by jimpo, committed by deadalnix). · Explain WhyMay 26 2019, 10:13

This revision was automatically updated to reflect the committed changes.

deadalnix added a commit: rABC1f2a6d15575b: Comments: More comments on functions/globals in standard.h..

deadalnix added a commit: rSTAGING1f2a6d15575b: Comments: More comments on functions/globals in standard.h..Sep 18 2019, 23:45

Revision Contents
Changeset List

Path

Size

src/

script/

standard.h

52 lines

standard.cpp

4 lines

Diff 8877

View Options

src/script/standard.h