no more -Wmissing-braces warnings
ClosedPublic
Actions

Authored by majcosta on Dec 14 2018, 09:21.

Details

Reviewers

deadalnix
jasonbcox

Group Reviewers

Restricted Owners Package	(Owns No Changed Paths)
Restricted Project

Commits

rSTAGINGe63fa0268ce5: no more -Wmissing-braces warnings
rABCe63fa0268ce5: no more -Wmissing-braces warnings

Summary

self-explanatory

Test Plan

run make with -Wmissing-braces and see no warnings

Diff Detail

Repository

rABC Bitcoin ABC

Branch

fix-missing-braces

Lint

Lint Skipped

Unit

No Test Coverage

Build Status

Buildable 4311
Build 6687: Bitcoin ABC Buildbot (legacy)
Build 6686: arc lint + arc unit

Event Timeline

majcosta created this revision.Dec 14 2018, 09:21

Owners added a reviewer: Restricted Owners Package.Dec 14 2018, 09:21

Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 14 2018, 09:21

Herald added a reviewer: deadalnix. · View Herald Transcript

Herald added a subscriber: schancel. · View Herald Transcript

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

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

Harbormaster completed remote builds in B4311: Diff 6339.Dec 14 2018, 09:32

jasonbcox accepted this revision.Dec 14 2018, 11:24

This revision is now accepted and ready to land.Dec 14 2018, 11:24

Closed by commit rABCe63fa0268ce5: no more -Wmissing-braces warnings (authored by majcosta). · Explain WhyDec 14 2018, 11:48

This revision was automatically updated to reflect the committed changes.

majcosta mentioned this in T191: Fix missing-braces warnings..Dec 14 2018, 13:17

Revision Contents
Changeset List

Path

Size

src/

test/

sigencoding_tests.cpp

5 lines

Diff 6339

View Options

src/test/sigencoding_tests.cpp

no more -Wmissing-braces warningsClosedPublicActions