about summary refs log tree commit diff
path: root/src/core/settings/plugins/ifcfg-rh
diff options
context:
space:
mode:
Diffstat (limited to 'src/core/settings/plugins/ifcfg-rh')
-rw-r--r--src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-plugin.c208
-rw-r--r--src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-reader.c754
-rw-r--r--src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-reader.h10
-rw-r--r--src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-storage.c14
-rw-r--r--src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-storage.h12
-rw-r--r--src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-utils.c14
-rw-r--r--src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-utils.h8
-rw-r--r--src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-writer.c370
-rw-r--r--src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-writer.h12
-rw-r--r--src/core/settings/plugins/ifcfg-rh/shvar.c46
-rw-r--r--src/core/settings/plugins/ifcfg-rh/shvar.h4
-rw-r--r--src/core/settings/plugins/ifcfg-rh/tests/test-ifcfg-rh.c1668
12 files changed, 1560 insertions, 1560 deletions
diff --git a/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-plugin.c b/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-plugin.c
index 8fdc01fb..eb0d733d 100644
--- a/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-plugin.c
+++ b/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-plugin.c
@@ -43,7 +43,7 @@ typedef struct {
 
     struct {
         GDBusConnection *connection;
-        GCancellable *   cancellable;
+        GCancellable    *cancellable;
         gulong           signal_id;
         guint            regist_id;
     } dbus;
@@ -99,9 +99,9 @@ nm_assert_self(NMSIfcfgRHPlugin *self, gboolean unhandled_specs_consistent)
 
 #if NM_MORE_ASSERTS > 5
     {
-        NMSIfcfgRHPluginPrivate *priv = NMS_IFCFG_RH_PLUGIN_GET_PRIVATE(self);
-        NMSIfcfgRHStorage *      storage;
-        gsize                    n_uuid;
+        NMSIfcfgRHPluginPrivate       *priv = NMS_IFCFG_RH_PLUGIN_GET_PRIVATE(self);
+        NMSIfcfgRHStorage             *storage;
+        gsize                          n_uuid;
         gs_unref_hashtable GHashTable *h_unmanaged    = NULL;
         gs_unref_hashtable GHashTable *h_unrecognized = NULL;
 
@@ -177,11 +177,11 @@ nm_assert_self(NMSIfcfgRHPlugin *self, gboolean unhandled_specs_consistent)
 static NMSIfcfgRHStorage *
 _load_file(NMSIfcfgRHPlugin *self, const char *filename, GError **error)
 {
-    gs_unref_object NMConnection *connection = NULL;
-    gs_free_error GError *load_error         = NULL;
-    gs_free char *        unhandled_spec     = NULL;
-    gboolean              load_error_ignore;
-    struct stat           st;
+    gs_unref_object NMConnection *connection     = NULL;
+    gs_free_error GError         *load_error     = NULL;
+    gs_free char                 *unhandled_spec = NULL;
+    gboolean                      load_error_ignore;
+    struct stat                   st;
 
     if (stat(filename, &st) != 0) {
         int errsv = errno;
@@ -240,9 +240,9 @@ static void
 _load_dir(NMSIfcfgRHPlugin *self, NMSettUtilStorages *storages)
 {
     gs_unref_hashtable GHashTable *dupl_filenames = NULL;
-    gs_free_error GError *local                   = NULL;
-    const char *          f_filename;
-    GDir *                dir;
+    gs_free_error GError          *local          = NULL;
+    const char                    *f_filename;
+    GDir                          *dir;
 
     dir = g_dir_open(IFCFG_DIR, 0, &local);
     if (!dir) {
@@ -253,9 +253,9 @@ _load_dir(NMSIfcfgRHPlugin *self, NMSettUtilStorages *storages)
     dupl_filenames = g_hash_table_new_full(nm_str_hash, g_str_equal, NULL, g_free);
 
     while ((f_filename = g_dir_read_name(dir))) {
-        gs_free char *     full_path = NULL;
+        gs_free char      *full_path = NULL;
         NMSIfcfgRHStorage *storage;
-        char *             full_filename;
+        char              *full_filename;
 
         full_path     = g_build_filename(IFCFG_DIR, f_filename, NULL);
         full_filename = utils_detect_ifcfg_path(full_path, TRUE);
@@ -275,21 +275,21 @@ _load_dir(NMSIfcfgRHPlugin *self, NMSettUtilStorages *storages)
 }
 
 static void
-_storages_consolidate(NMSIfcfgRHPlugin *                     self,
-                      NMSettUtilStorages *                   storages_new,
+_storages_consolidate(NMSIfcfgRHPlugin                      *self,
+                      NMSettUtilStorages                    *storages_new,
                       gboolean                               replace_all,
-                      GHashTable *                           storages_replaced,
+                      GHashTable                            *storages_replaced,
                       NMSettingsPluginConnectionLoadCallback callback,
                       gpointer                               user_data)
 {
-    NMSIfcfgRHPluginPrivate *priv                  = NMS_IFCFG_RH_PLUGIN_GET_PRIVATE(self);
-    CList                    lst_conn_info_deleted = C_LIST_INIT(lst_conn_info_deleted);
-    gs_unref_ptrarray GPtrArray *storages_modified = NULL;
+    NMSIfcfgRHPluginPrivate     *priv                  = NMS_IFCFG_RH_PLUGIN_GET_PRIVATE(self);
+    CList                        lst_conn_info_deleted = C_LIST_INIT(lst_conn_info_deleted);
+    gs_unref_ptrarray GPtrArray *storages_modified     = NULL;
     CList                        storages_deleted;
-    NMSIfcfgRHStorage *          storage_safe;
-    NMSIfcfgRHStorage *          storage_new;
-    NMSIfcfgRHStorage *          storage_old;
-    NMSIfcfgRHStorage *          storage;
+    NMSIfcfgRHStorage           *storage_safe;
+    NMSIfcfgRHStorage           *storage_new;
+    NMSIfcfgRHStorage           *storage_old;
+    NMSIfcfgRHStorage           *storage;
     guint                        i;
 
     /* when we reload all files, we must signal add/update/modify of profiles one-by-one.
@@ -402,20 +402,20 @@ _storages_consolidate(NMSIfcfgRHPlugin *                     self,
 /*****************************************************************************/
 
 static void
-load_connections(NMSettingsPlugin *                     plugin,
-                 NMSettingsPluginConnectionLoadEntry *  entries,
+load_connections(NMSettingsPlugin                      *plugin,
+                 NMSettingsPluginConnectionLoadEntry   *entries,
                  gsize                                  n_entries,
                  NMSettingsPluginConnectionLoadCallback callback,
                  gpointer                               user_data)
 {
-    NMSIfcfgRHPlugin *       self = NMS_IFCFG_RH_PLUGIN(plugin);
+    NMSIfcfgRHPlugin        *self = NMS_IFCFG_RH_PLUGIN(plugin);
     NMSIfcfgRHPluginPrivate *priv = NMS_IFCFG_RH_PLUGIN_GET_PRIVATE(self);
     nm_auto_clear_sett_util_storages NMSettUtilStorages storages_new =
         NM_SETT_UTIL_STORAGES_INIT(storages_new, nms_ifcfg_rh_storage_destroy);
     gs_unref_hashtable GHashTable *dupl_filenames    = NULL;
     gs_unref_hashtable GHashTable *storages_replaced = NULL;
     gs_unref_hashtable GHashTable *loaded_uuids      = NULL;
-    const char *                   loaded_uuid;
+    const char                    *loaded_uuid;
     GHashTableIter                 h_iter;
     gsize                          i;
 
@@ -430,12 +430,12 @@ load_connections(NMSettingsPlugin *                     plugin,
 
     for (i = 0; i < n_entries; i++) {
         NMSettingsPluginConnectionLoadEntry *const entry = &entries[i];
-        gs_free_error GError *               local       = NULL;
-        const char *                         full_filename;
-        const char *                         uuid;
-        gs_free char *                       full_filename_keep = NULL;
-        NMSettingsPluginConnectionLoadEntry *dupl_content_entry;
-        gs_unref_object NMSIfcfgRHStorage *storage = NULL;
+        gs_free_error GError                      *local = NULL;
+        const char                                *full_filename;
+        const char                                *uuid;
+        gs_free char                              *full_filename_keep = NULL;
+        NMSettingsPluginConnectionLoadEntry       *dupl_content_entry;
+        gs_unref_object NMSIfcfgRHStorage         *storage = NULL;
 
         if (entry->handled)
             continue;
@@ -499,7 +499,7 @@ load_connections(NMSettingsPlugin *                     plugin,
     /* now we visit all UUIDs that are about to change... */
     g_hash_table_iter_init(&h_iter, loaded_uuids);
     while (g_hash_table_iter_next(&h_iter, (gpointer *) &loaded_uuid, NULL)) {
-        NMSIfcfgRHStorage *          storage;
+        NMSIfcfgRHStorage           *storage;
         NMSettUtilStorageByUuidHead *sbuh;
 
         sbuh = nm_sett_util_storages_lookup_by_uuid(&priv->storages, loaded_uuid);
@@ -509,9 +509,9 @@ load_connections(NMSettingsPlugin *                     plugin,
         c_list_for_each_entry (storage,
                                &sbuh->_storage_by_uuid_lst_head,
                                parent._storage_by_uuid_lst) {
-            const char *    full_filename = nms_ifcfg_rh_storage_get_filename(storage);
+            const char *full_filename = nms_ifcfg_rh_storage_get_filename(storage);
             gs_unref_object NMSIfcfgRHStorage *storage_new = NULL;
-            gs_free_error GError *local                    = NULL;
+            gs_free_error GError              *local       = NULL;
 
             if (g_hash_table_contains(dupl_filenames, full_filename)) {
                 /* already re-loaded. */
@@ -544,11 +544,11 @@ load_connections(NMSettingsPlugin *                     plugin,
 }
 
 static void
-reload_connections(NMSettingsPlugin *                     plugin,
+reload_connections(NMSettingsPlugin                      *plugin,
                    NMSettingsPluginConnectionLoadCallback callback,
                    gpointer                               user_data)
 {
-    NMSIfcfgRHPlugin *                                  self = NMS_IFCFG_RH_PLUGIN(plugin);
+    NMSIfcfgRHPlugin                                   *self = NMS_IFCFG_RH_PLUGIN(plugin);
     nm_auto_clear_sett_util_storages NMSettUtilStorages storages_new =
         NM_SETT_UTIL_STORAGES_INIT(storages_new, nms_ifcfg_rh_storage_destroy);
 
@@ -581,20 +581,20 @@ load_connections_done(NMSettingsPlugin *plugin)
 /*****************************************************************************/
 
 static gboolean
-add_connection(NMSettingsPlugin *  plugin,
-               NMConnection *      connection,
+add_connection(NMSettingsPlugin   *plugin,
+               NMConnection       *connection,
                NMSettingsStorage **out_storage,
-               NMConnection **     out_connection,
-               GError **           error)
+               NMConnection      **out_connection,
+               GError            **error)
 {
-    NMSIfcfgRHPlugin *       self               = NMS_IFCFG_RH_PLUGIN(plugin);
-    NMSIfcfgRHPluginPrivate *priv               = NMS_IFCFG_RH_PLUGIN_GET_PRIVATE(self);
-    gs_unref_object NMSIfcfgRHStorage *storage  = NULL;
-    gs_unref_object NMConnection *reread        = NULL;
-    gs_free char *                full_filename = NULL;
-    GError *                      local         = NULL;
-    gboolean                      reread_same;
-    struct timespec               mtime;
+    NMSIfcfgRHPlugin                  *self          = NMS_IFCFG_RH_PLUGIN(plugin);
+    NMSIfcfgRHPluginPrivate           *priv          = NMS_IFCFG_RH_PLUGIN_GET_PRIVATE(self);
+    gs_unref_object NMSIfcfgRHStorage *storage       = NULL;
+    gs_unref_object NMConnection      *reread        = NULL;
+    gs_free char                      *full_filename = NULL;
+    GError                            *local         = NULL;
+    gboolean                           reread_same;
+    struct timespec                    mtime;
 
     nm_assert_self(self, TRUE);
     nm_assert(NM_IS_CONNECTION(connection));
@@ -646,19 +646,19 @@ add_connection(NMSettingsPlugin *  plugin,
 }
 
 static gboolean
-update_connection(NMSettingsPlugin *  plugin,
-                  NMSettingsStorage * storage_x,
-                  NMConnection *      connection,
+update_connection(NMSettingsPlugin   *plugin,
+                  NMSettingsStorage  *storage_x,
+                  NMConnection       *connection,
                   NMSettingsStorage **out_storage,
-                  NMConnection **     out_connection,
-                  GError **           error)
+                  NMConnection      **out_connection,
+                  GError            **error)
 {
-    NMSIfcfgRHPlugin *       self    = NMS_IFCFG_RH_PLUGIN(plugin);
-    NMSIfcfgRHPluginPrivate *priv    = NMS_IFCFG_RH_PLUGIN_GET_PRIVATE(self);
-    NMSIfcfgRHStorage *      storage = NMS_IFCFG_RH_STORAGE(storage_x);
-    const char *             full_filename;
-    const char *             uuid;
-    GError *                 local       = NULL;
+    NMSIfcfgRHPlugin             *self    = NMS_IFCFG_RH_PLUGIN(plugin);
+    NMSIfcfgRHPluginPrivate      *priv    = NMS_IFCFG_RH_PLUGIN_GET_PRIVATE(self);
+    NMSIfcfgRHStorage            *storage = NMS_IFCFG_RH_STORAGE(storage_x);
+    const char                   *full_filename;
+    const char                   *uuid;
+    GError                       *local  = NULL;
     gs_unref_object NMConnection *reread = NULL;
     gboolean                      reread_same;
     struct timespec               mtime;
@@ -718,11 +718,11 @@ update_connection(NMSettingsPlugin *  plugin,
 static gboolean
 delete_connection(NMSettingsPlugin *plugin, NMSettingsStorage *storage_x, GError **error)
 {
-    NMSIfcfgRHPlugin *       self    = NMS_IFCFG_RH_PLUGIN(plugin);
+    NMSIfcfgRHPlugin        *self    = NMS_IFCFG_RH_PLUGIN(plugin);
     NMSIfcfgRHPluginPrivate *priv    = NMS_IFCFG_RH_PLUGIN_GET_PRIVATE(self);
-    NMSIfcfgRHStorage *      storage = NMS_IFCFG_RH_STORAGE(storage_x);
-    const char *             operation_message;
-    const char *             full_filename;
+    NMSIfcfgRHStorage       *storage = NMS_IFCFG_RH_STORAGE(storage_x);
+    const char              *operation_message;
+    const char              *full_filename;
 
     nm_assert_self(self, TRUE);
     nm_assert(!error || !*error);
@@ -736,9 +736,9 @@ delete_connection(NMSettingsPlugin *plugin, NMSettingsStorage *storage_x, GError
     nm_assert(storage == nm_sett_util_storages_lookup_by_filename(&priv->storages, full_filename));
 
     {
-        gs_free char *    keyfile     = utils_get_keys_path(full_filename);
-        gs_free char *    routefile   = utils_get_route_path(full_filename);
-        gs_free char *    route6file  = utils_get_route6_path(full_filename);
+        gs_free char     *keyfile     = utils_get_keys_path(full_filename);
+        gs_free char     *routefile   = utils_get_route_path(full_filename);
+        gs_free char     *route6file  = utils_get_route6_path(full_filename);
         const char *const files[]     = {full_filename, keyfile, routefile, route6file};
         gboolean          any_deleted = FALSE;
         gboolean          any_failure = FALSE;
@@ -784,10 +784,10 @@ delete_connection(NMSettingsPlugin *plugin, NMSettingsStorage *storage_x, GError
 static void
 _unhandled_specs_reset(NMSIfcfgRHPlugin *self)
 {
-    NMSIfcfgRHPluginPrivate *priv                     = NMS_IFCFG_RH_PLUGIN_GET_PRIVATE(self);
+    NMSIfcfgRHPluginPrivate       *priv               = NMS_IFCFG_RH_PLUGIN_GET_PRIVATE(self);
     gs_unref_hashtable GHashTable *unmanaged_specs    = NULL;
     gs_unref_hashtable GHashTable *unrecognized_specs = NULL;
-    NMSIfcfgRHStorage *            storage;
+    NMSIfcfgRHStorage             *storage;
 
     unmanaged_specs    = g_hash_table_new_full(nm_str_hash, g_str_equal, g_free, NULL);
     unrecognized_specs = g_hash_table_new_full(nm_str_hash, g_str_equal, g_free, NULL);
@@ -820,7 +820,7 @@ _unhandled_specs_merge_storages(NMSIfcfgRHPlugin *self, NMSettUtilStorages *stor
     NMSIfcfgRHPluginPrivate *priv                 = NMS_IFCFG_RH_PLUGIN_GET_PRIVATE(self);
     gboolean                 unmanaged_changed    = FALSE;
     gboolean                 unrecognized_changed = FALSE;
-    NMSIfcfgRHStorage *      storage;
+    NMSIfcfgRHStorage       *storage;
 
     c_list_for_each_entry (storage, &storages->_storage_lst_head, parent._storage_lst) {
         if (storage->unmanaged_spec
@@ -845,7 +845,7 @@ static GSList *
 _unhandled_specs_from_hashtable(GHashTable *hash)
 {
     gs_free const char **keys = NULL;
-    GSList *             list = NULL;
+    GSList              *list = NULL;
     guint                i, l;
 
     keys = nm_strdict_get_keys(hash, TRUE, &l);
@@ -873,15 +873,15 @@ get_unrecognized_specs(NMSettingsPlugin *plugin)
 /*****************************************************************************/
 
 static void
-impl_ifcfgrh_get_ifcfg_details(NMSIfcfgRHPlugin *     self,
+impl_ifcfgrh_get_ifcfg_details(NMSIfcfgRHPlugin      *self,
                                GDBusMethodInvocation *context,
-                               const char *           in_ifcfg)
+                               const char            *in_ifcfg)
 {
     NMSIfcfgRHPluginPrivate *priv       = NMS_IFCFG_RH_PLUGIN_GET_PRIVATE(self);
-    gs_free char *           ifcfg_path = NULL;
-    NMSIfcfgRHStorage *      storage;
-    const char *             uuid;
-    const char *             path;
+    gs_free char            *ifcfg_path = NULL;
+    NMSIfcfgRHStorage       *storage;
+    const char              *uuid;
+    const char              *path;
 
     if (in_ifcfg[0] != '/') {
         g_dbus_method_invocation_return_error(context,
@@ -964,7 +964,7 @@ _dbus_clear(NMSIfcfgRHPlugin *self)
 static void
 _dbus_connection_closed(GDBusConnection *connection,
                         gboolean         remote_peer_vanished,
-                        GError *         error,
+                        GError          *error,
                         gpointer         user_data)
 {
     _LOGW("dbus: %s bus closed", IFCFGRH1_BUS_NAME);
@@ -974,12 +974,12 @@ _dbus_connection_closed(GDBusConnection *connection,
 }
 
 static void
-_method_call(GDBusConnection *      connection,
-             const char *           sender,
-             const char *           object_path,
-             const char *           interface_name,
-             const char *           method_name,
-             GVariant *             parameters,
+_method_call(GDBusConnection       *connection,
+             const char            *sender,
+             const char            *object_path,
+             const char            *interface_name,
+             const char            *method_name,
+             GVariant              *parameters,
              GDBusMethodInvocation *invocation,
              gpointer               user_data)
 {
@@ -1014,11 +1014,11 @@ static GDBusInterfaceInfo *const interface_info = NM_DEFINE_GDBUS_INTERFACE_INFO
 static void
 _dbus_request_name_done(GObject *source_object, GAsyncResult *res, gpointer user_data)
 {
-    GDBusConnection *        connection = G_DBUS_CONNECTION(source_object);
-    NMSIfcfgRHPlugin *       self;
-    NMSIfcfgRHPluginPrivate *priv;
-    gs_free_error GError *error    = NULL;
-    gs_unref_variant GVariant *ret = NULL;
+    GDBusConnection           *connection = G_DBUS_CONNECTION(source_object);
+    NMSIfcfgRHPlugin          *self;
+    NMSIfcfgRHPluginPrivate   *priv;
+    gs_free_error GError      *error = NULL;
+    gs_unref_variant GVariant *ret   = NULL;
     guint32                    result;
 
     ret = g_dbus_connection_call_finish(connection, res, &error);
@@ -1072,10 +1072,10 @@ _dbus_request_name_done(GObject *source_object, GAsyncResult *res, gpointer user
 static void
 _dbus_create_done(GObject *source_object, GAsyncResult *res, gpointer user_data)
 {
-    NMSIfcfgRHPlugin *       self;
+    NMSIfcfgRHPlugin        *self;
     NMSIfcfgRHPluginPrivate *priv;
-    gs_free_error GError *error = NULL;
-    GDBusConnection *     connection;
+    gs_free_error GError    *error = NULL;
+    GDBusConnection         *connection;
 
     connection = g_dbus_connection_new_for_address_finish(res, &error);
     if (g_error_matches(error, G_IO_ERROR, G_IO_ERROR_CANCELLED))
@@ -1117,8 +1117,8 @@ static void
 _dbus_setup(NMSIfcfgRHPlugin *self)
 {
     NMSIfcfgRHPluginPrivate *priv    = NMS_IFCFG_RH_PLUGIN_GET_PRIVATE(self);
-    gs_free char *           address = NULL;
-    gs_free_error GError *error      = NULL;
+    gs_free char            *address = NULL;
+    gs_free_error GError    *error   = NULL;
 
     _dbus_clear(self);
 
@@ -1147,11 +1147,11 @@ _dbus_setup(NMSIfcfgRHPlugin *self)
 }
 
 static void
-config_changed_cb(NMConfig *          config,
-                  NMConfigData *      config_data,
+config_changed_cb(NMConfig           *config,
+                  NMConfigData       *config_data,
                   NMConfigChangeFlags changes,
-                  NMConfigData *      old_data,
-                  NMSIfcfgRHPlugin *  self)
+                  NMConfigData       *old_data,
+                  NMSIfcfgRHPlugin   *self)
 {
     NMSIfcfgRHPluginPrivate *priv;
 
@@ -1188,7 +1188,7 @@ nms_ifcfg_rh_plugin_init(NMSIfcfgRHPlugin *self)
 static void
 constructed(GObject *object)
 {
-    NMSIfcfgRHPlugin *       self = NMS_IFCFG_RH_PLUGIN(object);
+    NMSIfcfgRHPlugin        *self = NMS_IFCFG_RH_PLUGIN(object);
     NMSIfcfgRHPluginPrivate *priv = NMS_IFCFG_RH_PLUGIN_GET_PRIVATE(self);
 
     G_OBJECT_CLASS(nms_ifcfg_rh_plugin_parent_class)->constructed(object);
@@ -1204,7 +1204,7 @@ constructed(GObject *object)
 static void
 dispose(GObject *object)
 {
-    NMSIfcfgRHPlugin *       self = NMS_IFCFG_RH_PLUGIN(object);
+    NMSIfcfgRHPlugin        *self = NMS_IFCFG_RH_PLUGIN(object);
     NMSIfcfgRHPluginPrivate *priv = NMS_IFCFG_RH_PLUGIN_GET_PRIVATE(self);
 
     if (priv->config)
@@ -1227,7 +1227,7 @@ dispose(GObject *object)
 static void
 nms_ifcfg_rh_plugin_class_init(NMSIfcfgRHPluginClass *klass)
 {
-    GObjectClass *         object_class = G_OBJECT_CLASS(klass);
+    GObjectClass          *object_class = G_OBJECT_CLASS(klass);
     NMSettingsPluginClass *plugin_class = NM_SETTINGS_PLUGIN_CLASS(klass);
 
     object_class->constructed = constructed;
@@ -1247,7 +1247,7 @@ nms_ifcfg_rh_plugin_class_init(NMSIfcfgRHPluginClass *klass)
 /*****************************************************************************/
 
 G_MODULE_EXPORT NMSettingsPlugin *
-                nm_settings_plugin_factory(void)
+nm_settings_plugin_factory(void)
 {
     return g_object_new(NMS_TYPE_IFCFG_RH_PLUGIN, NULL);
 }
diff --git a/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-reader.c b/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-reader.c
index 6dd5ef05..e62c0227 100644
--- a/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-reader.c
+++ b/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-reader.c
@@ -75,9 +75,9 @@
 static char *
 get_full_file_path(const char *ifcfg_path, const char *file_path)
 {
-    const char *  base    = file_path;
+    const char   *base    = file_path;
     gs_free char *dirname = NULL;
-    char *        p;
+    char         *p;
 
     g_return_val_if_fail(ifcfg_path != NULL, NULL);
     g_return_val_if_fail(file_path != NULL, NULL);
@@ -99,8 +99,8 @@ static NMSettingSecretFlags
 _secret_read_ifcfg_flags(shvarFile *ifcfg, const char *flags_key)
 {
     NMSettingSecretFlags flags    = NM_SETTING_SECRET_FLAG_NONE;
-    gs_free char *       val_free = NULL;
-    const char *         val;
+    gs_free char        *val_free = NULL;
+    const char          *val;
 
     nm_assert(flags_key);
     nm_assert(g_str_has_suffix(flags_key, "_FLAGS"));
@@ -118,10 +118,10 @@ _secret_read_ifcfg_flags(shvarFile *ifcfg, const char *flags_key)
 }
 
 static void
-_secret_read_ifcfg(shvarFile *           ifcfg,
-                   shvarFile *           keys_ifcfg,
-                   const char *          name,
-                   char **               value,
+_secret_read_ifcfg(shvarFile            *ifcfg,
+                   shvarFile            *keys_ifcfg,
+                   const char           *name,
+                   char                **value,
                    NMSettingSecretFlags *flags)
 {
     char flags_key[250];
@@ -141,8 +141,8 @@ _secret_read_ifcfg(shvarFile *           ifcfg,
 
 static void
 _secret_set_from_ifcfg(gpointer    setting,
-                       shvarFile * ifcfg,
-                       shvarFile * keys_ifcfg,
+                       shvarFile  *ifcfg,
+                       shvarFile  *keys_ifcfg,
                        const char *ifcfg_key,
                        const char *property_name)
 {
@@ -165,8 +165,8 @@ _secret_set_from_ifcfg(gpointer    setting,
 static gboolean
 _secret_password_raw_to_bytes(const char *ifcfg_key,
                               const char *password_raw,
-                              GBytes **   out_bytes,
-                              GError **   error)
+                              GBytes    **out_bytes,
+                              GError    **error)
 {
     nm_auto_free_secret_buf NMSecretBuf *secret = NULL;
     gsize                                len;
@@ -222,17 +222,17 @@ _cert_get_cert_bytes(const char *ifcfg_path, const char *value, GError **error)
 }
 
 static gboolean
-_cert_get_cert(shvarFile *             ifcfg,
-               const char *            ifcfg_key,
-               GBytes **               out_cert,
+_cert_get_cert(shvarFile              *ifcfg,
+               const char             *ifcfg_key,
+               GBytes                **out_cert,
                NMSetting8021xCKScheme *out_scheme,
-               GError **               error)
+               GError                **error)
 {
     nm_auto_free_secret char *val_free = NULL;
-    const char *              val;
-    gs_unref_bytes GBytes *cert  = NULL;
-    GError *               local = NULL;
-    NMSetting8021xCKScheme scheme;
+    const char               *val;
+    gs_unref_bytes GBytes    *cert  = NULL;
+    GError                   *local = NULL;
+    NMSetting8021xCKScheme    scheme;
 
     val = svGetValueStr(ifcfg, ifcfg_key, &val_free);
     if (!val) {
@@ -266,11 +266,11 @@ err:
 
 static gboolean
 _cert_set_from_ifcfg(gpointer    setting,
-                     shvarFile * ifcfg,
+                     shvarFile  *ifcfg,
                      const char *ifcfg_key,
                      const char *property_name,
-                     GBytes **   out_cert,
-                     GError **   error)
+                     GBytes    **out_cert,
+                     GError    **error)
 {
     gs_unref_bytes GBytes *cert = NULL;
 
@@ -289,8 +289,8 @@ static void
 check_if_bond_slave(shvarFile *ifcfg, NMSettingConnection *s_con)
 {
     gs_free char *value = NULL;
-    const char *  v;
-    const char *  master;
+    const char   *v;
+    const char   *master;
 
     v = svGetValueStr(ifcfg, "MASTER_UUID", &value);
     if (!v)
@@ -322,8 +322,8 @@ static void
 check_if_team_slave(shvarFile *ifcfg, NMSettingConnection *s_con)
 {
     gs_free char *value = NULL;
-    const char *  v;
-    const char *  master;
+    const char   *v;
+    const char   *master;
 
     v = svGetValueStr(ifcfg, "TEAM_MASTER_UUID", &value);
     if (!v)
@@ -348,7 +348,7 @@ check_if_team_slave(shvarFile *ifcfg, NMSettingConnection *s_con)
 }
 
 static char *
-make_connection_name(shvarFile * ifcfg,
+make_connection_name(shvarFile  *ifcfg,
                      const char *ifcfg_name,
                      const char *suggested,
                      const char *prefix)
@@ -378,21 +378,21 @@ make_connection_name(shvarFile * ifcfg,
 
 static NMSetting *
 make_connection_setting(const char *file,
-                        shvarFile * ifcfg,
+                        shvarFile  *ifcfg,
                         const char *type,
                         const char *suggested,
                         const char *prefix)
 {
-    NMSettingConnection *   s_con;
+    NMSettingConnection    *s_con;
     NMSettingConnectionLldp lldp;
-    const char *            ifcfg_name = NULL;
-    gs_free char *          new_id     = NULL;
-    const char *            uuid;
-    gs_free char *          uuid_free = NULL;
-    gs_free char *          value     = NULL;
-    const char *            v;
-    gs_free char *          stable_id = NULL;
-    const char *const *     iter;
+    const char             *ifcfg_name = NULL;
+    gs_free char           *new_id     = NULL;
+    const char             *uuid;
+    gs_free char           *uuid_free = NULL;
+    gs_free char           *value     = NULL;
+    const char             *v;
+    gs_free char           *stable_id = NULL;
+    const char *const      *iter;
     int                     vint64, i_val;
 
     ifcfg_name = utils_get_ifcfg_name(file, TRUE);
@@ -597,7 +597,7 @@ make_connection_setting(const char *file,
         if (vint64 != -1)
             vint64 *= 1000;
         else if (v[0] != '\0') {
-            char * endptr;
+            char  *endptr;
             double d;
 
             d      = nm_g_ascii_strtod(v, &endptr);
@@ -647,14 +647,14 @@ make_connection_setting(const char *file,
 }
 
 static gboolean
-read_ip4_address(shvarFile * ifcfg,
+read_ip4_address(shvarFile  *ifcfg,
                  const char *tag,
-                 gboolean *  out_has_key,
-                 guint32 *   out_addr,
-                 GError **   error)
+                 gboolean   *out_has_key,
+                 guint32    *out_addr,
+                 GError    **error)
 {
     gs_free char *value_to_free = NULL;
-    const char *  value;
+    const char   *value;
     in_addr_t     a;
 
     nm_assert(ifcfg);
@@ -714,18 +714,18 @@ is_any_ip4_address_defined(shvarFile *ifcfg, int *idx)
 
 /* Returns TRUE on missing address or valid address */
 static gboolean
-read_full_ip4_address(shvarFile *   ifcfg,
+read_full_ip4_address(shvarFile    *ifcfg,
                       gint32        which,
-                      NMIPAddress * base_addr,
+                      NMIPAddress  *base_addr,
                       NMIPAddress **out_address,
-                      char **       out_gateway,
-                      GError **     error)
+                      char        **out_gateway,
+                      GError      **error)
 {
     char          tag[256];
     char          prefix_tag[256];
     guint32       ipaddr;
     gs_free char *value = NULL;
-    const char *  v;
+    const char   *v;
     int           prefix = 0;
     gboolean      has_key;
     guint32       a;
@@ -918,9 +918,9 @@ enum {
 static int
 parse_route_line(const char *line,
                  int         addr_family,
-                 NMIPRoute * options_route,
+                 NMIPRoute  *options_route,
                  NMIPRoute **out_route,
-                 GError **   error)
+                 GError    **error)
 {
     static const ParseLineInfo parse_infos[] = {
         [PARSE_LINE_ATTR_ROUTE_TYPE] =
@@ -1016,9 +1016,9 @@ parse_route_line(const char *line,
             },
     };
     nm_auto_unref_ip_route NMIPRoute *route      = NULL;
-    gs_free const char **             words_free = NULL;
-    const char *const *               words;
-    const char *                      s;
+    gs_free const char              **words_free = NULL;
+    const char *const                *words;
+    const char                       *s;
     gsize                             i_words;
     guint                             i;
     char                              buf1[256];
@@ -1054,7 +1054,7 @@ parse_route_line(const char *line,
         const gsize          i_words0 = i_words;
         const char *const    w        = words[i_words0];
         const ParseLineInfo *p_info;
-        ParseLineData *      p_data;
+        ParseLineData       *p_data;
         gboolean             unqualified_addr = FALSE;
 
         for (i = 0; i < G_N_ELEMENTS(parse_infos); i++) {
@@ -1343,7 +1343,7 @@ next:;
 
     for (i = 0; i < G_N_ELEMENTS(parse_infos); i++) {
         const ParseLineInfo *p_info = &parse_infos[i];
-        ParseLineData *      p_data = &parse_datas[i];
+        ParseLineData       *p_data = &parse_datas[i];
 
         if (!p_data->has)
             continue;
@@ -1414,7 +1414,7 @@ read_one_ip4_route(shvarFile *ifcfg, guint32 which, NMIPRoute **out_route, GErro
     guint32       next_hop;
     guint32       netmask;
     gboolean      has_key;
-    const char *  v;
+    const char   *v;
     gs_free char *value = NULL;
     gint64        prefix, metric;
     char          inet_buf[NM_UTILS_INET_ADDRSTRLEN];
@@ -1499,11 +1499,11 @@ read_one_ip4_route(shvarFile *ifcfg, guint32 which, NMIPRoute **out_route, GErro
 
 static gboolean
 read_route_file_parse(int                addr_family,
-                      const char *       filename,
-                      const char *       contents,
+                      const char        *filename,
+                      const char        *contents,
                       gsize              len,
                       NMSettingIPConfig *s_ip,
-                      GError **          error)
+                      GError           **error)
 {
     gsize line_num;
 
@@ -1517,10 +1517,10 @@ read_route_file_parse(int                addr_family,
     line_num = 0;
     while (TRUE) {
         nm_auto_unref_ip_route NMIPRoute *route = NULL;
-        gs_free_error GError *local             = NULL;
-        const char *          line              = contents;
-        char *                eol;
-        int                   e;
+        gs_free_error GError             *local = NULL;
+        const char                       *line  = contents;
+        char                             *eol;
+        int                               e;
 
         eol = strchr(contents, '\n');
         if (eol) {
@@ -1582,7 +1582,7 @@ static void
 parse_dns_options(NMSettingIPConfig *ip_config, const char *value)
 {
     gs_free const char **options = NULL;
-    const char *const *  item;
+    const char *const   *item;
 
     g_return_if_fail(ip_config);
 
@@ -1602,11 +1602,11 @@ parse_dns_options(NMSettingIPConfig *ip_config, const char *value)
 }
 
 static gboolean
-parse_full_ip6_address(shvarFile *   ifcfg,
-                       const char *  addr_str,
+parse_full_ip6_address(shvarFile    *ifcfg,
+                       const char   *addr_str,
                        int           i,
                        NMIPAddress **out_address,
-                       GError **     error)
+                       GError      **error)
 {
     NMIPAddress *addr;
     NMIPAddr     addr_bin;
@@ -1639,12 +1639,12 @@ parse_full_ip6_address(shvarFile *   ifcfg,
 static NMSetting *
 make_user_setting(shvarFile *ifcfg)
 {
-    gboolean        has_user_data         = FALSE;
-    gs_unref_object NMSettingUser *s_user = NULL;
-    gs_unref_hashtable GHashTable *keys   = NULL;
+    gboolean                       has_user_data = FALSE;
+    gs_unref_object NMSettingUser *s_user        = NULL;
+    gs_unref_hashtable GHashTable *keys          = NULL;
     GHashTableIter                 iter;
-    const char *                   key;
-    nm_auto_free_gstring GString *str = NULL;
+    const char                    *key;
+    nm_auto_free_gstring GString  *str = NULL;
 
     keys = svGetKeys(ifcfg, SV_KEY_TYPE_USER);
     if (!keys)
@@ -1652,7 +1652,7 @@ make_user_setting(shvarFile *ifcfg)
 
     g_hash_table_iter_init(&iter, keys);
     while (g_hash_table_iter_next(&iter, (gpointer *) &key, NULL)) {
-        const char *  value;
+        const char   *value;
         gs_free char *value_to_free = NULL;
 
         value = svGetValue(ifcfg, key, &value_to_free);
@@ -1679,7 +1679,7 @@ make_user_setting(shvarFile *ifcfg)
 }
 
 static void
-make_match_setting_prop(const char *     v,
+make_match_setting_prop(const char      *v,
                         NMSettingMatch **s_match,
                         void (*add_fcn)(NMSettingMatch *s_match, const char *value))
 {
@@ -1700,11 +1700,11 @@ static NMSetting *
 make_match_setting(shvarFile *ifcfg)
 {
     NMSettingMatch *s_match   = NULL;
-    gs_free char *  value_ifn = NULL;
-    gs_free char *  value_kcl = NULL;
-    gs_free char *  value_d   = NULL;
-    gs_free char *  value_p   = NULL;
-    const char *    v;
+    gs_free char   *value_ifn = NULL;
+    gs_free char   *value_kcl = NULL;
+    gs_free char   *value_d   = NULL;
+    gs_free char   *value_p   = NULL;
+    const char     *v;
 
     v = svGetValueStr(ifcfg, "MATCH_INTERFACE_NAME", &value_ifn);
     make_match_setting_prop(v, &s_match, nm_setting_match_add_interface_name);
@@ -1721,9 +1721,9 @@ make_match_setting(shvarFile *ifcfg)
 static NMSetting *
 make_proxy_setting(shvarFile *ifcfg)
 {
-    NMSettingProxy *     s_proxy = NULL;
-    gs_free char *       value   = NULL;
-    const char *         v;
+    NMSettingProxy      *s_proxy = NULL;
+    gs_free char        *value   = NULL;
+    const char          *v;
     NMSettingProxyMethod method;
 
     v = svGetValueStr(ifcfg, "PROXY_METHOD", &value);
@@ -1767,24 +1767,24 @@ static NMSetting *
 make_ip4_setting(shvarFile *ifcfg,
                  shvarFile *network_ifcfg,
                  gboolean   routes_read,
-                 gboolean * out_has_defroute,
-                 GError **  error)
+                 gboolean  *out_has_defroute,
+                 GError   **error)
 {
     gs_unref_object NMSettingIPConfig *s_ip4      = NULL;
-    gs_free char *                     route_path = NULL;
-    gs_free char *                     value      = NULL;
-    const char *                       v;
-    char *                             method;
-    gs_free char *                     dns_options_free = NULL;
-    const char *                       dns_options      = NULL;
-    gs_free char *                     gateway          = NULL;
+    gs_free char                      *route_path = NULL;
+    gs_free char                      *value      = NULL;
+    const char                        *v;
+    char                              *method;
+    gs_free char                      *dns_options_free = NULL;
+    const char                        *dns_options      = NULL;
+    gs_free char                      *gateway          = NULL;
     int                                i;
     guint32                            a;
     gboolean                           has_key;
     gboolean                           never_default;
     gint64                             i64;
     int                                priority;
-    const char *const *                item;
+    const char *const                 *item;
     guint32                            route_table;
 
     nm_assert(out_has_defroute && !*out_has_defroute);
@@ -1807,7 +1807,7 @@ make_ip4_setting(shvarFile *ifcfg,
     /* Then check if GATEWAYDEV; it's global and overrides DEFROUTE */
     if (network_ifcfg) {
         gs_free char *gatewaydev_value = NULL;
-        const char *  gatewaydev;
+        const char   *gatewaydev;
 
         /* Get the connection ifcfg device name and the global gateway device */
         v           = svGetValueStr(ifcfg, "DEVICE", &value);
@@ -2115,11 +2115,11 @@ make_ip4_setting(shvarFile *ifcfg,
 static void
 read_aliases(NMSettingIPConfig *s_ip4, gboolean read_defroute, const char *filename)
 {
-    GDir *        dir;
+    GDir         *dir;
     gs_free char *dirname   = NULL;
     gs_free char *base      = NULL;
-    NMIPAddress * base_addr = NULL;
-    GError *      err       = NULL;
+    NMIPAddress  *base_addr = NULL;
+    GError       *err       = NULL;
 
     g_return_if_fail(s_ip4 != NULL);
     g_return_if_fail(filename != NULL);
@@ -2134,17 +2134,17 @@ read_aliases(NMSettingIPConfig *s_ip4, gboolean read_defroute, const char *filen
 
     dir = g_dir_open(dirname, 0, &err);
     if (dir) {
-        const char * item;
+        const char  *item;
         NMIPAddress *addr;
         gboolean     ok;
 
         while ((item = g_dir_read_name(dir))) {
             nm_auto_shvar_file_close shvarFile *parsed       = NULL;
-            gs_free char *                      gateway      = NULL;
-            gs_free char *                      device_value = NULL;
-            gs_free char *                      full_path    = NULL;
-            const char *                        device;
-            const char *                        p;
+            gs_free char                       *gateway      = NULL;
+            gs_free char                       *device_value = NULL;
+            gs_free char                       *full_path    = NULL;
+            const char                         *device;
+            const char                         *p;
 
             if (!utils_is_ifcfg_alias_file(item, base))
                 continue;
@@ -2239,22 +2239,22 @@ static NMSetting *
 make_ip6_setting(shvarFile *ifcfg, shvarFile *network_ifcfg, gboolean routes_read, GError **error)
 {
     gs_unref_object NMSettingIPConfig *s_ip6 = NULL;
-    const char *                       v;
-    gs_free char *                     value = NULL;
+    const char                        *v;
+    gs_free char                      *value = NULL;
     gboolean                           ipv6init;
     gboolean                           ipv6forwarding;
     gboolean                           disabled;
     gboolean                           dhcp6  = FALSE;
-    char *                             method = NM_SETTING_IP6_CONFIG_METHOD_MANUAL;
-    const char *                       ipv6addr, *ipv6addr_secondaries;
-    gs_free char *                     ipv6addr_to_free             = NULL;
-    gs_free char *                     ipv6addr_secondaries_to_free = NULL;
-    gs_free const char **              list                         = NULL;
-    const char *const *                iter;
+    char                              *method = NM_SETTING_IP6_CONFIG_METHOD_MANUAL;
+    const char                        *ipv6addr, *ipv6addr_secondaries;
+    gs_free char                      *ipv6addr_to_free             = NULL;
+    gs_free char                      *ipv6addr_secondaries_to_free = NULL;
+    gs_free const char               **list                         = NULL;
+    const char *const                 *iter;
     guint32                            i;
     gint64                             i64;
     int                                i_val;
-    GError *                           local = NULL;
+    GError                            *local = NULL;
     int                                priority;
     gboolean                           never_default = FALSE;
     gboolean                           ip6_privacy   = FALSE, ip6_privacy_prefer_public_ip;
@@ -2276,10 +2276,10 @@ make_ip6_setting(shvarFile *ifcfg, shvarFile *network_ifcfg, gboolean routes_rea
      * When both are set, the device specified in IPV6_DEFAULTGW takes preference.
      */
     if (network_ifcfg) {
-        const char *  ipv6_defaultgw, *ipv6_defaultdev;
+        const char   *ipv6_defaultgw, *ipv6_defaultdev;
         gs_free char *ipv6_defaultgw_to_free  = NULL;
         gs_free char *ipv6_defaultdev_to_free = NULL;
-        const char *  default_dev             = NULL;
+        const char   *default_dev             = NULL;
 
         /* Get the connection ifcfg device name and the global default route device */
         nm_clear_g_free(&value);
@@ -2629,10 +2629,10 @@ static NMSetting *
 make_sriov_setting(shvarFile *ifcfg)
 {
     gs_unref_hashtable GHashTable *keys = NULL;
-    gs_unref_ptrarray GPtrArray *vfs    = NULL;
-    int                          autoprobe_drivers;
-    NMSettingSriov *             s_sriov;
-    gint64                       total_vfs;
+    gs_unref_ptrarray GPtrArray   *vfs  = NULL;
+    int                            autoprobe_drivers;
+    NMSettingSriov                *s_sriov;
+    gint64                         total_vfs;
 
     total_vfs = svGetValueInt64(ifcfg, "SRIOV_TOTAL_VFS", 10, 0, G_MAXUINT32, -1);
 
@@ -2646,14 +2646,14 @@ make_sriov_setting(shvarFile *ifcfg)
     keys = svGetKeys(ifcfg, SV_KEY_TYPE_SRIOV_VF);
     if (keys) {
         GHashTableIter iter;
-        const char *   key;
+        const char    *key;
 
         g_hash_table_iter_init(&iter, keys);
         while (g_hash_table_iter_next(&iter, (gpointer *) &key, NULL)) {
             gs_free_error GError *error         = NULL;
-            gs_free char *        value_to_free = NULL;
-            const char *          value;
-            NMSriovVF *           vf;
+            gs_free char         *value_to_free = NULL;
+            const char           *value;
+            NMSriovVF            *vf;
 
             nm_assert(g_str_has_prefix(key, "SRIOV_VF"));
 
@@ -2705,10 +2705,10 @@ make_tc_setting(shvarFile *ifcfg)
     s_tc = (NMSettingTCConfig *) nm_setting_tc_config_new();
 
     for (i = 1;; i++) {
-        NMTCQdisc *   qdisc         = NULL;
+        NMTCQdisc    *qdisc         = NULL;
         gs_free char *value_to_free = NULL;
-        const char *  value         = NULL;
-        GError *      local         = NULL;
+        const char   *value         = NULL;
+        GError       *local         = NULL;
 
         value = svGetValueStr(ifcfg, numbered_tag(tag, "QDISC", i), &value_to_free);
         if (!value)
@@ -2727,10 +2727,10 @@ make_tc_setting(shvarFile *ifcfg)
     }
 
     for (i = 1;; i++) {
-        NMTCTfilter * tfilter       = NULL;
-        gs_free char *value_to_free = NULL;
-        const char *  value         = NULL;
-        gs_free_error GError *local = NULL;
+        NMTCTfilter          *tfilter       = NULL;
+        gs_free char         *value_to_free = NULL;
+        const char           *value         = NULL;
+        gs_free_error GError *local         = NULL;
 
         value = svGetValueStr(ifcfg, numbered_tag(tag, "FILTER", i), &value_to_free);
         if (!value)
@@ -2812,16 +2812,16 @@ read_dcb_flags(shvarFile *ifcfg, DcbFlagsProperty *property)
 }
 
 static gboolean
-read_dcb_app(shvarFile *       ifcfg,
-             NMSettingDcb *    s_dcb,
-             const char *      app,
+read_dcb_app(shvarFile        *ifcfg,
+             NMSettingDcb     *s_dcb,
+             const char       *app,
              DcbFlagsProperty *flags_prop,
-             const char *      priority_prop,
-             GError **         error)
+             const char       *priority_prop,
+             GError          **error)
 {
     NMSettingDcbFlags flags = NM_SETTING_DCB_FLAG_NONE;
-    gs_free char *    value = NULL;
-    const char *      v;
+    gs_free char     *value = NULL;
+    const char       *v;
     gboolean          success  = TRUE;
     int               priority = -1;
     char              key[255];
@@ -2862,16 +2862,16 @@ read_dcb_app(shvarFile *       ifcfg,
 typedef void (*DcbSetBoolFunc)(NMSettingDcb *, guint, gboolean);
 
 static gboolean
-read_dcb_bool_array(shvarFile *       ifcfg,
-                    NMSettingDcb *    s_dcb,
+read_dcb_bool_array(shvarFile        *ifcfg,
+                    NMSettingDcb     *s_dcb,
                     NMSettingDcbFlags flags,
-                    const char *      prop,
-                    const char *      desc,
+                    const char       *prop,
+                    const char       *desc,
                     DcbSetBoolFunc    set_func,
-                    GError **         error)
+                    GError          **error)
 {
     gs_free char *value = NULL;
-    const char *  v;
+    const char   *v;
     guint         i;
 
     v = svGetValueStr(ifcfg, prop, &value);
@@ -2910,14 +2910,14 @@ read_dcb_bool_array(shvarFile *       ifcfg,
 typedef void (*DcbSetUintFunc)(NMSettingDcb *, guint, guint);
 
 static gboolean
-read_dcb_uint_array(shvarFile *       ifcfg,
-                    NMSettingDcb *    s_dcb,
+read_dcb_uint_array(shvarFile        *ifcfg,
+                    NMSettingDcb     *s_dcb,
                     NMSettingDcbFlags flags,
-                    const char *      prop,
-                    const char *      desc,
+                    const char       *prop,
+                    const char       *desc,
                     gboolean          f_allowed,
                     DcbSetUintFunc    set_func,
-                    GError **         error)
+                    GError          **error)
 {
     gs_free char *val = NULL;
     guint         i;
@@ -2963,18 +2963,18 @@ read_dcb_uint_array(shvarFile *       ifcfg,
 }
 
 static gboolean
-read_dcb_percent_array(shvarFile *       ifcfg,
-                       NMSettingDcb *    s_dcb,
+read_dcb_percent_array(shvarFile        *ifcfg,
+                       NMSettingDcb     *s_dcb,
                        NMSettingDcbFlags flags,
-                       const char *      prop,
-                       const char *      desc,
+                       const char       *prop,
+                       const char       *desc,
                        gboolean          sum_pct,
                        DcbSetUintFunc    set_func,
-                       GError **         error)
+                       GError          **error)
 {
-    gs_free char *       val   = NULL;
+    gs_free char        *val   = NULL;
     gs_free const char **split = NULL;
-    const char *const *  iter;
+    const char *const   *iter;
     guint                i, sum = 0;
 
     val = svGetValueStr_cp(ifcfg, prop);
@@ -3030,7 +3030,7 @@ make_dcb_setting(shvarFile *ifcfg, NMSetting **out_setting, GError **error)
     gs_unref_object NMSettingDcb *s_dcb = NULL;
     gboolean                      dcb_on;
     NMSettingDcbFlags             flags = NM_SETTING_DCB_FLAG_NONE;
-    gs_free char *                val   = NULL;
+    gs_free char                 *val   = NULL;
 
     g_return_val_if_fail(out_setting, FALSE);
     *out_setting = NULL;
@@ -3165,16 +3165,16 @@ make_dcb_setting(shvarFile *ifcfg, NMSetting **out_setting, GError **error)
 }
 
 static gboolean
-add_one_wep_key(shvarFile *                ifcfg,
-                const char *               shvar_key,
+add_one_wep_key(shvarFile                 *ifcfg,
+                const char                *shvar_key,
                 guint8                     key_idx,
                 gboolean                   passphrase,
                 NMSettingWirelessSecurity *s_wsec,
-                GError **                  error)
+                GError                   **error)
 {
     gs_free char *value_free = NULL;
-    const char *  value;
-    const char *  key = NULL;
+    const char   *value;
+    const char   *key = NULL;
 
     g_return_val_if_fail(ifcfg != NULL, FALSE);
     g_return_val_if_fail(shvar_key != NULL, FALSE);
@@ -3233,11 +3233,11 @@ add_one_wep_key(shvarFile *                ifcfg,
 }
 
 static gboolean
-read_wep_keys(shvarFile *                ifcfg,
+read_wep_keys(shvarFile                 *ifcfg,
               NMWepKeyType               key_type,
               guint8                     def_idx,
               NMSettingWirelessSecurity *s_wsec,
-              GError **                  error)
+              GError                   **error)
 {
     if (key_type != NM_WEP_KEY_TYPE_PASSPHRASE) {
         if (!add_one_wep_key(ifcfg, "KEY1", 0, FALSE, s_wsec, error))
@@ -3270,8 +3270,8 @@ static NMSetting *
 make_wep_setting(shvarFile *ifcfg, const char *file, GError **error)
 {
     gs_unref_object NMSettingWirelessSecurity *s_wsec          = NULL;
-    gs_free char *                             value           = NULL;
-    shvarFile *                                keys_ifcfg      = NULL;
+    gs_free char                              *value           = NULL;
+    shvarFile                                 *keys_ifcfg      = NULL;
     int                                        default_key_idx = 0;
     gboolean                                   has_default_key = FALSE;
     NMSettingSecretFlags                       key_flags;
@@ -3306,7 +3306,7 @@ make_wep_setting(shvarFile *ifcfg, const char *file, GError **error)
     /* Read keys in the ifcfg file if they are system-owned */
     if (key_flags == NM_SETTING_SECRET_FLAG_NONE) {
         NMWepKeyType  key_type;
-        const char *  v;
+        const char   *v;
         gs_free char *to_free = NULL;
 
         v = svGetValueStr(ifcfg, "KEY_TYPE", &to_free);
@@ -3393,10 +3393,10 @@ make_wep_setting(shvarFile *ifcfg, const char *file, GError **error)
 static gboolean
 fill_wpa_ciphers(shvarFile *ifcfg, NMSettingWirelessSecurity *wsec, gboolean group, gboolean adhoc)
 {
-    gs_free char *       value = NULL;
-    const char *         p;
+    gs_free char        *value = NULL;
+    const char          *p;
     gs_free const char **list = NULL;
-    const char *const *  iter;
+    const char *const   *iter;
     int                  i = 0;
 
     p = svGetValueStr(ifcfg, group ? "CIPHER_GROUP" : "CIPHER_PAIRWISE", &value);
@@ -3434,7 +3434,7 @@ fill_wpa_ciphers(shvarFile *ifcfg, NMSettingWirelessSecurity *wsec, gboolean gro
 static char *
 parse_wpa_psk(shvarFile *ifcfg, const char *file, GBytes *ssid, GError **error)
 {
-    shvarFile *   keys_ifcfg;
+    shvarFile    *keys_ifcfg;
     gs_free char *psk = NULL;
     size_t        plen;
 
@@ -3484,17 +3484,17 @@ parse_wpa_psk(shvarFile *ifcfg, const char *file, GBytes *ssid, GError **error)
 }
 
 static gboolean
-eap_simple_reader(const char *    eap_method,
-                  shvarFile *     ifcfg,
-                  shvarFile *     keys_ifcfg,
+eap_simple_reader(const char     *eap_method,
+                  shvarFile      *ifcfg,
+                  shvarFile      *keys_ifcfg,
                   NMSetting8021x *s_8021x,
                   gboolean        phase2,
-                  GError **       error)
+                  GError        **error)
 {
     NMSettingSecretFlags      flags;
-    gs_free char *            identity_free    = NULL;
-    nm_auto_free_secret char *password_raw_str = NULL;
-    gs_unref_bytes GBytes *password_raw_bytes  = NULL;
+    gs_free char             *identity_free      = NULL;
+    nm_auto_free_secret char *password_raw_str   = NULL;
+    gs_unref_bytes GBytes    *password_raw_bytes = NULL;
 
     g_object_set(s_8021x,
                  NM_SETTING_802_1X_IDENTITY,
@@ -3525,19 +3525,19 @@ eap_simple_reader(const char *    eap_method,
 }
 
 static gboolean
-eap_tls_reader(const char *    eap_method,
-               shvarFile *     ifcfg,
-               shvarFile *     keys_ifcfg,
+eap_tls_reader(const char     *eap_method,
+               shvarFile      *ifcfg,
+               shvarFile      *keys_ifcfg,
                NMSetting8021x *s_8021x,
                gboolean        phase2,
-               GError **       error)
+               GError        **error)
 {
     gs_unref_bytes GBytes *privkey       = NULL;
     gs_unref_bytes GBytes *client_cert   = NULL;
-    gs_free char *         identity_free = NULL;
-    gs_free char *         value_to_free = NULL;
-    const char *           client_cert_var;
-    const char *           client_cert_prop;
+    gs_free char          *identity_free = NULL;
+    gs_free char          *value_to_free = NULL;
+    const char            *client_cert_var;
+    const char            *client_cert_prop;
     NMSetting8021xCKFormat format;
 
     g_object_set(s_8021x,
@@ -3616,16 +3616,16 @@ eap_tls_reader(const char *    eap_method,
 }
 
 static gboolean
-parse_8021x_phase2_auth(shvarFile *     ifcfg,
-                        shvarFile *     keys_ifcfg,
+parse_8021x_phase2_auth(shvarFile      *ifcfg,
+                        shvarFile      *keys_ifcfg,
                         NMSetting8021x *s_8021x,
-                        GError **       error)
+                        GError        **error)
 {
-    gs_free char *       inner_auth = NULL;
-    gs_free char *       v_free     = NULL;
-    const char *         v;
+    gs_free char        *inner_auth = NULL;
+    gs_free char        *v_free     = NULL;
+    const char          *v;
     gs_free const char **list = NULL;
-    const char *const *  iter;
+    const char *const   *iter;
     guint                num_auth    = 0;
     guint                num_autheap = 0;
 
@@ -3699,15 +3699,15 @@ parse_8021x_phase2_auth(shvarFile *     ifcfg,
 }
 
 static gboolean
-eap_peap_reader(const char *    eap_method,
-                shvarFile *     ifcfg,
-                shvarFile *     keys_ifcfg,
+eap_peap_reader(const char     *eap_method,
+                shvarFile      *ifcfg,
+                shvarFile      *keys_ifcfg,
                 NMSetting8021x *s_8021x,
                 gboolean        phase2,
-                GError **       error)
+                GError        **error)
 {
     gs_free char *value = NULL;
-    const char *  v;
+    const char   *v;
 
     if (!_cert_set_from_ifcfg(s_8021x,
                               ifcfg,
@@ -3754,15 +3754,15 @@ eap_peap_reader(const char *    eap_method,
 }
 
 static gboolean
-eap_ttls_reader(const char *    eap_method,
-                shvarFile *     ifcfg,
-                shvarFile *     keys_ifcfg,
+eap_ttls_reader(const char     *eap_method,
+                shvarFile      *ifcfg,
+                shvarFile      *keys_ifcfg,
                 NMSetting8021x *s_8021x,
                 gboolean        phase2,
-                GError **       error)
+                GError        **error)
 {
     gs_free char *value = NULL;
-    const char *  v;
+    const char   *v;
 
     if (!_cert_set_from_ifcfg(s_8021x,
                               ifcfg,
@@ -3789,19 +3789,19 @@ eap_ttls_reader(const char *    eap_method,
 }
 
 static gboolean
-eap_fast_reader(const char *    eap_method,
-                shvarFile *     ifcfg,
-                shvarFile *     keys_ifcfg,
+eap_fast_reader(const char     *eap_method,
+                shvarFile      *ifcfg,
+                shvarFile      *keys_ifcfg,
                 NMSetting8021x *s_8021x,
                 gboolean        phase2,
-                GError **       error)
+                GError        **error)
 {
-    gs_free char *     anon_ident        = NULL;
-    gs_free char *     pac_file          = NULL;
-    gs_free char *     real_pac_path     = NULL;
-    gs_free char *     fast_provisioning = NULL;
+    gs_free char      *anon_ident        = NULL;
+    gs_free char      *pac_file          = NULL;
+    gs_free char      *real_pac_path     = NULL;
+    gs_free char      *fast_provisioning = NULL;
     const char *const *iter;
-    const char *       pac_prov_str;
+    const char        *pac_prov_str;
     gboolean           allow_unauth = FALSE, allow_auth = FALSE;
 
     pac_file = svGetValueStr_cp(ifcfg, "IEEE_8021X_PAC_FILE");
@@ -3852,12 +3852,12 @@ eap_fast_reader(const char *    eap_method,
 
 typedef struct {
     const char *method;
-    gboolean (*reader)(const char *    eap_method,
-                       shvarFile *     ifcfg,
-                       shvarFile *     keys_ifcfg,
+    gboolean (*reader)(const char     *eap_method,
+                       shvarFile      *ifcfg,
+                       shvarFile      *keys_ifcfg,
                        NMSetting8021x *s_8021x,
                        gboolean        phase2,
-                       GError **       error);
+                       GError        **error);
     gboolean wifi_phase2_only;
 } EAPReader;
 
@@ -3875,14 +3875,14 @@ static EAPReader eap_readers[] = {{"md5", eap_simple_reader, TRUE},
                                   {NULL, NULL}};
 
 static void
-read_8021x_list_value(shvarFile *     ifcfg,
-                      const char *    ifcfg_var_name,
+read_8021x_list_value(shvarFile      *ifcfg,
+                      const char     *ifcfg_var_name,
                       NMSetting8021x *setting,
-                      const char *    prop_name)
+                      const char     *prop_name)
 {
-    gs_free char *       value = NULL;
+    gs_free char        *value = NULL;
     gs_free const char **strv  = NULL;
-    const char *         v;
+    const char          *v;
 
     g_return_if_fail(ifcfg != NULL);
     g_return_if_fail(ifcfg_var_name != NULL);
@@ -3901,13 +3901,13 @@ static NMSetting8021x *
 fill_8021x(shvarFile *ifcfg, const char *file, const char *key_mgmt, gboolean wifi, GError **error)
 {
     nm_auto_shvar_file_close shvarFile *keys_ifcfg = NULL;
-    gs_unref_object NMSetting8021x *s_8021x        = NULL;
-    gs_free char *                  value          = NULL;
-    const char *                    v;
-    gs_free const char **           list = NULL;
-    const char *const *             iter;
-    gint64                          timeout;
-    int                             i_val;
+    gs_unref_object NMSetting8021x     *s_8021x    = NULL;
+    gs_free char                       *value      = NULL;
+    const char                         *v;
+    gs_free const char                **list = NULL;
+    const char *const                  *iter;
+    gint64                              timeout;
+    int                                 i_val;
 
     v = svGetValueStr(ifcfg, "IEEE_8021X_EAP_METHODS", &value);
     if (!v) {
@@ -3928,7 +3928,7 @@ fill_8021x(shvarFile *ifcfg, const char *file, const char *key_mgmt, gboolean wi
 
     /* Validate and handle each EAP method */
     for (iter = list; iter && *iter; iter++) {
-        EAPReader *   eap   = &eap_readers[0];
+        EAPReader    *eap   = &eap_readers[0];
         gboolean      found = FALSE;
         gs_free char *lower = NULL;
 
@@ -4040,16 +4040,16 @@ next:
 }
 
 static NMSetting *
-make_wpa_setting(shvarFile *      ifcfg,
-                 const char *     file,
-                 GBytes *         ssid,
+make_wpa_setting(shvarFile       *ifcfg,
+                 const char      *file,
+                 GBytes          *ssid,
                  gboolean         adhoc,
                  NMSetting8021x **s_8021x,
-                 GError **        error)
+                 GError         **error)
 {
     gs_unref_object NMSettingWirelessSecurity *wsec  = NULL;
-    gs_free char *                             value = NULL;
-    const char *                               v;
+    gs_free char                              *value = NULL;
+    const char                                *v;
     gboolean wpa_psk = FALSE, wpa_sae = FALSE, wpa_owe = FALSE, wpa_eap = FALSE, ieee8021x = FALSE,
              wpa3_eap = FALSE;
     int     i_val;
@@ -4089,7 +4089,7 @@ make_wpa_setting(shvarFile *      ifcfg,
         nm_setting_wireless_security_add_proto(wsec, "rsn");
     } else {
         gs_free char *value2 = NULL;
-        const char *  v2;
+        const char   *v2;
 
         v2 = svGetValueStr(ifcfg, "WPA_ALLOW_WPA", &value2);
         if (v2 && svParseBoolean(v2, TRUE))
@@ -4175,8 +4175,8 @@ static NMSetting *
 make_leap_setting(shvarFile *ifcfg, const char *file, GError **error)
 {
     gs_unref_object NMSettingWirelessSecurity *wsec = NULL;
-    shvarFile *                                keys_ifcfg;
-    gs_free char *                             value = NULL;
+    shvarFile                                 *keys_ifcfg;
+    gs_free char                              *value = NULL;
     NMSettingSecretFlags                       flags;
 
     wsec = NM_SETTING_WIRELESS_SECURITY(nm_setting_wireless_security_new());
@@ -4232,12 +4232,12 @@ make_leap_setting(shvarFile *ifcfg, const char *file, GError **error)
 }
 
 static NMSetting *
-make_wireless_security_setting(shvarFile *      ifcfg,
-                               const char *     file,
-                               GBytes *         ssid,
+make_wireless_security_setting(shvarFile       *ifcfg,
+                               const char      *file,
+                               GBytes          *ssid,
                                gboolean         adhoc,
                                NMSetting8021x **s_8021x,
-                               GError **        error)
+                               GError         **error)
 {
     NMSetting *wsec;
 
@@ -4291,9 +4291,9 @@ transform_hwaddr_blacklist(const char *blacklist)
 static NMSetting *
 make_wireless_setting(shvarFile *ifcfg, GError **error)
 {
-    NMSettingWireless *        s_wireless;
-    const char *               cvalue;
-    char *                     value = NULL;
+    NMSettingWireless         *s_wireless;
+    const char                *cvalue;
+    char                      *value = NULL;
     gint64                     chan  = 0;
     NMSettingMacRandomization  mac_randomization;
     NMSettingWirelessPowersave powersave = NM_SETTING_WIRELESS_POWERSAVE_DEFAULT;
@@ -4360,7 +4360,7 @@ make_wireless_setting(shvarFile *ifcfg, GError **error)
 
     value = svGetValueStr_cp(ifcfg, "MODE");
     if (value) {
-        char *      lcase;
+        char       *lcase;
         const char *mode = NULL;
 
         lcase = g_ascii_strdown(value, -1);
@@ -4540,16 +4540,16 @@ error:
 static NMConnection *
 wireless_connection_from_ifcfg(const char *file, shvarFile *ifcfg, GError **error)
 {
-    NMConnection *  connection       = NULL;
-    NMSetting *     con_setting      = NULL;
-    NMSetting *     wireless_setting = NULL;
+    NMConnection   *connection       = NULL;
+    NMSetting      *con_setting      = NULL;
+    NMSetting      *wireless_setting = NULL;
     NMSetting8021x *s_8021x          = NULL;
-    GBytes *        ssid;
-    NMSetting *     security_setting = NULL;
-    gs_free char *  ssid_utf8        = NULL;
-    const char *    mode;
+    GBytes         *ssid;
+    NMSetting      *security_setting = NULL;
+    gs_free char   *ssid_utf8        = NULL;
+    const char     *mode;
     gboolean        adhoc = FALSE;
-    GError *        local = NULL;
+    GError         *local = NULL;
 
     g_return_val_if_fail(file != NULL, NULL);
     g_return_val_if_fail(ifcfg != NULL, NULL);
@@ -4619,12 +4619,12 @@ typedef struct {
  * pointing to the next unprocessed option or NULL
  * in case of failure */
 static const char **
-_next_ethtool_options_nmternary(const char **         words,
+_next_ethtool_options_nmternary(const char          **words,
                                 NMEthtoolType         ethtool_type,
                                 NMEthtoolIfcfgOption *out_value)
 {
-    const char *         opt;
-    const char *         opt_val;
+    const char          *opt;
+    const char          *opt_val;
     const NMEthtoolData *d     = NULL;
     NMTernary            onoff = NM_TERNARY_DEFAULT;
 
@@ -4671,13 +4671,13 @@ _next_ethtool_options_nmternary(const char **         words,
  * pointing to the next unprocessed option or NULL
  * in case of failure */
 static const char **
-_next_ethtool_options_uint32(const char **         words,
+_next_ethtool_options_uint32(const char          **words,
                              NMEthtoolType         ethtool_type,
                              NMEthtoolIfcfgOption *out_value)
 {
     gint64               i64;
-    const char *         opt;
-    const char *         opt_val;
+    const char          *opt;
+    const char          *opt_val;
     const NMEthtoolData *d = NULL;
 
     nm_assert(out_value);
@@ -4727,16 +4727,16 @@ static NM_UTILS_STRING_TABLE_LOOKUP_DEFINE(
     {"-K", NM_ETHTOOL_TYPE_FEATURE}, );
 
 static void
-parse_ethtool_option(const char *             value,
+parse_ethtool_option(const char              *value,
                      NMSettingWiredWakeOnLan *out_flags,
-                     char **                  out_password,
-                     gboolean *               out_autoneg,
-                     guint32 *                out_speed,
-                     const char **            out_duplex,
-                     NMSettingEthtool **      out_s_ethtool)
+                     char                   **out_password,
+                     gboolean                *out_autoneg,
+                     guint32                 *out_speed,
+                     const char             **out_duplex,
+                     NMSettingEthtool       **out_s_ethtool)
 {
     guint                i;
-    const char **        w_iter;
+    const char         **w_iter;
     NMEthtoolIfcfgOption ifcfg_option;
     gs_free const char **words        = NULL;
     NMEthtoolType        ethtool_type = NM_ETHTOOL_TYPE_UNKNOWN;
@@ -4904,7 +4904,7 @@ static GPtrArray *
 read_routing_rules_parse(shvarFile *ifcfg, gboolean routes_read)
 {
     gs_unref_ptrarray GPtrArray *arr  = NULL;
-    gs_free const char **        keys = NULL;
+    gs_free const char         **keys = NULL;
     guint                        i, len;
 
     keys = svGetKeysSorted(ifcfg, SV_KEY_TYPE_ROUTING_RULE4 | SV_KEY_TYPE_ROUTING_RULE6, &len);
@@ -4919,12 +4919,12 @@ read_routing_rules_parse(shvarFile *ifcfg, gboolean routes_read)
 
     arr = g_ptr_array_new_full(len, (GDestroyNotify) nm_ip_routing_rule_unref);
     for (i = 0; i < len; i++) {
-        const char *                  key                   = keys[i];
-        nm_auto_unref_ip_routing_rule NMIPRoutingRule *rule = NULL;
-        gs_free_error GError *local                         = NULL;
-        gs_free char *        value_to_free                 = NULL;
-        const char *          value;
-        gboolean              key_is_ipv4;
+        const char                                    *key           = keys[i];
+        nm_auto_unref_ip_routing_rule NMIPRoutingRule *rule          = NULL;
+        gs_free_error GError                          *local         = NULL;
+        gs_free char                                  *value_to_free = NULL;
+        const char                                    *value;
+        gboolean                                       key_is_ipv4;
 
         key_is_ipv4 = (key[NM_STRLEN("ROUTING_RULE")] == '_');
         nm_assert(key_is_ipv4 == NM_STR_HAS_PREFIX(key, "ROUTING_RULE_"));
@@ -4956,7 +4956,7 @@ read_routing_rules_parse(shvarFile *ifcfg, gboolean routes_read)
 }
 
 static void
-read_routing_rules(shvarFile *        ifcfg,
+read_routing_rules(shvarFile         *ifcfg,
                    gboolean           routes_read,
                    NMSettingIPConfig *s_ip4,
                    NMSettingIPConfig *s_ip6)
@@ -4980,17 +4980,17 @@ read_routing_rules(shvarFile *        ifcfg,
 static void
 parse_ethtool_options(shvarFile *ifcfg, NMConnection *connection)
 {
-    NMSettingWired *s_wired;
+    NMSettingWired                   *s_wired;
     gs_unref_object NMSettingEthtool *s_ethtool         = NULL;
     NMSettingWiredWakeOnLan           wol_flags         = NM_SETTING_WIRED_WAKE_ON_LAN_DEFAULT;
-    gs_free char *                    ethtool_opts_free = NULL;
-    const char *                      ethtool_opts;
-    gs_free char *                    wol_password   = NULL;
-    gs_free char *                    wol_value_free = NULL;
-    const char *                      tmp;
+    gs_free char                     *ethtool_opts_free = NULL;
+    const char                       *ethtool_opts;
+    gs_free char                     *wol_password   = NULL;
+    gs_free char                     *wol_value_free = NULL;
+    const char                       *tmp;
     gboolean                          autoneg     = FALSE;
     guint32                           speed       = 0;
-    const char *                      duplex      = NULL;
+    const char                       *duplex      = NULL;
     gboolean                          wired_found = FALSE;
 
     ethtool_opts = svGetValue(ifcfg, "ETHTOOL_OPTS", &ethtool_opts_free);
@@ -5001,7 +5001,7 @@ parse_ethtool_options(shvarFile *ifcfg, NMConnection *connection)
             wol_flags = NM_SETTING_WIRED_WAKE_ON_LAN_IGNORE;
         else {
             gs_free const char **opts = NULL;
-            const char *const *  iter;
+            const char *const   *iter;
 
             opts = nm_strsplit_set(ethtool_opts, ";");
             for (iter = opts; iter && iter[0]; iter++) {
@@ -5061,8 +5061,8 @@ static NMSetting *
 make_wired_setting(shvarFile *ifcfg, const char *file, NMSetting8021x **s_8021x, GError **error)
 {
     gs_unref_object NMSettingWired *s_wired = NULL;
-    const char *                    cvalue;
-    gs_free char *                  value = NULL;
+    const char                     *cvalue;
+    gs_free char                   *value = NULL;
     gboolean                        found = FALSE;
     NMTernary                       accept_all_mac_addresses;
 
@@ -5246,11 +5246,11 @@ make_wired_setting(shvarFile *ifcfg, const char *file, NMSetting8021x **s_8021x,
 static NMConnection *
 wired_connection_from_ifcfg(const char *file, shvarFile *ifcfg, GError **error)
 {
-    NMConnection *  connection    = NULL;
-    NMSetting *     con_setting   = NULL;
-    NMSetting *     wired_setting = NULL;
+    NMConnection   *connection    = NULL;
+    NMSetting      *con_setting   = NULL;
+    NMSetting      *wired_setting = NULL;
     NMSetting8021x *s_8021x       = NULL;
-    GError *        local         = NULL;
+    GError         *local         = NULL;
 
     g_return_val_if_fail(file != NULL, NULL);
     g_return_val_if_fail(ifcfg != NULL, NULL);
@@ -5288,8 +5288,8 @@ wired_connection_from_ifcfg(const char *file, shvarFile *ifcfg, GError **error)
 static gboolean
 parse_infiniband_p_key(shvarFile *ifcfg, int *out_p_key, char **out_parent, GError **error)
 {
-    char *   device = NULL, *physdev = NULL, *pkey_id = NULL;
-    char *   ifname = NULL;
+    char    *device = NULL, *physdev = NULL, *pkey_id = NULL;
+    char    *ifname = NULL;
     int      id;
     gboolean ret = FALSE;
 
@@ -5347,7 +5347,7 @@ static NMSetting *
 make_infiniband_setting(shvarFile *ifcfg, const char *file, GError **error)
 {
     NMSettingInfiniband *s_infiniband;
-    char *               value = NULL;
+    char                *value = NULL;
 
     s_infiniband = NM_SETTING_INFINIBAND(nm_setting_infiniband_new());
 
@@ -5399,8 +5399,8 @@ static NMConnection *
 infiniband_connection_from_ifcfg(const char *file, shvarFile *ifcfg, GError **error)
 {
     NMConnection *connection         = NULL;
-    NMSetting *   con_setting        = NULL;
-    NMSetting *   infiniband_setting = NULL;
+    NMSetting    *con_setting        = NULL;
+    NMSetting    *infiniband_setting = NULL;
 
     g_return_val_if_fail(file != NULL, NULL);
     g_return_val_if_fail(ifcfg != NULL, NULL);
@@ -5433,7 +5433,7 @@ static void
 handle_bond_option(NMSettingBond *s_bond, const char *key, const char *value)
 {
     gs_free char *sanitized = NULL;
-    const char *  p         = value;
+    const char   *p         = value;
 
     /* Remove any quotes or +/- from arp_ip_target */
     if (nm_streq0(key, NM_SETTING_BOND_OPTION_ARP_IP_TARGET) && value && value[0]) {
@@ -5463,8 +5463,8 @@ static NMSetting *
 make_bond_setting(shvarFile *ifcfg, const char *file, GError **error)
 {
     NMSettingBond *s_bond;
-    gs_free char * value = NULL;
-    const char *   v;
+    gs_free char  *value = NULL;
+    const char    *v;
 
     v = svGetValueStr(ifcfg, "DEVICE", &value);
     if (!v) {
@@ -5481,12 +5481,12 @@ make_bond_setting(shvarFile *ifcfg, const char *file, GError **error)
     v = svGetValueStr(ifcfg, "BONDING_OPTS", &value);
     if (v) {
         gs_free const char **items = NULL;
-        const char *const *  iter;
+        const char *const   *iter;
 
         items = nm_strsplit_set(v, " ");
         for (iter = items; iter && *iter; iter++) {
             gs_free char *key = NULL;
-            const char *  val;
+            const char   *val;
 
             val = strchr(*iter, '=');
             if (!val)
@@ -5504,9 +5504,9 @@ make_bond_setting(shvarFile *ifcfg, const char *file, GError **error)
 static NMSetting *
 make_bond_port_setting(shvarFile *ifcfg)
 {
-    NMSetting *   s_port        = NULL;
+    NMSetting    *s_port        = NULL;
     gs_free char *value_to_free = NULL;
-    const char *  value;
+    const char   *value;
     guint         queue_id;
 
     g_return_val_if_fail(ifcfg != NULL, FALSE);
@@ -5529,12 +5529,12 @@ make_bond_port_setting(shvarFile *ifcfg)
 static NMConnection *
 bond_connection_from_ifcfg(const char *file, shvarFile *ifcfg, GError **error)
 {
-    NMConnection *  connection    = NULL;
-    NMSetting *     con_setting   = NULL;
-    NMSetting *     bond_setting  = NULL;
-    NMSetting *     wired_setting = NULL;
+    NMConnection   *connection    = NULL;
+    NMSetting      *con_setting   = NULL;
+    NMSetting      *bond_setting  = NULL;
+    NMSetting      *wired_setting = NULL;
     NMSetting8021x *s_8021x       = NULL;
-    GError *        local         = NULL;
+    GError         *local         = NULL;
 
     g_return_val_if_fail(file != NULL, NULL);
     g_return_val_if_fail(ifcfg != NULL, NULL);
@@ -5580,7 +5580,7 @@ bond_connection_from_ifcfg(const char *file, shvarFile *ifcfg, GError **error)
 static NMSetting *
 make_team_setting(shvarFile *ifcfg, const char *file, GError **error)
 {
-    NMSetting *   s_team;
+    NMSetting    *s_team;
     gs_free char *value_device = NULL;
     gs_free char *value        = NULL;
 
@@ -5600,12 +5600,12 @@ make_team_setting(shvarFile *ifcfg, const char *file, GError **error)
 static NMConnection *
 team_connection_from_ifcfg(const char *file, shvarFile *ifcfg, GError **error)
 {
-    NMConnection *  connection    = NULL;
-    NMSetting *     con_setting   = NULL;
-    NMSetting *     team_setting  = NULL;
-    NMSetting *     wired_setting = NULL;
+    NMConnection   *connection    = NULL;
+    NMSetting      *con_setting   = NULL;
+    NMSetting      *team_setting  = NULL;
+    NMSetting      *wired_setting = NULL;
     NMSetting8021x *s_8021x       = NULL;
-    GError *        local         = NULL;
+    GError         *local         = NULL;
 
     g_return_val_if_fail(file != NULL, NULL);
     g_return_val_if_fail(ifcfg != NULL, NULL);
@@ -5655,22 +5655,22 @@ typedef enum {
     BRIDGE_OPT_TYPE_PORT_OPTION,
 } BridgeOptType;
 
-typedef void (*BridgeOptFunc)(NMSetting *   setting,
+typedef void (*BridgeOptFunc)(NMSetting    *setting,
                               gboolean      stp,
-                              const char *  key,
-                              const char *  value,
+                              const char   *key,
+                              const char   *value,
                               BridgeOptType opt_type);
 
 static void
-handle_bridge_option(NMSetting *   setting,
+handle_bridge_option(NMSetting    *setting,
                      gboolean      stp,
-                     const char *  key,
-                     const char *  value,
+                     const char   *key,
+                     const char   *value,
                      BridgeOptType opt_type)
 {
     static const struct {
-        const char *  key;
-        const char *  property_name;
+        const char   *key;
+        const char   *property_name;
         BridgeOptType opt_type;
         gboolean      only_with_stp;
         gboolean      extended_bool;
@@ -5809,19 +5809,19 @@ warn:
 }
 
 static void
-handle_bridging_opts(NMSetting *   setting,
+handle_bridging_opts(NMSetting    *setting,
                      gboolean      stp,
-                     const char *  value,
+                     const char   *value,
                      BridgeOptFunc func,
                      BridgeOptType opt_type)
 {
     gs_free const char **items = NULL;
-    const char *const *  iter;
+    const char *const   *iter;
 
     items = nm_strsplit_set(value, " ");
     for (iter = items; iter && *iter; iter++) {
         gs_free char *key = NULL;
-        const char *  val;
+        const char   *val;
 
         val = strchr(*iter, '=');
         if (!val)
@@ -5837,15 +5837,15 @@ static void
 read_bridge_vlans(shvarFile *ifcfg, const char *key, NMSetting *setting, const char *property)
 {
     gs_unref_ptrarray GPtrArray *array         = NULL;
-    gs_free char *               value_to_free = NULL;
-    const char *                 value;
+    gs_free char                *value_to_free = NULL;
+    const char                  *value;
 
     value = svGetValueStr(ifcfg, key, &value_to_free);
     if (value) {
         gs_free const char **strv = NULL;
-        const char *const *  iter;
-        GError *             local = NULL;
-        NMBridgeVlan *       vlan;
+        const char *const   *iter;
+        GError              *local = NULL;
+        NMBridgeVlan        *vlan;
 
         array = g_ptr_array_new_with_free_func((GDestroyNotify) nm_bridge_vlan_unref);
 
@@ -5871,8 +5871,8 @@ static NMSetting *
 make_bridge_setting(shvarFile *ifcfg, const char *file, GError **error)
 {
     gs_unref_object NMSettingBridge *s_bridge      = NULL;
-    gs_free char *                   value_to_free = NULL;
-    const char *                     value;
+    gs_free char                    *value_to_free = NULL;
+    const char                      *value;
     gboolean                         stp     = FALSE;
     gboolean                         stp_set = FALSE;
 
@@ -5937,12 +5937,12 @@ make_bridge_setting(shvarFile *ifcfg, const char *file, GError **error)
 static NMConnection *
 bridge_connection_from_ifcfg(const char *file, shvarFile *ifcfg, GError **error)
 {
-    NMConnection *  connection     = NULL;
-    NMSetting *     con_setting    = NULL;
-    NMSetting *     bridge_setting = NULL;
-    NMSetting *     wired_setting  = NULL;
+    NMConnection   *connection     = NULL;
+    NMSetting      *con_setting    = NULL;
+    NMSetting      *bridge_setting = NULL;
+    NMSetting      *wired_setting  = NULL;
     NMSetting8021x *s_8021x        = NULL;
-    GError *        local          = NULL;
+    GError         *local          = NULL;
 
     g_return_val_if_fail(file != NULL, NULL);
     g_return_val_if_fail(ifcfg != NULL, NULL);
@@ -5988,9 +5988,9 @@ bridge_connection_from_ifcfg(const char *file, shvarFile *ifcfg, GError **error)
 static NMSetting *
 make_bridge_port_setting(shvarFile *ifcfg)
 {
-    NMSetting *   s_port        = NULL;
+    NMSetting    *s_port        = NULL;
     gs_free char *value_to_free = NULL;
-    const char *  value;
+    const char   *value;
 
     g_return_val_if_fail(ifcfg != NULL, FALSE);
 
@@ -6020,7 +6020,7 @@ make_bridge_port_setting(shvarFile *ifcfg)
 static NMSetting *
 make_team_port_setting(shvarFile *ifcfg)
 {
-    NMSetting *   s_port;
+    NMSetting    *s_port;
     gs_free char *value = NULL;
 
     value = svGetValueStr_cp(ifcfg, "TEAM_PORT_CONFIG");
@@ -6071,10 +6071,10 @@ is_wifi_device(const char *name, shvarFile *parsed)
 static void
 parse_prio_map_list(NMSettingVlan *s_vlan, shvarFile *ifcfg, const char *key, NMVlanPriorityMap map)
 {
-    gs_free char *       value = NULL;
+    gs_free char        *value = NULL;
     gs_free const char **list  = NULL;
-    const char *const *  iter;
-    const char *         v;
+    const char *const   *iter;
+    const char          *v;
 
     v = svGetValueStr(ifcfg, key, &value);
     if (!v)
@@ -6093,10 +6093,10 @@ static NMSetting *
 make_vlan_setting(shvarFile *ifcfg, const char *file, GError **error)
 {
     gs_unref_object NMSettingVlan *s_vlan     = NULL;
-    gs_free char *                 parent     = NULL;
-    gs_free char *                 iface_name = NULL;
-    gs_free char *                 value      = NULL;
-    const char *                   v          = NULL;
+    gs_free char                  *parent     = NULL;
+    gs_free char                  *iface_name = NULL;
+    gs_free char                  *value      = NULL;
+    const char                    *v          = NULL;
     int                            vlan_id    = -1;
     guint32                        vlan_flags = 0;
     int                            gvrp, reorder_hdr;
@@ -6188,7 +6188,7 @@ make_vlan_setting(shvarFile *ifcfg, const char *file, GError **error)
     v = svGetValueStr(ifcfg, "VLAN_FLAGS", &value);
     if (v) {
         gs_free const char **strv = NULL;
-        const char *const *  ptr;
+        const char *const   *ptr;
 
         strv = nm_strsplit_set(v, ", ");
         for (ptr = strv; ptr && *ptr; ptr++) {
@@ -6219,12 +6219,12 @@ make_vlan_setting(shvarFile *ifcfg, const char *file, GError **error)
 static NMConnection *
 vlan_connection_from_ifcfg(const char *file, shvarFile *ifcfg, GError **error)
 {
-    NMConnection *  connection    = NULL;
-    NMSetting *     con_setting   = NULL;
-    NMSetting *     wired_setting = NULL;
-    NMSetting *     vlan_setting  = NULL;
+    NMConnection   *connection    = NULL;
+    NMSetting      *con_setting   = NULL;
+    NMSetting      *wired_setting = NULL;
+    NMSetting      *vlan_setting  = NULL;
     NMSetting8021x *s_8021x       = NULL;
-    GError *        local         = NULL;
+    GError         *local         = NULL;
 
     g_return_val_if_fail(file != NULL, NULL);
     g_return_val_if_fail(ifcfg != NULL, NULL);
@@ -6268,14 +6268,14 @@ vlan_connection_from_ifcfg(const char *file, shvarFile *ifcfg, GError **error)
 
 static NMConnection *
 create_unhandled_connection(const char *filename,
-                            shvarFile * ifcfg,
+                            shvarFile  *ifcfg,
                             const char *type,
-                            char **     out_spec)
+                            char      **out_spec)
 {
     NMConnection *connection;
-    NMSetting *   s_con;
+    NMSetting    *s_con;
     gs_free char *value = NULL;
-    const char *  v;
+    const char   *v;
 
     nm_assert(out_spec && !*out_spec);
 
@@ -6329,12 +6329,12 @@ check_dns_search_domains(shvarFile *ifcfg, NMSetting *s_ip4, NMSetting *s_ip6)
     if (!s_ip4 || nm_setting_ip_config_get_num_dns_searches(NM_SETTING_IP_CONFIG(s_ip4)) == 0) {
         /* DNS searches */
         gs_free char *value = NULL;
-        const char *  v;
+        const char   *v;
 
         v = svGetValueStr(ifcfg, "DOMAIN", &value);
         if (v) {
             gs_free const char **searches = NULL;
-            const char *const *  item;
+            const char *const   *item;
 
             searches = nm_strsplit_set(v, " ");
             if (searches) {
@@ -6351,23 +6351,23 @@ static NMConnection *
 connection_from_file_full(const char *filename,
                           const char *network_file, /* for unit tests only */
                           const char *test_type,    /* for unit tests only */
-                          char **     out_unhandled,
-                          GError **   error,
-                          gboolean *  out_ignore_error)
+                          char      **out_unhandled,
+                          GError    **error,
+                          gboolean   *out_ignore_error)
 {
     nm_auto_shvar_file_close shvarFile *main_ifcfg    = NULL;
     nm_auto_shvar_file_close shvarFile *network_ifcfg = NULL;
-    gs_unref_object NMConnection *connection          = NULL;
-    gs_free char *                type                = NULL;
-    char *                        devtype, *bootproto;
-    NMSetting *                   setting;
-    NMSetting *                   s_ip4;
-    NMSetting *                   s_ip6;
-    const char *                  ifcfg_name       = NULL;
-    gs_free char *                s_tmp            = NULL;
-    gboolean                      has_ip4_defroute = FALSE;
-    gboolean                      has_complex_routes_v4;
-    gboolean                      has_complex_routes_v6;
+    gs_unref_object NMConnection       *connection    = NULL;
+    gs_free char                       *type          = NULL;
+    char                               *devtype, *bootproto;
+    NMSetting                          *setting;
+    NMSetting                          *s_ip4;
+    NMSetting                          *s_ip6;
+    const char                         *ifcfg_name       = NULL;
+    gs_free char                       *s_tmp            = NULL;
+    gboolean                            has_ip4_defroute = FALSE;
+    gboolean                            has_complex_routes_v4;
+    gboolean                            has_complex_routes_v6;
 
     g_return_val_if_fail(filename != NULL, NULL);
     g_return_val_if_fail(out_unhandled && !*out_unhandled, NULL);
@@ -6463,7 +6463,7 @@ connection_from_file_full(const char *filename,
 
     if (!type) {
         gs_free char *tmp = NULL;
-        char *        device;
+        char         *device;
 
         if ((tmp = svGetValueStr_cp(main_ifcfg, "IPV6TUNNELIPV4"))) {
             NM_SET_OUT(out_ignore_error, TRUE);
@@ -6503,7 +6503,7 @@ connection_from_file_full(const char *filename,
                 type = g_strdup(TYPE_WIRELESS);
             else {
                 gs_free char *p_path = NULL;
-                char *        p_device;
+                char         *p_device;
                 gsize         i;
 
                 /* network-functions detects DEVICETYPE based on the ifcfg-* name and the existence
@@ -6714,9 +6714,9 @@ connection_from_file_full(const char *filename,
 
 NMConnection *
 connection_from_file(const char *filename,
-                     char **     out_unhandled,
-                     GError **   error,
-                     gboolean *  out_ignore_error)
+                     char      **out_unhandled,
+                     GError    **error,
+                     gboolean   *out_ignore_error)
 {
     return connection_from_file_full(filename, NULL, NULL, out_unhandled, error, out_ignore_error);
 }
@@ -6725,8 +6725,8 @@ NMConnection *
 nmtst_connection_from_file(const char *filename,
                            const char *network_file,
                            const char *test_type,
-                           char **     out_unhandled,
-                           GError **   error)
+                           char      **out_unhandled,
+                           GError    **error)
 {
     return connection_from_file_full(filename, network_file, test_type, out_unhandled, error, NULL);
 }
diff --git a/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-reader.h b/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-reader.h
index c7dc2240..c38432b7 100644
--- a/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-reader.h
+++ b/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-reader.h
@@ -9,14 +9,14 @@
 #include "nm-connection.h"
 
 NMConnection *connection_from_file(const char *filename,
-                                   char **     out_unhandled,
-                                   GError **   error,
-                                   gboolean *  out_ignore_error);
+                                   char      **out_unhandled,
+                                   GError    **error,
+                                   gboolean   *out_ignore_error);
 
 NMConnection *nmtst_connection_from_file(const char *filename,
                                          const char *network_file,
                                          const char *test_type,
-                                         char **     out_unhandled,
-                                         GError **   error);
+                                         char      **out_unhandled,
+                                         GError    **error);
 
 #endif /* __NMS_IFCFG_RH_READER_H__ */
diff --git a/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-storage.c b/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-storage.c
index 3feb4440..16632861 100644
--- a/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-storage.c
+++ b/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-storage.c
@@ -103,9 +103,9 @@ _storage_new(NMSIfcfgRHPlugin *plugin, const char *uuid, const char *filename)
 }
 
 NMSIfcfgRHStorage *
-nms_ifcfg_rh_storage_new_connection(NMSIfcfgRHPlugin *     plugin,
-                                    const char *           filename,
-                                    NMConnection *         connection_take,
+nms_ifcfg_rh_storage_new_connection(NMSIfcfgRHPlugin      *plugin,
+                                    const char            *filename,
+                                    NMConnection          *connection_take,
                                     const struct timespec *mtime)
 {
     NMSIfcfgRHStorage *self;
@@ -123,9 +123,9 @@ nms_ifcfg_rh_storage_new_connection(NMSIfcfgRHPlugin *     plugin,
 
 NMSIfcfgRHStorage *
 nms_ifcfg_rh_storage_new_unhandled(NMSIfcfgRHPlugin *plugin,
-                                   const char *      filename,
-                                   const char *      unmanaged_spec,
-                                   const char *      unrecognized_spec)
+                                   const char       *filename,
+                                   const char       *unmanaged_spec,
+                                   const char       *unrecognized_spec)
 {
     NMSIfcfgRHStorage *self;
 
@@ -167,7 +167,7 @@ nms_ifcfg_rh_storage_destroy(NMSIfcfgRHStorage *self)
 static void
 nms_ifcfg_rh_storage_class_init(NMSIfcfgRHStorageClass *klass)
 {
-    GObjectClass *          object_class  = G_OBJECT_CLASS(klass);
+    GObjectClass           *object_class  = G_OBJECT_CLASS(klass);
     NMSettingsStorageClass *storage_class = NM_SETTINGS_STORAGE_CLASS(klass);
 
     object_class->dispose = dispose;
diff --git a/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-storage.h b/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-storage.h
index 6fb9838f..2ab6beb6 100644
--- a/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-storage.h
+++ b/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-storage.h
@@ -45,14 +45,14 @@ GType nms_ifcfg_rh_storage_get_type(void);
 struct _NMSIfcfgRHPlugin;
 
 NMSIfcfgRHStorage *nms_ifcfg_rh_storage_new_connection(struct _NMSIfcfgRHPlugin *plugin,
-                                                       const char *              filename,
-                                                       NMConnection *            connection_take,
-                                                       const struct timespec *   mtime);
+                                                       const char               *filename,
+                                                       NMConnection             *connection_take,
+                                                       const struct timespec    *mtime);
 
 NMSIfcfgRHStorage *nms_ifcfg_rh_storage_new_unhandled(struct _NMSIfcfgRHPlugin *plugin,
-                                                      const char *              filename,
-                                                      const char *              unmanaged_spec,
-                                                      const char *              unrecognized_spec);
+                                                      const char               *filename,
+                                                      const char               *unmanaged_spec,
+                                                      const char               *unrecognized_spec);
 
 void nms_ifcfg_rh_storage_destroy(NMSIfcfgRHStorage *self);
 
diff --git a/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-utils.c b/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-utils.c
index 1afdf8ee..15e01ba9 100644
--- a/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-utils.c
+++ b/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-utils.c
@@ -17,7 +17,7 @@
 /*****************************************************************************/
 
 gboolean
-nms_ifcfg_rh_utils_parse_unhandled_spec(const char * unhandled_spec,
+nms_ifcfg_rh_utils_parse_unhandled_spec(const char  *unhandled_spec,
                                         const char **out_unmanaged_spec,
                                         const char **out_unrecognized_spec)
 {
@@ -108,7 +108,7 @@ char *
 utils_cert_path(const char *parent, const char *suffix, const char *extension)
 {
     gs_free char *dir = NULL;
-    const char *  name;
+    const char   *name;
 
     g_return_val_if_fail(parent, NULL);
     g_return_val_if_fail(suffix, NULL);
@@ -171,7 +171,7 @@ utils_get_ifcfg_name(const char *file, gboolean only_ifcfg)
 static char *
 utils_get_extra_path(const char *parent, const char *tag)
 {
-    char *      item_path = NULL, *dirname;
+    char       *item_path = NULL, *dirname;
     const char *name;
 
     g_return_val_if_fail(parent != NULL, NULL);
@@ -221,7 +221,7 @@ shvarFile *
 utils_get_extra_ifcfg(const char *parent, const char *tag, gboolean should_create)
 {
     shvarFile *ifcfg = NULL;
-    char *     path;
+    char      *path;
 
     path = utils_get_extra_path(parent, tag);
     if (!path)
@@ -275,7 +275,7 @@ utils_has_route_file_new_syntax_content(const char *contents, gsize len)
 
     while (TRUE) {
         const char *line = contents;
-        char *      eol;
+        char       *eol;
         gboolean    found = FALSE;
 
         /* matches regex "^[[:space:]]*ADDRESS[0-9]+=" */
@@ -374,7 +374,7 @@ utils_detect_ifcfg_path(const char *path, gboolean only_ifcfg)
             return NULL;
         if (utils_is_ifcfg_alias_file(base, NULL)) {
             gs_free char *ifcfg = NULL;
-            char *        ptr;
+            char         *ptr;
 
             ifcfg = g_strdup(path);
             ptr   = strrchr(ifcfg, ':');
@@ -773,7 +773,7 @@ gboolean
 nms_ifcfg_rh_utils_is_numbered_tag_impl(const char *key,
                                         const char *tag,
                                         gsize       tag_len,
-                                        gint64 *    out_idx)
+                                        gint64     *out_idx)
 {
     gint64 idx;
 
diff --git a/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-utils.h b/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-utils.h
index 6b396a9a..d7a1cfc6 100644
--- a/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-utils.h
+++ b/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-utils.h
@@ -29,7 +29,7 @@ typedef enum {
 } NMSIfcfgKeyTypeFlags;
 
 typedef struct {
-    const char *         key_name;
+    const char          *key_name;
     NMSIfcfgKeyTypeFlags key_flags;
 } NMSIfcfgKeyTypeInfo;
 
@@ -50,7 +50,7 @@ nms_ifcfg_well_known_key_find_info_flags(const char *key)
 
 /*****************************************************************************/
 
-gboolean nms_ifcfg_rh_utils_parse_unhandled_spec(const char * unhandled_spec,
+gboolean nms_ifcfg_rh_utils_parse_unhandled_spec(const char  *unhandled_spec,
                                                  const char **out_unmanaged_spec,
                                                  const char **out_unrecognized_spec);
 
@@ -127,7 +127,7 @@ _nms_ifcfg_rh_utils_numbered_tag(char *buf, gsize buf_len, const char *tag_name,
 gboolean nms_ifcfg_rh_utils_is_numbered_tag_impl(const char *key,
                                                  const char *tag,
                                                  gsize       tag_len,
-                                                 gint64 *    out_idx);
+                                                 gint64     *out_idx);
 
 static inline gboolean
 nms_ifcfg_rh_utils_is_numbered_tag(const char *key, const char *tag, gint64 *out_idx)
@@ -158,7 +158,7 @@ nms_ifcfg_rh_utils_get_ethtool_name(NMEthtoolID ethtool_id)
     return _nm_ethtool_ifcfg_names[ethtool_id];
 }
 
-const NMEthtoolData *nms_ifcfg_rh_utils_get_ethtool_by_name(const char *  name,
+const NMEthtoolData *nms_ifcfg_rh_utils_get_ethtool_by_name(const char   *name,
                                                             NMEthtoolType ethtool_type);
 
 #endif /* _UTILS_H_ */
diff --git a/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-writer.c b/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-writer.c
index 263c9253..48c4cae9 100644
--- a/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-writer.c
+++ b/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-writer.c
@@ -94,11 +94,11 @@ save_secret_flags(shvarFile *ifcfg, const char *key, NMSettingSecretFlags flags)
 }
 
 static void
-set_secret(shvarFile *          ifcfg,
-           GHashTable *         secrets,
-           const char *         key,
-           const char *         value,
-           const char *         flags_key,
+set_secret(shvarFile           *ifcfg,
+           GHashTable          *secrets,
+           const char          *key,
+           const char          *value,
+           const char          *flags_key,
            NMSettingSecretFlags flags)
 {
     /* Save secret flags */
@@ -114,14 +114,14 @@ set_secret(shvarFile *          ifcfg,
 static gboolean
 write_secrets(shvarFile *ifcfg, GHashTable *secrets, GError **error)
 {
-    nm_auto_shvar_file_close shvarFile *keyfile = NULL;
-    gs_free NMUtilsNamedValue *secrets_arr_free = NULL;
-    NMUtilsNamedValue          secrets_arr_static[30];
-    const NMUtilsNamedValue *  secrets_arr;
-    guint                      secrets_len;
-    GError *                   local       = NULL;
-    gboolean                   any_secrets = FALSE;
-    guint                      i;
+    nm_auto_shvar_file_close shvarFile *keyfile          = NULL;
+    gs_free NMUtilsNamedValue          *secrets_arr_free = NULL;
+    NMUtilsNamedValue                   secrets_arr_static[30];
+    const NMUtilsNamedValue            *secrets_arr;
+    guint                               secrets_len;
+    GError                             *local       = NULL;
+    gboolean                            any_secrets = FALSE;
+    guint                               i;
 
     keyfile = utils_get_keys_ifcfg(svFileGetName(ifcfg), TRUE);
     if (!keyfile) {
@@ -164,7 +164,7 @@ write_secrets(shvarFile *ifcfg, GHashTable *secrets, GError **error)
 
 typedef struct {
     const NMSetting8021xSchemeVtable *vtable;
-    const char *                      ifcfg_rh_key;
+    const char                       *ifcfg_rh_key;
 } Setting8021xSchemeVtable;
 
 static const Setting8021xSchemeVtable setting_8021x_scheme_vtable[] = {
@@ -184,23 +184,23 @@ static const Setting8021xSchemeVtable setting_8021x_scheme_vtable[] = {
 };
 
 static gboolean
-write_object(NMSetting8021x *                s_8021x,
-             shvarFile *                     ifcfg,
-             GHashTable *                    secrets,
-             GHashTable *                    blobs,
+write_object(NMSetting8021x                 *s_8021x,
+             shvarFile                      *ifcfg,
+             GHashTable                     *secrets,
+             GHashTable                     *blobs,
              const Setting8021xSchemeVtable *objtype,
              gboolean                        force_write,
-             GError **                       error)
+             GError                        **error)
 {
     NMSetting8021xCKScheme scheme;
-    const char *           value    = NULL;
-    GBytes *               blob     = NULL;
-    const char *           password = NULL;
+    const char            *value    = NULL;
+    GBytes                *blob     = NULL;
+    const char            *password = NULL;
     NMSettingSecretFlags   flags    = NM_SETTING_SECRET_FLAG_NONE;
     char                   secret_name[100];
     char                   secret_flags[sizeof(secret_name) + NM_STRLEN("_FLAGS")];
-    const char *           extension;
-    char *                 standard_file;
+    const char            *extension;
+    char                  *standard_file;
 
     g_return_val_if_fail(ifcfg != NULL, FALSE);
     g_return_val_if_fail(objtype != NULL, FALSE);
@@ -277,8 +277,8 @@ static gboolean
 write_blobs(GHashTable *blobs, GError **error)
 {
     GHashTableIter iter;
-    const char *   filename;
-    GBytes *       blob;
+    const char    *filename;
+    GBytes        *blob;
 
     if (!blobs)
         return TRUE;
@@ -318,14 +318,14 @@ write_blobs(GHashTable *blobs, GError **error)
 
 static gboolean
 write_8021x_certs(NMSetting8021x *s_8021x,
-                  GHashTable *    secrets,
-                  GHashTable *    blobs,
+                  GHashTable     *secrets,
+                  GHashTable     *blobs,
                   gboolean        phase2,
-                  shvarFile *     ifcfg,
-                  GError **       error)
+                  shvarFile      *ifcfg,
+                  GError        **error)
 {
     const Setting8021xSchemeVtable *pk_otype      = NULL;
-    gs_free char *                  value_to_free = NULL;
+    gs_free char                   *value_to_free = NULL;
 
     /* CA certificate */
     if (!write_object(
@@ -371,20 +371,20 @@ write_8021x_certs(NMSetting8021x *s_8021x,
 
 static gboolean
 write_8021x_setting(NMConnection *connection,
-                    shvarFile *   ifcfg,
-                    GHashTable *  secrets,
-                    GHashTable *  blobs,
+                    shvarFile    *ifcfg,
+                    GHashTable   *secrets,
+                    GHashTable   *blobs,
                     gboolean      wired,
-                    GError **     error)
+                    GError      **error)
 {
-    NMSetting8021x *        s_8021x;
+    NMSetting8021x         *s_8021x;
     NMSetting8021xAuthFlags auth_flags;
-    const char *            value, *match;
+    const char             *value, *match;
     gconstpointer           ptr;
-    GBytes *                bytes;
-    char *                  tmp = NULL;
-    GString *               phase2_auth;
-    GString *               str;
+    GBytes                 *bytes;
+    char                   *tmp = NULL;
+    GString                *phase2_auth;
+    GString                *str;
     guint32                 i, num;
     gsize                   size;
     int                     vint;
@@ -565,20 +565,20 @@ write_8021x_setting(NMConnection *connection,
 
 static gboolean
 write_wireless_security_setting(NMConnection *connection,
-                                shvarFile *   ifcfg,
-                                GHashTable *  secrets,
+                                shvarFile    *ifcfg,
+                                GHashTable   *secrets,
                                 gboolean      adhoc,
-                                gboolean *    no_8021x,
-                                GError **     error)
+                                gboolean     *no_8021x,
+                                GError      **error)
 {
-    NMSettingWirelessSecurity *        s_wsec;
-    const char *                       key_mgmt, *auth_alg, *key, *proto, *cipher;
-    const char *                       psk = NULL;
+    NMSettingWirelessSecurity         *s_wsec;
+    const char                        *key_mgmt, *auth_alg, *key, *proto, *cipher;
+    const char                        *psk = NULL;
     gboolean                           wep = FALSE, wpa = FALSE, dynamic_wep = FALSE;
     NMSettingWirelessSecurityWpsMethod wps_method;
-    char *                             tmp;
+    char                              *tmp;
     guint32                            i, num;
-    GString *                          str;
+    GString                           *str;
 
     s_wsec = nm_connection_get_setting_wireless_security(connection);
     if (!s_wsec) {
@@ -667,7 +667,7 @@ write_wireless_security_setting(NMConnection *connection,
     /* And write the new ones out */
     if (wep) {
         NMWepKeyType key_type;
-        const char * key_type_str = NULL;
+        const char  *key_type_str = NULL;
 
         /* Default WEP TX key index */
         svSetValueInt64(ifcfg,
@@ -711,7 +711,7 @@ write_wireless_security_setting(NMConnection *connection,
                     numbered_tag(tag, "KEY", i + 1);
 
                     /* Add 's:' prefix for ASCII keys */
-                    if (strlen(key) == 5 || strlen(key) == 13) {
+                    if (NM_IN_SET(strlen(key), 5, 13)) {
                         ascii_key = g_strdup_printf("s:%s", key);
                         key       = ascii_key;
                     }
@@ -759,7 +759,7 @@ write_wireless_security_setting(NMConnection *connection,
             g_free(tmp);
         }
     }
-    if (strlen(str->str) && (dynamic_wep == FALSE))
+    if (nm_str_not_empty(str->str) && !dynamic_wep)
         svSetValueStr(ifcfg, "CIPHER_PAIRWISE", str->str);
     g_string_free(str, TRUE);
 
@@ -809,17 +809,17 @@ write_wireless_security_setting(NMConnection *connection,
 
 static gboolean
 write_wireless_setting(NMConnection *connection,
-                       shvarFile *   ifcfg,
-                       GHashTable *  secrets,
-                       gboolean *    no_8021x,
-                       GError **     error)
+                       shvarFile    *ifcfg,
+                       GHashTable   *secrets,
+                       gboolean     *no_8021x,
+                       GError      **error)
 {
     NMSettingWireless *s_wireless;
-    GBytes *           ssid;
-    const guint8 *     ssid_data;
+    GBytes            *ssid;
+    const guint8      *ssid_data;
     gsize              ssid_len;
-    const char *       mode, *bssid;
-    const char *       device_mac, *cloned_mac;
+    const char        *mode, *bssid;
+    const char        *device_mac, *cloned_mac;
     guint32            mtu, chan, i;
     gboolean           adhoc = FALSE, hex_ssid = FALSE;
     const char *const *macaddr_blacklist;
@@ -1018,7 +1018,7 @@ static gboolean
 write_infiniband_setting(NMConnection *connection, shvarFile *ifcfg, GError **error)
 {
     NMSettingInfiniband *s_infiniband;
-    const char *         mac, *transport_mode, *parent;
+    const char          *mac, *transport_mode, *parent;
     guint32              mtu;
     int                  p_key;
 
@@ -1223,14 +1223,14 @@ _ethtool_gstring_prepare(GString **str, gboolean *is_first, char cmdline_flag, c
 static gboolean
 write_ethtool_setting(NMConnection *connection, shvarFile *ifcfg, GError **error)
 {
-    NMSettingWired *        s_wired;
-    NMSettingEthtool *      s_ethtool;
-    const char *            duplex;
+    NMSettingWired         *s_wired;
+    NMSettingEthtool       *s_ethtool;
+    const char             *duplex;
     guint32                 speed;
-    GString *               str = NULL;
+    GString                *str = NULL;
     gboolean                auto_negotiate;
     NMSettingWiredWakeOnLan wol;
-    const char *            wol_password;
+    const char             *wol_password;
 
     s_wired   = nm_connection_get_setting_wired(connection);
     s_ethtool = NM_SETTING_ETHTOOL(nm_connection_get_setting(connection, NM_TYPE_SETTING_ETHTOOL));
@@ -1300,7 +1300,7 @@ write_ethtool_setting(NMConnection *connection, shvarFile *ifcfg, GError **error
     if (s_ethtool) {
         NMEthtoolID          ethtool_id;
         NMSettingConnection *s_con;
-        const char *         iface;
+        const char          *iface;
         gboolean             is_first;
         guint32              u32;
         gboolean             b;
@@ -1398,7 +1398,7 @@ static char *
 vlan_priority_maplist_to_stringlist(NMSettingVlan *s_vlan, NMVlanPriorityMap map)
 {
     char **strlist;
-    char * value;
+    char  *value;
 
     if (map == NM_VLAN_INGRESS_MAP)
         g_object_get(G_OBJECT(s_vlan), NM_SETTING_VLAN_INGRESS_PRIORITY_MAP, &strlist, NULL);
@@ -1420,7 +1420,7 @@ static gboolean
 write_vlan_setting(NMConnection *connection, shvarFile *ifcfg, gboolean *wired, GError **error)
 {
     NMSettingVlan *s_vlan;
-    char *         tmp;
+    char          *tmp;
     guint32        vlan_flags = 0;
     gsize          s_buf_len;
     char           s_buf[50], *s_buf_ptr;
@@ -1486,7 +1486,7 @@ write_bond_setting(NMConnection *connection, shvarFile *ifcfg, gboolean *wired,
     num_opts = nm_setting_bond_get_num_options(s_bond);
     if (num_opts) {
         nm_auto_free_gstring GString *str = NULL;
-        const char *                  name, *value;
+        const char                   *name, *value;
 
         str = g_string_sized_new(64);
         for (i = 0; i < num_opts; i++) {
@@ -1511,7 +1511,7 @@ static gboolean
 write_team_setting(NMConnection *connection, shvarFile *ifcfg, gboolean *wired, GError **error)
 {
     NMSettingTeam *s_team;
-    const char *   config;
+    const char    *config;
 
     s_team = nm_connection_get_setting_team(connection);
     if (!s_team) {
@@ -1562,14 +1562,14 @@ write_team_setting(NMConnection *connection, shvarFile *ifcfg, gboolean *wired,
     })
 
 static gboolean
-write_bridge_vlans(NMSetting * setting,
+write_bridge_vlans(NMSetting  *setting,
                    gboolean    is_port,
-                   shvarFile * ifcfg,
+                   shvarFile  *ifcfg,
                    const char *key,
-                   GError **   error)
+                   GError    **error)
 {
-    GPtrArray *              vlans;
-    NMBridgeVlan *           vlan;
+    GPtrArray               *vlans;
+    NMBridgeVlan            *vlan;
     nm_auto_str_buf NMStrBuf strbuf = NM_STR_BUF_INIT(0, FALSE);
     guint                    i;
 
@@ -1608,8 +1608,8 @@ write_bridge_setting(NMConnection *connection, shvarFile *ifcfg, gboolean *wired
     guint64          u64;
     guint            u;
     gboolean         b;
-    const char *     s;
-    GString *        opts;
+    const char      *s;
+    GString         *opts;
 
     s_bridge = nm_connection_get_setting_bridge(connection);
     if (!s_bridge) {
@@ -1847,7 +1847,7 @@ write_bridge_port_setting(NMConnection *connection, shvarFile *ifcfg, GError **e
 {
     NMSettingBridgePort *s_port;
     guint32              u32;
-    GString *            string;
+    GString             *string;
 
     s_port = nm_connection_get_setting_bridge_port(connection);
     if (!s_port)
@@ -1903,7 +1903,7 @@ static gboolean
 write_team_port_setting(NMConnection *connection, shvarFile *ifcfg, GError **error)
 {
     NMSettingTeamPort *s_port;
-    const char *       config;
+    const char        *config;
 
     s_port = nm_connection_get_setting_team_port(connection);
     if (!s_port)
@@ -1946,9 +1946,9 @@ write_dcb_app(shvarFile *ifcfg, const char *tag, NMSettingDcbFlags flags, int pr
 typedef gboolean (*DcbGetBoolFunc)(NMSettingDcb *, guint);
 
 static void
-write_dcb_bool_array(shvarFile *       ifcfg,
-                     const char *      key,
-                     NMSettingDcb *    s_dcb,
+write_dcb_bool_array(shvarFile        *ifcfg,
+                     const char       *key,
+                     NMSettingDcb     *s_dcb,
                      NMSettingDcbFlags flags,
                      DcbGetBoolFunc    get_func)
 {
@@ -1967,9 +1967,9 @@ write_dcb_bool_array(shvarFile *       ifcfg,
 typedef guint (*DcbGetUintFunc)(NMSettingDcb *, guint);
 
 static void
-write_dcb_uint_array(shvarFile *       ifcfg,
-                     const char *      key,
-                     NMSettingDcb *    s_dcb,
+write_dcb_uint_array(shvarFile        *ifcfg,
+                     const char       *key,
+                     NMSettingDcb     *s_dcb,
                      NMSettingDcbFlags flags,
                      DcbGetUintFunc    get_func)
 {
@@ -1993,9 +1993,9 @@ write_dcb_uint_array(shvarFile *       ifcfg,
 }
 
 static void
-write_dcb_percent_array(shvarFile *       ifcfg,
-                        const char *      key,
-                        NMSettingDcb *    s_dcb,
+write_dcb_percent_array(shvarFile        *ifcfg,
+                        const char       *key,
+                        NMSettingDcb     *s_dcb,
                         NMSettingDcbFlags flags,
                         DcbGetUintFunc    get_func)
 {
@@ -2018,7 +2018,7 @@ write_dcb_percent_array(shvarFile *       ifcfg,
 static gboolean
 write_dcb_setting(NMConnection *connection, shvarFile *ifcfg, GError **error)
 {
-    NMSettingDcb *    s_dcb;
+    NMSettingDcb     *s_dcb;
     NMSettingDcbFlags flags;
 
     s_dcb = nm_connection_get_setting_dcb(connection);
@@ -2079,16 +2079,16 @@ write_dcb_setting(NMConnection *connection, shvarFile *ifcfg, GError **error)
 static void
 write_connection_setting(NMSettingConnection *s_con, shvarFile *ifcfg)
 {
-    guint32              n, i;
+    guint32                       n, i;
     nm_auto_free_gstring GString *str = NULL;
-    const char *                  master, *master_iface = NULL, *type;
+    const char                   *master, *master_iface = NULL, *type;
     int                           vint;
     gint32                        vint32;
     NMSettingConnectionMdns       mdns;
     NMSettingConnectionLlmnr      llmnr;
     NMSettingConnectionDnsOverTls dns_over_tls;
     guint32                       vuint32;
-    const char *                  tmp, *mud_url;
+    const char                   *tmp, *mud_url;
 
     svSetValueStr(ifcfg, "NAME", nm_setting_connection_get_id(s_con));
     svSetValueStr(ifcfg, "UUID", nm_setting_connection_get_uuid(s_con));
@@ -2284,8 +2284,8 @@ static char *
 get_route_attributes_string(NMIPRoute *route, int family)
 {
     gs_free const char **names = NULL;
-    GVariant *           attr, *lock;
-    GString *            str;
+    GVariant            *attr, *lock;
+    GString             *str;
     guint                i, len;
 
     names = _nm_ip_route_get_attribute_names(route, TRUE, &len);
@@ -2369,7 +2369,7 @@ write_route_file_svformat(const char *filename, NMSettingIPConfig *s_ip4)
     num = nm_setting_ip_config_get_num_routes(s_ip4);
     for (i = 0; i < num; i++) {
         char          buf[INET_ADDRSTRLEN];
-        NMIPRoute *   route;
+        NMIPRoute    *route;
         guint32       netmask;
         gint64        metric;
         char          addr_key[64];
@@ -2409,7 +2409,7 @@ write_route_file_svformat(const char *filename, NMSettingIPConfig *s_ip4)
 static GString *
 write_route_file(NMSettingIPConfig *s_ip)
 {
-    GString *  contents;
+    GString   *contents;
     NMIPRoute *route;
     guint32    i, num;
     int        addr_family;
@@ -2424,7 +2424,7 @@ write_route_file(NMSettingIPConfig *s_ip)
 
     for (i = 0; i < num; i++) {
         gs_free char *options = NULL;
-        const char *  next_hop;
+        const char   *next_hop;
         gint64        metric;
 
         route    = nm_setting_ip_config_get_route(s_ip, i);
@@ -2454,9 +2454,9 @@ write_route_file(NMSettingIPConfig *s_ip)
 static gboolean
 write_proxy_setting(NMConnection *connection, shvarFile *ifcfg, GError **error)
 {
-    NMSettingProxy *     s_proxy;
+    NMSettingProxy      *s_proxy;
     NMSettingProxyMethod method;
-    const char *         pac_url, *pac_script;
+    const char          *pac_url, *pac_script;
 
     s_proxy = nm_connection_get_setting_proxy(connection);
     if (!s_proxy)
@@ -2489,7 +2489,7 @@ write_proxy_setting(NMConnection *connection, shvarFile *ifcfg, GError **error)
 static gboolean
 write_user_setting(NMConnection *connection, shvarFile *ifcfg, GError **error)
 {
-    NMSettingUser *    s_user;
+    NMSettingUser     *s_user;
     guint              i, len;
     const char *const *keys;
 
@@ -2521,9 +2521,9 @@ write_sriov_setting(NMConnection *connection, shvarFile *ifcfg)
     NMSettingSriov *s_sriov;
     guint           i, num = 0;
     NMTernary       b;
-    NMSriovVF *     vf;
+    NMSriovVF      *vf;
     char            key[32];
-    char *          str;
+    char           *str;
 
     s_sriov = NM_SETTING_SRIOV(nm_connection_get_setting(connection, NM_TYPE_SETTING_SRIOV));
     if (!s_sriov) {
@@ -2562,7 +2562,7 @@ write_tc_setting(NMConnection *connection, shvarFile *ifcfg)
 
     num_qdiscs = nm_setting_tc_config_get_num_qdiscs(s_tc);
     for (n = 1, i = 0; i < num_qdiscs; i++) {
-        NMTCQdisc *   qdisc;
+        NMTCQdisc    *qdisc;
         gs_free char *str = NULL;
 
         qdisc = nm_setting_tc_config_get_qdisc(s_tc, i);
@@ -2574,7 +2574,7 @@ write_tc_setting(NMConnection *connection, shvarFile *ifcfg)
 
     num_filters = nm_setting_tc_config_get_num_tfilters(s_tc);
     for (n = 1, i = 0; i < num_filters; i++) {
-        NMTCTfilter * tfilter;
+        NMTCTfilter  *tfilter;
         gs_free char *str = NULL;
 
         tfilter = nm_setting_tc_config_get_tfilter(s_tc, i);
@@ -2592,9 +2592,9 @@ static void
 write_match_setting(NMConnection *connection, shvarFile *ifcfg)
 {
     nm_auto_free_gstring GString *str = NULL;
-    NMSettingMatch *              s_match;
+    NMSettingMatch               *s_match;
     guint                         i, num;
-    const char *                  name;
+    const char                   *name;
 
     s_match = (NMSettingMatch *) nm_connection_get_setting(connection, NM_TYPE_SETTING_MATCH);
     if (!s_match)
@@ -2706,25 +2706,25 @@ write_dns_setting(shvarFile *ifcfg, NMConnection *connection, int addr_family)
 
 static void
 write_ip4_setting(NMConnection *connection,
-                  shvarFile *   ifcfg,
-                  shvarFile **  out_route_content_svformat,
-                  GString **    out_route_content)
+                  shvarFile    *ifcfg,
+                  shvarFile   **out_route_content_svformat,
+                  GString     **out_route_content)
 {
-    NMSettingIPConfig *    s_ip4;
-    const char *           value;
-    char *                 tmp;
-    char                   tag[64];
-    int                    j;
-    guint                  i, num, n;
-    gint64                 route_metric;
-    NMIPRouteTableSyncMode route_table;
-    int                    priority;
-    int                    timeout;
+    NMSettingIPConfig            *s_ip4;
+    const char                   *value;
+    char                         *tmp;
+    char                          tag[64];
+    int                           j;
+    guint                         i, num, n;
+    gint64                        route_metric;
+    NMIPRouteTableSyncMode        route_table;
+    int                           priority;
+    int                           timeout;
     nm_auto_free_gstring GString *str    = NULL;
-    const char *                  method = NULL;
+    const char                   *method = NULL;
     gboolean                      has_netmask;
     NMDhcpHostnameFlags           flags;
-    const char *const *           strv;
+    const char *const            *strv;
 
     NM_SET_OUT(out_route_content_svformat, NULL);
     NM_SET_OUT(out_route_content, NULL);
@@ -2905,17 +2905,17 @@ static void
 write_ip4_aliases(NMConnection *connection, const char *base_ifcfg_path)
 {
     NMSettingIPConfig *s_ip4;
-    gs_free char *     base_ifcfg_dir = NULL, *base_ifcfg_name = NULL;
-    const char *       base_name;
+    gs_free char      *base_ifcfg_dir = NULL, *base_ifcfg_name = NULL;
+    const char        *base_name;
     int                i, num, base_ifcfg_name_len, base_name_len;
-    GDir *             dir;
+    GDir              *dir;
 
     base_ifcfg_dir      = g_path_get_dirname(base_ifcfg_path);
     base_ifcfg_name     = g_path_get_basename(base_ifcfg_path);
     base_ifcfg_name_len = strlen(base_ifcfg_name);
     if (!g_str_has_prefix(base_ifcfg_name, IFCFG_TAG))
         g_return_if_reached();
-    base_name     = base_ifcfg_name + strlen(IFCFG_TAG);
+    base_name     = base_ifcfg_name + NM_STRLEN(IFCFG_TAG);
     base_name_len = strlen(base_name);
 
     /* Remove all existing aliases for this file first */
@@ -2946,11 +2946,11 @@ write_ip4_aliases(NMConnection *connection, const char *base_ifcfg_path)
 
     num = nm_setting_ip_config_get_num_addresses(s_ip4);
     for (i = 0; i < num; i++) {
-        GVariant *   label_var;
-        const char * label, *p;
-        char *       path;
+        GVariant    *label_var;
+        const char  *label, *p;
+        char        *path;
         NMIPAddress *addr;
-        shvarFile *  ifcfg;
+        shvarFile   *ifcfg;
 
         addr = nm_setting_ip_config_get_address(s_ip4, i);
 
@@ -2987,17 +2987,17 @@ write_ip4_aliases(NMConnection *connection, const char *base_ifcfg_path)
 static void
 write_ip6_setting(NMConnection *connection, shvarFile *ifcfg, GString **out_route6_content)
 {
-    NMSettingIPConfig *           s_ip6;
-    const char *                  value;
+    NMSettingIPConfig            *s_ip6;
+    const char                   *value;
     guint                         i, num;
     int                           priority;
-    NMIPAddress *                 addr;
+    NMIPAddress                  *addr;
     gint64                        route_metric;
     NMIPRouteTableSyncMode        route_table;
-    GString *                     ip_str1, *ip_str2, *ip_ptr;
+    GString                      *ip_str1, *ip_str2, *ip_ptr;
     NMSettingIP6ConfigAddrGenMode addr_gen_mode;
     NMDhcpHostnameFlags           flags;
-    const char *                  hostname;
+    const char                   *hostname;
     int                           timeout;
 
     NM_SET_OUT(out_route6_content, NULL);
@@ -3183,7 +3183,7 @@ write_ip_routing_rules(NMConnection *connection, shvarFile *ifcfg, gboolean rout
 
         num = nm_setting_ip_config_get_num_routing_rules(s_ip);
         for (i = 0; i < num; i++) {
-            NMIPRoutingRule *   rule = nm_setting_ip_config_get_routing_rule(s_ip, i);
+            NMIPRoutingRule    *rule = nm_setting_ip_config_get_routing_rule(s_ip, i);
             gs_free const char *s    = NULL;
             char                key[64];
 
@@ -3222,38 +3222,38 @@ escape_id(const char *id)
 }
 
 static gboolean
-do_write_construct(NMConnection *                  connection,
-                   const char *                    ifcfg_dir,
-                   const char *                    filename,
+do_write_construct(NMConnection                   *connection,
+                   const char                     *ifcfg_dir,
+                   const char                     *filename,
                    NMSIfcfgRHWriterAllowFilenameCb allow_filename_cb,
                    gpointer                        allow_filename_user_data,
-                   shvarFile **                    out_ifcfg,
-                   GHashTable **                   out_blobs,
-                   GHashTable **                   out_secrets,
-                   gboolean *                      out_route_ignore,
-                   shvarFile **                    out_route_content_svformat,
-                   GString **                      out_route_content,
-                   GString **                      out_route6_content,
-                   GError **                       error)
+                   shvarFile                     **out_ifcfg,
+                   GHashTable                    **out_blobs,
+                   GHashTable                    **out_secrets,
+                   gboolean                       *out_route_ignore,
+                   shvarFile                     **out_route_content_svformat,
+                   GString                       **out_route_content,
+                   GString                       **out_route6_content,
+                   GError                        **error)
 {
-    NMSettingConnection *    s_con;
+    NMSettingConnection                *s_con;
     nm_auto_shvar_file_close shvarFile *ifcfg = NULL;
-    const char *                        ifcfg_name;
-    gs_free char *                      ifcfg_name_free = NULL;
-    gs_free char *                      route_path      = NULL;
-    gs_free char *                      route6_path     = NULL;
-    const char *                        type;
-    gs_unref_hashtable GHashTable *blobs   = NULL;
-    gs_unref_hashtable GHashTable *secrets = NULL;
-    gboolean                       wired;
-    gboolean                       no_8021x;
-    gboolean                       route_path_is_svformat;
-    gboolean                       has_complex_routes_v4;
-    gboolean                       has_complex_routes_v6;
-    gboolean                       route_ignore;
+    const char                         *ifcfg_name;
+    gs_free char                       *ifcfg_name_free = NULL;
+    gs_free char                       *route_path      = NULL;
+    gs_free char                       *route6_path     = NULL;
+    const char                         *type;
+    gs_unref_hashtable GHashTable      *blobs   = NULL;
+    gs_unref_hashtable GHashTable      *secrets = NULL;
+    gboolean                            wired;
+    gboolean                            no_8021x;
+    gboolean                            route_path_is_svformat;
+    gboolean                            has_complex_routes_v4;
+    gboolean                            has_complex_routes_v6;
+    gboolean                            route_ignore;
     nm_auto_shvar_file_close shvarFile *route_content_svformat = NULL;
-    nm_auto_free_gstring GString *route_content                = NULL;
-    nm_auto_free_gstring GString *route6_content               = NULL;
+    nm_auto_free_gstring GString       *route_content          = NULL;
+    nm_auto_free_gstring GString       *route6_content         = NULL;
 
     nm_assert(NM_IS_CONNECTION(connection));
     nm_assert(_nm_connection_verify(connection, NULL) == NM_SETTING_VERIFY_SUCCESS);
@@ -3478,14 +3478,14 @@ do_write_construct(NMConnection *                  connection,
 
 static gboolean
 do_write_to_disk(NMConnection *connection,
-                 shvarFile *   ifcfg,
-                 GHashTable *  blobs,
-                 GHashTable *  secrets,
+                 shvarFile    *ifcfg,
+                 GHashTable   *blobs,
+                 GHashTable   *secrets,
                  gboolean      route_ignore,
-                 shvarFile *   route_content_svformat,
-                 GString *     route_content,
-                 GString *     route6_content,
-                 GError **     error)
+                 shvarFile    *route_content_svformat,
+                 GString      *route_content,
+                 GString      *route6_content,
+                 GError      **error)
 {
     /* From here on, we persist data to disk. Before, it was all in-memory
      * only. But we loaded the ifcfg files from disk, and managled our
@@ -3549,23 +3549,23 @@ do_write_to_disk(NMConnection *connection,
 }
 
 gboolean
-nms_ifcfg_rh_writer_write_connection(NMConnection *                  connection,
-                                     const char *                    ifcfg_dir,
-                                     const char *                    filename,
+nms_ifcfg_rh_writer_write_connection(NMConnection                   *connection,
+                                     const char                     *ifcfg_dir,
+                                     const char                     *filename,
                                      NMSIfcfgRHWriterAllowFilenameCb allow_filename_cb,
                                      gpointer                        allow_filename_user_data,
-                                     char **                         out_filename,
-                                     NMConnection **                 out_reread,
-                                     gboolean *                      out_reread_same,
-                                     GError **                       error)
+                                     char                          **out_filename,
+                                     NMConnection                  **out_reread,
+                                     gboolean                       *out_reread_same,
+                                     GError                        **error)
 {
     nm_auto_shvar_file_close shvarFile *ifcfg                  = NULL;
-    nm_auto_free_gstring GString *route_content                = NULL;
-    gboolean                      route_ignore                 = FALSE;
+    nm_auto_free_gstring GString       *route_content          = NULL;
+    gboolean                            route_ignore           = FALSE;
     nm_auto_shvar_file_close shvarFile *route_content_svformat = NULL;
-    nm_auto_free_gstring GString *route6_content               = NULL;
-    gs_unref_hashtable GHashTable *secrets                     = NULL;
-    gs_unref_hashtable GHashTable *blobs                       = NULL;
+    nm_auto_free_gstring GString       *route6_content         = NULL;
+    gs_unref_hashtable GHashTable      *secrets                = NULL;
+    gs_unref_hashtable GHashTable      *blobs                  = NULL;
 
     nm_assert(!out_reread || !*out_reread);
 
@@ -3611,8 +3611,8 @@ nms_ifcfg_rh_writer_write_connection(NMConnection *                  connection,
     if (out_reread || out_reread_same) {
         gs_unref_object NMConnection *reread      = NULL;
         gboolean                      reread_same = FALSE;
-        gs_free_error GError *local               = NULL;
-        gs_free char *        unhandled           = NULL;
+        gs_free_error GError         *local       = NULL;
+        gs_free char                 *unhandled   = NULL;
 
         reread = connection_from_file(svFileGetName(ifcfg), &unhandled, &local, NULL);
         nm_assert((NM_IS_CONNECTION(reread) && !local) || (!reread && local));
diff --git a/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-writer.h b/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-writer.h
index d0d8a4cd..9b5162f8 100644
--- a/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-writer.h
+++ b/src/core/settings/plugins/ifcfg-rh/nms-ifcfg-rh-writer.h
@@ -13,14 +13,14 @@ typedef gboolean (*NMSIfcfgRHWriterAllowFilenameCb)(const char *check_filename,
 
 gboolean nms_ifcfg_rh_writer_can_write_connection(NMConnection *connection, GError **error);
 
-gboolean nms_ifcfg_rh_writer_write_connection(NMConnection *                  connection,
-                                              const char *                    ifcfg_dir,
-                                              const char *                    filename,
+gboolean nms_ifcfg_rh_writer_write_connection(NMConnection                   *connection,
+                                              const char                     *ifcfg_dir,
+                                              const char                     *filename,
                                               NMSIfcfgRHWriterAllowFilenameCb allow_filename_cb,
                                               gpointer       allow_filename_user_data,
-                                              char **        out_filename,
+                                              char         **out_filename,
                                               NMConnection **out_reread,
-                                              gboolean *     out_reread_same,
-                                              GError **      error);
+                                              gboolean      *out_reread_same,
+                                              GError       **error);
 
 #endif /* __NMS_IFCFG_RH_WRITER_H__ */
diff --git a/src/core/settings/plugins/ifcfg-rh/shvar.c b/src/core/settings/plugins/ifcfg-rh/shvar.c
index d82efb3c..2ca49558 100644
--- a/src/core/settings/plugins/ifcfg-rh/shvar.c
+++ b/src/core/settings/plugins/ifcfg-rh/shvar.c
@@ -62,7 +62,7 @@ struct _shvarLine {
 typedef struct _shvarLine shvarLine;
 
 struct _shvarFile {
-    char *      fileName;
+    char       *fileName;
     CList       lst_head;
     GHashTable *lst_idx;
     int         fd;
@@ -151,8 +151,8 @@ static char *
 _escape_ansic(const char *source)
 {
     const char *p;
-    char *      dest;
-    char *      q;
+    char       *dest;
+    char       *q;
     gsize       n_alloc;
 
     nm_assert(source);
@@ -705,7 +705,7 @@ out_error:
 shvarFile *
 svFile_new(const char *name, int fd, const char *content)
 {
-    shvarFile * s;
+    shvarFile  *s;
     const char *p;
     const char *q;
 
@@ -820,9 +820,9 @@ line_new_parse(const char *value, gsize len)
 static shvarLine *
 line_new_build(const char *key, const char *value)
 {
-    char *     value_escaped = NULL;
+    char      *value_escaped = NULL;
     shvarLine *line;
-    char *     new_key;
+    char      *new_key;
 
     value = svEscape(value, &value_escaped);
 
@@ -841,7 +841,7 @@ line_new_build(const char *key, const char *value)
 static gboolean
 line_set(shvarLine *line, const char *value)
 {
-    char *   value_escaped = NULL;
+    char    *value_escaped = NULL;
     gboolean changed       = FALSE;
 
     ASSERT_shvarLine(line);
@@ -926,11 +926,11 @@ do_link:
 static shvarFile *
 svOpenFileInternal(const char *name, gboolean create, GError **error)
 {
-    gboolean      closefd       = FALSE;
-    int           errsv         = 0;
-    gs_free char *content       = NULL;
-    gs_free_error GError *local = NULL;
-    nm_auto_close int     fd    = -1;
+    gboolean              closefd = FALSE;
+    int                   errsv   = 0;
+    gs_free char         *content = NULL;
+    gs_free_error GError *local   = NULL;
+    nm_auto_close int     fd      = -1;
 
     if (create)
         fd = open(name, O_RDWR | O_CLOEXEC); /* NOT O_CREAT */
@@ -1063,8 +1063,8 @@ svNumberedParseKey(const char *key)
 GHashTable *
 svGetKeys(shvarFile *s, SvKeyType match_key_type)
 {
-    GHashTable *     keys = NULL;
-    CList *          current;
+    GHashTable      *keys = NULL;
+    CList           *current;
     const shvarLine *line;
 
     nm_assert(s);
@@ -1137,7 +1137,7 @@ static const char *
 _svGetValue(shvarFile *s, const char *key, char **to_free)
 {
     const shvarLine *line;
-    const char *     v;
+    const char      *v;
 
     nm_assert(s);
     nm_assert(_shell_is_name(key, -1));
@@ -1209,7 +1209,7 @@ svGetValueStr(shvarFile *s, const char *key, char **to_free)
 char *
 svGetValue_cp(shvarFile *s, const char *key)
 {
-    char *      to_free;
+    char       *to_free;
     const char *value;
 
     g_return_val_if_fail(s, NULL);
@@ -1233,7 +1233,7 @@ svGetValue_cp(shvarFile *s, const char *key)
 char *
 svGetValueStr_cp(shvarFile *s, const char *key)
 {
-    char *      to_free;
+    char       *to_free;
     const char *value;
 
     g_return_val_if_fail(s, NULL);
@@ -1260,7 +1260,7 @@ int
 svGetValueBoolean(shvarFile *s, const char *key, int fallback)
 {
     gs_free char *to_free = NULL;
-    const char *  value;
+    const char   *value;
 
     value = _svGetValue(s, key, &to_free);
     return svParseBoolean(value, fallback);
@@ -1293,7 +1293,7 @@ svGetValueTernary(shvarFile *s, const char *key)
 gint64
 svGetValueInt64(shvarFile *s, const char *key, guint base, gint64 min, gint64 max, gint64 fallback)
 {
-    char *      to_free;
+    char       *to_free;
     const char *value;
     gint64      result;
     int         errsv;
@@ -1320,7 +1320,7 @@ gboolean
 svGetValueEnum(shvarFile *s, const char *key, GType gtype, int *out_value, GError **error)
 {
     gs_free char *to_free = NULL;
-    const char *  svalue;
+    const char   *svalue;
     gs_free char *err_token = NULL;
     int           value;
 
@@ -1618,7 +1618,7 @@ svWarnInvalid(shvarFile *s, const char *file_type, NMLogDomain log_domain)
 gboolean
 svWriteFile(shvarFile *s, int mode, GError **error)
 {
-    FILE * f;
+    FILE  *f;
     int    tmpfd;
     CList *current;
     int    errsv;
@@ -1672,8 +1672,8 @@ svWriteFile(shvarFile *s, int mode, GError **error)
         fseek(f, 0, SEEK_SET);
         c_list_for_each (current, &s->lst_head) {
             const shvarLine *line = c_list_entry(current, shvarLine, lst);
-            const char *     str;
-            char *           s_tmp;
+            const char      *str;
+            char            *s_tmp;
             gboolean         valid_value;
 
             ASSERT_shvarLine(line);
diff --git a/src/core/settings/plugins/ifcfg-rh/shvar.h b/src/core/settings/plugins/ifcfg-rh/shvar.h
index cf91642f..4b838eea 100644
--- a/src/core/settings/plugins/ifcfg-rh/shvar.h
+++ b/src/core/settings/plugins/ifcfg-rh/shvar.h
@@ -43,10 +43,10 @@ const char *svFindFirstNumberedKey(shvarFile *s, const char *key_prefix);
  * be freed by the caller.
  */
 const char *svGetValue(shvarFile *s, const char *key, char **to_free);
-char *      svGetValue_cp(shvarFile *s, const char *key);
+char       *svGetValue_cp(shvarFile *s, const char *key);
 
 const char *svGetValueStr(shvarFile *s, const char *key, char **to_free);
-char *      svGetValueStr_cp(shvarFile *s, const char *key);
+char       *svGetValueStr_cp(shvarFile *s, const char *key);
 
 int svParseBoolean(const char *value, int def);
 
diff --git a/src/core/settings/plugins/ifcfg-rh/tests/test-ifcfg-rh.c b/src/core/settings/plugins/ifcfg-rh/tests/test-ifcfg-rh.c
index b1cfdfec..7960ea2a 100644
--- a/src/core/settings/plugins/ifcfg-rh/tests/test-ifcfg-rh.c
+++ b/src/core/settings/plugins/ifcfg-rh/tests/test-ifcfg-rh.c
@@ -61,8 +61,8 @@
 
 #define _svOpenFile(testfile)                \
     ({                                       \
-        shvarFile * _f;                      \
-        GError *    _error    = NULL;        \
+        shvarFile  *_f;                      \
+        GError     *_error    = NULL;        \
         const char *_testfile = (testfile);  \
                                              \
         g_assert(_testfile);                 \
@@ -74,9 +74,9 @@
 #define _svGetValue_check(f, key, expected_value)                   \
     G_STMT_START                                                    \
     {                                                               \
-        const char *      _val;                                     \
-        gs_free char *    _to_free    = NULL;                       \
-        gs_free char *    _val_string = NULL;                       \
+        const char       *_val;                                     \
+        gs_free char     *_to_free    = NULL;                       \
+        gs_free char     *_val_string = NULL;                       \
         shvarFile *const  _f          = (f);                        \
         const char *const _key        = (key);                      \
                                                                     \
@@ -98,8 +98,8 @@ _assert_reread_same(NMConnection *connection, NMConnection *reread)
 static void
 _assert_reread_same_FIXME(NMConnection *connection, NMConnection *reread)
 {
-    gs_unref_object NMConnection *connection_normalized = NULL;
-    gs_unref_hashtable GHashTable *settings             = NULL;
+    gs_unref_object NMConnection  *connection_normalized = NULL;
+    gs_unref_hashtable GHashTable *settings              = NULL;
 
     /* FIXME: these assertion failures should not happen as we expect
      * that re-reading a connection after write yields the same result.
@@ -125,11 +125,11 @@ _assert_expected_content(NMConnection *connection, const char *filename, const c
 {
     gs_free char *content_expectd = NULL;
     gs_free char *content_written = NULL;
-    GError *      error           = NULL;
+    GError       *error           = NULL;
     gsize         len_expectd     = 0;
     gsize         len_written     = 0;
     gboolean      success;
-    const char *  uuid = NULL;
+    const char   *uuid = NULL;
 
     g_assert(NM_IS_CONNECTION(connection));
     g_assert(filename);
@@ -196,9 +196,9 @@ _assert_expected_content(NMConnection *connection, const char *filename, const c
         if (rewrite > 0) {
             if (uuid) {
                 gs_free char *search = g_strdup_printf("UUID=%s\n", uuid);
-                const char *  s;
+                const char   *s;
                 gsize         i;
-                GString *     str;
+                GString      *str;
 
                 s = content_written;
                 while (TRUE) {
@@ -246,11 +246,11 @@ _assert_expected_content(NMConnection *connection, const char *filename, const c
         gs_unref_object NMConnection *_connection =                       \
             nmtst_connection_duplicate_and_normalize(connection);         \
         NMConnection **_out_reread      = (out_reread);                   \
-        gboolean *     _out_reread_same = (out_reread_same);              \
-        const char *   _ifcfg_dir       = (ifcfg_dir);                    \
-        const char *   _filename        = (filename);                     \
-        const char *   _expected        = (expected);                     \
-        GError *       _error           = NULL;                           \
+        gboolean      *_out_reread_same = (out_reread_same);              \
+        const char    *_ifcfg_dir       = (ifcfg_dir);                    \
+        const char    *_filename        = (filename);                     \
+        const char    *_expected        = (expected);                     \
+        GError        *_error           = NULL;                           \
         gboolean       _success;                                          \
                                                                           \
         g_assert(_ifcfg_dir &&_ifcfg_dir[0]);                             \
@@ -274,7 +274,7 @@ _assert_expected_content(NMConnection *connection, const char *filename, const c
     G_STMT_START                                                             \
     {                                                                        \
         gs_unref_object NMConnection *_reread      = NULL;                   \
-        NMConnection *                _c           = (connection);           \
+        NMConnection                 *_c           = (connection);           \
         gboolean                      _reread_same = FALSE;                  \
                                                                              \
         _writer_update_connection_reread(_c,                                 \
@@ -292,11 +292,11 @@ static NMConnection *
 _connection_from_file(const char *filename,
                       const char *network_file,
                       const char *test_type,
-                      char **     out_unhandled)
+                      char      **out_unhandled)
 {
     NMConnection *connection;
-    GError *      error              = NULL;
-    char *        unhandled_fallback = NULL;
+    GError       *error              = NULL;
+    char         *unhandled_fallback = NULL;
 
     g_assert(!out_unhandled || !*out_unhandled);
 
@@ -319,11 +319,11 @@ static void
 _connection_from_file_fail(const char *filename,
                            const char *network_file,
                            const char *test_type,
-                           GError **   error)
+                           GError    **error)
 {
     NMConnection *connection;
-    GError *      local     = NULL;
-    char *        unhandled = NULL;
+    GError       *local     = NULL;
+    char         *unhandled = NULL;
 
     connection = nmtst_connection_from_file(filename, network_file, test_type, &unhandled, &local);
 
@@ -334,16 +334,16 @@ _connection_from_file_fail(const char *filename,
 }
 
 static void
-_writer_new_connection_reread(NMConnection * connection,
-                              const char *   ifcfg_dir,
-                              char **        out_filename,
-                              const char *   expected,
+_writer_new_connection_reread(NMConnection  *connection,
+                              const char    *ifcfg_dir,
+                              char         **out_filename,
+                              const char    *expected,
                               NMConnection **out_reread,
-                              gboolean *     out_reread_same)
+                              gboolean      *out_reread_same)
 {
-    gboolean        success;
-    GError *        error                      = NULL;
-    gs_free char *  filename                   = NULL;
+    gboolean                      success;
+    GError                       *error        = NULL;
+    gs_free char                 *filename     = NULL;
     gs_unref_object NMConnection *con_verified = NULL;
     gs_unref_object NMConnection *reread_copy  = NULL;
     NMConnection **reread = out_reread ?: ((nmtst_get_rand_uint32() % 2) ? &reread_copy : NULL);
@@ -378,9 +378,9 @@ _writer_new_connection_reread(NMConnection * connection,
 
 static void
 _writer_new_connec_exp(NMConnection *connection,
-                       const char *  ifcfg_dir,
-                       const char *  expected,
-                       char **       out_filename)
+                       const char   *ifcfg_dir,
+                       const char   *expected,
+                       char        **out_filename)
 {
     gs_unref_object NMConnection *reread      = NULL;
     gboolean                      reread_same = FALSE;
@@ -425,8 +425,8 @@ _writer_new_connection_fail(NMConnection *connection, const char *ifcfg_dir, GEr
     gs_unref_object NMConnection *connection_normalized = NULL;
     gs_unref_object NMConnection *reread                = NULL;
     gboolean                      success;
-    GError *                      local    = NULL;
-    char *                        filename = NULL;
+    GError                       *local    = NULL;
+    char                         *filename = NULL;
 
     g_assert(NM_IS_CONNECTION(connection));
     g_assert(ifcfg_dir);
@@ -454,13 +454,13 @@ _writer_new_connection_fail(NMConnection *connection, const char *ifcfg_dir, GEr
 static void
 test_read_netmask_1(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
-    gs_free char *                content    = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    NMIPAddress *                 ip4_addr;
-    const char *                  FILENAME = TEST_IFCFG_DIR "/ifcfg-netmask-1";
+    gs_free char                 *content    = NULL;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    NMIPAddress                  *ip4_addr;
+    const char                   *FILENAME = TEST_IFCFG_DIR "/ifcfg-netmask-1";
 
     connection = _connection_from_file(FILENAME, NULL, TYPE_ETHERNET, NULL);
 
@@ -492,14 +492,14 @@ test_read_netmask_1(void)
 
 static gboolean
 verify_cert_or_key(NMSetting8021x *s_compare,
-                   const char *    file,
-                   const char *    privkey_password,
-                   const char *    property)
+                   const char     *file,
+                   const char     *privkey_password,
+                   const char     *property)
 {
     gs_unref_object NMSetting8021x *s_8021x  = NULL;
-    GError *                        error    = NULL;
+    GError                         *error    = NULL;
     gboolean                        success  = FALSE;
-    const char *                    expected = NULL, *setting = NULL;
+    const char                     *expected = NULL, *setting = NULL;
     gboolean                        phase2 = FALSE;
     NMSetting8021xCKScheme          scheme = NM_SETTING_802_1X_CK_SCHEME_UNKNOWN;
 
@@ -617,11 +617,11 @@ static void
 test_read_basic(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    const char *                  mac;
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    const char                   *mac;
     char expected_mac_address[ETH_ALEN] = {0x00, 0x16, 0x41, 0x11, 0x22, 0x33};
 
     connection =
@@ -662,10 +662,10 @@ static void
 test_read_miscellaneous_variables(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
-    NMSettingIPConfig *           s_ip4;
-    char *                        expected_mac_blacklist[3] = {"00:16:41:11:22:88",
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
+    NMSettingIPConfig            *s_ip4;
+    char                         *expected_mac_blacklist[3] = {"00:16:41:11:22:88",
                                        "00:16:41:11:22:99",
                                        "6a:5d:5a:fa:dd:f0"};
     int                           mac_blacklist_num, i;
@@ -705,10 +705,10 @@ static void
 test_read_variables_corner_cases(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
-    NMSettingIPConfig *           s_ip4;
-    const char *                  mac;
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
+    NMSettingIPConfig            *s_ip4;
+    const char                   *mac;
     char expected_mac_address[ETH_ALEN] = {0x00, 0x16, 0x41, 0x11, 0x22, 0x33};
 
     NMTST_EXPECT_NM_WARN("*key NAME is badly quoted and is treated as \"\"*");
@@ -745,8 +745,8 @@ static void
 test_read_unmanaged(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    gs_free char *                unhandled_spec     = NULL;
+    NMSettingConnection          *s_con;
+    gs_free char                 *unhandled_spec     = NULL;
     guint64                       expected_timestamp = 0;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-nm-controlled",
@@ -765,8 +765,8 @@ static void
 test_read_unmanaged_unrecognized(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    gs_free char *                unhandled_spec     = NULL;
+    NMSettingConnection          *s_con;
+    gs_free char                 *unhandled_spec     = NULL;
     guint64                       expected_timestamp = 0;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-nm-controlled-unrecognized",
@@ -784,8 +784,8 @@ static void
 test_read_unrecognized(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    gs_free char *                unhandled_spec     = NULL;
+    NMSettingConnection          *s_con;
+    gs_free char                 *unhandled_spec     = NULL;
     guint64                       expected_timestamp = 0;
 
     NMTST_EXPECT_NM_WARN("*key NAME is badly quoted and is treated as \"\"*");
@@ -805,16 +805,16 @@ static void
 test_read_wired_static(gconstpointer test_data)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    char *                        unmanaged = NULL;
-    const char *                  mac;
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    char                         *unmanaged = NULL;
+    const char                   *mac;
     char         expected_mac_address[ETH_ALEN] = {0x00, 0x11, 0x22, 0x33, 0x44, 0xee};
     NMIPAddress *ip4_addr;
     NMIPAddress *ip6_addr;
-    const char * file, *expected_id;
+    const char  *file, *expected_id;
     gpointer     expect_ip6_p;
 
     nmtst_test_data_unpack(test_data, &file, &expected_id, &expect_ip6_p);
@@ -895,13 +895,13 @@ test_read_wired_static(gconstpointer test_data)
 static void
 test_read_wired_static_no_prefix(gconstpointer user_data)
 {
-    guint32         expected_prefix          = GPOINTER_TO_UINT(user_data);
-    gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    NMIPAddress *                 ip4_addr;
-    gs_free char *                file        = NULL;
-    gs_free char *                expected_id = NULL;
+    guint32                       expected_prefix = GPOINTER_TO_UINT(user_data);
+    gs_unref_object NMConnection *connection      = NULL;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    NMIPAddress                  *ip4_addr;
+    gs_free char                 *file        = NULL;
+    gs_free char                 *expected_id = NULL;
 
     file = g_strdup_printf(TEST_IFCFG_DIR "/ifcfg-test-wired-static-no-prefix-%u", expected_prefix);
     expected_id = g_strdup_printf("System test-wired-static-no-prefix-%u", expected_prefix);
@@ -931,11 +931,11 @@ static void
 test_read_wired_dhcp(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    char *                        unmanaged    = NULL;
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    char                         *unmanaged    = NULL;
     char        expected_mac_address[ETH_ALEN] = {0x00, 0x11, 0x22, 0x33, 0x44, 0xee};
     const char *mac;
 
@@ -983,10 +983,10 @@ static void
 test_read_wired_dhcp_plus_ip(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    NMIPAddress *                 ip4_addr;
-    NMIPAddress *                 ip6_addr;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    NMIPAddress                  *ip4_addr;
+    NMIPAddress                  *ip6_addr;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wired-dhcp-plus-ip",
                                        NULL,
@@ -1048,8 +1048,8 @@ static void
 test_read_wired_shared_plus_ip(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingIPConfig *           s_ip4;
-    NMIPAddress *                 ip4_addr;
+    NMSettingIPConfig            *s_ip4;
+    NMIPAddress                  *ip4_addr;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wired-shared-plus-ip",
                                        NULL,
@@ -1077,10 +1077,10 @@ static void
 test_read_wired_global_gateway(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    NMIPAddress *                 ip4_addr;
-    char *                        unmanaged = NULL;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    NMIPAddress                  *ip4_addr;
+    char                         *unmanaged = NULL;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wired-global-gateway",
                                        TEST_IFCFG_DIR "/network-test-wired-global-gateway",
@@ -1113,9 +1113,9 @@ static void
 test_read_wired_global_gateway_ignore(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    char *                        unmanaged = NULL;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    char                         *unmanaged = NULL;
 
     NMTST_EXPECT_NM_WARN("*ignoring GATEWAY (/etc/sysconfig/network) for * because the connection "
                          "has no static addresses");
@@ -1147,8 +1147,8 @@ static void
 test_read_wired_obsolete_gateway_n(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingIPConfig *           s_ip4;
-    NMIPAddress *                 ip4_addr;
+    NMSettingIPConfig            *s_ip4;
+    NMIPAddress                  *ip4_addr;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wired-obsolete-gateway-n",
                                        NULL,
@@ -1174,10 +1174,10 @@ test_read_wired_obsolete_gateway_n(void)
 static void
 test_user_1(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingUser *               s_user;
+    NMSettingUser                *s_user;
 
     connection =
         nmtst_create_minimal_connection("Test User 1", NULL, NM_SETTING_WIRED_SETTING_NAME, NULL);
@@ -1187,7 +1187,7 @@ test_user_1(void)
 #define _USER_SET_DATA(s_user, key, val)                                      \
     G_STMT_START                                                              \
     {                                                                         \
-        GError * _error = NULL;                                               \
+        GError  *_error = NULL;                                               \
         gboolean _success;                                                    \
                                                                               \
         _success = nm_setting_user_set_data((s_user), (key), (val), &_error); \
@@ -1232,8 +1232,8 @@ static void
 test_read_wired_never_default(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wired-never-default",
                                        TEST_IFCFG_DIR "/network-test-wired-never-default",
@@ -1256,10 +1256,10 @@ static void
 test_read_wired_defroute_no(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    char *                        unmanaged = NULL;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    char                         *unmanaged = NULL;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wired-defroute-no",
                                        NULL,
@@ -1285,10 +1285,10 @@ static void
 test_read_wired_defroute_no_gatewaydev_yes(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    char *                        unmanaged = NULL;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    char                         *unmanaged = NULL;
 
     connection =
         _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wired-defroute-no-gatewaydev-yes",
@@ -1316,9 +1316,9 @@ static void
 test_read_wired_static_routes(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    NMIPRoute *                   ip4_route;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    NMIPRoute                    *ip4_route;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wired-static-routes",
                                        NULL,
@@ -1391,10 +1391,10 @@ static void
 test_read_wired_static_routes_legacy(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    char *                        unmanaged = NULL;
-    NMIPRoute *                   ip4_route;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    char                         *unmanaged = NULL;
+    NMIPRoute                    *ip4_route;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wired-static-routes-legacy",
                                        NULL,
@@ -1466,11 +1466,11 @@ static void
 test_read_wired_ipv4_manual(gconstpointer data)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    char *                        unmanaged = NULL;
-    NMIPAddress *                 ip4_addr;
-    const char *                  file, *expected_id;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    char                         *unmanaged = NULL;
+    NMIPAddress                  *ip4_addr;
+    const char                   *file, *expected_id;
 
     nmtst_test_data_unpack(data, &file, &expected_id);
 
@@ -1516,12 +1516,12 @@ static void
 test_read_wired_ipv6_manual(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    char *                        unmanaged = NULL;
-    NMIPAddress *                 ip6_addr;
-    NMIPRoute *                   ip6_route;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    char                         *unmanaged = NULL;
+    NMIPAddress                  *ip6_addr;
+    NMIPRoute                    *ip6_route;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wired-ipv6-manual",
                                        NULL,
@@ -1621,13 +1621,13 @@ static void
 test_read_wired_ipv6_only(gconstpointer test_data)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    char *                        unmanaged = NULL;
-    NMIPAddress *                 ip6_addr;
-    const char *                  method;
-    const char *                  file, *expected_id;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    char                         *unmanaged = NULL;
+    NMIPAddress                  *ip6_addr;
+    const char                   *method;
+    const char                   *file, *expected_id;
 
     nmtst_test_data_unpack(test_data, &file, &expected_id);
 
@@ -1675,11 +1675,11 @@ static void
 test_read_wired_dhcp6_only(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    char *                        unmanaged = NULL;
-    const char *                  method;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    char                         *unmanaged = NULL;
+    const char                   *method;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wired-dhcp6-only",
                                        NULL,
@@ -1705,9 +1705,9 @@ static void
 test_read_wired_autoip(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    char *                        unmanaged = NULL;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    char                         *unmanaged = NULL;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wired-autoip",
                                        NULL,
@@ -1730,8 +1730,8 @@ static void
 test_read_onboot_no(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    char *                        unmanaged = NULL;
+    NMSettingConnection          *s_con;
+    char                         *unmanaged = NULL;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-onboot-no",
                                        NULL,
@@ -1748,8 +1748,8 @@ static void
 test_read_noip(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
 
     connection =
         _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-noip", NULL, TYPE_ETHERNET, NULL);
@@ -1772,15 +1772,15 @@ test_read_noip(void)
 static void
 test_read_wired_8021x_peap_mschapv2(void)
 {
-    gs_unref_object NMConnection *connection = NULL;
-    NMSettingIPConfig *           s_ip4;
-    NMSetting8021x *              s_8021x;
+    gs_unref_object NMConnection   *connection = NULL;
+    NMSettingIPConfig              *s_ip4;
+    NMSetting8021x                 *s_8021x;
     gs_unref_object NMSetting8021x *tmp_8021x = NULL;
-    char *                          unmanaged = NULL;
-    GError *                        error     = NULL;
+    char                           *unmanaged = NULL;
+    GError                         *error     = NULL;
     gboolean                        success   = FALSE;
-    const char *                    expected_ca_cert_path;
-    const char *                    read_ca_cert_path;
+    const char                     *expected_ca_cert_path;
+    const char                     *read_ca_cert_path;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wired-8021x-peap-mschapv2",
                                        NULL,
@@ -1827,11 +1827,11 @@ static void
 test_read_wired_8021x_tls_secret_flags(gconstpointer test_data)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSetting8021x *              s_8021x;
-    gs_free char *                dirname = NULL;
-    gs_free char *                tmp1    = NULL;
-    gs_free char *                tmp2    = NULL;
-    const char *                  ifcfg;
+    NMSetting8021x               *s_8021x;
+    gs_free char                 *dirname = NULL;
+    gs_free char                 *tmp1    = NULL;
+    gs_free char                 *tmp2    = NULL;
+    const char                   *ifcfg;
     gpointer                      expected_flags_p;
 
     nmtst_test_data_unpack(test_data, &ifcfg, &expected_flags_p);
@@ -1861,10 +1861,10 @@ test_read_wired_8021x_tls_secret_flags(gconstpointer test_data)
 static void
 test_read_write_802_1X_subj_matches(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSetting8021x *              s_8021x;
+    NMSetting8021x               *s_8021x;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wired-802-1X-subj-matches",
                                        NULL,
@@ -1926,7 +1926,7 @@ static void
 test_read_802_1x_ttls_eapgtc(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSetting8021x *              s_8021x;
+    NMSetting8021x               *s_8021x;
 
     /* Test that EAP-* inner methods are correctly read into the
      * NMSetting8021x::autheap property.
@@ -1952,8 +1952,8 @@ static void
 test_read_802_1x_tls_p12_no_client_cert(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSetting8021x *              s_8021x;
-    const char *                  path;
+    NMSetting8021x               *s_8021x;
+    const char                   *path;
 
     connection =
         _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wired-8021x-tls-p12-no-client-cert",
@@ -1978,12 +1978,12 @@ test_read_802_1x_tls_p12_no_client_cert(void)
 static void
 test_read_write_802_1x_password_raw(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
-    nmtst_auto_unlinkfile char *keyfile      = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
+    nmtst_auto_unlinkfile char   *keyfile    = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSetting8021x *              s_8021x;
-    GBytes *                      bytes;
+    NMSetting8021x               *s_8021x;
+    GBytes                       *bytes;
     gconstpointer                 data;
     gsize                         size;
 
@@ -2019,18 +2019,18 @@ test_read_write_802_1x_password_raw(void)
 static void
 test_read_wired_aliases_good(gconstpointer test_data)
 {
-    const int       N                        = GPOINTER_TO_INT(test_data);
+    const int                     N          = GPOINTER_TO_INT(test_data);
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
     int                           expected_num_addresses;
-    const char *                  expected_address_0[] = {"192.168.1.5",
+    const char                   *expected_address_0[] = {"192.168.1.5",
                                         "192.168.1.6",
                                         "192.168.1.9",
                                         "192.168.1.99",
                                         NULL};
-    const char *                  expected_address_3[] = {"192.168.1.5", "192.168.1.6", NULL};
-    const char *                  expected_label_0[]   = {
+    const char                   *expected_address_3[] = {"192.168.1.5", "192.168.1.6", NULL};
+    const char                   *expected_label_0[]   = {
         NULL,
         "aliasem0:1",
         "aliasem0:2",
@@ -2070,8 +2070,8 @@ test_read_wired_aliases_good(gconstpointer test_data)
     /* Addresses */
     for (i = 0; i < expected_num_addresses; i++) {
         NMIPAddress *ip4_addr;
-        const char * addr;
-        GVariant *   label;
+        const char  *addr;
+        GVariant    *label;
 
         ip4_addr = nm_setting_ip_config_get_address(s_ip4, i);
         g_assert(ip4_addr != NULL);
@@ -2108,9 +2108,9 @@ static void
 test_read_wired_aliases_bad(const char *base, const char *expected_id)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    NMIPAddress *                 ip4_addr;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    NMIPAddress                  *ip4_addr;
 
     g_assert(expected_id);
 
@@ -2156,12 +2156,12 @@ static void
 test_read_dns_options(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    char *                        unmanaged = NULL;
-    const char *                  option;
-    const char *                  options4[] = {"ndots:3", "single-request-reopen"};
-    const char *                  options6[] = {"inet6"};
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    char                         *unmanaged = NULL;
+    const char                   *option;
+    const char                   *options4[] = {"ndots:3", "single-request-reopen"};
+    const char                   *options6[] = {"inet6"};
     guint32                       num;
     guint32                       i;
 
@@ -2195,12 +2195,12 @@ test_read_dns_options(void)
 static void
 test_clear_master(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
-    gs_free char *              keyfile      = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
+    gs_free char                 *keyfile    = NULL;
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    char *                        unmanaged = NULL;
-    shvarFile *                   f;
+    NMSettingConnection          *s_con;
+    char                         *unmanaged = NULL;
+    shvarFile                    *f;
 
     /* 1. load the bridge slave connection from disk */
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-bridge-component",
@@ -2250,18 +2250,18 @@ test_clear_master(void)
 static void
 test_write_dns_options(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    static const char *           mac = "31:33:33:37:be:cd";
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    static const char            *mac = "31:33:33:37:be:cd";
     guint32                       mtu = 1492;
-    NMIPAddress *                 addr;
-    NMIPAddress *                 addr6;
-    GError *                      error = NULL;
+    NMIPAddress                  *addr;
+    NMIPAddress                  *addr6;
+    GError                       *error = NULL;
 
     connection = nm_simple_connection_new();
 
@@ -2328,12 +2328,12 @@ static void
 test_read_wifi_open(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    GBytes *                      ssid;
-    const char *                  mac;
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    GBytes                       *ssid;
+    const char                   *mac;
     char        expected_mac_address[ETH_ALEN] = {0x00, 0x16, 0x41, 0x11, 0x22, 0x33};
     const char *expected_ssid                  = "blahblah";
 
@@ -2385,8 +2385,8 @@ static void
 test_read_wifi_open_auto(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wifi-open-auto",
                                        NULL,
@@ -2406,10 +2406,10 @@ static void
 test_read_wifi_open_ssid_hex(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    GBytes *                      ssid;
-    const char *                  expected_ssid = "blahblah";
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    GBytes                       *ssid;
+    const char                   *expected_ssid = "blahblah";
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wifi-open-ssid-hex",
                                        NULL,
@@ -2435,10 +2435,10 @@ static void
 test_read_wifi_open_ssid_hex_bad(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    GBytes *                      ssid;
-    const char *                  expected_ssid = "0x626cxx";
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    GBytes                       *ssid;
+    const char                   *expected_ssid = "0x626cxx";
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wifi-open-ssid-bad-hex",
                                        NULL,
@@ -2470,10 +2470,10 @@ static void
 test_read_wifi_open_ssid_quoted(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    GBytes *                      ssid;
-    const char *                  expected_ssid = "foo\"bar\\";
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    GBytes                       *ssid;
+    const char                   *expected_ssid = "foo\"bar\\";
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wifi-open-ssid-quoted",
                                        NULL,
@@ -2499,14 +2499,14 @@ static void
 test_read_wifi_wep(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingWirelessSecurity *   s_wsec;
-    NMSettingIPConfig *           s_ip4;
-    GBytes *                      ssid;
-    const char *                  mac;
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    NMSettingWirelessSecurity    *s_wsec;
+    NMSettingIPConfig            *s_ip4;
+    GBytes                       *ssid;
+    const char                   *mac;
     char         expected_mac_address[ETH_ALEN] = {0x00, 0x16, 0x41, 0x11, 0x22, 0x33};
-    const char * expected_ssid                  = "blahblah";
+    const char  *expected_ssid                  = "blahblah";
     NMWepKeyType key_type;
 
     connection =
@@ -2566,12 +2566,12 @@ static void
 test_read_wifi_wep_adhoc(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingWirelessSecurity *   s_wsec;
-    NMSettingIPConfig *           s_ip4;
-    GBytes *                      ssid;
-    const char *                  expected_ssid = "blahblah";
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    NMSettingWirelessSecurity    *s_wsec;
+    NMSettingIPConfig            *s_ip4;
+    GBytes                       *ssid;
+    const char                   *expected_ssid = "blahblah";
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wifi-wep-adhoc",
                                        NULL,
@@ -2631,7 +2631,7 @@ static void
 test_read_wifi_wep_passphrase(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingWirelessSecurity *   s_wsec;
+    NMSettingWirelessSecurity    *s_wsec;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wifi-wep-passphrase",
                                        NULL,
@@ -2658,7 +2658,7 @@ static void
 test_read_wifi_wep_40_ascii(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingWirelessSecurity *   s_wsec;
+    NMSettingWirelessSecurity    *s_wsec;
     NMWepKeyType                  key_type;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wifi-wep-40-ascii",
@@ -2687,7 +2687,7 @@ static void
 test_read_wifi_wep_104_ascii(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingWirelessSecurity *   s_wsec;
+    NMSettingWirelessSecurity    *s_wsec;
     NMWepKeyType                  key_type;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wifi-wep-104-ascii",
@@ -2716,8 +2716,8 @@ static void
 test_read_wifi_leap(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWirelessSecurity *   s_wsec;
+    NMSettingConnection          *s_con;
+    NMSettingWirelessSecurity    *s_wsec;
 
     connection =
         _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wifi-leap", NULL, TYPE_WIRELESS, NULL);
@@ -2738,8 +2738,8 @@ static void
 test_read_wifi_leap_secret_flags(gconstpointer test_data)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingWirelessSecurity *   s_wsec;
-    const char *                  file;
+    NMSettingWirelessSecurity    *s_wsec;
+    const char                   *file;
     gpointer                      expected_flags_p;
 
     nmtst_test_data_unpack(test_data, &file, &expected_flags_p);
@@ -2773,12 +2773,12 @@ static void
 test_read_wifi_wpa_psk(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingWirelessSecurity *   s_wsec;
-    NMSettingIPConfig *           s_ip4;
-    GBytes *                      ssid;
-    const char *                  mac;
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    NMSettingWirelessSecurity    *s_wsec;
+    NMSettingIPConfig            *s_ip4;
+    GBytes                       *ssid;
+    const char                   *mac;
     char        expected_mac_address[ETH_ALEN] = {0x00, 0x16, 0x41, 0x11, 0x22, 0x33};
     const char *expected_ssid                  = "blahblah";
     guint32     n, i;
@@ -2887,11 +2887,11 @@ static void
 test_read_wifi_sae(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingWirelessSecurity *   s_wsec;
-    GBytes *                      ssid;
-    const char *                  expected_ssid = "blahblah";
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    NMSettingWirelessSecurity    *s_wsec;
+    GBytes                       *ssid;
+    const char                   *expected_ssid = "blahblah";
 
     connection =
         _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wifi-sae", NULL, TYPE_WIRELESS, NULL);
@@ -2926,11 +2926,11 @@ static void
 test_read_wifi_owe(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingWirelessSecurity *   s_wsec;
-    GBytes *                      ssid;
-    const char *                  expected_ssid = "blahblah_owe";
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    NMSettingWirelessSecurity    *s_wsec;
+    GBytes                       *ssid;
+    const char                   *expected_ssid = "blahblah_owe";
 
     connection =
         _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wifi-owe", NULL, TYPE_WIRELESS, NULL);
@@ -2965,8 +2965,8 @@ static void
 test_read_wifi_wpa_psk_2(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWirelessSecurity *   s_wsec;
+    NMSettingConnection          *s_con;
+    NMSettingWirelessSecurity    *s_wsec;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wifi-wpa-psk-2",
                                        NULL,
@@ -2988,8 +2988,8 @@ static void
 test_read_wifi_wpa_psk_unquoted(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWirelessSecurity *   s_wsec;
+    NMSettingConnection          *s_con;
+    NMSettingWirelessSecurity    *s_wsec;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wifi-wpa-psk-unquoted",
                                        NULL,
@@ -3022,10 +3022,10 @@ static void
 test_read_wifi_wpa_psk_adhoc(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingWirelessSecurity *   s_wsec;
-    NMSettingIPConfig *           s_ip4;
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    NMSettingWirelessSecurity    *s_wsec;
+    NMSettingIPConfig            *s_ip4;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wifi-wpa-psk-adhoc",
                                        NULL,
@@ -3064,12 +3064,12 @@ static void
 test_read_wifi_wpa_psk_hex(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingWirelessSecurity *   s_wsec;
-    NMSettingIPConfig *           s_ip4;
-    GBytes *                      ssid;
-    const char *                  expected_ssid = "blahblah";
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    NMSettingWirelessSecurity    *s_wsec;
+    NMSettingIPConfig            *s_ip4;
+    GBytes                       *ssid;
+    const char                   *expected_ssid = "blahblah";
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wifi-wpa-psk-hex",
                                        NULL,
@@ -3105,10 +3105,10 @@ static void
 test_read_wifi_wpa_eap_tls(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingIPConfig *           s_ip4;
-    NMSetting8021x *              s_8021x;
-    char *                        unmanaged                 = NULL;
-    const char *                  expected_privkey_password = "test1";
+    NMSettingIPConfig            *s_ip4;
+    NMSetting8021x               *s_8021x;
+    char                         *unmanaged                 = NULL;
+    const char                   *expected_privkey_password = "test1";
 
     NMTST_EXPECT_NM_WARN("*key ONBOOT is duplicated and the early occurrence ignored*");
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wifi-wpa-eap-tls",
@@ -3158,10 +3158,10 @@ static void
 test_read_wifi_wpa_eap_ttls_tls(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingIPConfig *           s_ip4;
-    NMSetting8021x *              s_8021x;
-    char *                        unmanaged                 = NULL;
-    const char *                  expected_privkey_password = "test1";
+    NMSettingIPConfig            *s_ip4;
+    NMSetting8021x               *s_8021x;
+    char                         *unmanaged                 = NULL;
+    const char                   *expected_privkey_password = "test1";
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wifi-wpa-eap-ttls-tls",
                                        NULL,
@@ -3216,7 +3216,7 @@ static void
 test_read_wifi_wpa_eap_suite_b_192_tls(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingWirelessSecurity *   s_wsec;
+    NMSettingWirelessSecurity    *s_wsec;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wifi-wpa-eap-suite-b-192-tls",
                                        NULL,
@@ -3233,8 +3233,8 @@ static void
 test_read_wifi_dynamic_wep_leap(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingWirelessSecurity *   s_wsec;
-    NMSetting8021x *              s_8021x;
+    NMSettingWirelessSecurity    *s_wsec;
+    NMSetting8021x               *s_8021x;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wifi-dynamic-wep-leap",
                                        NULL,
@@ -3272,10 +3272,10 @@ static void
 test_read_wifi_wep_eap_ttls_chap(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingWirelessSecurity *   s_wsec;
-    NMSettingIPConfig *           s_ip4;
-    NMSetting8021x *              s_8021x;
-    char *                        unmanaged = NULL;
+    NMSettingWirelessSecurity    *s_wsec;
+    NMSettingIPConfig            *s_ip4;
+    NMSetting8021x               *s_8021x;
+    char                         *unmanaged = NULL;
 
     NMTST_EXPECT_NM_WARN("*key ONBOOT is duplicated and the early occurrence ignored*");
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wifi-wep-eap-ttls-chap",
@@ -3314,8 +3314,8 @@ static void
 test_read_wired_wake_on_lan(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wired-wake-on-lan",
                                        NULL,
@@ -3340,8 +3340,8 @@ static void
 test_read_wired_auto_negotiate_off(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wired-wake-on-lan",
                                        NULL,
@@ -3364,8 +3364,8 @@ static void
 test_read_wired_auto_negotiate_on(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wired-auto-negotiate-on",
                                        NULL,
@@ -3388,8 +3388,8 @@ static void
 test_read_wired_unknown_ethtool_opt(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wired-unknown-ethtool-opt",
                                        NULL,
@@ -3418,8 +3418,8 @@ static void
 test_roundtrip_ethtool(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSetting *                   s_ethtool;
-    NMSetting *                   s_wired;
+    NMSetting                    *s_ethtool;
+    NMSetting                    *s_wired;
 
     connection = nmtst_create_minimal_connection("test_roundtrip_ethtool",
                                                  NULL,
@@ -3488,8 +3488,8 @@ static void
 test_read_wifi_hidden(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
 
     connection =
         _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wifi-hidden", NULL, TYPE_WIRELESS, NULL);
@@ -3506,13 +3506,13 @@ test_read_wifi_hidden(void)
 static void
 test_write_wifi_hidden(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    shvarFile *                   f;
-    gs_unref_bytes GBytes *ssid =
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    shvarFile                    *f;
+    gs_unref_bytes GBytes        *ssid =
         nmtst_gbytes_from_arr(0x54, 0x65, 0x73, 0x74, 0x20, 0x53, 0x53, 0x49, 0x44);
 
     connection = nm_simple_connection_new();
@@ -3550,9 +3550,9 @@ static void
 test_read_wifi_mac_random(gconstpointer user_data)
 {
     gs_unref_object NMConnection *connection = NULL;
-    gs_free char *                path       = NULL;
-    NMSettingWireless *           s_wifi;
-    const char *                  name;
+    gs_free char                 *path       = NULL;
+    NMSettingWireless            *s_wifi;
+    const char                   *name;
     gpointer                      value_p;
     NMSettingMacRandomization     value;
 
@@ -3569,16 +3569,16 @@ test_read_wifi_mac_random(gconstpointer user_data)
 static void
 test_write_wifi_mac_random(gconstpointer user_data)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    gs_free char *                val = NULL;
-    shvarFile *                   f;
-    gs_unref_bytes GBytes *ssid =
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    gs_free char                 *val = NULL;
+    shvarFile                    *f;
+    gs_unref_bytes GBytes        *ssid =
         nmtst_gbytes_from_arr(0x54, 0x65, 0x73, 0x74, 0x20, 0x53, 0x53, 0x49, 0x44);
-    const char *              name, *write_expected;
+    const char               *name, *write_expected;
     gpointer                  value_p;
     NMSettingMacRandomization value;
     char                      cexpected[NM_STRLEN(TEST_IFCFG_DIR) + 100];
@@ -3632,13 +3632,13 @@ test_write_wifi_mac_random(gconstpointer user_data)
 static void
 test_write_wired_wake_on_lan(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
-    gs_free char *                val = NULL;
-    shvarFile *                   f;
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
+    gs_free char                 *val = NULL;
+    shvarFile                    *f;
 
     connection = nm_simple_connection_new();
 
@@ -3684,12 +3684,12 @@ test_write_wired_wake_on_lan(void)
 static void
 test_write_wired_auto_negotiate_off(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingWired *              s_wired;
-    gs_free char *                val = NULL;
-    shvarFile *                   f;
+    NMSettingWired               *s_wired;
+    gs_free char                 *val = NULL;
+    shvarFile                    *f;
 
     connection = nmtst_create_minimal_connection("Test Write Wired Auto-Negotiate",
                                                  NULL,
@@ -3726,13 +3726,13 @@ test_write_wired_auto_negotiate_off(void)
 static void
 test_write_wired_auto_negotiate_on(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingWired *              s_wired;
-    NMSettingEthtool *            s_ethtool;
-    gs_free char *                val = NULL;
-    shvarFile *                   f;
+    NMSettingWired               *s_wired;
+    NMSettingEthtool             *s_ethtool;
+    gs_free char                 *val = NULL;
+    shvarFile                    *f;
 
     connection = nmtst_create_minimal_connection("Test Write Wired Auto-Negotiate",
                                                  NULL,
@@ -3780,8 +3780,8 @@ static void
 test_read_wifi_band_a(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
 
     connection =
         _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wifi-band-a", NULL, TYPE_WIRELESS, NULL);
@@ -3798,13 +3798,13 @@ test_read_wifi_band_a(void)
 static void
 test_write_wifi_band_a(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    shvarFile *                   f;
-    gs_unref_bytes GBytes *ssid =
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    shvarFile                    *f;
+    gs_unref_bytes GBytes        *ssid =
         nmtst_gbytes_from_arr(0x54, 0x65, 0x73, 0x74, 0x20, 0x53, 0x53, 0x49, 0x44);
 
     connection = nm_simple_connection_new();
@@ -3848,12 +3848,12 @@ test_write_wifi_band_a(void)
 static void
 test_write_wifi_ap_mode(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    gs_unref_bytes GBytes *ssid = nmtst_gbytes_from_str("MySSID");
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    gs_unref_bytes GBytes        *ssid = nmtst_gbytes_from_str("MySSID");
 
     connection = nm_simple_connection_new();
 
@@ -3921,11 +3921,11 @@ static void
 test_read_wired_qeth_static(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
-    NMSettingIPConfig *           s_ip4;
-    char *                        unmanaged = NULL;
-    const char *const *           subchannels;
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
+    NMSettingIPConfig            *s_ip4;
+    char                         *unmanaged = NULL;
+    const char *const            *subchannels;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wired-qeth-static",
                                        NULL,
@@ -3964,10 +3964,10 @@ static void
 test_read_wired_ctc_static(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
-    char *                        unmanaged = NULL;
-    const char *const *           subchannels;
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
+    char                         *unmanaged = NULL;
+    const char *const            *subchannels;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wired-ctc-static",
                                        NULL,
@@ -3998,8 +3998,8 @@ static void
 test_read_wifi_wep_no_keys(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWirelessSecurity *   s_wsec;
+    NMSettingConnection          *s_con;
+    NMSettingWirelessSecurity    *s_wsec;
     NMWepKeyType                  key_type;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wifi-wep-no-keys",
@@ -4039,10 +4039,10 @@ static void
 test_read_permissions(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
+    NMSettingConnection          *s_con;
     gboolean                      success;
     guint32                       num;
-    const char *                  tmp;
+    const char                   *tmp;
 
     connection =
         _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-permissions", NULL, TYPE_ETHERNET, NULL);
@@ -4073,7 +4073,7 @@ static void
 test_read_wifi_wep_agent_keys(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingWirelessSecurity *   s_wsec;
+    NMSettingWirelessSecurity    *s_wsec;
     NMWepKeyType                  key_type;
     NMSettingSecretFlags          flags;
 
@@ -4106,18 +4106,18 @@ test_read_wifi_wep_agent_keys(void)
 static void
 test_write_wired_static(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
-    nmtst_auto_unlinkfile char *route6file   = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
+    nmtst_auto_unlinkfile char   *route6file = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
-    NMSettingIPConfig *           s_ip4, *reread_s_ip4;
-    NMSettingIPConfig *           s_ip6, *reread_s_ip6;
-    NMIPAddress *                 addr;
-    NMIPAddress *                 addr6;
-    NMIPRoute *                   route6;
-    GError *                      error = NULL;
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
+    NMSettingIPConfig            *s_ip4, *reread_s_ip4;
+    NMSettingIPConfig            *s_ip6, *reread_s_ip6;
+    NMIPAddress                  *addr;
+    NMIPAddress                  *addr6;
+    NMIPRoute                    *route6;
+    GError                       *error = NULL;
 
     connection = nm_simple_connection_new();
 
@@ -4249,18 +4249,18 @@ test_write_wired_static(void)
 static void
 test_write_wired_static_with_generic(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
-    nmtst_auto_unlinkfile char *route6file   = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
+    nmtst_auto_unlinkfile char   *route6file = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
-    NMSettingIPConfig *           s_ip4, *reread_s_ip4;
-    NMSettingIPConfig *           s_ip6, *reread_s_ip6;
-    NMIPAddress *                 addr;
-    NMIPAddress *                 addr6;
-    NMIPRoute *                   route6;
-    GError *                      error = NULL;
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
+    NMSettingIPConfig            *s_ip4, *reread_s_ip4;
+    NMSettingIPConfig            *s_ip6, *reread_s_ip6;
+    NMIPAddress                  *addr;
+    NMIPAddress                  *addr6;
+    NMIPRoute                    *route6;
+    GError                       *error = NULL;
 
     connection = nm_simple_connection_new();
 
@@ -4411,12 +4411,12 @@ test_write_wired_static_with_generic(void)
 static void
 test_write_wired_dhcp(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
 
     connection = nm_simple_connection_new();
 
@@ -4477,8 +4477,8 @@ static NMIPRoutingRule *
 _ip_routing_rule_new(int addr_family, const char *str)
 {
     NMIPRoutingRuleAsStringFlags flags = NM_IP_ROUTING_RULE_AS_STRING_FLAGS_NONE;
-    gs_free_error GError *local        = NULL;
-    NMIPRoutingRule *     rule;
+    gs_free_error GError        *local = NULL;
+    NMIPRoutingRule             *rule;
 
     if (addr_family != AF_UNSPEC) {
         if (addr_family == AF_INET)
@@ -4512,12 +4512,12 @@ _ip_routing_rule_add_to_setting(NMSettingIPConfig *s_ip, const char *str)
 static void
 test_write_routing_rules(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
 
     connection = nm_simple_connection_new();
 
@@ -4560,13 +4560,13 @@ test_write_routing_rules(void)
 static void
 test_write_wired_match(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingMatch *              s_match;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
+    NMSettingConnection          *s_con;
+    NMSettingMatch               *s_match;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
 
     connection = nm_simple_connection_new();
 
@@ -4615,7 +4615,7 @@ test_write_wired_match(void)
 static void
 test_write_wired_dhcp_plus_ip(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
 
@@ -4634,12 +4634,12 @@ test_write_wired_dhcp_plus_ip(void)
 static void
 test_read_write_wired_dhcp_send_hostname(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    const char *                  dhcp_hostname = "kamil-patka";
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    const char                   *dhcp_hostname = "kamil-patka";
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wired-dhcp-send-hostname",
                                        NULL,
@@ -4676,7 +4676,7 @@ static void
 test_read_wired_dhcpv6_hostname_fallback(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingIPConfig *           s_ip6;
+    NMSettingIPConfig            *s_ip6;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-wired-dhcpv6-hostname-fallback",
                                        NULL,
@@ -4691,15 +4691,15 @@ test_read_wired_dhcpv6_hostname_fallback(void)
 static void
 test_write_wired_static_ip6_only(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    NMIPAddress *                 addr6;
-    GError *                      error = NULL;
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    NMIPAddress                  *addr6;
+    GError                       *error = NULL;
 
     connection = nm_simple_connection_new();
 
@@ -4745,12 +4745,12 @@ test_write_wired_static_ip6_only(void)
 static void
 test_write_ip6_disabled(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
 
     connection = nmtst_create_minimal_connection("Test Write Wired Disabled IP6",
                                                  NULL,
@@ -4792,19 +4792,19 @@ test_write_ip6_disabled(void)
 static void
 test_write_wired_static_ip6_only_gw(gconstpointer user_data)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    NMIPAddress *                 addr6;
-    GError *                      error                 = NULL;
-    gs_free char *                id                    = NULL;
-    gs_free char *                written_ifcfg_gateway = NULL;
-    const char *                  gateway6              = user_data;
-    shvarFile *                   ifcfg;
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    NMIPAddress                  *addr6;
+    GError                       *error                 = NULL;
+    gs_free char                 *id                    = NULL;
+    gs_free char                 *written_ifcfg_gateway = NULL;
+    const char                   *gateway6              = user_data;
+    shvarFile                    *ifcfg;
 
     connection = nm_simple_connection_new();
 
@@ -4877,14 +4877,14 @@ test_write_wired_static_ip6_only_gw(gconstpointer user_data)
 static void
 test_read_write_static_routes_legacy(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
-    nmtst_auto_unlinkfile char *routefile    = NULL;
-    gs_free char *              route6file   = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
+    nmtst_auto_unlinkfile char   *routefile  = NULL;
+    gs_free char                 *route6file = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    const char *                  tmp;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    const char                   *tmp;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-static-routes-legacy",
                                        NULL,
@@ -4927,17 +4927,17 @@ test_read_write_static_routes_legacy(void)
 static void
 test_write_wired_static_routes(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
-    nmtst_auto_unlinkfile char *routefile    = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
+    nmtst_auto_unlinkfile char   *routefile  = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    NMIPAddress *                 addr;
-    NMIPRoute *                   route;
-    GError *                      error       = NULL;
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    NMIPAddress                  *addr;
+    NMIPRoute                    *route;
+    GError                       *error       = NULL;
     gboolean                      reread_same = FALSE;
 
     connection = nm_simple_connection_new();
@@ -5041,16 +5041,16 @@ test_write_wired_static_routes(void)
 static void
 test_write_wired_dhcp_8021x_peap_mschapv2(void)
 {
-    nmtst_auto_unlinkfile char *keyfile      = NULL;
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *keyfile    = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    NMSetting8021x *              s_8021x;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    NMSetting8021x               *s_8021x;
     gboolean                      success;
-    GError *                      error = NULL;
+    GError                       *error = NULL;
 
     connection = nm_simple_connection_new();
 
@@ -5119,21 +5119,21 @@ test_write_wired_dhcp_8021x_peap_mschapv2(void)
 static void
 test_write_wired_8021x_tls(gconstpointer test_data)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
-    gs_free char *              keyfile      = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
+    gs_free char                 *keyfile    = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    NMSetting8021x *              s_8021x;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    NMSetting8021x               *s_8021x;
     gboolean                      success;
-    GError *                      error  = NULL;
+    GError                       *error  = NULL;
     NMSetting8021xCKFormat        format = NM_SETTING_802_1X_CK_FORMAT_UNKNOWN;
-    const char *                  pw;
-    gs_free char *                tmp1 = NULL;
-    gs_free char *                tmp2 = NULL;
-    gs_free char *                tmp3 = NULL;
+    const char                   *pw;
+    gs_free char                 *tmp1 = NULL;
+    gs_free char                 *tmp2 = NULL;
+    gs_free char                 *tmp3 = NULL;
     gpointer                      scheme_p, flags_p;
     NMSetting8021xCKScheme        scheme;
     NMSettingSecretFlags          flags;
@@ -5277,17 +5277,17 @@ test_write_wired_8021x_tls(gconstpointer test_data)
 static void
 test_write_wired_aliases(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
     int                           num_addresses = 4;
-    const char *                  ip[]          = {"1.1.1.1", "1.1.1.2", "1.1.1.3", "1.1.1.4"};
-    const char *                  label[]       = {NULL, "alias0:2", NULL, "alias0:3"};
-    NMIPAddress *                 addr;
-    GError *                      error = NULL;
-    shvarFile *                   ifcfg;
+    const char                   *ip[]          = {"1.1.1.1", "1.1.1.2", "1.1.1.3", "1.1.1.4"};
+    const char                   *label[]       = {NULL, "alias0:2", NULL, "alias0:3"};
+    NMIPAddress                  *addr;
+    GError                       *error = NULL;
+    shvarFile                    *ifcfg;
     int                           i, j;
 
     nmtst_file_unlink_if_exists(TEST_SCRATCH_ALIAS_BASE ":2");
@@ -5404,14 +5404,14 @@ test_write_wired_aliases(void)
 static void
 test_write_gateway(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    GError *                      error = NULL;
-    shvarFile *                   f;
-    NMIPAddress *                 addr;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    GError                       *error = NULL;
+    shvarFile                    *f;
+    NMIPAddress                  *addr;
 
     connection = nm_simple_connection_new();
 
@@ -5471,14 +5471,14 @@ test_write_gateway(void)
 static void
 test_write_wifi_open(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    gs_unref_bytes GBytes *ssid =
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    gs_unref_bytes GBytes        *ssid =
         nmtst_gbytes_from_arr(0x54, 0x65, 0x73, 0x74, 0x20, 0x53, 0x53, 0x49, 0x44);
     shvarFile *ifcfg;
 
@@ -5543,14 +5543,14 @@ test_write_wifi_open(void)
 static void
 test_write_wifi_open_hex_ssid(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    gs_unref_bytes GBytes *ssid = nmtst_gbytes_from_arr(0x11,
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    gs_unref_bytes GBytes        *ssid = nmtst_gbytes_from_arr(0x11,
                                                         0x22,
                                                         0x33,
                                                         0x44,
@@ -5609,17 +5609,17 @@ test_write_wifi_open_hex_ssid(void)
 static void
 test_write_wifi_wep(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
-    nmtst_auto_unlinkfile char *keyfile      = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
+    nmtst_auto_unlinkfile char   *keyfile    = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingWirelessSecurity *   s_wsec;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    gs_unref_bytes GBytes *ssid = nmtst_gbytes_from_str("blahblah");
-    struct stat            statbuf;
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    NMSettingWirelessSecurity    *s_wsec;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    gs_unref_bytes GBytes        *ssid = nmtst_gbytes_from_str("blahblah");
+    struct stat                   statbuf;
 
     connection = nm_simple_connection_new();
 
@@ -5687,19 +5687,19 @@ test_write_wifi_wep(void)
 static void
 test_write_wifi_wep_adhoc(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
-    nmtst_auto_unlinkfile char *keyfile      = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
+    nmtst_auto_unlinkfile char   *keyfile    = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingWirelessSecurity *   s_wsec;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    GError *                      error = NULL;
-    gs_unref_bytes GBytes *ssid         = nmtst_gbytes_from_str("blahblah");
-    struct stat            statbuf;
-    NMIPAddress *          addr;
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    NMSettingWirelessSecurity    *s_wsec;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    GError                       *error = NULL;
+    gs_unref_bytes GBytes        *ssid  = nmtst_gbytes_from_str("blahblah");
+    struct stat                   statbuf;
+    NMIPAddress                  *addr;
 
     connection = nm_simple_connection_new();
 
@@ -5763,17 +5763,17 @@ test_write_wifi_wep_adhoc(void)
 static void
 test_write_wifi_wep_passphrase(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
-    nmtst_auto_unlinkfile char *keyfile      = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
+    nmtst_auto_unlinkfile char   *keyfile    = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingWirelessSecurity *   s_wsec;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    gs_unref_bytes GBytes *ssid = nmtst_gbytes_from_str("blahblah");
-    struct stat            statbuf;
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    NMSettingWirelessSecurity    *s_wsec;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    gs_unref_bytes GBytes        *ssid = nmtst_gbytes_from_str("blahblah");
+    struct stat                   statbuf;
 
     connection = nm_simple_connection_new();
 
@@ -5838,17 +5838,17 @@ test_write_wifi_wep_passphrase(void)
 static void
 test_write_wifi_wep_40_ascii(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
-    nmtst_auto_unlinkfile char *keyfile      = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
+    nmtst_auto_unlinkfile char   *keyfile    = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingWirelessSecurity *   s_wsec;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    gs_unref_bytes GBytes *ssid = nmtst_gbytes_from_str("blahblah40");
-    struct stat            statbuf;
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    NMSettingWirelessSecurity    *s_wsec;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    gs_unref_bytes GBytes        *ssid = nmtst_gbytes_from_str("blahblah40");
+    struct stat                   statbuf;
 
     connection = nm_simple_connection_new();
 
@@ -5916,17 +5916,17 @@ test_write_wifi_wep_40_ascii(void)
 static void
 test_write_wifi_wep_104_ascii(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
-    nmtst_auto_unlinkfile char *keyfile      = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
+    nmtst_auto_unlinkfile char   *keyfile    = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingWirelessSecurity *   s_wsec;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    gs_unref_bytes GBytes *ssid = nmtst_gbytes_from_str("blahblah104");
-    struct stat            statbuf;
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    NMSettingWirelessSecurity    *s_wsec;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    gs_unref_bytes GBytes        *ssid = nmtst_gbytes_from_str("blahblah104");
+    struct stat                   statbuf;
 
     connection = nm_simple_connection_new();
 
@@ -5997,17 +5997,17 @@ test_write_wifi_wep_104_ascii(void)
 static void
 test_write_wifi_leap(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
-    nmtst_auto_unlinkfile char *keyfile      = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
+    nmtst_auto_unlinkfile char   *keyfile    = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingWirelessSecurity *   s_wsec;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    gs_unref_bytes GBytes *ssid = nmtst_gbytes_from_str("blahblah");
-    struct stat            statbuf;
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    NMSettingWirelessSecurity    *s_wsec;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    gs_unref_bytes GBytes        *ssid = nmtst_gbytes_from_str("blahblah");
+    struct stat                   statbuf;
 
     connection = nm_simple_connection_new();
 
@@ -6074,17 +6074,17 @@ test_write_wifi_leap(void)
 static void
 test_write_wifi_leap_secret_flags(gconstpointer data)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
-    gs_free char *              keyfile      = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
+    gs_free char                 *keyfile    = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingWirelessSecurity *   s_wsec;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    gs_unref_bytes GBytes *ssid  = nmtst_gbytes_from_str("blahblah");
-    NMSettingSecretFlags   flags = GPOINTER_TO_UINT(data);
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    NMSettingWirelessSecurity    *s_wsec;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    gs_unref_bytes GBytes        *ssid  = nmtst_gbytes_from_str("blahblah");
+    NMSettingSecretFlags          flags = GPOINTER_TO_UINT(data);
 
     connection = nm_simple_connection_new();
     g_assert(connection);
@@ -6152,16 +6152,16 @@ test_write_wifi_leap_secret_flags(gconstpointer data)
 static void
 test_write_wifi_wpa_psk(gconstpointer test_data)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
-    nmtst_auto_unlinkfile char *keyfile      = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
+    nmtst_auto_unlinkfile char   *keyfile    = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingWirelessSecurity *   s_wsec;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    gs_unref_bytes GBytes *ssid = nmtst_gbytes_from_str("blahblah");
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    NMSettingWirelessSecurity    *s_wsec;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    gs_unref_bytes GBytes        *ssid = nmtst_gbytes_from_str("blahblah");
     struct {
         const char *name, *psk;
         gpointer    wep_group_p, wpa_p, wpa2_p;
@@ -6248,18 +6248,18 @@ test_write_wifi_wpa_psk(gconstpointer test_data)
 static void
 test_write_wifi_wpa_psk_adhoc(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
-    nmtst_auto_unlinkfile char *keyfile      = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
+    nmtst_auto_unlinkfile char   *keyfile    = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingWirelessSecurity *   s_wsec;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    GError *                      error = NULL;
-    gs_unref_bytes GBytes *ssid         = nmtst_gbytes_from_str("blahblah");
-    NMIPAddress *          addr;
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    NMSettingWirelessSecurity    *s_wsec;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    GError                       *error = NULL;
+    gs_unref_bytes GBytes        *ssid  = nmtst_gbytes_from_str("blahblah");
+    NMIPAddress                  *addr;
 
     connection = nm_simple_connection_new();
 
@@ -6337,19 +6337,19 @@ test_write_wifi_wpa_psk_adhoc(void)
 static void
 test_write_wifi_wpa_eap_tls(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
-    nmtst_auto_unlinkfile char *keyfile      = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
+    nmtst_auto_unlinkfile char   *keyfile    = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingWirelessSecurity *   s_wsec;
-    NMSetting8021x *              s_8021x;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    NMSettingWirelessSecurity    *s_wsec;
+    NMSetting8021x               *s_8021x;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
     gboolean                      success;
-    GError *                      error = NULL;
-    gs_unref_bytes GBytes *ssid         = nmtst_gbytes_from_str("blahblah");
+    GError                       *error = NULL;
+    gs_unref_bytes GBytes        *ssid  = nmtst_gbytes_from_str("blahblah");
 
     connection = nm_simple_connection_new();
 
@@ -6441,19 +6441,19 @@ test_write_wifi_wpa_eap_tls(void)
 static void
 test_write_wifi_wpa_eap_ttls_tls(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
-    nmtst_auto_unlinkfile char *keyfile      = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
+    nmtst_auto_unlinkfile char   *keyfile    = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingWirelessSecurity *   s_wsec;
-    NMSetting8021x *              s_8021x;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    NMSettingWirelessSecurity    *s_wsec;
+    NMSetting8021x               *s_8021x;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
     gboolean                      success;
-    GError *                      error = NULL;
-    gs_unref_bytes GBytes *ssid         = nmtst_gbytes_from_str("blahblah");
+    GError                       *error = NULL;
+    gs_unref_bytes GBytes        *ssid  = nmtst_gbytes_from_str("blahblah");
 
     connection = nm_simple_connection_new();
 
@@ -6553,19 +6553,19 @@ test_write_wifi_wpa_eap_ttls_tls(void)
 static void
 test_write_wifi_wpa_eap_ttls_mschapv2(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
-    nmtst_auto_unlinkfile char *keyfile      = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
+    nmtst_auto_unlinkfile char   *keyfile    = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingWirelessSecurity *   s_wsec;
-    NMSetting8021x *              s_8021x;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    NMSettingWirelessSecurity    *s_wsec;
+    NMSetting8021x               *s_8021x;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
     gboolean                      success;
-    GError *                      error = NULL;
-    gs_unref_bytes GBytes *ssid         = nmtst_gbytes_from_str("blahblah");
+    GError                       *error = NULL;
+    gs_unref_bytes GBytes        *ssid  = nmtst_gbytes_from_str("blahblah");
 
     connection = nm_simple_connection_new();
 
@@ -6643,17 +6643,17 @@ test_write_wifi_wpa_eap_ttls_mschapv2(void)
 static void
 test_write_wifi_wpa_then_open(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
-    gs_free char *              keyfile      = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
+    gs_free char                 *keyfile    = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread1    = NULL;
     gs_unref_object NMConnection *reread2    = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingWirelessSecurity *   s_wsec;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    gs_unref_bytes GBytes *ssid = nmtst_gbytes_from_str("blahblah");
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    NMSettingWirelessSecurity    *s_wsec;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    gs_unref_bytes GBytes        *ssid = nmtst_gbytes_from_str("blahblah");
 
     /* Test that writing out a WPA config then changing that to an open
      * config doesn't leave various WPA-related keys lying around in the ifcfg.
@@ -6739,18 +6739,18 @@ test_write_wifi_wpa_then_open(void)
 static void
 test_write_wifi_wpa_then_wep_with_perms(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
-    nmtst_auto_unlinkfile char *keyfile      = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
+    nmtst_auto_unlinkfile char   *keyfile    = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread1    = NULL;
     gs_unref_object NMConnection *reread2    = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingWirelessSecurity *   s_wsec;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    NMSettingWirelessSecurity    *s_wsec;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
     gboolean                      success;
-    gs_unref_bytes GBytes *ssid = nmtst_gbytes_from_str("SomeSSID");
+    gs_unref_bytes GBytes        *ssid = nmtst_gbytes_from_str("SomeSSID");
 
     /* Test that writing out a WPA config then changing that to a WEP
      * config works and doesn't cause infinite loop or other issues.
@@ -6848,18 +6848,18 @@ test_write_wifi_wpa_then_wep_with_perms(void)
 static void
 test_write_wifi_dynamic_wep_leap(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
-    nmtst_auto_unlinkfile char *keyfile      = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
+    nmtst_auto_unlinkfile char   *keyfile    = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingWirelessSecurity *   s_wsec;
-    NMSetting8021x *              s_8021x;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    gs_unref_bytes GBytes *ssid = nmtst_gbytes_from_str("blahblah");
-    shvarFile *            ifcfg;
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    NMSettingWirelessSecurity    *s_wsec;
+    NMSetting8021x               *s_8021x;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    gs_unref_bytes GBytes        *ssid = nmtst_gbytes_from_str("blahblah");
+    shvarFile                    *ifcfg;
 
     connection = nm_simple_connection_new();
 
@@ -6927,13 +6927,13 @@ test_write_wifi_dynamic_wep_leap(void)
 static void
 test_write_wired_qeth_dhcp(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
 
     connection = nm_simple_connection_new();
 
@@ -6985,14 +6985,14 @@ test_write_wired_qeth_dhcp(void)
 static void
 test_write_wired_ctc_dhcp(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    shvarFile *                   ifcfg;
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    shvarFile                    *ifcfg;
 
     connection = nm_simple_connection_new();
 
@@ -7048,12 +7048,12 @@ test_write_wired_ctc_dhcp(void)
 static void
 test_write_permissions(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
 
     connection = nm_simple_connection_new();
 
@@ -7101,15 +7101,15 @@ test_write_permissions(void)
 static void
 test_write_wifi_wep_agent_keys(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWireless *           s_wifi;
-    NMSettingWirelessSecurity *   s_wsec;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    gs_unref_bytes GBytes *ssid = nmtst_gbytes_from_str("foobarbaz");
+    NMSettingConnection          *s_con;
+    NMSettingWireless            *s_wifi;
+    NMSettingWirelessSecurity    *s_wsec;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    gs_unref_bytes GBytes        *ssid = nmtst_gbytes_from_str("foobarbaz");
 
     connection = nm_simple_connection_new();
     g_assert(connection != NULL);
@@ -7177,9 +7177,9 @@ static void
 test_write_wired_pppoe(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingPppoe *              s_pppoe;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingPppoe               *s_pppoe;
 
     connection = nm_simple_connection_new();
 
@@ -7221,9 +7221,9 @@ static void
 test_write_vpn(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingVpn *                s_vpn;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingVpn                 *s_vpn;
 
     connection = nm_simple_connection_new();
 
@@ -7262,10 +7262,10 @@ static void
 test_write_mobile_broadband(gconstpointer data)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingCdma *               s_cdma;
-    NMSettingSerial *             s_serial;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingCdma                *s_cdma;
+    NMSettingSerial              *s_serial;
     gboolean                      gsm = GPOINTER_TO_UINT(data);
 
     connection = nm_simple_connection_new();
@@ -7315,9 +7315,9 @@ static void
 test_read_bridge_main(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingBridge *             s_bridge;
-    NMSettingWired *              s_wired;
-    const char *                  mac;
+    NMSettingBridge              *s_bridge;
+    NMSettingWired               *s_wired;
+    const char                   *mac;
     char expected_mac_address[ETH_ALEN] = {0x00, 0x16, 0x41, 0x11, 0x22, 0x33};
 
     connection =
@@ -7346,18 +7346,18 @@ test_read_bridge_main(void)
 static void
 test_write_bridge_main(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingBridge *             s_bridge;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    NMIPAddress *                 addr;
-    static const char *           mac   = "31:33:33:37:be:cd";
-    GError *                      error = NULL;
-    gs_unref_ptrarray GPtrArray *vlans  = NULL;
-    NMBridgeVlan *               vlan;
+    NMSettingConnection          *s_con;
+    NMSettingBridge              *s_bridge;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    NMIPAddress                  *addr;
+    static const char            *mac   = "31:33:33:37:be:cd";
+    GError                       *error = NULL;
+    gs_unref_ptrarray GPtrArray  *vlans = NULL;
+    NMBridgeVlan                 *vlan;
 
     connection = nm_simple_connection_new();
     g_assert(connection);
@@ -7437,8 +7437,8 @@ static void
 test_read_bridge_component(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingBridgePort *         s_port;
+    NMSettingConnection          *s_con;
+    NMSettingBridgePort          *s_port;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-bridge-component",
                                        NULL,
@@ -7460,16 +7460,16 @@ test_read_bridge_component(void)
 static void
 test_write_bridge_component(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
-    NMSetting *                   s_port;
-    static const char *           mac  = "31:33:33:37:be:cd";
-    guint32                       mtu  = 1492;
-    gs_unref_ptrarray GPtrArray *vlans = NULL;
-    NMBridgeVlan *               vlan;
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
+    NMSetting                    *s_port;
+    static const char            *mac   = "31:33:33:37:be:cd";
+    guint32                       mtu   = 1492;
+    gs_unref_ptrarray GPtrArray  *vlans = NULL;
+    NMBridgeVlan                 *vlan;
 
     connection = nm_simple_connection_new();
     g_assert(connection);
@@ -7531,7 +7531,7 @@ static void
 test_read_bridge_missing_stp(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingBridge *             s_bridge;
+    NMSettingBridge              *s_bridge;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-bridge-missing-stp",
                                        NULL,
@@ -7550,7 +7550,7 @@ static void
 test_read_vlan_interface(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingVlan *               s_vlan;
+    NMSettingVlan                *s_vlan;
     guint32                       from = 0;
     guint32                       to   = 0;
 
@@ -7599,7 +7599,7 @@ static void
 test_read_vlan_only_vlan_id(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingVlan *               s_vlan;
+    NMSettingVlan                *s_vlan;
 
     connection = _connection_from_file(TEST_IFCFG_VLAN_ONLY_VLANID, NULL, TYPE_ETHERNET, NULL);
 
@@ -7615,10 +7615,10 @@ test_read_vlan_only_vlan_id(void)
 static void
 test_read_vlan_vlanid_use(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingVlan *               s_vlan;
+    NMSettingVlan                *s_vlan;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-vlan-vlanid-use",
                                        NULL,
@@ -7647,7 +7647,7 @@ static void
 test_read_vlan_only_device(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingVlan *               s_vlan;
+    NMSettingVlan                *s_vlan;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-vlan-only-device",
                                        NULL,
@@ -7666,7 +7666,7 @@ static void
 test_read_vlan_physdev(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingVlan *               s_vlan;
+    NMSettingVlan                *s_vlan;
 
     connection =
         _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-vlan-physdev", NULL, TYPE_ETHERNET, NULL);
@@ -7683,7 +7683,7 @@ static void
 test_read_vlan_reorder_hdr_1(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingVlan *               s_vlan;
+    NMSettingVlan                *s_vlan;
 
     NMTST_EXPECT_NM_WARN("*REORDER_HDR key is deprecated, use VLAN_FLAGS*");
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-vlan-reorder-hdr-1",
@@ -7706,7 +7706,7 @@ static void
 test_read_vlan_reorder_hdr_2(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingVlan *               s_vlan;
+    NMSettingVlan                *s_vlan;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-vlan-reorder-hdr-2",
                                        NULL,
@@ -7727,7 +7727,7 @@ static void
 test_read_vlan_flags_1(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingVlan *               s_vlan;
+    NMSettingVlan                *s_vlan;
 
     connection =
         _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-vlan-flags-1", NULL, TYPE_ETHERNET, NULL);
@@ -7747,7 +7747,7 @@ static void
 test_read_vlan_flags_2(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingVlan *               s_vlan;
+    NMSettingVlan                *s_vlan;
 
     connection =
         _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-vlan-flags-2", NULL, TYPE_ETHERNET, NULL);
@@ -7766,7 +7766,7 @@ test_read_vlan_flags_2(void)
 static void
 test_write_vlan(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
 
     connection = _connection_from_file(TEST_IFCFG_VLAN_INTERFACE, NULL, TYPE_VLAN, NULL);
@@ -7780,7 +7780,7 @@ test_write_vlan(void)
 static void
 test_write_vlan_flags(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
 
@@ -7797,7 +7797,7 @@ test_write_vlan_flags(void)
 static void
 test_write_vlan_only_vlanid(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
 
@@ -7813,11 +7813,11 @@ test_write_vlan_only_vlanid(void)
 static void
 test_write_vlan_reorder_hdr(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingVlan *               s_vlan;
+    NMSettingConnection          *s_con;
+    NMSettingVlan                *s_vlan;
 
     connection = nm_simple_connection_new();
 
@@ -7858,11 +7858,11 @@ test_write_vlan_reorder_hdr(void)
 static void
 test_write_ethernet_missing_ipv6(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
 
     connection = nm_simple_connection_new();
     g_assert(connection);
@@ -7921,7 +7921,7 @@ static void
 test_read_bond_main(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingBond *               s_bond;
+    NMSettingBond                *s_bond;
 
     connection =
         _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-bond-main", NULL, TYPE_ETHERNET, NULL);
@@ -7939,7 +7939,7 @@ static void
 test_read_bond_eth_type(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingBond *               s_bond;
+    NMSettingBond                *s_bond;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-bond-eth-type",
                                        NULL,
@@ -7961,15 +7961,15 @@ test_read_bond_eth_type(void)
 static void
 test_write_bond_main(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingBond *               s_bond;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    NMIPAddress *                 addr;
-    GError *                      error = NULL;
+    NMSettingConnection          *s_con;
+    NMSettingBond                *s_bond;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    NMIPAddress                  *addr;
+    GError                       *error = NULL;
 
     connection = nm_simple_connection_new();
 
@@ -8030,7 +8030,7 @@ static void
 test_read_bond_slave(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
+    NMSettingConnection          *s_con;
 
     connection =
         _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-bond-slave", NULL, TYPE_ETHERNET, NULL);
@@ -8046,12 +8046,12 @@ test_read_bond_slave(void)
 static void
 test_write_bond_slave(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingWired *              s_wired;
-    static const char *           mac = "31:33:33:37:be:cd";
+    NMSettingConnection          *s_con;
+    NMSettingWired               *s_wired;
+    static const char            *mac = "31:33:33:37:be:cd";
     guint32                       mtu = 1492;
 
     connection = nm_simple_connection_new();
@@ -8088,8 +8088,8 @@ static void
 test_read_bond_port(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con      = NULL;
-    NMSettingBondPort *           s_port     = NULL;
+    NMSettingConnection          *s_con      = NULL;
+    NMSettingBondPort            *s_port     = NULL;
 
     connection =
         _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-bond-port", NULL, TYPE_ETHERNET, NULL);
@@ -8105,11 +8105,11 @@ test_read_bond_port(void)
 static void
 test_write_bond_port(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingBondPort *           s_bond_port;
+    NMSettingConnection          *s_con;
+    NMSettingBondPort            *s_bond_port;
 
     connection = nm_simple_connection_new();
 
@@ -8147,9 +8147,9 @@ static void
 test_read_infiniband(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingInfiniband *         s_infiniband;
-    char *                        unmanaged = NULL;
-    const char *                  mac;
+    NMSettingInfiniband          *s_infiniband;
+    char                         *unmanaged = NULL;
+    const char                   *mac;
     char        expected_mac_address[INFINIBAND_ALEN] = {0x80, 0x00, 0x11, 0x22, 0x33, 0x44, 0x55,
                                                   0x66, 0x77, 0x88, 0x99, 0xaa, 0xbb, 0xcc,
                                                   0xdd, 0xee, 0xff, 0x00, 0x11, 0x22};
@@ -8175,17 +8175,17 @@ test_read_infiniband(void)
 static void
 test_write_infiniband(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingInfiniband *         s_infiniband;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    const char * mac = "80:00:11:22:33:44:55:66:77:88:99:aa:bb:cc:dd:ee:ff:00:11:22";
+    NMSettingConnection          *s_con;
+    NMSettingInfiniband          *s_infiniband;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    const char  *mac = "80:00:11:22:33:44:55:66:77:88:99:aa:bb:cc:dd:ee:ff:00:11:22";
     guint32      mtu = 65520;
     NMIPAddress *addr;
-    GError *     error = NULL;
+    GError      *error = NULL;
 
     connection = nm_simple_connection_new();
 
@@ -8242,7 +8242,7 @@ static void
 test_read_bond_slave_ib(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
+    NMSettingConnection          *s_con;
 
     connection =
         _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-bond-slave-ib", NULL, NULL, NULL);
@@ -8256,11 +8256,11 @@ test_read_bond_slave_ib(void)
 static void
 test_write_bond_slave_ib(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingInfiniband *         s_infiniband;
+    NMSettingConnection          *s_con;
+    NMSettingInfiniband          *s_infiniband;
     static const char *mac = "80:00:11:22:33:44:55:66:77:88:99:aa:bb:cc:dd:ee:ff:00:11:22";
 
     connection = nm_simple_connection_new();
@@ -8304,8 +8304,8 @@ static void
 test_read_bond_opts_mode_numeric(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingBond *               s_bond;
+    NMSettingConnection          *s_con;
+    NMSettingBond                *s_bond;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-bond-mode-numeric",
                                        NULL,
@@ -8332,7 +8332,7 @@ static void
 test_read_dcb_basic(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingDcb *                s_dcb;
+    NMSettingDcb                 *s_dcb;
     guint                         i;
     guint                         expected_group_ids[8]        = {0, 0, 0, 0, 1, 1, 1, 0xF};
     guint                         expected_group_bandwidths[8] = {25, 0, 0, 75, 0, 0, 0, 0};
@@ -8394,13 +8394,13 @@ test_read_dcb_basic(void)
 static void
 test_write_dcb_basic(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingDcb *                s_dcb;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
+    NMSettingConnection          *s_con;
+    NMSettingDcb                 *s_dcb;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
     guint                         i;
     const guint                   group_ids[8]        = {4, 0xF, 6, 0xF, 1, 7, 3, 0xF};
     const guint                   group_bandwidths[8] = {10, 20, 15, 10, 2, 3, 35, 5};
@@ -8476,7 +8476,7 @@ static void
 test_read_dcb_default_app_priorities(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingDcb *                s_dcb;
+    NMSettingDcb                 *s_dcb;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-dcb-default-app-priorities",
                                        NULL,
@@ -8611,9 +8611,9 @@ static void
 test_read_fcoe_mode(gconstpointer user_data)
 {
     gs_unref_object NMConnection *connection    = NULL;
-    gs_free char *                file          = NULL;
-    const char *                  expected_mode = user_data;
-    NMSettingDcb *                s_dcb;
+    gs_free char                 *file          = NULL;
+    const char                   *expected_mode = user_data;
+    NMSettingDcb                 *s_dcb;
 
     file       = g_strdup_printf(TEST_IFCFG_DIR "/ifcfg-test-fcoe-%s", expected_mode);
     connection = _connection_from_file(file, NULL, TYPE_ETHERNET, NULL);
@@ -8627,15 +8627,15 @@ test_read_fcoe_mode(gconstpointer user_data)
 static void
 test_write_fcoe_mode(gconstpointer user_data)
 {
-    nmtst_auto_unlinkfile char *testfile        = NULL;
+    nmtst_auto_unlinkfile char   *testfile      = NULL;
     gs_unref_object NMConnection *connection    = NULL;
     gs_unref_object NMConnection *reread        = NULL;
-    const char *                  expected_mode = user_data;
-    NMSettingConnection *         s_con;
-    NMSettingDcb *                s_dcb;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    shvarFile *                   ifcfg;
+    const char                   *expected_mode = user_data;
+    NMSettingConnection          *s_con;
+    NMSettingDcb                 *s_dcb;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    shvarFile                    *ifcfg;
 
     connection = nm_simple_connection_new();
 
@@ -8683,11 +8683,11 @@ test_write_fcoe_mode(gconstpointer user_data)
 static void
 test_read_team_master(gconstpointer user_data)
 {
-    const char *const PATH_NAME              = user_data;
+    const char *const             PATH_NAME  = user_data;
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingTeam *               s_team;
-    const char *                  expected_config =
+    NMSettingConnection          *s_con;
+    NMSettingTeam                *s_team;
+    const char                   *expected_config =
         "{\"device\": \"team0\", \"link_watch\": {\"name\": \"ethtool\"}}";
 
     connection = _connection_from_file(PATH_NAME, NULL, TYPE_ETHERNET, NULL);
@@ -8706,8 +8706,8 @@ test_read_team_master(gconstpointer user_data)
 static void
 test_read_team_master_invalid(gconstpointer user_data)
 {
-    const char *const PATH_NAME              = user_data;
-    gs_free_error GError *error              = NULL;
+    const char *const             PATH_NAME  = user_data;
+    gs_free_error GError         *error      = NULL;
     gs_unref_object NMConnection *connection = NULL;
 
     if (nm_json_vt()) {
@@ -8722,14 +8722,14 @@ test_read_team_master_invalid(gconstpointer user_data)
 static void
 test_write_team_master(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingTeam *               s_team;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    const char *                  expected_config =
+    NMSettingConnection          *s_con;
+    NMSettingTeam                *s_team;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    const char                   *expected_config =
         "{\"device\": \"team0\", \"link_watch\": {\"name\": \"ethtool\"}}";
     shvarFile *f;
 
@@ -8777,11 +8777,11 @@ test_write_team_master(void)
 static void
 test_read_team_port(gconstpointer user_data)
 {
-    const char *const PATH_NAME              = user_data;
+    const char *const             PATH_NAME  = user_data;
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingTeamPort *           s_team_port;
-    const char *                  expected_config = "{\"p4p1\": {\"prio\": -10, \"sticky\": true}}";
+    NMSettingConnection          *s_con;
+    NMSettingTeamPort            *s_team_port;
+    const char                   *expected_config = "{\"p4p1\": {\"prio\": -10, \"sticky\": true}}";
 
     connection = _connection_from_file(PATH_NAME, NULL, TYPE_ETHERNET, NULL);
 
@@ -8798,13 +8798,13 @@ test_read_team_port(gconstpointer user_data)
 static void
 test_write_team_port(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingTeamPort *           s_team_port;
-    const char *                  expected_config = "{\"p4p1\": {\"prio\": -10, \"sticky\": true}}";
-    shvarFile *                   f;
+    NMSettingConnection          *s_con;
+    NMSettingTeamPort            *s_team_port;
+    const char                   *expected_config = "{\"p4p1\": {\"prio\": -10, \"sticky\": true}}";
+    shvarFile                    *f;
 
     connection = nm_simple_connection_new();
 
@@ -8849,14 +8849,14 @@ test_write_team_port(void)
 static void
 test_write_team_infiniband_port(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingTeamPort *           s_team_port;
-    NMSettingInfiniband *         s_infiniband;
-    const char *                  expected_config = "{\"inf1\": {\"prio\": -10, \"sticky\": true}}";
-    shvarFile *                   f;
+    NMSettingConnection          *s_con;
+    NMSettingTeamPort            *s_team_port;
+    NMSettingInfiniband          *s_infiniband;
+    const char                   *expected_config = "{\"inf1\": {\"prio\": -10, \"sticky\": true}}";
+    shvarFile                    *f;
 
     connection = nm_simple_connection_new();
 
@@ -8905,8 +8905,8 @@ static void
 test_read_team_port_empty_config(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingTeamPort *           s_team_port;
+    NMSettingConnection          *s_con;
+    NMSettingTeamPort            *s_team_port;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-team-port-empty-config",
                                        NULL,
@@ -8929,12 +8929,12 @@ test_read_team_port_empty_config(void)
 static void
 test_team_reread_slave(void)
 {
-    nmtst_auto_unlinkfile char *testfile       = NULL;
+    nmtst_auto_unlinkfile char   *testfile     = NULL;
     gs_unref_object NMConnection *connection_1 = NULL;
     gs_unref_object NMConnection *connection_2 = NULL;
     gs_unref_object NMConnection *reread       = NULL;
     gboolean                      reread_same  = FALSE;
-    NMSettingConnection *         s_con;
+    NMSettingConnection          *s_con;
 
     connection_1 =
         nmtst_create_connection_from_keyfile("[connection]\n"
@@ -9006,7 +9006,7 @@ static void
 test_read_proxy_basic(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingProxy *              s_proxy;
+    NMSettingProxy               *s_proxy;
 
     /* Test basic proxy configuration */
 
@@ -9027,13 +9027,13 @@ test_read_proxy_basic(void)
 static void
 test_write_proxy_basic(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingProxy *              s_proxy;
-    const char *                  expected_url = "https://wpad.neverland.org/wpad.dat";
-    shvarFile *                   f;
+    NMSettingConnection          *s_con;
+    NMSettingProxy               *s_proxy;
+    const char                   *expected_url = "https://wpad.neverland.org/wpad.dat";
+    shvarFile                    *f;
 
     connection = nm_simple_connection_new();
 
@@ -9076,7 +9076,7 @@ test_write_proxy_basic(void)
 static const char *
 _svUnescape(const char *str, char **to_free)
 {
-    const char *  s;
+    const char   *s;
     gs_free char *str_free = NULL;
 
     g_assert(str);
@@ -9097,7 +9097,7 @@ _svUnescape(const char *str, char **to_free)
     }
 
     {
-        const char *  s2;
+        const char   *s2;
         gs_free char *to_free2 = NULL;
 
         gboolean is_utf8 = s && g_utf8_validate(s, -1, NULL);
@@ -9140,7 +9140,7 @@ static void
 do_svUnescape_assert(const char *str, const char *expected)
 {
     gs_free char *to_free = NULL;
-    const char *  s;
+    const char   *s;
 
     s = _svUnescape(str, &to_free);
     g_assert_cmpstr(s, ==, expected);
@@ -9149,7 +9149,7 @@ do_svUnescape_assert(const char *str, const char *expected)
     if (expected) {
         gs_free char *s1_free = NULL;
         gs_free char *s2_free = NULL;
-        const char *  s1, *s2;
+        const char   *s1, *s2;
 
         s1 = svEscape(expected, &s1_free);
         g_assert(s1);
@@ -9162,8 +9162,8 @@ do_svUnescape_assert(const char *str, const char *expected)
 }
 
 static void
-do_svUnescape_combine_ansi_append(GString *               str_val,
-                                  GString *               str_exp,
+do_svUnescape_combine_ansi_append(GString                *str_val,
+                                  GString                *str_exp,
                                   const UnescapeTestData *data,
                                   gboolean                honor_needs_ascii_separator)
 {
@@ -9180,8 +9180,8 @@ do_svUnescape_combine_ansi_append(GString *               str_val,
 }
 
 static void
-do_svUnescape_combine_ansi(GString *               str_val,
-                           GString *               str_exp,
+do_svUnescape_combine_ansi(GString                *str_val,
+                           GString                *str_exp,
                            const UnescapeTestData *data_ansi,
                            gsize                   data_len,
                            gssize                  idx)
@@ -9401,7 +9401,7 @@ test_svUnescape(void)
 #undef V0
 #undef V1
 #undef V2
-    gsize                i;
+    gsize                         i;
     nm_auto_free_gstring GString *str_val  = g_string_new(NULL);
     nm_auto_free_gstring GString *str_val2 = g_string_new(NULL);
     nm_auto_free_gstring GString *str_exp  = g_string_new(NULL);
@@ -9470,13 +9470,13 @@ static void
 test_write_unknown(gconstpointer test_data)
 {
     nmtst_auto_unlinkfile char *filename_tmp_1    = g_strdup(TEST_SCRATCH_DIR_TMP "/tmp-1");
-    const char *                testfile          = test_data;
-    gs_free char *              testfile_expected = g_strconcat(testfile, ".expected", NULL);
-    shvarFile *                 sv;
-    gs_free_error GError *error = NULL;
-    gboolean              success;
-    gs_free char *        file_contents_out = NULL;
-    gs_free char *        file_contents_exp = NULL;
+    const char                 *testfile          = test_data;
+    gs_free char               *testfile_expected = g_strconcat(testfile, ".expected", NULL);
+    shvarFile                  *sv;
+    gs_free_error GError       *error = NULL;
+    gboolean                    success;
+    gs_free char               *file_contents_out = NULL;
+    gs_free char               *file_contents_exp = NULL;
 
     sv = _svOpenFile(testfile);
 
@@ -9519,12 +9519,12 @@ test_write_unknown(gconstpointer test_data)
 static void
 test_read_vlan_trailing_spaces(void)
 {
-    const char *    testfile                 = TEST_IFCFG_DIR "/ifcfg-test-vlan-trailing-spaces";
+    const char                   *testfile   = TEST_IFCFG_DIR "/ifcfg-test-vlan-trailing-spaces";
     gs_unref_object NMConnection *connection = NULL;
     gboolean                      success;
-    GError *                      error = NULL;
-    NMSettingVlan *               s_vlan;
-    gs_free char *                contents = NULL;
+    GError                       *error = NULL;
+    NMSettingVlan                *s_vlan;
+    gs_free char                 *contents = NULL;
 
     /* Ensure there is whitespace at the end of the VLAN interface name,
      * to prevent the whitespace getting stripped off and committed mistakenly
@@ -9681,12 +9681,12 @@ static void
 test_sriov_read(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingSriov *              s_sriov;
-    NMSriovVF *                   vf;
-    GVariant *                    variant;
-    GError *                      error = NULL;
-    gs_free char *                str1  = NULL;
-    gs_free char *                str2  = NULL;
+    NMSettingSriov               *s_sriov;
+    NMSriovVF                    *vf;
+    GVariant                     *variant;
+    GError                       *error = NULL;
+    gs_free char                 *str1  = NULL;
+    gs_free char                 *str2  = NULL;
 
     connection =
         _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-sriov", NULL, TYPE_ETHERNET, NULL);
@@ -9730,17 +9730,17 @@ test_sriov_read(void)
 static void
 test_sriov_write(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    NMSettingSriov *              s_sriov;
-    NMSriovVF *                   vf;
-    gs_unref_ptrarray GPtrArray *vfs = NULL;
-    NMIPAddress *                addr;
-    GError *                     error = NULL;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    NMSettingSriov               *s_sriov;
+    NMSriovVF                    *vf;
+    gs_unref_ptrarray GPtrArray  *vfs = NULL;
+    NMIPAddress                  *addr;
+    GError                       *error = NULL;
 
     connection = nm_simple_connection_new();
 
@@ -9819,10 +9819,10 @@ static void
 test_tc_read(void)
 {
     gs_unref_object NMConnection *connection = NULL;
-    NMSettingTCConfig *           s_tc;
-    NMTCQdisc *                   qdisc;
-    NMTCTfilter *                 filter;
-    gs_free char *                str = NULL;
+    NMSettingTCConfig            *s_tc;
+    NMTCQdisc                    *qdisc;
+    NMTCTfilter                  *filter;
+    gs_free char                 *str = NULL;
 
     connection = _connection_from_file(TEST_IFCFG_DIR "/ifcfg-test-tc", NULL, TYPE_ETHERNET, NULL);
 
@@ -9847,14 +9847,14 @@ test_tc_read(void)
 static void
 test_tc_write_empty(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    NMIPAddress *                 addr;
-    GError *                      error = NULL;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    NMIPAddress                  *addr;
+    GError                       *error = NULL;
 
     connection = nm_simple_connection_new();
 
@@ -9911,17 +9911,17 @@ test_tc_write_empty(void)
 static void
 test_tc_write(void)
 {
-    nmtst_auto_unlinkfile char *testfile     = NULL;
+    nmtst_auto_unlinkfile char   *testfile   = NULL;
     gs_unref_object NMConnection *connection = NULL;
     gs_unref_object NMConnection *reread     = NULL;
-    NMSettingConnection *         s_con;
-    NMSettingIPConfig *           s_ip4;
-    NMSettingIPConfig *           s_ip6;
-    NMSettingTCConfig *           s_tc;
-    NMTCQdisc *                   qdisc;
-    NMTCTfilter *                 tfilter;
-    NMIPAddress *                 addr;
-    GError *                      error = NULL;
+    NMSettingConnection          *s_con;
+    NMSettingIPConfig            *s_ip4;
+    NMSettingIPConfig            *s_ip6;
+    NMSettingTCConfig            *s_tc;
+    NMTCQdisc                    *qdisc;
+    NMTCTfilter                  *tfilter;
+    NMIPAddress                  *addr;
+    GError                       *error = NULL;
 
     connection = nm_simple_connection_new();
 
@@ -10014,7 +10014,7 @@ test_well_known_keys(void)
 
     for (i = 0; i < G_N_ELEMENTS(nms_ifcfg_well_known_keys); i++) {
         const NMSIfcfgKeyTypeInfo *ti       = &nms_ifcfg_well_known_keys[i];
-        gs_free char *             key_name = NULL;
+        gs_free char              *key_name = NULL;
         gssize                     idx;
 
         g_assert(ti == nms_ifcfg_well_known_key_find_info(ti->key_name, &idx));
@@ -10029,7 +10029,7 @@ test_well_known_keys(void)
     G_STMT_START                                               \
     {                                                          \
         const NMSIfcfgKeyTypeInfo *_ti;                        \
-        const char *               _expected = (expected);     \
+        const char                *_expected = (expected);     \
                                                                \
         _ti = nms_ifcfg_rh_utils_is_well_known_key("" key ""); \
         if (!_expected) {                                      \