From: Elyes Haouas <ehaouas@noos.fr>
To: devel@edk2.groups.io
Cc: Elyes Haouas <ehaouas@noos.fr>
Subject: [edk2-devel] [PATCH 4/4] UefiPayloadPkg: Use C99 flexible arrays
Date: Thu, 24 Aug 2023 07:42:14 +0200 [thread overview]
Message-ID: <20230824054214.2839-4-ehaouas@noos.fr> (raw)
In-Reply-To: <20230824054214.2839-1-ehaouas@noos.fr>
One-element or zero-length arrays have been deprecated since
last millennium.
Use C99 flexible arrays instead, it allows the compiler to
generate errors when the flexible array does not occur at the
end in the structure.
Signed-off-by: Elyes Haouas <ehaouas@noos.fr>
---
UefiPayloadPkg/Include/Coreboot.h | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/UefiPayloadPkg/Include/Coreboot.h b/UefiPayloadPkg/Include/Coreboot.h
index 2d454f7c89..a53ede390c 100644
--- a/UefiPayloadPkg/Include/Coreboot.h
+++ b/UefiPayloadPkg/Include/Coreboot.h
@@ -59,7 +59,7 @@ struct cbmem_root {
UINT32 num_entries;
UINT32 locked;
UINT32 size;
- struct cbmem_entry entries[0];
+ struct cbmem_entry entries[];
};
struct imd_entry {
@@ -75,7 +75,7 @@ struct imd_root {
UINT32 flags;
UINT32 entry_align;
UINT32 max_offset;
- struct imd_entry entries[0];
+ struct imd_entry entries[];
};
struct cbuint64 {
@@ -119,7 +119,7 @@ struct cb_memory_range {
struct cb_memory {
UINT32 tag;
UINT32 size;
- struct cb_memory_range map[0];
+ struct cb_memory_range map[];
};
#define CB_TAG_MAINBOARD 0x0003
@@ -129,7 +129,7 @@ struct cb_mainboard {
UINT32 size;
UINT8 vendor_idx;
UINT8 part_number_idx;
- UINT8 strings[0];
+ UINT8 strings[];
};
#define CB_TAG_VERSION 0x0004
@@ -146,7 +146,7 @@ struct cb_mainboard {
struct cb_string {
UINT32 tag;
UINT32 size;
- UINT8 string[0];
+ UINT8 string[];
};
#define CB_TAG_SERIAL 0x000f
--
2.40.1
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#108006): https://edk2.groups.io/g/devel/message/108006
Mute This Topic: https://groups.io/mt/100935952/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
prev parent reply other threads:[~2023-08-24 14:15 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-24 5:42 [edk2-devel] [PATCH 1/4] EmbeddedPkg: Use C99 flexible arrays Elyes Haouas
2023-08-24 5:42 ` [edk2-devel] [PATCH 2/4] MdePkg/Include: " Elyes Haouas
2023-08-24 5:42 ` [edk2-devel] [PATCH 3/4] MdeModulePkg/Library/FrameBufferBltLib: " Elyes Haouas
2023-08-24 5:42 ` Elyes Haouas [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230824054214.2839-4-ehaouas@noos.fr \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox