michaelh/zephyr
1
0
Fork 0
Code Releases Activity

style: tests: comply with MISRA C:2012 Rule 15.6

Browse source 
Add missing braces to comply with MISRA C:2012 Rule 15.6 and
also following Zephyr's style guideline.

Signed-off-by: Pisit Sawangvonganan <pisit@ndrsolution.com>
This commit is contained in:
Pisit Sawangvonganan 2024-08-16 13:35:07 +07:00 • committed by Henrik Brix Andersen
commit b7d55d8ec6
8 changed files with 26 additions and 13 deletions
Download patch file Download diff file Expand all files Collapse all files

3
tests/bluetooth/controller/common/src/helper_util.c
View file

@ -389,8 +389,9 @@ uint16_t event_counter(struct ll_conn *conn)
* return the current event counter value (i.e. -1); * return the current event counter value (i.e. -1);
* otherwise return the next event counter value * otherwise return the next event counter value
*/ */
if (*evt_active) if (*evt_active) {
event_counter--; event_counter--;
}
return event_counter; return event_counter;
} }

3
tests/bsim/bluetooth/ll/edtt/gatt_test_app/src/gatt/service_b_3_1.c
View file

@ -127,8 +127,9 @@ void service_b_3_1_remove(void)
*/ */
void service_b_3_1_value_v6_notify(void) void service_b_3_1_value_v6_notify(void)
{ {
if (!value_v6_ntf_active) if (!value_v6_ntf_active) {
return; return;
}
bt_gatt_notify(NULL, &service_b_3_1_attrs[1], &value_v6_value, bt_gatt_notify(NULL, &service_b_3_1_attrs[1], &value_v6_value,
sizeof(value_v6_value)); sizeof(value_v6_value));

6
tests/bsim/bluetooth/ll/edtt/gatt_test_app/src/gatt/service_b_3_2.c
View file

@ -131,8 +131,9 @@ void service_b_3_2_remove(void)
*/ */
void service_b_3_2_value_v6_notify(void) void service_b_3_2_value_v6_notify(void)
{ {
if (!value_v6_ntf_active) if (!value_v6_ntf_active) {
return; return;
}
bt_gatt_notify(NULL, &service_b_3_2_attrs[1], &value_v6_value, bt_gatt_notify(NULL, &service_b_3_2_attrs[1], &value_v6_value,
sizeof(value_v6_value)); sizeof(value_v6_value));
@ -160,8 +161,9 @@ static void value_v6_indicate_cb(struct bt_conn *conn,
*/ */
void service_b_3_2_value_v6_indicate(void) void service_b_3_2_value_v6_indicate(void)
{ {
if (!value_v6_ind_active) if (!value_v6_ind_active) {
return; return;
}
/* /*
* NOTE: Zephyr doesn't automatically bump up the attribute pointer for * NOTE: Zephyr doesn't automatically bump up the attribute pointer for
* indications as it does for notifications. * indications as it does for notifications.

6
tests/bsim/bluetooth/ll/edtt/gatt_test_app/src/gatt/service_b_3_3.c
View file

@ -131,8 +131,9 @@ void service_b_3_3_remove(void)
*/ */
void service_b_3_3_value_v6_notify(void) void service_b_3_3_value_v6_notify(void)
{ {
if (!value_v6_ntf_active) if (!value_v6_ntf_active) {
return; return;
}
bt_gatt_notify(NULL, &service_b_3_3_attrs[1], &value_v6_value, bt_gatt_notify(NULL, &service_b_3_3_attrs[1], &value_v6_value,
sizeof(value_v6_value)); sizeof(value_v6_value));
@ -160,8 +161,9 @@ static void value_v6_indicate_cb(struct bt_conn *conn,
*/ */
void service_b_3_3_value_v6_indicate(void) void service_b_3_3_value_v6_indicate(void)
{ {
if (!value_v6_ind_active) if (!value_v6_ind_active) {
return; return;
}
/* /*
* NOTE: Zephyr doesn't automatically bump up the attribute pointer for * NOTE: Zephyr doesn't automatically bump up the attribute pointer for
* indications as it does for notifications. * indications as it does for notifications.

3
tests/drivers/build_all/sensor/src/generic_test.c
View file

@ -63,8 +63,9 @@ static void before(void *args)
/* Flush the SQ and CQ */ /* Flush the SQ and CQ */
rtio_sqe_drop_all(&sensor_read_rtio_ctx); rtio_sqe_drop_all(&sensor_read_rtio_ctx);
while (rtio_cqe_consume(&sensor_read_rtio_ctx)) while (rtio_cqe_consume(&sensor_read_rtio_ctx)) {
; ;
}
} }
/** /**

9
tests/net/lib/lwm2m/lwm2m_engine/src/main.c
View file

@ -32,12 +32,15 @@ static int lwm2m_get_res_buf_custom_fake(const struct lwm2m_obj_path *path, void
uint16_t *buffer_len, uint16_t *data_len, uint16_t *buffer_len, uint16_t *data_len,
uint8_t *data_flags) uint8_t *data_flags)
{ {
if (buffer_ptr) if (buffer_ptr) {
*buffer_ptr = my_buf; *buffer_ptr = my_buf;
if (buffer_len) }
if (buffer_len) {
*buffer_len = sizeof(my_buf); *buffer_len = sizeof(my_buf);
if (data_len) }
if (data_len) {
*data_len = my_data_len; *data_len = my_data_len;
}
return 0; return 0;
} }

3
tests/net/virtual/src/main.c
View file

@ -309,8 +309,9 @@ static void iface_cb(struct net_if *iface, void *user_data)
if ((iface != net_if_lookup_by_dev(DEVICE_GET(eth_test_dummy1))) && if ((iface != net_if_lookup_by_dev(DEVICE_GET(eth_test_dummy1))) &&
(iface != net_if_lookup_by_dev(DEVICE_GET(eth_test_dummy2))) && (iface != net_if_lookup_by_dev(DEVICE_GET(eth_test_dummy2))) &&
(iface != net_if_lookup_by_dev(DEVICE_GET(eth_test))) && (iface != net_if_lookup_by_dev(DEVICE_GET(eth_test))) &&
(net_if_l2(iface) != &NET_L2_GET_NAME(VIRTUAL))) (net_if_l2(iface) != &NET_L2_GET_NAME(VIRTUAL))) {
return; return;
}
DBG("Interface %p (%s) [%d]\n", iface, iface2str(iface), DBG("Interface %p (%s) [%d]\n", iface, iface2str(iface),
net_if_get_by_iface(iface)); net_if_get_by_iface(iface));

6
tests/posix/fs/src/test_fs_file.c
View file

@ -145,8 +145,9 @@ static int test_file_fsync(void)
{ {
int res = 0; int res = 0;
if (file < 0) if (file < 0) {
return res; return res;
}
res = fsync(file); res = fsync(file);
if (res < 0) { if (res < 0) {
@ -184,8 +185,9 @@ static int test_file_truncate(void)
int res = 0; int res = 0;
size_t truncate_size = sizeof(test_str) - 4; size_t truncate_size = sizeof(test_str) - 4;
if (file < 0) if (file < 0) {
return res; return res;
}
res = ftruncate(file, truncate_size); res = ftruncate(file, truncate_size);
if (res) { if (res) {