Differential D12584

for * comments
ClosedPublic
Actions

Authored by Fabien on Nov 22 2022, 16:03.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABC8769f8facbbe: Remove all #include // for * comments

Summary

Most of them have been removed along the backport already.

Partial backport of core#23280:
https://github.com/bitcoin/bitcoin/pull/23280/commits/3b1584b794499158e0df07bd1bee1b803c568614

Depends on D12583.

Test Plan

Read the code.

Diff Detail

Repository

rABC Bitcoin ABC

Branch

PR23280_p21

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 21057
Build 41768: Build Diff	build-without-wallet · lint-circular-dependencies · build-diff · build-debug · build-clang-tidy · build-clang
Build 41767: arc lint + arc unit

Event Timeline

Fabien created this revision.Nov 22 2022, 16:03

Herald added a reviewer: Restricted Project. · View Herald TranscriptNov 22 2022, 16:03

Fabien requested review of this revision.Nov 22 2022, 16:03

teamcity edited the summary of this revision. (Show Details)Nov 22 2022, 16:03

Fabien added a child revision: D12585: style-only: Remove redundant scope in *Chainstate.Nov 22 2022, 16:03

Harbormaster completed remote builds in B21057: Diff 36510.Nov 22 2022, 16:50

PiRK accepted this revision.Nov 22 2022, 17:20

This revision is now accepted and ready to land.Nov 22 2022, 17:20

Closed by commit rABC8769f8facbbe: Remove all #include // for * comments (authored by Carl Dong <contact@carldong.me>, committed by Fabien). · Explain WhyNov 22 2022, 19:38

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC8769f8facbbe: Remove all #include // for * comments.

Revision Contents
Changeset List

Path

Size

src/

init.cpp

4 lines

Diff 36510

View Options

Remove all #include // for * commentsClosedPublicActions