Remove extra newline that leads to linter warning
ClosedPublic
Actions

Authored by Fabien on Sep 5 2019, 09:03.

Details

Reviewers

deadalnix
jasonbcox

Group Reviewers

Restricted Project

Commits

rSTAGING267b632ca393: Remove extra newline that leads to linter warning
rABC267b632ca393: Remove extra newline that leads to linter warning

Summary

D3979 removed the freopen() function and leaded to an extra newline.
Most likely the linter did some other autofix that made this being
hidden at submission time.

Test Plan

Should return no error:

arc lint --everything

Diff Detail

Repository

rABC Bitcoin ABC

Branch

fix_arcanist_extra_newline

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 7324
Build 12691: Bitcoin ABC Buildbot (legacy)
Build 12690: arc lint + arc unit

Event Timeline

Fabien created this revision.Sep 5 2019, 09:03

Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 5 2019, 09:03

Herald added a reviewer: deadalnix. · View Herald Transcript

Build Bitcoin-ABC / Diffs / Diff Testing started.

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Harbormaster completed remote builds in B7324: Diff 11092.Sep 5 2019, 09:11

jasonbcox accepted this revision.Sep 6 2019, 00:37

This revision is now accepted and ready to land.Sep 6 2019, 00:37

Closed by commit rABC267b632ca393: Remove extra newline that leads to linter warning (authored by Fabien). · Explain WhySep 6 2019, 07:56

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC267b632ca393: Remove extra newline that leads to linter warning.

Fabien added a commit: rSTAGING267b632ca393: Remove extra newline that leads to linter warning.Sep 19 2019, 02:08

Revision Contents
Changeset List

Path

Size

src/

fs.cpp

1 line

Diff 11092

View Options

Remove extra newline that leads to linter warningClosedPublicActions