bootstrap: Ratchet up compiler strictness
Replace the specific warning flags with -Wall -Wextra and turn on
-Werror.
Covered by -Wall:
* -Wformat
* -Warray-bounds
* -Wimplicit-function-declaration
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
Change-Id: Iab21e67026d2453648aa6ebe4c299e1162fc5ebf
diff --git a/tests/test_astlpc.c b/tests/test_astlpc.c
index c5f9e93..369702e 100644
--- a/tests/test_astlpc.c
+++ b/tests/test_astlpc.c
@@ -122,7 +122,10 @@
return 0;
}
-static void rx_message(uint8_t eid, void *data, void *msg, size_t len)
+#define __unused __attribute__((unused))
+
+static void rx_message(uint8_t eid __unused, void *data __unused, void *msg,
+ size_t len)
{
uint8_t type;
diff --git a/tests/test_cmds.c b/tests/test_cmds.c
index 7a05c35..1772734 100644
--- a/tests/test_cmds.c
+++ b/tests/test_cmds.c
@@ -30,8 +30,11 @@
};
};
-static void control_message_transport_callback(mctp_eid_t src, void *data,
- void *buf, size_t len)
+#define __unused __attribute__((unused))
+
+static void control_message_transport_callback(mctp_eid_t src __unused,
+ void *data, void *buf,
+ size_t len __unused)
{
struct callback_data *ctx = data;
struct mctp_ctrl_msg_hdr *msg_hdr = buf;
@@ -98,7 +101,7 @@
mctp_destroy(endpoint);
}
-int main(int argc, char *argv[])
+int main(void)
{
send_transport_control_message();
diff --git a/tests/test_serial.c b/tests/test_serial.c
index eef66d0..12d1476 100644
--- a/tests/test_serial.c
+++ b/tests/test_serial.c
@@ -33,7 +33,8 @@
ssize_t rc;
rc = write(ctx->egress, buf, len);
- assert(rc == len);
+ assert(rc >= 0);
+ assert((size_t)rc == len);
return rc;
}
@@ -42,7 +43,10 @@
static bool seen;
-static void rx_message(uint8_t eid, void *data, void *msg, size_t len)
+#define __unused __attribute__((unused))
+
+static void rx_message(uint8_t eid __unused, void *data __unused, void *msg,
+ size_t len)
{
uint8_t type;