summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorThomas Haller <thaller@redhat.com>2017-11-24 16:24:40 +0100
committerThomas Haller <thaller@redhat.com>2017-11-24 17:51:05 +0100
commit2d70beccaae536860ac8155d8d35e4186e74bf0e (patch)
tree599efdd3c6b9580321509da3b0dbe0ada5bbbe30
parentefed5254cde007efcb48e353bd00651f37555e6d (diff)
downloadNetworkManager-2d70beccaae536860ac8155d8d35e4186e74bf0e.tar.gz
core: refactor NMSettingsConnectionCallId typedef not to be a pointer to struct
Typedefs to structs are fine, but a typedef for a pointer seems confusing to me. Let's avoid it.
-rw-r--r--src/nm-act-request.c6
-rw-r--r--src/settings/nm-settings-connection.c6
-rw-r--r--src/settings/nm-settings-connection.h22
-rw-r--r--src/vpn/nm-vpn-connection.c4
4 files changed, 19 insertions, 19 deletions
diff --git a/src/nm-act-request.c b/src/nm-act-request.c
index 25a1dd9b4f..29bb3a05bb 100644
--- a/src/nm-act-request.c
+++ b/src/nm-act-request.c
@@ -96,7 +96,7 @@ struct _NMActRequestGetSecretsCallId {
NMActRequest *self;
NMActRequestSecretsFunc callback;
gpointer callback_data;
- NMSettingsConnectionCallId call_id;
+ NMSettingsConnectionCallId *call_id;
bool has_ref;
};
@@ -113,7 +113,7 @@ _get_secrets_call_id_free (NMActRequestGetSecretsCallId *call_id)
static void
get_secrets_cb (NMSettingsConnection *connection,
- NMSettingsConnectionCallId call_id_s,
+ NMSettingsConnectionCallId *call_id_s,
const char *agent_username,
const char *setting_name,
GError *error,
@@ -172,7 +172,7 @@ nm_act_request_get_secrets (NMActRequest *self,
{
NMActRequestPrivate *priv;
NMActRequestGetSecretsCallId *call_id;
- NMSettingsConnectionCallId call_id_s;
+ NMSettingsConnectionCallId *call_id_s;
NMSettingsConnection *settings_connection;
NMConnection *applied_connection;
const char *hints[2] = { hint, NULL };
diff --git a/src/settings/nm-settings-connection.c b/src/settings/nm-settings-connection.c
index 1185aef905..77b10aae6e 100644
--- a/src/settings/nm-settings-connection.c
+++ b/src/settings/nm-settings-connection.c
@@ -1222,7 +1222,7 @@ get_secrets_idle_cb (GetSecretsInfo *info)
*
* Returns: a call ID which may be used to cancel the ongoing secrets request.
**/
-NMSettingsConnectionCallId
+NMSettingsConnectionCallId *
nm_settings_connection_get_secrets (NMSettingsConnection *self,
NMConnection *applied_connection,
NMAuthSubject *subject,
@@ -1337,7 +1337,7 @@ _get_secrets_cancel (NMSettingsConnection *self,
void
nm_settings_connection_cancel_secrets (NMSettingsConnection *self,
- NMSettingsConnectionCallId call_id)
+ NMSettingsConnectionCallId *call_id)
{
_LOGD ("(%p) secrets canceled", call_id);
@@ -1951,7 +1951,7 @@ out_err:
static void
dbus_get_agent_secrets_cb (NMSettingsConnection *self,
- NMSettingsConnectionCallId call_id,
+ NMSettingsConnectionCallId *call_id,
const char *agent_username,
const char *setting_name,
GError *error,
diff --git a/src/settings/nm-settings-connection.h b/src/settings/nm-settings-connection.h
index 7a9e383259..aa79628f7f 100644
--- a/src/settings/nm-settings-connection.h
+++ b/src/settings/nm-settings-connection.h
@@ -90,7 +90,7 @@ typedef enum {
} NMSettingsAutoconnectBlockedReason;
struct _NMSettingsConnectionCallId;
-typedef struct _NMSettingsConnectionCallId *NMSettingsConnectionCallId;
+typedef struct _NMSettingsConnectionCallId NMSettingsConnectionCallId;
typedef struct _NMSettingsConnectionClass NMSettingsConnectionClass;
@@ -150,7 +150,7 @@ gboolean nm_settings_connection_delete (NMSettingsConnection *self,
GError **error);
typedef void (*NMSettingsConnectionSecretsFunc) (NMSettingsConnection *self,
- NMSettingsConnectionCallId call_id,
+ NMSettingsConnectionCallId *call_id,
const char *agent_username,
const char *setting_name,
GError *error,
@@ -162,17 +162,17 @@ gboolean nm_settings_connection_new_secrets (NMSettingsConnection *self,
GVariant *secrets,
GError **error);
-NMSettingsConnectionCallId nm_settings_connection_get_secrets (NMSettingsConnection *self,
- NMConnection *applied_connection,
- NMAuthSubject *subject,
- const char *setting_name,
- NMSecretAgentGetSecretsFlags flags,
- const char *const*hints,
- NMSettingsConnectionSecretsFunc callback,
- gpointer callback_data);
+NMSettingsConnectionCallId *nm_settings_connection_get_secrets (NMSettingsConnection *self,
+ NMConnection *applied_connection,
+ NMAuthSubject *subject,
+ const char *setting_name,
+ NMSecretAgentGetSecretsFlags flags,
+ const char *const*hints,
+ NMSettingsConnectionSecretsFunc callback,
+ gpointer callback_data);
void nm_settings_connection_cancel_secrets (NMSettingsConnection *self,
- NMSettingsConnectionCallId call_id);
+ NMSettingsConnectionCallId *call_id);
gboolean nm_settings_connection_is_visible (NMSettingsConnection *self);
diff --git a/src/vpn/nm-vpn-connection.c b/src/vpn/nm-vpn-connection.c
index e441bbd051..6c600c46a6 100644
--- a/src/vpn/nm-vpn-connection.c
+++ b/src/vpn/nm-vpn-connection.c
@@ -103,7 +103,7 @@ typedef struct {
gboolean service_can_persist;
gboolean connection_can_persist;
- NMSettingsConnectionCallId secrets_id;
+ NMSettingsConnectionCallId *secrets_id;
SecretsReq secrets_idx;
char *username;
@@ -2571,7 +2571,7 @@ plugin_new_secrets_cb (GDBusProxy *proxy, GAsyncResult *result, gpointer user_da
static void
get_secrets_cb (NMSettingsConnection *connection,
- NMSettingsConnectionCallId call_id,
+ NMSettingsConnectionCallId *call_id,
const char *agent_username,
const char *setting_name,
GError *error,