summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorÍñigo Huguet <ihuguet@redhat.com>2024-04-05 08:20:02 +0200
committerÍñigo Huguet <inigohuguet@hotmail.com>2024-04-08 06:35:20 +0000
commit56179465dfa375805af1f36e9012e9575516189f (patch)
tree624c49dd7a16f8f03cecafcffd31845b3f62e3cf
parentff101ada3ab65ce452f0ef235e486432fd9160fc (diff)
Updated code formatih/ci-distros
The CI will use Fedora 40 for code formatting check. Update the code formatting so it passes.
-rw-r--r--src/core/devices/nm-device-factory.c3
-rw-r--r--src/core/devices/nm-device.c2
-rw-r--r--src/core/devices/wifi/nm-wifi-ap.c15
-rw-r--r--src/core/dhcp/nm-dhcp-client.c7
-rw-r--r--src/core/ndisc/nm-ndisc.h2
-rw-r--r--src/core/nm-connectivity.h8
-rw-r--r--src/core/nm-core-utils.c3
-rw-r--r--src/core/nm-core-utils.h2
-rw-r--r--src/core/nm-manager.c7
-rw-r--r--src/core/nm-types.h2
-rw-r--r--src/libnm-base/nm-base.h4
-rw-r--r--src/libnm-client-impl/nm-conn-utils.c4
-rw-r--r--src/libnm-client-impl/tests/test-libnm.c8
-rw-r--r--src/libnm-client-impl/tests/test-nm-client.c2
-rw-r--r--src/libnm-core-impl/nm-setting-private.h2
-rw-r--r--src/libnm-core-impl/nm-team-utils.c48
-rw-r--r--src/libnm-core-impl/nm-utils.c3
-rw-r--r--src/libnm-crypto/nm-crypto.c10
-rw-r--r--src/libnm-glib-aux/nm-macros-internal.h6
-rw-r--r--src/libnm-platform/nm-platform.c17
-rw-r--r--src/libnm-std-aux/nm-linux-compat.h2
21 files changed, 82 insertions, 75 deletions
diff --git a/src/core/devices/nm-device-factory.c b/src/core/devices/nm-device-factory.c
index 69c2a38f13..22c8fa5aa5 100644
--- a/src/core/devices/nm-device-factory.c
+++ b/src/core/devices/nm-device-factory.c
@@ -183,7 +183,8 @@ nm_device_factory_class_init(NMDeviceFactoryClass *klass)
static GHashTable *factories_by_link = NULL;
static GHashTable *factories_by_setting = NULL;
-static void __attribute__((destructor)) _cleanup(void)
+static void __attribute__((destructor))
+_cleanup(void)
{
nm_clear_pointer(&factories_by_link, g_hash_table_unref);
nm_clear_pointer(&factories_by_setting, g_hash_table_unref);
diff --git a/src/core/devices/nm-device.c b/src/core/devices/nm-device.c
index b26b13e861..5b377529f8 100644
--- a/src/core/devices/nm-device.c
+++ b/src/core/devices/nm-device.c
@@ -15693,7 +15693,7 @@ nm_device_update_firewall_zone(NMDevice *self)
void
nm_device_update_metered(NMDevice *self)
{
-#define NM_METERED_INVALID ((NMMetered) -1)
+#define NM_METERED_INVALID ((NMMetered) - 1)
NMDevicePrivate *priv = NM_DEVICE_GET_PRIVATE(self);
NMSettingConnection *setting;
NMMetered conn_value, value = NM_METERED_INVALID;
diff --git a/src/core/devices/wifi/nm-wifi-ap.c b/src/core/devices/wifi/nm-wifi-ap.c
index d4d3815e7a..ceb954b797 100644
--- a/src/core/devices/wifi/nm-wifi-ap.c
+++ b/src/core/devices/wifi/nm-wifi-ap.c
@@ -719,13 +719,14 @@ get_property(GObject *object, guint prop_id, GValue *value, GParamSpec *pspec)
g_value_set_uchar(value, priv->strength);
break;
case PROP_LAST_SEEN:
- g_value_set_int(value,
- priv->last_seen_msec != G_MININT64 ? (int) NM_MAX(
- nm_utils_monotonic_timestamp_as_boottime(priv->last_seen_msec,
- NM_UTILS_NSEC_PER_MSEC)
- / 1000,
- 1)
- : -1);
+ g_value_set_int(
+ value,
+ priv->last_seen_msec != G_MININT64
+ ? (int) NM_MAX(nm_utils_monotonic_timestamp_as_boottime(priv->last_seen_msec,
+ NM_UTILS_NSEC_PER_MSEC)
+ / 1000,
+ 1)
+ : -1);
break;
default:
G_OBJECT_WARN_INVALID_PROPERTY_ID(object, prop_id, pspec);
diff --git a/src/core/dhcp/nm-dhcp-client.c b/src/core/dhcp/nm-dhcp-client.c
index 8770656b73..4ebc17543f 100644
--- a/src/core/dhcp/nm-dhcp-client.c
+++ b/src/core/dhcp/nm-dhcp-client.c
@@ -824,9 +824,10 @@ _nm_dhcp_client_notify(NMDhcpClient *self,
_acd_check_lease(self, &acd_state);
- options = priv->l3cd_next ? nm_dhcp_lease_get_options(
- nm_l3_config_data_get_dhcp_lease(priv->l3cd_next, priv->config.addr_family))
- : NULL;
+ options = priv->l3cd_next
+ ? nm_dhcp_lease_get_options(
+ nm_l3_config_data_get_dhcp_lease(priv->l3cd_next, priv->config.addr_family))
+ : NULL;
if (_LOGI_ENABLED()) {
const char *req_str =
diff --git a/src/core/ndisc/nm-ndisc.h b/src/core/ndisc/nm-ndisc.h
index 8f1a12a267..b8f8b06eed 100644
--- a/src/core/ndisc/nm-ndisc.h
+++ b/src/core/ndisc/nm-ndisc.h
@@ -40,7 +40,7 @@ typedef enum {
const char *nm_ndisc_dhcp_level_to_string(NMNDiscDHCPLevel level);
-#define NM_NDISC_INFINITY_U32 ((uint32_t) -1)
+#define NM_NDISC_INFINITY_U32 ((uint32_t) - 1)
/* It's important that this is G_MAXINT64, so that we can meaningfully do
* MIN(e1, e2) to find the minimum expiry time (and properly handle if any
diff --git a/src/core/nm-connectivity.h b/src/core/nm-connectivity.h
index d9ea95c269..e0353f9fd9 100644
--- a/src/core/nm-connectivity.h
+++ b/src/core/nm-connectivity.h
@@ -25,10 +25,10 @@ nm_connectivity_state_cmp(NMConnectivityState a, NMConnectivityState b)
/*****************************************************************************/
-#define NM_CONNECTIVITY_ERROR ((NMConnectivityState) -1)
-#define NM_CONNECTIVITY_FAKE ((NMConnectivityState) -2)
-#define NM_CONNECTIVITY_CANCELLED ((NMConnectivityState) -3)
-#define NM_CONNECTIVITY_DISPOSING ((NMConnectivityState) -4)
+#define NM_CONNECTIVITY_ERROR ((NMConnectivityState) - 1)
+#define NM_CONNECTIVITY_FAKE ((NMConnectivityState) - 2)
+#define NM_CONNECTIVITY_CANCELLED ((NMConnectivityState) - 3)
+#define NM_CONNECTIVITY_DISPOSING ((NMConnectivityState) - 4)
#define NM_TYPE_CONNECTIVITY (nm_connectivity_get_type())
#define NM_CONNECTIVITY(obj) \
diff --git a/src/core/nm-core-utils.c b/src/core/nm-core-utils.c
index 178ea3c4c6..dd1e9939d9 100644
--- a/src/core/nm-core-utils.c
+++ b/src/core/nm-core-utils.c
@@ -121,7 +121,8 @@ _nm_singleton_instance_weak_cb(gpointer data, GObject *where_the_object_was)
_singletons = g_slist_remove(_singletons, where_the_object_was);
}
-static void __attribute__((destructor)) _nm_singleton_instance_destroy(void)
+static void __attribute__((destructor))
+_nm_singleton_instance_destroy(void)
{
_singletons_shutdown = TRUE;
diff --git a/src/core/nm-core-utils.h b/src/core/nm-core-utils.h
index f015236de8..fdfed5f65d 100644
--- a/src/core/nm-core-utils.h
+++ b/src/core/nm-core-utils.h
@@ -306,7 +306,7 @@ typedef enum {
NM_UTILS_STABLE_TYPE_RANDOM = 3,
} NMUtilsStableType;
-#define NM_UTILS_STABLE_TYPE_NONE ((NMUtilsStableType) -1)
+#define NM_UTILS_STABLE_TYPE_NONE ((NMUtilsStableType) - 1)
NMUtilsStableType nm_utils_stable_id_parse(const char *stable_id,
const char *deviceid,
diff --git a/src/core/nm-manager.c b/src/core/nm-manager.c
index 198b7e14c4..b2a827e38b 100644
--- a/src/core/nm-manager.c
+++ b/src/core/nm-manager.c
@@ -9045,9 +9045,10 @@ get_property(GObject *object, guint prop_id, GValue *value, GParamSpec *pspec)
case PROP_CHECKPOINTS:
g_value_take_boxed(
value,
- priv->checkpoint_mgr ? nm_strv_make_deep_copied(
- nm_checkpoint_manager_get_checkpoint_paths(priv->checkpoint_mgr, NULL))
- : NULL);
+ priv->checkpoint_mgr
+ ? nm_strv_make_deep_copied(
+ nm_checkpoint_manager_get_checkpoint_paths(priv->checkpoint_mgr, NULL))
+ : NULL);
break;
default:
G_OBJECT_WARN_INVALID_PROPERTY_ID(object, prop_id, pspec);
diff --git a/src/core/nm-types.h b/src/core/nm-types.h
index 29b391fc88..6dfdc8e240 100644
--- a/src/core/nm-types.h
+++ b/src/core/nm-types.h
@@ -96,6 +96,6 @@ typedef struct _NMSecretAgent NMSecretAgent;
typedef struct _NMSettings NMSettings;
typedef struct _NMSettingsConnection NMSettingsConnection;
-#define NM_SETTING_CONNECTION_MDNS_UNKNOWN ((NMSettingConnectionMdns) -42)
+#define NM_SETTING_CONNECTION_MDNS_UNKNOWN ((NMSettingConnectionMdns) - 42)
#endif /* NM_TYPES_H */
diff --git a/src/libnm-base/nm-base.h b/src/libnm-base/nm-base.h
index e1cc27332e..4b1bff547c 100644
--- a/src/libnm-base/nm-base.h
+++ b/src/libnm-base/nm-base.h
@@ -147,8 +147,8 @@ typedef enum {
_NM_ETHTOOL_ID_NUM = (_NM_ETHTOOL_ID_LAST - _NM_ETHTOOL_ID_FIRST + 1),
} NMEthtoolID;
-#define _NM_ETHTOOL_ID_FEATURE_AS_IDX(ethtool_id) ((ethtool_id) -_NM_ETHTOOL_ID_FEATURE_FIRST)
-#define _NM_ETHTOOL_ID_COALESCE_AS_IDX(ethtool_id) ((ethtool_id) -_NM_ETHTOOL_ID_COALESCE_FIRST)
+#define _NM_ETHTOOL_ID_FEATURE_AS_IDX(ethtool_id) ((ethtool_id) - _NM_ETHTOOL_ID_FEATURE_FIRST)
+#define _NM_ETHTOOL_ID_COALESCE_AS_IDX(ethtool_id) ((ethtool_id) - _NM_ETHTOOL_ID_COALESCE_FIRST)
typedef enum {
NM_ETHTOOL_TYPE_UNKNOWN,
diff --git a/src/libnm-client-impl/nm-conn-utils.c b/src/libnm-client-impl/nm-conn-utils.c
index 08f8403c07..e548a96185 100644
--- a/src/libnm-client-impl/nm-conn-utils.c
+++ b/src/libnm-client-impl/nm-conn-utils.c
@@ -194,8 +194,8 @@ nm_conn_wireguard_import(const char *filename, GError **error)
* This code here instead generates a NetworkManager connection profile so that
* NetworkManager will apply a similar configuration when later activating the profile. */
-#define _TABLE_AUTO ((gint64) -1)
-#define _TABLE_OFF ((gint64) -2)
+#define _TABLE_AUTO ((gint64) - 1)
+#define _TABLE_OFF ((gint64) - 2)
data_table = _TABLE_AUTO;
diff --git a/src/libnm-client-impl/tests/test-libnm.c b/src/libnm-client-impl/tests/test-libnm.c
index 59774f966b..687e6c1717 100644
--- a/src/libnm-client-impl/tests/test-libnm.c
+++ b/src/libnm-client-impl/tests/test-libnm.c
@@ -2397,10 +2397,10 @@ _do_read_vpn_details_impl1(const char *file,
g_print(">>>> n_read=%zd; \"%s\"",
n_read,
- n_read > 0 ? (
- ss = nm_utils_buf_utf8safe_escape_cp(read_buf,
- n_read,
- NM_UTILS_STR_UTF8_SAFE_FLAG_ESCAPE_CTRL))
+ n_read > 0 ? (ss = nm_utils_buf_utf8safe_escape_cp(
+ read_buf,
+ n_read,
+ NM_UTILS_STR_UTF8_SAFE_FLAG_ESCAPE_CTRL))
: "");
}
diff --git a/src/libnm-client-impl/tests/test-nm-client.c b/src/libnm-client-impl/tests/test-nm-client.c
index 216e3e426c..3d52732456 100644
--- a/src/libnm-client-impl/tests/test-nm-client.c
+++ b/src/libnm-client-impl/tests/test-nm-client.c
@@ -1023,7 +1023,7 @@ _test_connection_invalid_find_connections(gpointer element, gpointer needle, gpo
G_STMT_START \
{ \
g_assert_cmpint(idx[i], >=, 0); \
- g_assert(path##i &&*path##i); \
+ g_assert(path##i && *path##i); \
g_assert(NM_IS_REMOTE_CONNECTION(connections->pdata[idx[i]])); \
g_assert_cmpstr(nm_connection_get_path(connections->pdata[idx[i]]), ==, path##i); \
} \
diff --git a/src/libnm-core-impl/nm-setting-private.h b/src/libnm-core-impl/nm-setting-private.h
index f784fc0eec..a1ae6825ed 100644
--- a/src/libnm-core-impl/nm-setting-private.h
+++ b/src/libnm-core-impl/nm-setting-private.h
@@ -755,7 +755,7 @@ _nm_properties_override(GArray *properties_override, const NMSettInfoProperty *p
!NM_FLAGS_ANY((param_flags), \
~(NM_SETTING_PARAM_FUZZY_IGNORE | NM_SETTING_PARAM_INFERRABLE))); \
G_STATIC_ASSERT((min_value) <= (default_value)); \
- G_STATIC_ASSERT((default_value) == 0 || (default_value) -1u < (max_value)); \
+ G_STATIC_ASSERT((default_value) == 0 || (default_value) - 1u < (max_value)); \
G_STATIC_ASSERT((max_value) <= G_MAXUINT64); \
\
_param_spec = g_param_spec_uint64("" prop_name "", \
diff --git a/src/libnm-core-impl/nm-team-utils.c b/src/libnm-core-impl/nm-team-utils.c
index 21562163d2..83452569b0 100644
--- a/src/libnm-core-impl/nm-team-utils.c
+++ b/src/libnm-core-impl/nm-team-utils.c
@@ -172,17 +172,17 @@ static const TeamAttrData team_attr_datas[] = {
#define _VAL_INT32_RANGE(_default, _min, _max) \
_VAL_INT32(_default), .has_range = TRUE, \
- .range.r_int32 = { \
- .min = _min, \
- .max = _max, \
+ .range.r_int32 = { \
+ .min = _min, \
+ .max = _max, \
}
#define _VAL_STRING() .default_val.v_string = NULL
-#define _VAL_STRING_RANGE(_valid_names) \
- _VAL_STRING(), .has_range = TRUE, \
- .range.r_string = { \
- .valid_names = (_valid_names), \
+#define _VAL_STRING_RANGE(_valid_names) \
+ _VAL_STRING(), .has_range = TRUE, \
+ .range.r_string = { \
+ .valid_names = (_valid_names), \
}
#define _VAL_UNSPEC() .default_val.v_string = (NULL)
@@ -1957,23 +1957,23 @@ _js_parse_locate_keys(const NMJsonVt *vt,
nm_assert(vt);
-#define _handle(_self, _cur_key, _cur_val, _keys, _level, _found_keys, _out_unrecognized_content) \
- ({ \
- const TeamAttrData *_attr_data; \
- gboolean _handled = FALSE; \
- \
- (_keys)[(_level) -1] = (_cur_key); \
- _attr_data = _attr_data_find_by_json_key((_self)->d.is_port, (_keys), (_level)); \
- if (_attr_data && _attr_data->js_keys_len == (_level)) { \
- if ((_found_keys)[_attr_data->team_attr]) \
- *(_out_unrecognized_content) = TRUE; \
- (_found_keys)[_attr_data->team_attr] = (_cur_val); \
- _handled = TRUE; \
- } else if (!_attr_data || !nm_json_is_object((_cur_val))) { \
- *(_out_unrecognized_content) = TRUE; \
- _handled = TRUE; \
- } \
- _handled; \
+#define _handle(_self, _cur_key, _cur_val, _keys, _level, _found_keys, _out_unrecognized_content) \
+ ({ \
+ const TeamAttrData *_attr_data; \
+ gboolean _handled = FALSE; \
+ \
+ (_keys)[(_level) - 1] = (_cur_key); \
+ _attr_data = _attr_data_find_by_json_key((_self)->d.is_port, (_keys), (_level)); \
+ if (_attr_data && _attr_data->js_keys_len == (_level)) { \
+ if ((_found_keys)[_attr_data->team_attr]) \
+ *(_out_unrecognized_content) = TRUE; \
+ (_found_keys)[_attr_data->team_attr] = (_cur_val); \
+ _handled = TRUE; \
+ } else if (!_attr_data || !nm_json_is_object((_cur_val))) { \
+ *(_out_unrecognized_content) = TRUE; \
+ _handled = TRUE; \
+ } \
+ _handled; \
})
nm_json_object_foreach (vt, root_js_obj, cur_key1, cur_val1) {
diff --git a/src/libnm-core-impl/nm-utils.c b/src/libnm-core-impl/nm-utils.c
index 761f74bdf1..b6e2414106 100644
--- a/src/libnm-core-impl/nm-utils.c
+++ b/src/libnm-core-impl/nm-utils.c
@@ -506,7 +506,8 @@ nmtst_system_encodings_get(void)
/*****************************************************************************/
-static void __attribute__((constructor)) _nm_utils_init(void)
+static void __attribute__((constructor))
+_nm_utils_init(void)
{
static int initialized = 0;
diff --git a/src/libnm-crypto/nm-crypto.c b/src/libnm-crypto/nm-crypto.c
index 7fcff4a9f7..883cf3c9cd 100644
--- a/src/libnm-crypto/nm-crypto.c
+++ b/src/libnm-crypto/nm-crypto.c
@@ -47,11 +47,11 @@
static const NMCryptoCipherInfo cipher_infos[] = {
#define _CI(_cipher, _name, _digest_len, _real_iv_len) \
- [(_cipher) -1] = {.cipher = _cipher, \
- .name = ""_name \
- "", \
- .digest_len = _digest_len, \
- .real_iv_len = _real_iv_len}
+ [(_cipher) - 1] = {.cipher = _cipher, \
+ .name = ""_name \
+ "", \
+ .digest_len = _digest_len, \
+ .real_iv_len = _real_iv_len}
_CI(NM_CRYPTO_CIPHER_DES_EDE3_CBC, "DES-EDE3-CBC", 24, 8),
_CI(NM_CRYPTO_CIPHER_DES_CBC, "DES-CBC", 8, 8),
_CI(NM_CRYPTO_CIPHER_AES_128_CBC, "AES-128-CBC", 16, 16),
diff --git a/src/libnm-glib-aux/nm-macros-internal.h b/src/libnm-glib-aux/nm-macros-internal.h
index 1c8c85e668..0b39271e1e 100644
--- a/src/libnm-glib-aux/nm-macros-internal.h
+++ b/src/libnm-glib-aux/nm-macros-internal.h
@@ -964,8 +964,8 @@ nm_g_variant_equal(GVariant *a, GVariant *b)
/* check if @flags has exactly one flag (@check) set. You should call this
* only with @check being a compile time constant and a power of two. */
-#define NM_FLAGS_HAS(flags, check) \
- (G_STATIC_ASSERT_EXPR((check) > 0 && ((check) & ((check) -1)) == 0), \
+#define NM_FLAGS_HAS(flags, check) \
+ (G_STATIC_ASSERT_EXPR((check) > 0 && ((check) & ((check) - 1)) == 0), \
NM_FLAGS_ANY((flags), (check)))
#define NM_FLAGS_ANY(flags, check) (((flags) & (check)) != 0)
@@ -1695,7 +1695,7 @@ nm_decode_version(guint version, guint *major, guint *minor, guint *micro)
/*****************************************************************************/
-#define NM_PID_T_INVAL ((pid_t) -1)
+#define NM_PID_T_INVAL ((pid_t) - 1)
/*****************************************************************************/
diff --git a/src/libnm-platform/nm-platform.c b/src/libnm-platform/nm-platform.c
index b89b035925..cd5a54bb87 100644
--- a/src/libnm-platform/nm-platform.c
+++ b/src/libnm-platform/nm-platform.c
@@ -6147,9 +6147,9 @@ nm_platform_link_to_string(const NMPlatformLink *link, char *buf, gsize len)
link->initialized ? " init" : " not-init",
link->inet6_addr_gen_mode_inv ? " addrgenmode " : "",
link->inet6_addr_gen_mode_inv ? nm_platform_link_inet6_addrgenmode2str(
- _nm_platform_uint8_inv(link->inet6_addr_gen_mode_inv),
- str_addrmode,
- sizeof(str_addrmode))
+ _nm_platform_uint8_inv(link->inet6_addr_gen_mode_inv),
+ str_addrmode,
+ sizeof(str_addrmode))
: "",
str_address[0] ? " addr " : "",
str_address[0] ? str_address : "",
@@ -7385,11 +7385,12 @@ nm_platform_ip6_route_to_string(const NMPlatformIP6Route *route, char *buf, gsiz
route->lock_mtu ? "lock " : "",
route->mtu)
: "",
- route->rt_pref ? nm_sprintf_buf(
- str_pref,
- " pref %s",
- nm_icmpv6_router_pref_to_string(route->rt_pref, str_pref2, sizeof(str_pref2)))
- : "");
+ route->rt_pref
+ ? nm_sprintf_buf(
+ str_pref,
+ " pref %s",
+ nm_icmpv6_router_pref_to_string(route->rt_pref, str_pref2, sizeof(str_pref2)))
+ : "");
return buf;
}
diff --git a/src/libnm-std-aux/nm-linux-compat.h b/src/libnm-std-aux/nm-linux-compat.h
index 2b04b0dfb6..9bfdb36562 100644
--- a/src/libnm-std-aux/nm-linux-compat.h
+++ b/src/libnm-std-aux/nm-linux-compat.h
@@ -16,7 +16,7 @@
#include <linux/const.h>
#ifndef __KERNEL_DIV_ROUND_UP
-#define __KERNEL_DIV_ROUND_UP(n, d) (((n) + (d) -1) / (d))
+#define __KERNEL_DIV_ROUND_UP(n, d) (((n) + (d) - 1) / (d))
#endif
#include "linux-headers/ethtool.h"