From a01aa55fada72e214a0005a7e0b0d6cb502741fe Mon Sep 17 00:00:00 2001 From: Elliott Sales de Andrade Date: Fri, 10 Mar 2023 17:42:23 -0600 Subject: Make PurpleRequestFieldDatasheet into a GObject Not sure that we want to keep all of this, but probably we don't need this extra wrapper object long term, and `PurpleRequestDatasheet` (once made into a `GObject`) and `PurpleRequestFieldDatasheet` could merge. Testing Done: Compiled only. Reviewed at https://reviews.imfreedom.org/r/2340/ --- pidgin/gtkrequest.c | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-) (limited to 'pidgin') diff --git a/pidgin/gtkrequest.c b/pidgin/gtkrequest.c index bdb88586cd..58f8355929 100644 --- a/pidgin/gtkrequest.c +++ b/pidgin/gtkrequest.c @@ -1772,6 +1772,7 @@ datasheet_action_clicked(GtkButton *btn, PurpleRequestDatasheetAction *act) static GtkWidget * create_datasheet_field(PurpleRequestField *field, GtkSizeGroup *buttons_sg) { + PurpleRequestFieldDatasheet *dfield = PURPLE_REQUEST_FIELD_DATASHEET(field); PurpleRequestDatasheet *sheet; guint i, col_count; GType *col_types; @@ -1785,7 +1786,7 @@ create_datasheet_field(PurpleRequestField *field, GtkSizeGroup *buttons_sg) GtkWidget *buttons_box; const GList *it; - sheet = purple_request_field_datasheet_get_sheet(field); + sheet = purple_request_field_datasheet_get_sheet(dfield); main_box = gtk_box_new(GTK_ORIENTATION_HORIZONTAL, 0); col_count = purple_request_datasheet_get_column_count(sheet); @@ -2044,12 +2045,10 @@ pidgin_request_fields(const char *title, const char *primary, for (fl = field_list; fl != NULL; fl = fl->next) { PurpleRequestField *field = PURPLE_REQUEST_FIELD(fl->data); - PurpleRequestFieldType type; - type = purple_request_field_get_field_type(field); - - if (type == PURPLE_REQUEST_FIELD_DATASHEET) + if(PURPLE_IS_REQUEST_FIELD_DATASHEET(field)) { contains_resizable = TRUE; + } if(PURPLE_IS_REQUEST_FIELD_LABEL(field)) { rows++; @@ -2089,7 +2088,6 @@ pidgin_request_fields(const char *title, const char *primary, { size_t col_offset = 0; PurpleRequestField *field = PURPLE_REQUEST_FIELD(fl->data); - PurpleRequestFieldType type; GtkWidget *widget = NULL; gchar *field_label; @@ -2099,7 +2097,6 @@ pidgin_request_fields(const char *title, const char *primary, continue; } - type = purple_request_field_get_field_type(field); field_label = pidgin_request_escape(cpar, purple_request_field_get_label(field)); @@ -2160,10 +2157,11 @@ pidgin_request_fields(const char *title, const char *primary, widget = create_image_field(field); } else if(PURPLE_IS_REQUEST_FIELD_ACCOUNT(field)) { widget = create_account_field(field); - } else if (type == PURPLE_REQUEST_FIELD_DATASHEET) + } else if(PURPLE_IS_REQUEST_FIELD_DATASHEET(field)) { widget = create_datasheet_field(field, datasheet_buttons_sg); - else + } else { continue; + } } gtk_widget_set_sensitive(widget, -- cgit v1.2.1