Differential D18981

doc: Remove comments that just repeat what the code does
AcceptedPublic
Actions

Authored by PiRK on Fri, Nov 21, 16:49.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Summary

No need to artificially bloat the code and waste space.

This is a partial backport of core#28060
https://github.com/bitcoin/bitcoin/pull/28060/commits/fa8d227d58f7baa5a9be1b88930f4813bf6eedb1

Depends on D18980

Test Plan

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 35141
Build 69740: Build Diff	lint-circular-dependencies · build-without-wallet · build-diff · build-debug · build-clang-tidy · build-clang
Build 69739: arc lint + arc unit

Event Timeline

PiRK created this revision.Fri, Nov 21, 16:49

Herald added a reviewer: Restricted Project. · View Herald TranscriptFri, Nov 21, 16:49

PiRK requested review of this revision.Fri, Nov 21, 16:49

teamcity edited the summary of this revision. (Show Details)Fri, Nov 21, 16:49

PiRK edited the summary of this revision. (Show Details)Fri, Nov 21, 16:50

PiRK added a parent revision: D18980: refactor: Remove redundant file check from AutoFile shift operators.

Harbormaster completed remote builds in B35141: Diff 56720.Fri, Nov 21, 17:33

Fabien accepted this revision.Fri, Nov 21, 19:28

This revision is now accepted and ready to land.Fri, Nov 21, 19:28

Revision Contents
Changeset List

Path

Size

src/

hash.h

2 lines

streams.h

7 lines

Diff 56720

View Options

src/hash.h