[PATCH 2/4] plugins: remove explicit GDestroyNotify cast on g_free / g_object_unref
Ben Chan
benchan at chromium.org
Wed Mar 29 01:35:55 UTC 2017
g_free and g_object_unref are in form of `void (*)(gpointer)`, which
matches the GDestroyNotify signature. An explicit GDestroyNotify cast on
g_free and g_object_unref is thus not needed.
---
plugins/altair/mm-broadband-modem-altair-lte.c | 4 ++--
plugins/cinterion/mm-broadband-modem-cinterion.c | 6 +++---
plugins/huawei/mm-broadband-modem-huawei.c | 14 +++++++-------
plugins/icera/mm-broadband-bearer-icera.c | 2 +-
plugins/icera/mm-broadband-modem-icera.c | 6 +++---
plugins/iridium/mm-broadband-modem-iridium.c | 2 +-
plugins/longcheer/mm-broadband-modem-longcheer.c | 2 +-
plugins/mbm/mm-broadband-bearer-mbm.c | 4 ++--
plugins/mbm/mm-broadband-modem-mbm.c | 2 +-
plugins/mtk/mm-broadband-modem-mtk.c | 2 +-
plugins/novatel/mm-broadband-modem-novatel-lte.c | 2 +-
plugins/option/mm-broadband-bearer-hso.c | 4 ++--
plugins/option/mm-broadband-modem-hso.c | 6 +++---
plugins/sierra/mm-broadband-bearer-sierra.c | 2 +-
plugins/sierra/mm-broadband-modem-sierra-icera.c | 2 +-
plugins/sierra/mm-broadband-modem-sierra.c | 2 +-
plugins/telit/mm-broadband-modem-telit.c | 2 +-
plugins/tests/test-fixture.c | 2 +-
plugins/ublox/mm-broadband-bearer-ublox.c | 4 ++--
plugins/ublox/mm-broadband-modem-ublox.c | 2 +-
plugins/zte/mm-broadband-modem-zte.c | 2 +-
21 files changed, 37 insertions(+), 37 deletions(-)
diff --git a/plugins/altair/mm-broadband-modem-altair-lte.c b/plugins/altair/mm-broadband-modem-altair-lte.c
index 1155a7a1..572a1bfd 100644
--- a/plugins/altair/mm-broadband-modem-altair-lte.c
+++ b/plugins/altair/mm-broadband-modem-altair-lte.c
@@ -128,7 +128,7 @@ broadband_bearer_new_ready (GObject *source,
else
g_simple_async_result_set_op_res_gpointer (simple,
bearer,
- (GDestroyNotify)g_object_unref);
+ g_object_unref);
g_simple_async_result_complete (simple);
g_object_unref (simple);
}
@@ -198,7 +198,7 @@ load_unlock_retries_ready (MMBaseModem *self,
mm_unlock_retries_set (retries, MM_MODEM_LOCK_SIM_PUK2, puk2);
g_simple_async_result_set_op_res_gpointer (operation_result,
retries,
- (GDestroyNotify)g_object_unref);
+ g_object_unref);
} else {
g_simple_async_result_set_error (operation_result,
MM_CORE_ERROR,
diff --git a/plugins/cinterion/mm-broadband-modem-cinterion.c b/plugins/cinterion/mm-broadband-modem-cinterion.c
index ae092a0e..4a4e18bf 100644
--- a/plugins/cinterion/mm-broadband-modem-cinterion.c
+++ b/plugins/cinterion/mm-broadband-modem-cinterion.c
@@ -1438,7 +1438,7 @@ load_unlock_retries_context_step (LoadUnlockRetriesContext *ctx)
if (ctx->i == G_N_ELEMENTS (unlock_retries_map)) {
g_simple_async_result_set_op_res_gpointer (ctx->result,
g_object_ref (ctx->retries),
- (GDestroyNotify)g_object_unref);
+ g_object_unref);
load_unlock_retries_context_complete_and_free (ctx);
return;
}
@@ -1643,7 +1643,7 @@ broadband_bearer_cinterion_new_ready (GObject *source,
if (!bearer)
g_simple_async_result_take_error (ctx->result, error);
else
- g_simple_async_result_set_op_res_gpointer (ctx->result, bearer, (GDestroyNotify)g_object_unref);
+ g_simple_async_result_set_op_res_gpointer (ctx->result, bearer, g_object_unref);
create_bearer_context_complete_and_free (ctx);
}
@@ -1659,7 +1659,7 @@ broadband_bearer_new_ready (GObject *source,
if (!bearer)
g_simple_async_result_take_error (ctx->result, error);
else
- g_simple_async_result_set_op_res_gpointer (ctx->result, bearer, (GDestroyNotify)g_object_unref);
+ g_simple_async_result_set_op_res_gpointer (ctx->result, bearer, g_object_unref);
create_bearer_context_complete_and_free (ctx);
}
diff --git a/plugins/huawei/mm-broadband-modem-huawei.c b/plugins/huawei/mm-broadband-modem-huawei.c
index 74e680b4..49c77e9a 100644
--- a/plugins/huawei/mm-broadband-modem-huawei.c
+++ b/plugins/huawei/mm-broadband-modem-huawei.c
@@ -1933,7 +1933,7 @@ set_3gpp_unsolicited_events_handlers (MMBroadbandModemHuawei *self,
NULL);
}
- g_list_free_full (ports, (GDestroyNotify)g_object_unref);
+ g_list_free_full (ports, g_object_unref);
}
static gboolean
@@ -2266,7 +2266,7 @@ broadband_bearer_huawei_new_ready (GObject *source,
if (!bearer)
g_simple_async_result_take_error (ctx->result, error);
else
- g_simple_async_result_set_op_res_gpointer (ctx->result, bearer, (GDestroyNotify)g_object_unref);
+ g_simple_async_result_set_op_res_gpointer (ctx->result, bearer, g_object_unref);
create_bearer_context_complete_and_free (ctx);
}
@@ -2282,7 +2282,7 @@ broadband_bearer_new_ready (GObject *source,
if (!bearer)
g_simple_async_result_take_error (ctx->result, error);
else
- g_simple_async_result_set_op_res_gpointer (ctx->result, bearer, (GDestroyNotify)g_object_unref);
+ g_simple_async_result_set_op_res_gpointer (ctx->result, bearer, g_object_unref);
create_bearer_context_complete_and_free (ctx);
}
@@ -2667,7 +2667,7 @@ set_cdma_unsolicited_events_handlers (MMBroadbandModemHuawei *self,
NULL);
}
- g_list_free_full (ports, (GDestroyNotify)g_object_unref);
+ g_list_free_full (ports, g_object_unref);
}
static gboolean
@@ -3158,7 +3158,7 @@ set_voice_unsolicited_events_handlers (MMBroadbandModemHuawei *self,
NULL, NULL, NULL);
}
- g_list_free_full (ports, (GDestroyNotify)g_object_unref);
+ g_list_free_full (ports, g_object_unref);
}
static gboolean
@@ -3486,7 +3486,7 @@ enable_disable_unsolicited_rfswitch_event_handler (MMBroadbandModemHuawei *self,
enable);
}
- g_list_free_full (ports, (GDestroyNotify)g_object_unref);
+ g_list_free_full (ports, g_object_unref);
}
static void
@@ -4333,7 +4333,7 @@ set_ignored_unsolicited_events_handlers (MMBroadbandModemHuawei *self)
NULL, NULL, NULL);
}
- g_list_free_full (ports, (GDestroyNotify)g_object_unref);
+ g_list_free_full (ports, g_object_unref);
}
static void
diff --git a/plugins/icera/mm-broadband-bearer-icera.c b/plugins/icera/mm-broadband-bearer-icera.c
index c08acc4f..3b3a49a0 100644
--- a/plugins/icera/mm-broadband-bearer-icera.c
+++ b/plugins/icera/mm-broadband-bearer-icera.c
@@ -660,7 +660,7 @@ report_connect_status (MMBroadbandBearerIcera *self,
g_simple_async_result_set_op_res_gpointer (ctx->result,
g_object_ref (ctx->data),
- (GDestroyNotify)g_object_unref);
+ g_object_unref);
dial_3gpp_context_complete_and_free (ctx);
return;
}
diff --git a/plugins/icera/mm-broadband-modem-icera.c b/plugins/icera/mm-broadband-modem-icera.c
index 730a79fb..562b9e0d 100644
--- a/plugins/icera/mm-broadband-modem-icera.c
+++ b/plugins/icera/mm-broadband-modem-icera.c
@@ -883,7 +883,7 @@ broadband_bearer_icera_new_ready (GObject *source,
else
g_simple_async_result_set_op_res_gpointer (simple,
bearer,
- (GDestroyNotify)g_object_unref);
+ g_object_unref);
g_simple_async_result_complete (simple);
g_object_unref (simple);
}
@@ -902,7 +902,7 @@ broadband_bearer_new_ready (GObject *source,
else
g_simple_async_result_set_op_res_gpointer (simple,
bearer,
- (GDestroyNotify)g_object_unref);
+ g_object_unref);
g_simple_async_result_complete (simple);
g_object_unref (simple);
}
@@ -1090,7 +1090,7 @@ load_unlock_retries_ready (MMBaseModem *self,
mm_unlock_retries_set (retries, MM_MODEM_LOCK_SIM_PUK2, puk2);
g_simple_async_result_set_op_res_gpointer (operation_result,
retries,
- (GDestroyNotify)g_object_unref);
+ g_object_unref);
} else {
g_simple_async_result_set_error (operation_result,
MM_CORE_ERROR,
diff --git a/plugins/iridium/mm-broadband-modem-iridium.c b/plugins/iridium/mm-broadband-modem-iridium.c
index c64e8d2d..0e2d670e 100644
--- a/plugins/iridium/mm-broadband-modem-iridium.c
+++ b/plugins/iridium/mm-broadband-modem-iridium.c
@@ -311,7 +311,7 @@ create_bearer (MMIfaceModem *self,
properties);
g_simple_async_result_set_op_res_gpointer (result,
bearer,
- (GDestroyNotify)g_object_unref);
+ g_object_unref);
g_simple_async_result_complete_in_idle (result);
g_object_unref (result);
}
diff --git a/plugins/longcheer/mm-broadband-modem-longcheer.c b/plugins/longcheer/mm-broadband-modem-longcheer.c
index 27da1868..3bb19190 100644
--- a/plugins/longcheer/mm-broadband-modem-longcheer.c
+++ b/plugins/longcheer/mm-broadband-modem-longcheer.c
@@ -363,7 +363,7 @@ load_unlock_retries_ready (MMBaseModem *self,
mm_unlock_retries_set (retries, MM_MODEM_LOCK_SIM_PUK2, puk2);
g_simple_async_result_set_op_res_gpointer (operation_result,
retries,
- (GDestroyNotify)g_object_unref);
+ g_object_unref);
} else {
g_simple_async_result_set_error (operation_result,
MM_CORE_ERROR,
diff --git a/plugins/mbm/mm-broadband-bearer-mbm.c b/plugins/mbm/mm-broadband-bearer-mbm.c
index 0fea7190..fa1a4902 100644
--- a/plugins/mbm/mm-broadband-bearer-mbm.c
+++ b/plugins/mbm/mm-broadband-bearer-mbm.c
@@ -178,7 +178,7 @@ handle_e2nap_connect_status (Dial3gppContext *ctx)
mm_dbg ("Connected status indicated already by an unsolicited message");
g_simple_async_result_set_op_res_gpointer (ctx->result,
g_object_ref (ctx->data),
- (GDestroyNotify)g_object_unref);
+ g_object_unref);
dial_3gpp_context_complete_and_free (ctx);
return TRUE;
case MM_BEARER_CONNECTION_STATUS_DISCONNECTED:
@@ -220,7 +220,7 @@ connect_poll_ready (MMBaseModem *modem,
/* Success! Connected... */
g_simple_async_result_set_op_res_gpointer (ctx->result,
g_object_ref (ctx->data),
- (GDestroyNotify)g_object_unref);
+ g_object_unref);
dial_3gpp_context_complete_and_free (ctx);
return;
}
diff --git a/plugins/mbm/mm-broadband-modem-mbm.c b/plugins/mbm/mm-broadband-modem-mbm.c
index c015c669..dcc91699 100644
--- a/plugins/mbm/mm-broadband-modem-mbm.c
+++ b/plugins/mbm/mm-broadband-modem-mbm.c
@@ -111,7 +111,7 @@ broadband_bearer_mbm_new_ready (GObject *source,
else
g_simple_async_result_set_op_res_gpointer (simple,
bearer,
- (GDestroyNotify)g_object_unref);
+ g_object_unref);
g_simple_async_result_complete (simple);
g_object_unref (simple);
}
diff --git a/plugins/mtk/mm-broadband-modem-mtk.c b/plugins/mtk/mm-broadband-modem-mtk.c
index f652a65b..181497bb 100644
--- a/plugins/mtk/mm-broadband-modem-mtk.c
+++ b/plugins/mtk/mm-broadband-modem-mtk.c
@@ -125,7 +125,7 @@ load_unlock_retries_ready (MMBaseModem *self,
g_simple_async_result_set_op_res_gpointer (operation_result,
retries,
- (GDestroyNotify)g_object_unref);
+ g_object_unref);
}
if (match_info)
diff --git a/plugins/novatel/mm-broadband-modem-novatel-lte.c b/plugins/novatel/mm-broadband-modem-novatel-lte.c
index e5be522d..94030a97 100644
--- a/plugins/novatel/mm-broadband-modem-novatel-lte.c
+++ b/plugins/novatel/mm-broadband-modem-novatel-lte.c
@@ -98,7 +98,7 @@ broadband_bearer_new_ready (GObject *source,
else
g_simple_async_result_set_op_res_gpointer (simple,
bearer,
- (GDestroyNotify)g_object_unref);
+ g_object_unref);
g_simple_async_result_complete (simple);
g_object_unref (simple);
}
diff --git a/plugins/option/mm-broadband-bearer-hso.c b/plugins/option/mm-broadband-bearer-hso.c
index 746f96e5..953277aa 100644
--- a/plugins/option/mm-broadband-bearer-hso.c
+++ b/plugins/option/mm-broadband-bearer-hso.c
@@ -173,7 +173,7 @@ ip_config_ready (MMBaseModem *modem,
g_simple_async_result_set_op_res_gpointer (ctx->result,
ip_config,
- (GDestroyNotify)g_object_unref);
+ g_object_unref);
}
get_ip_config_context_complete_and_free (ctx);
@@ -378,7 +378,7 @@ report_connection_status (MMBaseBearer *bearer,
g_simple_async_result_set_op_res_gpointer (ctx->result,
g_object_ref (ctx->data),
- (GDestroyNotify)g_object_unref);
+ g_object_unref);
dial_3gpp_context_complete_and_free (ctx);
return;
}
diff --git a/plugins/option/mm-broadband-modem-hso.c b/plugins/option/mm-broadband-modem-hso.c
index d2d46891..e48bcbab 100644
--- a/plugins/option/mm-broadband-modem-hso.c
+++ b/plugins/option/mm-broadband-modem-hso.c
@@ -86,7 +86,7 @@ broadband_bearer_new_ready (GObject *source,
else
g_simple_async_result_set_op_res_gpointer (simple,
bearer,
- (GDestroyNotify)g_object_unref);
+ g_object_unref);
g_simple_async_result_complete (simple);
g_object_unref (simple);
}
@@ -105,7 +105,7 @@ broadband_bearer_hso_new_ready (GObject *source,
else
g_simple_async_result_set_op_res_gpointer (simple,
bearer,
- (GDestroyNotify)g_object_unref);
+ g_object_unref);
g_simple_async_result_complete (simple);
g_object_unref (simple);
}
@@ -182,7 +182,7 @@ load_unlock_retries_ready (MMBaseModem *self,
mm_unlock_retries_set (retries, MM_MODEM_LOCK_SIM_PUK, puk1);
g_simple_async_result_set_op_res_gpointer (operation_result,
retries,
- (GDestroyNotify)g_object_unref);
+ g_object_unref);
} else {
g_simple_async_result_set_error (operation_result,
MM_CORE_ERROR,
diff --git a/plugins/sierra/mm-broadband-bearer-sierra.c b/plugins/sierra/mm-broadband-bearer-sierra.c
index 1f590a37..6f999bf2 100644
--- a/plugins/sierra/mm-broadband-bearer-sierra.c
+++ b/plugins/sierra/mm-broadband-bearer-sierra.c
@@ -311,7 +311,7 @@ dial_3gpp_context_step (Dial3gppContext *ctx)
case DIAL_3GPP_STEP_LAST:
g_simple_async_result_set_op_res_gpointer (ctx->result,
g_object_ref (ctx->data),
- (GDestroyNotify)g_object_unref);
+ g_object_unref);
dial_3gpp_context_complete_and_free (ctx);
return;
}
diff --git a/plugins/sierra/mm-broadband-modem-sierra-icera.c b/plugins/sierra/mm-broadband-modem-sierra-icera.c
index bd01cc4d..5489ddb8 100644
--- a/plugins/sierra/mm-broadband-modem-sierra-icera.c
+++ b/plugins/sierra/mm-broadband-modem-sierra-icera.c
@@ -66,7 +66,7 @@ broadband_bearer_sierra_new_ready (GObject *source,
else
g_simple_async_result_set_op_res_gpointer (simple,
bearer,
- (GDestroyNotify)g_object_unref);
+ g_object_unref);
g_simple_async_result_complete (simple);
g_object_unref (simple);
}
diff --git a/plugins/sierra/mm-broadband-modem-sierra.c b/plugins/sierra/mm-broadband-modem-sierra.c
index 8616f1c9..5d3d5935 100644
--- a/plugins/sierra/mm-broadband-modem-sierra.c
+++ b/plugins/sierra/mm-broadband-modem-sierra.c
@@ -1115,7 +1115,7 @@ broadband_bearer_sierra_new_ready (GObject *source,
else
g_simple_async_result_set_op_res_gpointer (simple,
bearer,
- (GDestroyNotify)g_object_unref);
+ g_object_unref);
g_simple_async_result_complete (simple);
g_object_unref (simple);
}
diff --git a/plugins/telit/mm-broadband-modem-telit.c b/plugins/telit/mm-broadband-modem-telit.c
index 6ef340f1..8b873106 100644
--- a/plugins/telit/mm-broadband-modem-telit.c
+++ b/plugins/telit/mm-broadband-modem-telit.c
@@ -665,7 +665,7 @@ load_unlock_retries_step (LoadUnlockRetriesContext *ctx)
} else {
g_simple_async_result_set_op_res_gpointer (ctx->result,
g_object_ref (ctx->retries),
- (GDestroyNotify)g_object_unref);
+ g_object_unref);
}
load_unlock_retries_context_complete_and_free (ctx);
diff --git a/plugins/tests/test-fixture.c b/plugins/tests/test-fixture.c
index 3013fe04..d496fee7 100644
--- a/plugins/tests/test-fixture.c
+++ b/plugins/tests/test-fixture.c
@@ -135,7 +135,7 @@ common_get_modem (TestFixture *fixture,
ready = TRUE;
}
- g_list_free_full (modems, (GDestroyNotify) g_object_unref);
+ g_list_free_full (modems, g_object_unref);
g_object_unref (manager);
if (ready)
diff --git a/plugins/ublox/mm-broadband-bearer-ublox.c b/plugins/ublox/mm-broadband-bearer-ublox.c
index e3ba90d0..62f3d7ea 100644
--- a/plugins/ublox/mm-broadband-bearer-ublox.c
+++ b/plugins/ublox/mm-broadband-bearer-ublox.c
@@ -340,7 +340,7 @@ cgact_activate_ready (MMBaseModem *modem,
if (!response)
g_task_return_error (task, error);
else
- g_task_return_pointer (task, g_object_ref (ctx->data), (GDestroyNotify) g_object_unref);
+ g_task_return_pointer (task, g_object_ref (ctx->data), g_object_unref);
g_object_unref (task);
}
@@ -595,7 +595,7 @@ ugcntrd_ready (MMBaseModem *modem,
result = g_new (StatsResult, 1);
result->bytes_rx = rx_bytes;
result->bytes_tx = tx_bytes;
- g_task_return_pointer (task, result, (GDestroyNotify) g_free);
+ g_task_return_pointer (task, result, g_free);
}
g_object_unref (task);
diff --git a/plugins/ublox/mm-broadband-modem-ublox.c b/plugins/ublox/mm-broadband-modem-ublox.c
index be2e6e22..64dd9dec 100644
--- a/plugins/ublox/mm-broadband-modem-ublox.c
+++ b/plugins/ublox/mm-broadband-modem-ublox.c
@@ -871,7 +871,7 @@ create_bearer_step (GTask *task)
case CREATE_BEARER_STEP_LAST:
g_assert (ctx->bearer);
- g_task_return_pointer (task, g_object_ref (ctx->bearer), (GDestroyNotify) g_object_unref);
+ g_task_return_pointer (task, g_object_ref (ctx->bearer), g_object_unref);
g_object_unref (task);
return;
}
diff --git a/plugins/zte/mm-broadband-modem-zte.c b/plugins/zte/mm-broadband-modem-zte.c
index 3ea59b59..3d10e389 100644
--- a/plugins/zte/mm-broadband-modem-zte.c
+++ b/plugins/zte/mm-broadband-modem-zte.c
@@ -88,7 +88,7 @@ load_unlock_retries_ready (MMBaseModem *self,
mm_unlock_retries_set (retries, MM_MODEM_LOCK_SIM_PUK, puk1);
g_simple_async_result_set_op_res_gpointer (operation_result,
retries,
- (GDestroyNotify)g_object_unref);
+ g_object_unref);
} else {
g_simple_async_result_set_error (operation_result,
MM_CORE_ERROR,
--
2.12.2.564.g063fe858b8-goog
More information about the ModemManager-devel
mailing list