diff options
| author | Daniel Gröber <dxld@darkboxed.org> | 2020-11-30 13:30:10 +0100 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2020-12-02 10:38:25 +0000 |
| commit | 2aa15872d2468c6244d4c4a1dd0a325a2c9002e9 (patch) | |
| tree | 27217c89903fc9051b61a0c0ffc4564555c12630 | |
| parent | 7c0f007cf589ac01e35d555d0c8d98f4a72396d4 (diff) | |
edist-test: Fix _Static_assert missing message string
Older GCCs don't support _Static_assert without a message string as the
second argument. AFAICT _Static_assert with two arguments is in C11 but
omitting the message argument is an extension.
The tests appear to be built with the system gcc rather than our
crossgcc so that's probably why this was not cought by CI.
Change-Id: I41fd0ffc42ded8b6d145c3ec30cc7407a78b9a43
Signed-off-by: Daniel Gröber <dxld@darkboxed.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/48151
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
| -rw-r--r-- | tests/include/lib/edid-test.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/tests/include/lib/edid-test.h b/tests/include/lib/edid-test.h index 073905d5f0..8327748b66 100644 --- a/tests/include/lib/edid-test.h +++ b/tests/include/lib/edid-test.h @@ -43,7 +43,8 @@ struct edid_raw { uint8_t checksum; } __packed; -_Static_assert(sizeof(struct edid_raw) == 128); +_Static_assert(sizeof(struct edid_raw) == 128, + "assert failed: edid_raw size mismatch"); #define EDID_HEADER_RAW { 0x00, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0x00 } #define EDID_HEADER_INVALID_RAW { 0, 0, 0, 0, 0, 0, 0, 0 } |