Remove unnecessary casts for GObject methods
Testing Done:
Compiled with the turtles and verified no new warnings appeared.
Reviewed at https://reviews.imfreedom.org/r/2956/
--- a/finch/gntaccount.c Tue Feb 20 00:40:30 2024 -0600
+++ b/finch/gntaccount.c Tue Feb 20 00:55:28 2024 -0600
@@ -146,7 +146,7 @@
* only unref this new instance here. The pointer will remain valid as
* the account manager adds a reference.
- g_object_unref(G_OBJECT(account));
+ g_object_unref(account); account = dialog->account;
info = PURPLE_CONTACT_INFO(account);
--- a/finch/gntmedia.c Tue Feb 20 00:40:30 2024 -0600
+++ b/finch/gntmedia.c Tue Feb 20 00:55:28 2024 -0600
@@ -315,11 +315,12 @@
finch_media_new(PurpleMedia *media)
- return GNT_WIDGET(g_object_new(finch_media_get_type(),
--- a/finch/gntroomlist.c Tue Feb 20 00:40:30 2024 -0600
+++ b/finch/gntroomlist.c Tue Feb 20 00:55:28 2024 -0600
@@ -217,7 +217,7 @@
froomlist.window = window = gnt_window_new();
- g_object_set(G_OBJECT(window), "vertical", TRUE, NULL);
+ g_object_set(window, "vertical", TRUE, NULL); gnt_box_set_pad(GNT_BOX(window), 0);
gnt_box_set_title(GNT_BOX(window), _("Room List"));
gnt_box_set_alignment(GNT_BOX(window), GNT_ALIGN_MID);
--- a/finch/plugins/grouping/grouping.c Tue Feb 20 00:40:30 2024 -0600
+++ b/finch/plugins/grouping/grouping.c Tue Feb 20 00:55:28 2024 -0600
@@ -193,7 +193,7 @@
GntTree *tree = finch_blist_get_tree();
- g_object_set(G_OBJECT(tree), "expander-level", 0, NULL);
+ g_object_set(tree, "expander-level", 0, NULL); @@ -201,7 +201,7 @@
GntTree *tree = finch_blist_get_tree();
- g_object_set(G_OBJECT(tree), "expander-level", 1, NULL);
+ g_object_set(tree, "expander-level", 1, NULL); --- a/libpurple/media.c Tue Feb 20 00:40:30 2024 -0600
+++ b/libpurple/media.c Tue Feb 20 00:55:28 2024 -0600
@@ -495,7 +495,7 @@
g_return_val_if_fail(PURPLE_IS_MEDIA(media), NULL);
- g_object_get(G_OBJECT(media), "account", &account, NULL);
+ g_object_get(media, "account", &account, NULL); @@ -504,7 +504,7 @@
g_return_val_if_fail(PURPLE_IS_MEDIA(media), NULL);
- g_object_get(G_OBJECT(media), "protocol-data", &protocol_data, NULL);
+ g_object_get(media, "protocol-data", &protocol_data, NULL); @@ -512,7 +512,7 @@
purple_media_set_protocol_data(PurpleMedia *media, gpointer protocol_data)
g_return_if_fail(PURPLE_IS_MEDIA(media));
- g_object_set(G_OBJECT(media), "protocol-data", protocol_data, NULL);
+ g_object_set(media, "protocol-data", protocol_data, NULL); --- a/libpurple/mediamanager.c Tue Feb 20 00:40:30 2024 -0600
+++ b/libpurple/mediamanager.c Tue Feb 20 00:55:28 2024 -0600
@@ -215,7 +215,7 @@
static PurpleMediaManager *manager = NULL;
- manager = PURPLE_MEDIA_MANAGER(g_object_new(purple_media_manager_get_type(), NULL));
+ manager = g_object_new(PURPLE_TYPE_MEDIA_MANAGER, NULL); @@ -286,12 +286,13 @@
- media = PURPLE_MEDIA(g_object_new(purple_media_get_type(),
- "conference-type", conference_type,
- "initiator", initiator,
+ "conference-type", conference_type, + "initiator", initiator, signals[SIG_INIT_PRIVATE_MEDIA] :
@@ -591,7 +592,7 @@
GstElement *capsfilter = gst_bin_get_by_name(GST_BIN(src), "protocol_video_caps");
- g_object_set(G_OBJECT(capsfilter), "caps", caps, NULL);
+ g_object_set(capsfilter, "caps", caps, NULL); gst_object_unref (capsfilter);
@@ -1058,7 +1059,7 @@
videoscale = gst_element_factory_make("videoscale", NULL);
capsfilter = gst_element_factory_make("capsfilter", "protocol_video_caps");
- g_object_set(G_OBJECT(capsfilter),
+ g_object_set(capsfilter, "caps", purple_media_manager_get_video_caps(manager), NULL);
gst_bin_add_many(GST_BIN(bin), videoscale, capsfilter, NULL);
@@ -1353,12 +1354,10 @@
G_OBJECT_GET_CLASS(ow->sink);
if (g_object_class_find_property(klass, "sync")) {
- g_object_set(G_OBJECT(ow->sink),
+ g_object_set(ow->sink, "sync", FALSE, NULL); if (g_object_class_find_property(klass, "async")) {
- g_object_set(G_OBJECT(ow->sink),
+ g_object_set(ow->sink, "async", FALSE, NULL); --- a/libpurple/plugins/kwallet/purplekwallet.cpp Tue Feb 20 00:40:30 2024 -0600
+++ b/libpurple/plugins/kwallet/purplekwallet.cpp Tue Feb 20 00:55:28 2024 -0600
@@ -114,7 +114,7 @@
*****************************************************************************/
PurpleKWalletPlugin::Request::Request(const QString &key, GTask *task) {
- this->task = G_TASK(g_object_ref(G_OBJECT(task)));
+ this->task = g_object_ref(task); PurpleKWalletPlugin::Request::~Request(void) {
@@ -414,7 +414,7 @@
key = purple_kwallet_provider_account_key(account);
- task = g_task_new(G_OBJECT(provider), cancellable, callback, data);
+ task = g_task_new(provider, cancellable, callback, data); /* We manually set the task name otherwise the (gpointer) cast ends up in
@@ -450,7 +450,7 @@
- task = g_task_new(G_OBJECT(provider), cancellable, callback, data);
+ task = g_task_new(provider, cancellable, callback, data); /* We manually set the task name otherwise the (gpointer) cast ends up in
@@ -487,7 +487,7 @@
- task = g_task_new(G_OBJECT(provider), cancellable, callback, data);
+ task = g_task_new(provider, cancellable, callback, data); /* We manually set the task name otherwise the (gpointer) cast ends up in
--- a/libpurple/plugins/libsecret/libsecret.c Tue Feb 20 00:40:30 2024 -0600
+++ b/libpurple/plugins/libsecret/libsecret.c Tue Feb 20 00:55:28 2024 -0600
@@ -93,7 +93,7 @@
g_task_return_pointer(task, password, g_free);
- g_object_unref(G_OBJECT(task));
@@ -112,7 +112,7 @@
g_task_return_boolean(task, ret);
- g_object_unref(G_OBJECT(task));
@@ -135,7 +135,7 @@
g_task_return_boolean(task, TRUE);
- g_object_unref(G_OBJECT(task));
/******************************************************************************
@@ -151,7 +151,7 @@
PurpleContactInfo *info = PURPLE_CONTACT_INFO(account);
- task = g_task_new(G_OBJECT(provider), cancellable, callback, data);
+ task = g_task_new(provider, cancellable, callback, data); g_task_set_source_tag(task, purple_libsecret_read_password_async);
secret_password_lookup(&purple_libsecret_schema, cancellable,
@@ -185,7 +185,7 @@
const gchar *username = NULL;
- task = g_task_new(G_OBJECT(provider), cancellable, callback, data);
+ task = g_task_new(provider, cancellable, callback, data); g_task_set_source_tag(task, purple_libsecret_write_password_async);
username = purple_contact_info_get_username(info);
@@ -221,7 +221,7 @@
PurpleContactInfo *info = PURPLE_CONTACT_INFO(account);
- task = g_task_new(G_OBJECT(provider), cancellable, callback, data);
+ task = g_task_new(provider, cancellable, callback, data); g_task_set_source_tag(task, purple_libsecret_clear_password_async);
secret_password_clear(&purple_libsecret_schema, cancellable,
--- a/libpurple/plugins/purple-toast/purple-toast.c Tue Feb 20 00:40:30 2024 -0600
+++ b/libpurple/plugins/purple-toast/purple-toast.c Tue Feb 20 00:55:28 2024 -0600
@@ -64,7 +64,7 @@
- g_object_unref(G_OBJECT(notification));
+ g_object_unref(notification); @@ -102,9 +102,9 @@
icon = g_file_icon_new(file);
- g_object_unref(G_OBJECT(file));
- g_object_unref(G_OBJECT(image));
@@ -137,7 +137,7 @@
purple_toast_show_notification(title, message, icon);
- g_object_unref(G_OBJECT(icon));
@@ -182,7 +182,7 @@
- g_object_unref(G_OBJECT(icon));
--- a/libpurple/plugins/wincred/wincred.c Tue Feb 20 00:40:30 2024 -0600
+++ b/libpurple/plugins/wincred/wincred.c Tue Feb 20 00:55:28 2024 -0600
@@ -95,13 +95,13 @@
PCREDENTIALW credential = NULL;
- task = g_task_new(G_OBJECT(provider), cancellable, callback, data);
+ task = g_task_new(provider, cancellable, callback, data); g_task_set_source_tag(task, purple_wincred_read_password_async);
target_name = wincred_get_target_name(account, &error);
if (target_name == NULL) {
g_task_return_error(task, error);
- g_object_unref(G_OBJECT(task));
@@ -136,7 +136,7 @@
g_task_return_error(task, error);
- g_object_unref(G_OBJECT(task));
@@ -154,7 +154,7 @@
_("Cannot read password (unicode error)."));
g_task_return_error(task, error);
- g_object_unref(G_OBJECT(task));
PurpleContactInfo *info = PURPLE_CONTACT_INFO(account);
@@ -164,7 +164,7 @@
g_task_return_pointer(task, password, g_free);
- g_object_unref(G_OBJECT(task));
@@ -194,13 +194,13 @@
- task = g_task_new(G_OBJECT(provider), cancellable, callback, data);
+ task = g_task_new(provider, cancellable, callback, data); g_task_set_source_tag(task, purple_wincred_write_password_async);
target_name = wincred_get_target_name(account, &error);
if (target_name == NULL) {
g_task_return_error(task, error);
- g_object_unref(G_OBJECT(task));
@@ -210,7 +210,7 @@
purple_debug_error("keyring-wincred", "Couldn't convert username");
g_task_return_error(task, error);
- g_object_unref(G_OBJECT(task));
@@ -220,7 +220,7 @@
purple_debug_error("keyring-wincred", "Couldn't convert password");
g_task_return_error(task, error);
- g_object_unref(G_OBJECT(task));
@@ -263,7 +263,7 @@
g_task_return_boolean(task, TRUE);
- g_object_unref(G_OBJECT(task));
@@ -286,13 +286,13 @@
gunichar2 *target_name = NULL;
- task = g_task_new(G_OBJECT(provider), cancellable, callback, data);
+ task = g_task_new(provider, cancellable, callback, data); g_task_set_source_tag(task, purple_wincred_clear_password_async);
target_name = wincred_get_target_name(account, &error);
if (target_name == NULL) {
g_task_return_error(task, error);
- g_object_unref(G_OBJECT(task));
@@ -328,7 +328,7 @@
- g_object_unref(G_OBJECT(task));
@@ -376,13 +376,12 @@
static PurpleCredentialProvider *
- return PURPLE_CREDENTIAL_PROVIDER(g_object_new(
"description", _(WINCRED_DESCRIPTION),
/******************************************************************************
--- a/libpurple/protocols/bonjour/bonjour.c Tue Feb 20 00:40:30 2024 -0600
+++ b/libpurple/protocols/bonjour/bonjour.c Tue Feb 20 00:55:28 2024 -0600
@@ -724,7 +724,7 @@
bonjour_protocol_new(void) {
- return PURPLE_PROTOCOL(g_object_new(
"name", "Bonjour (Deprecated)",
@@ -733,7 +733,7 @@
"icon-name", "im-bonjour",
"icon-resource-path", "/im/pidgin/libpurple/bonjour/icons",
"options", OPT_PROTO_NO_PASSWORD,
static GPluginPluginInfo *
--- a/libpurple/protocols/bonjour/xmpp.c Tue Feb 20 00:40:30 2024 -0600
+++ b/libpurple/protocols/bonjour/xmpp.c Tue Feb 20 00:55:28 2024 -0600
@@ -1262,7 +1262,7 @@
g_clear_object(&bconv->input);
g_clear_object(&bconv->output);
- g_object_unref(G_OBJECT(bconv->tx_buf));
+ g_object_unref(bconv->tx_buf); if (bconv->stream_data != NULL) {
struct _stream_start_data *ss = bconv->stream_data;
--- a/libpurple/protocols/gg/gg.c Tue Feb 20 00:40:30 2024 -0600
+++ b/libpurple/protocols/gg/gg.c Tue Feb 20 00:55:28 2024 -0600
@@ -1348,14 +1348,14 @@
- return PURPLE_PROTOCOL(g_object_new(
"description", _("Gadu-Gadu is a Polish instant messaging network."),
"icon-name", "im-gadu-gadu",
"icon-resource-path", "/im/pidgin/libpurple/gg/icons",
static GPluginPluginInfo *
--- a/libpurple/protocols/gg/message-prpl.c Tue Feb 20 00:40:30 2024 -0600
+++ b/libpurple/protocols/gg/message-prpl.c Tue Feb 20 00:55:28 2024 -0600
@@ -257,7 +257,7 @@
purple_conversation_write_message(im, pmsg);
- g_object_unref(G_OBJECT(pmsg));
purple_debug_error("gg", "ggp_message_got_display: "
"unexpected message type: %d\n", msg->type);
--- a/libpurple/protocols/gg/resolver-purple.c Tue Feb 20 00:40:30 2024 -0600
+++ b/libpurple/protocols/gg/resolver-purple.c Tue Feb 20 00:55:28 2024 -0600
@@ -84,7 +84,7 @@
- g_object_unref(G_OBJECT(data->cancellable));
+ g_object_unref(data->cancellable); data->cancellable = NULL;
for(l = addresses; l; l = l->next) {
@@ -193,7 +193,7 @@
if (G_IS_CANCELLABLE(data->cancellable)) {
g_cancellable_cancel(data->cancellable);
- g_object_unref(G_OBJECT(data->cancellable));
+ g_object_unref(data->cancellable); --- a/libpurple/protocols/jabber/jabber.c Tue Feb 20 00:40:30 2024 -0600
+++ b/libpurple/protocols/jabber/jabber.c Tue Feb 20 00:55:28 2024 -0600
@@ -1056,7 +1056,7 @@
g_clear_handle_id(&js->conn_close_timeout, g_source_remove);
g_cancellable_cancel(js->cancellable);
- g_object_unref(G_OBJECT(js->cancellable));
+ g_object_unref(js->cancellable); --- a/libpurple/protocols/jabber/si.c Tue Feb 20 00:40:30 2024 -0600
+++ b/libpurple/protocols/jabber/si.c Tue Feb 20 00:55:28 2024 -0600
@@ -1783,7 +1783,7 @@
if(jsx->stream_method == STREAM_METHOD_UNKNOWN) {
- g_object_unref(G_OBJECT(jsx));
@@ -1857,7 +1857,7 @@
- g_object_unref(G_OBJECT(jsx->ibb_buffer));
+ g_object_unref(jsx->ibb_buffer); purple_debug_info("jabber", "jabber_si_xfer_free(): freeing jsx %p\n", jsx);
--- a/libpurple/purpleattachment.c Tue Feb 20 00:40:30 2024 -0600
+++ b/libpurple/purpleattachment.c Tue Feb 20 00:55:28 2024 -0600
@@ -224,12 +224,11 @@
purple_attachment_new(guint64 id, const gchar *content_type) {
g_return_val_if_fail(content_type != NULL, NULL);
- return PURPLE_ATTACHMENT(g_object_new(
"content-type", content_type,
--- a/libpurple/purpleconversation.c Tue Feb 20 00:40:30 2024 -0600
+++ b/libpurple/purpleconversation.c Tue Feb 20 00:55:28 2024 -0600
@@ -295,7 +295,7 @@
purple_signal_emit(handle, "sent-im-msg", account, msg);
- g_object_unref(G_OBJECT(msg));
} else if(PURPLE_IS_CHAT_CONVERSATION(conv)) {
gint id = purple_chat_conversation_get_id(PURPLE_CHAT_CONVERSATION(conv));
@@ -310,7 +310,7 @@
purple_signal_emit(handle, "sent-chat-msg", account, msg, id);
- g_object_unref(G_OBJECT(msg));
--- a/libpurple/purplecredentialmanager.c Tue Feb 20 00:40:30 2024 -0600
+++ b/libpurple/purplecredentialmanager.c Tue Feb 20 00:55:28 2024 -0600
@@ -72,7 +72,7 @@
/* Clean up our initial reference to the task. */
- g_object_unref(G_OBJECT(task));
@@ -95,7 +95,7 @@
/* Clean up our initial reference to the task. */
- g_object_unref(G_OBJECT(task));
@@ -118,7 +118,7 @@
/* Clean up our initial reference to the task. */
- g_object_unref(G_OBJECT(task));
/******************************************************************************
@@ -426,7 +426,7 @@
if(PURPLE_IS_CREDENTIAL_PROVIDER(manager->active)) {
- previous = PURPLE_CREDENTIAL_PROVIDER(g_object_ref(manager->active));
+ previous = g_object_ref(manager->active); if(g_set_object(&manager->active, provider)) {
@@ -494,7 +494,7 @@
g_task_return_new_error(task, PURPLE_CREDENTIAL_MANAGER_DOMAIN, 0,
_("can not read password, no active "
- g_object_unref(G_OBJECT(task));
@@ -533,7 +533,7 @@
g_task_return_new_error(task, PURPLE_CREDENTIAL_MANAGER_DOMAIN, 0,
_("account \"%s\" is not marked to be stored"),
- g_object_unref(G_OBJECT(task));
@@ -549,7 +549,7 @@
_("can not write password, no active "
- g_object_unref(G_OBJECT(task));
@@ -591,7 +591,7 @@
_("can not clear password, no active "
- g_object_unref(G_OBJECT(task));
--- a/libpurple/purplegdkpixbuf.c Tue Feb 20 00:40:30 2024 -0600
+++ b/libpurple/purplegdkpixbuf.c Tue Feb 20 00:55:28 2024 -0600
@@ -137,7 +137,7 @@
"failed with size=%" G_GSIZE_FORMAT ": %s\n", count,
error ? error->message : "(no error message)");
- g_object_unref(G_OBJECT(loader));
+ g_object_unref(loader); @@ -146,7 +146,7 @@
"failed for image of size %" G_GSIZE_FORMAT ": %s\n", count,
error ? error->message : "(no error message)");
- g_object_unref(G_OBJECT(loader));
+ g_object_unref(loader); @@ -159,12 +159,12 @@
"of size %" G_GSIZE_FORMAT "\n",
animated ? "gdk_pixbuf_loader_get_animation"
: "gdk_pixbuf_loader_get_pixbuf", count);
- g_object_unref(G_OBJECT(loader));
+ g_object_unref(loader); - g_object_unref(G_OBJECT(loader));
+ g_object_unref(loader); --- a/libpurple/purplehistorymanager.c Tue Feb 20 00:40:30 2024 -0600
+++ b/libpurple/purplehistorymanager.c Tue Feb 20 00:55:28 2024 -0600
@@ -190,7 +190,7 @@
- g_object_ref(G_OBJECT(adapter));
id = purple_history_adapter_get_id(adapter);
@@ -206,7 +206,7 @@
- g_object_unref(G_OBJECT(adapter));
+ g_object_unref(adapter); --- a/libpurple/purplemessage.c Tue Feb 20 00:40:30 2024 -0600
+++ b/libpurple/purplemessage.c Tue Feb 20 00:55:28 2024 -0600
@@ -450,13 +450,14 @@
dt = g_date_time_new_now_local();
/* who may be NULL for outgoing MUC messages */
- message = PURPLE_MESSAGE(g_object_new(PURPLE_TYPE_MESSAGE,
+ message = g_object_new(
@@ -481,13 +482,14 @@
dt = g_date_time_new_from_unix_local((gint64)timestamp);
- message = PURPLE_MESSAGE(g_object_new(PURPLE_TYPE_MESSAGE,
+ message = g_object_new(
@@ -505,11 +507,12 @@
flags |= PURPLE_MESSAGE_SYSTEM;
dt = g_date_time_new_now_local();
- message = PURPLE_MESSAGE(g_object_new(PURPLE_TYPE_MESSAGE,
+ message = g_object_new(
@@ -702,7 +705,7 @@
return g_hash_table_insert(message->attachments,
purple_attachment_get_hash_key(attachment),
- g_object_ref(G_OBJECT(attachment)));
+ g_object_ref(attachment)); @@ -720,7 +723,7 @@
attachment = g_hash_table_lookup(message->attachments, &id);
if(PURPLE_IS_ATTACHMENT(attachment)) {
- return PURPLE_ATTACHMENT(g_object_ref(G_OBJECT(attachment)));
+ return g_object_ref(attachment); --- a/libpurple/purplenoopcredentialprovider.c Tue Feb 20 00:40:30 2024 -0600
+++ b/libpurple/purplenoopcredentialprovider.c Tue Feb 20 00:55:28 2024 -0600
@@ -42,14 +42,14 @@
- task = g_task_new(G_OBJECT(provider), cancellable, callback, data);
+ task = g_task_new(provider, cancellable, callback, data); g_task_set_source_tag(task,
purple_noop_credential_provider_read_password_async);
g_task_return_new_error(task, PURPLE_CREDENTIAL_MANAGER_DOMAIN, 0,
_("provider does not store passwords"));
- g_object_unref(G_OBJECT(task));
@@ -70,14 +70,14 @@
- task = g_task_new(G_OBJECT(provider), cancellable, callback, data);
+ task = g_task_new(provider, cancellable, callback, data); g_task_set_source_tag(task,
purple_noop_credential_provider_write_password_async);
g_task_return_new_error(task, PURPLE_CREDENTIAL_MANAGER_DOMAIN, 0,
_("provider does not store passwords"));
- g_object_unref(G_OBJECT(task));
@@ -117,11 +117,10 @@
*****************************************************************************/
PurpleCredentialProvider *
purple_noop_credential_provider_new(void) {
- return PURPLE_CREDENTIAL_PROVIDER(g_object_new(
PURPLE_TYPE_NOOP_CREDENTIAL_PROVIDER,
"description", _("Passwords will not be saved."),
--- a/libpurple/purpleprotocolmanager.c Tue Feb 20 00:40:30 2024 -0600
+++ b/libpurple/purpleprotocolmanager.c Tue Feb 20 00:55:28 2024 -0600
@@ -260,7 +260,7 @@
* will need to pass it to the signal emission after it's removed from the
* hash table that'll unref it.
- g_object_ref(G_OBJECT(protocol));
+ g_object_ref(protocol); id = purple_protocol_get_id(protocol);
@@ -292,7 +292,7 @@
- g_object_unref(G_OBJECT(protocol));
+ g_object_unref(protocol); --- a/libpurple/purplesavedpresence.c Tue Feb 20 00:40:30 2024 -0600
+++ b/libpurple/purplesavedpresence.c Tue Feb 20 00:55:28 2024 -0600
@@ -145,7 +145,7 @@
if(G_IS_SETTINGS(settings)) {
- g_object_get(G_OBJECT(settings), "schema-id", &schema_id, NULL);
+ g_object_get(settings, "schema-id", &schema_id, NULL); if(!purple_strequal("im.pidgin.Purple.SavedPresence", schema_id)) {
g_warning("expected schema id of im.pidgin.Purple.SavedPresence, "
--- a/libpurple/server.c Tue Feb 20 00:40:30 2024 -0600
+++ b/libpurple/server.c Tue Feb 20 00:55:28 2024 -0600
@@ -390,7 +390,7 @@
pmsg = purple_message_new_incoming(name, message, flags, mtime);
purple_conversation_write_message(im, pmsg);
- g_object_unref(G_OBJECT(pmsg));
@@ -678,7 +678,7 @@
- g_object_unref(G_OBJECT(pmsg));
void purple_serv_send_file(PurpleConnection *gc, const char *who, const char *file)
--- a/libpurple/tests/test_channel_join_details.c Tue Feb 20 00:40:30 2024 -0600
+++ b/libpurple/tests/test_channel_join_details.c Tue Feb 20 00:55:28 2024 -0600
@@ -52,7 +52,8 @@
"password-supported", TRUE,
- g_object_get(G_OBJECT(details),
"nickname-supported", &nickname_supported,
--- a/libpurple/tests/test_credential_provider.c Tue Feb 20 00:40:30 2024 -0600
+++ b/libpurple/tests/test_credential_provider.c Tue Feb 20 00:55:28 2024 -0600
@@ -109,7 +109,7 @@
g_assert_error(error, PURPLE_CREDENTIAL_PROVIDER_DOMAIN, 0);
- g_object_unref(G_OBJECT(provider));
+ g_object_unref(provider); @@ -126,7 +126,7 @@
g_assert_error(error, PURPLE_CREDENTIAL_PROVIDER_DOMAIN, 1);
- g_object_unref(G_OBJECT(provider));
+ g_object_unref(provider); @@ -151,7 +151,7 @@
g_assert_error(error, PURPLE_CREDENTIAL_PROVIDER_DOMAIN, 2);
- g_object_unref(G_OBJECT(provider));
+ g_object_unref(provider); @@ -176,7 +176,7 @@
g_assert_error(error, PURPLE_CREDENTIAL_PROVIDER_DOMAIN, 3);
- g_object_unref(G_OBJECT(provider));
+ g_object_unref(provider); @@ -202,7 +202,7 @@
g_assert_no_error(error);
- g_object_unref(G_OBJECT(provider));
+ g_object_unref(provider); /******************************************************************************
@@ -242,7 +242,7 @@
task = g_task_new(p, cancellable, callback, data);
g_task_return_pointer(task, NULL, NULL);
- g_object_unref(G_OBJECT(task));
@@ -272,7 +272,7 @@
task = g_task_new(p, cancellable, callback, data);
g_task_return_boolean(task, TRUE);
- g_object_unref(G_OBJECT(task));
@@ -301,7 +301,7 @@
task = g_task_new(p, cancellable, callback, data);
g_task_return_boolean(task, TRUE);
- g_object_unref(G_OBJECT(task));
--- a/libpurple/tests/test_image.c Tue Feb 20 00:40:30 2024 -0600
+++ b/libpurple/tests/test_image.c Tue Feb 20 00:55:28 2024 -0600
@@ -88,7 +88,7 @@
g_assert_cmpstr(purple_image_get_extension(image), ==, ext);
g_assert_cmpstr(purple_image_get_mimetype(image), ==, mimetype);
- g_object_unref(G_OBJECT(image));
/******************************************************************************
--- a/pidgin/gtkmedia.c Tue Feb 20 00:40:30 2024 -0600
+++ b/pidgin/gtkmedia.c Tue Feb 20 00:55:28 2024 -0600
@@ -681,7 +681,7 @@
sink = gst_bin_get_by_name(GST_BIN(pipeline), "gtksink");
- g_object_get(G_OBJECT(sink), "widget", &remote_video, NULL);
+ g_object_get(sink, "widget", &remote_video, NULL); gtk_widget_set_visible(remote_video, TRUE);
gtk_widget_set_vexpand(remote_video, TRUE);
gtk_box_append(GTK_BOX(recv_widget), remote_video);
@@ -703,7 +703,7 @@
sink = gst_bin_get_by_name(GST_BIN(pipeline), "gtksink");
- g_object_get(G_OBJECT(sink), "widget", &local_video, NULL);
+ g_object_get(sink, "widget", &local_video, NULL); gtk_widget_set_visible(local_video, TRUE);
gtk_widget_set_vexpand(local_video, TRUE);
gtk_box_append(GTK_BOX(send_widget), local_video);
--- a/pidgin/gtknotify.c Tue Feb 20 00:40:30 2024 -0600
+++ b/pidgin/gtknotify.c Tue Feb 20 00:55:28 2024 -0600
@@ -428,7 +428,7 @@
treeview = gtk_tree_view_new_with_model(GTK_TREE_MODEL(model));
- g_object_unref(G_OBJECT(model));
gtk_widget_set_size_request(treeview, 500, 400);
gtk_tree_selection_set_mode(gtk_tree_view_get_selection(GTK_TREE_VIEW(treeview)),
--- a/pidgin/gtkrequest.c Tue Feb 20 00:40:30 2024 -0600
+++ b/pidgin/gtkrequest.c Tue Feb 20 00:55:28 2024 -0600
@@ -1181,8 +1181,8 @@
purple_request_field_image_get_scale_y(ifield) * gdk_pixbuf_get_height(buf),
widget = gtk_image_new_from_pixbuf(scale);
- g_object_unref(G_OBJECT(buf));
- g_object_unref(G_OBJECT(scale));
@@ -1678,7 +1678,7 @@
view = GTK_TREE_VIEW(gtk_tree_view_new_with_model(
g_object_set_data(G_OBJECT(model), "sheet-widget", main_box);
- g_object_unref(G_OBJECT(model));
id_column = gtk_tree_view_column_new();
gtk_tree_view_column_set_visible(id_column, FALSE);
--- a/pidgin/gtkwhiteboard.c Tue Feb 20 00:40:30 2024 -0600
+++ b/pidgin/gtkwhiteboard.c Tue Feb 20 00:55:28 2024 -0600
@@ -476,7 +476,7 @@
- gtkwb = PIDGIN_WHITEBOARD(g_object_new(PIDGIN_TYPE_WHITEBOARD, NULL));
+ gtkwb = g_object_new(PIDGIN_TYPE_WHITEBOARD, NULL); g_object_set_data_full(G_OBJECT(wb), UI_DATA, gtkwb, g_object_unref);
--- a/pidgin/pidginabout.c Tue Feb 20 00:40:30 2024 -0600
+++ b/pidgin/pidginabout.c Tue Feb 20 00:55:28 2024 -0600
@@ -141,7 +141,7 @@
- g_object_unref(G_OBJECT(parser));
+ g_object_unref(parser); g_input_stream_close(istream, NULL, NULL);
--- a/pidgin/pidginavatar.c Tue Feb 20 00:40:30 2024 -0600
+++ b/pidgin/pidginavatar.c Tue Feb 20 00:55:28 2024 -0600
@@ -548,7 +548,7 @@
*****************************************************************************/
pidgin_avatar_new(void) {
- return GTK_WIDGET(g_object_new(PIDGIN_TYPE_AVATAR, NULL));
+ return g_object_new(PIDGIN_TYPE_AVATAR, NULL); --- a/pidgin/pidgindebug.c Tue Feb 20 00:40:30 2024 -0600
+++ b/pidgin/pidgindebug.c Tue Feb 20 00:55:28 2024 -0600
@@ -459,9 +459,7 @@
scroll = view_near_bottom(win);
for (i = 0; i <= PURPLE_DEBUG_FATAL; i++) {
- g_object_set(G_OBJECT(win->tags.level[i]),
- "invisible", i < level,
+ g_object_set(win->tags.level[i], "invisible", i < level, NULL); gtk_text_view_scroll_to_mark(GTK_TEXT_VIEW(win->textview),
--- a/pidgin/pidgininvitedialog.c Tue Feb 20 00:40:30 2024 -0600
+++ b/pidgin/pidgininvitedialog.c Tue Feb 20 00:55:28 2024 -0600
@@ -175,10 +175,10 @@
*****************************************************************************/
pidgin_invite_dialog_new(PurpleChatConversation *conversation) {
- return GTK_WIDGET(g_object_new(
PIDGIN_TYPE_INVITE_DIALOG,
"conversation", conversation,
--- a/pidgin/pidginpluginsdialog.c Tue Feb 20 00:40:30 2024 -0600
+++ b/pidgin/pidginpluginsdialog.c Tue Feb 20 00:55:28 2024 -0600
@@ -96,5 +96,5 @@
*****************************************************************************/
pidgin_plugins_dialog_new(void) {
- return GTK_WIDGET(g_object_new(PIDGIN_TYPE_PLUGINS_DIALOG, NULL));
+ return g_object_new(PIDGIN_TYPE_PLUGINS_DIALOG, NULL); --- a/pidgin/prefs/pidginconversationprefs.c Tue Feb 20 00:40:30 2024 -0600
+++ b/pidgin/prefs/pidginconversationprefs.c Tue Feb 20 00:55:28 2024 -0600
@@ -105,5 +105,5 @@
*****************************************************************************/
pidgin_conversation_prefs_new(void) {
- return GTK_WIDGET(g_object_new(PIDGIN_TYPE_CONVERSATION_PREFS, NULL));
+ return g_object_new(PIDGIN_TYPE_CONVERSATION_PREFS, NULL); --- a/pidgin/prefs/pidgincredentialprefs.c Tue Feb 20 00:40:30 2024 -0600
+++ b/pidgin/prefs/pidgincredentialprefs.c Tue Feb 20 00:55:28 2024 -0600
@@ -189,5 +189,5 @@
*****************************************************************************/
pidgin_credential_prefs_new(void) {
- return GTK_WIDGET(g_object_new(PIDGIN_TYPE_CREDENTIAL_PREFS, NULL));
+ return g_object_new(PIDGIN_TYPE_CREDENTIAL_PREFS, NULL); --- a/pidgin/prefs/pidgincredentialproviderrow.c Tue Feb 20 00:40:30 2024 -0600
+++ b/pidgin/prefs/pidgincredentialproviderrow.c Tue Feb 20 00:55:28 2024 -0600
@@ -187,9 +187,10 @@
pidgin_credential_provider_row_new(PurpleCredentialProvider *provider) {
g_return_val_if_fail(PURPLE_IS_CREDENTIAL_PROVIDER(provider), NULL);
- return GTK_WIDGET(g_object_new(PIDGIN_TYPE_CREDENTIAL_PROVIDER_ROW,
+ PIDGIN_TYPE_CREDENTIAL_PROVIDER_ROW, PurpleCredentialProvider *
--- a/pidgin/prefs/pidginnetworkprefs.c Tue Feb 20 00:40:30 2024 -0600
+++ b/pidgin/prefs/pidginnetworkprefs.c Tue Feb 20 00:55:28 2024 -0600
@@ -175,5 +175,5 @@
*****************************************************************************/
pidgin_network_prefs_new(void) {
- return GTK_WIDGET(g_object_new(PIDGIN_TYPE_NETWORK_PREFS, NULL));
+ return g_object_new(PIDGIN_TYPE_NETWORK_PREFS, NULL); --- a/pidgin/prefs/pidginvvprefs.c Tue Feb 20 00:40:30 2024 -0600
+++ b/pidgin/prefs/pidginvvprefs.c Tue Feb 20 00:55:28 2024 -0600
@@ -603,7 +603,7 @@
*****************************************************************************/
pidgin_vv_prefs_new(void) {
- return GTK_WIDGET(g_object_new(PIDGIN_TYPE_VV_PREFS, NULL));
+ return g_object_new(PIDGIN_TYPE_VV_PREFS, NULL);