Page MenuHomePhabricator
Differential D816

Enable warning in the cmake/ninja build
ClosedPublic
Actions

Authored by deadalnix on Dec 16 2017, 16:18.

Details

Reviewers
schancel
Group Reviewers
Restricted Project
Commits
rSTAGING9edddc558bf7: Enable warning in the cmake/ninja build
rABC9edddc558bf7: Enable warning in the cmake/ninja build
Summary

As per title.

Test Plan
ninja

Checks that there are no warning - except when cross compiling.

Diff Detail

Repository
rABC Bitcoin ABC
Branch
cmakewarning
Lint
Lint Passed
SeverityLocationCodeMessage
Auto-Fixsrc/rpc/mining.cpp:1CFMTCode style violation
Unit
No Test Coverage
Build Status
Buildable 1392
Build 1392: arc lint + arc unit

Event Timeline

deadalnix created this revision.Dec 16 2017, 16:18
Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 16 2017, 16:18
schancel accepted this revision.Dec 17 2017, 07:04
This revision is now accepted and ready to land.Dec 17 2017, 07:04
Closed by commit rABC9edddc558bf7: Enable warning in the cmake/ninja build (authored by deadalnix). · Explain WhyDec 17 2017, 13:09
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
cmake/
modules/
35 lines
src/
21 lines
leveldb/
6 lines
rpc/
9 lines
secp256k1/
9 lines

Diff 2156

View Options

cmake/modules/AddCompilerFlags.cmake

Loading...
View Options

src/CMakeLists.txt

Loading...
View Options

src/leveldb/CMakeLists.txt

Loading...
View Options

src/rpc/mining.cpp

Loading...
View Options

src/secp256k1/CMakeLists.txt

Loading...