This was a long-standing bug in clang that is fixed in clang 6 and newer: https://bugs.llvm.org/show_bug.cgi?id=21689

Thanks for looking this up (the linked commit also addresses
-Wmissing-braces).

Using {0} is *the* standard way to tell the compiler to initialize all fields (recursively) to its neutral value. Back in the times I would have put a -2 on this.

No worries, nobody thought about merging this.

Alex, I guess this change can be abandoned.

View Change

To view, visit change 31973. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I3cbf6bb6bdc34b812c2321c6674051af991362b4
Gerrit-Change-Number: 31973
Gerrit-PatchSet: 1
Gerrit-Owner: Alex James <theracermaster@gmail.com>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: Stefan Tauner <stefan.tauner@gmx.at>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Comment-Date: Sat, 11 May 2019 15:28:39 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment