[Libguestfs] [p2v PATCH 05/11] gui: factor out tgl_btn_is_act()

Laszlo Ersek lersek at redhat.com
Mon Jan 30 14:22:22 UTC 2023


Repeat the previous refactoring for

  gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (toggle_button))

which is another source of overlong lines in "gui.c". The new function is
again used in both the connection dialog and the conversion dialog.

Cc: Alban Lecorps <alban.lecorps at ubisoft.com>
Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1792141
Signed-off-by: Laszlo Ersek <lersek at redhat.com>
---
 gui.c | 16 +++++++++++++---
 1 file changed, 13 insertions(+), 3 deletions(-)

diff --git a/gui.c b/gui.c
index 5c53883ff2ae..3c0f21ca87d0 100644
--- a/gui.c
+++ b/gui.c
@@ -160,6 +160,16 @@ entry_text_dup (GtkWidget *entry)
   return strdup (gtk_entry_get_text (GTK_ENTRY (entry)));
 }
 
+/**
+ * Trivial helper (shorthand) function for getting the active/inactive state
+ * of a GTK_TOGGLE_BUTTON.
+ */
+static bool
+tgl_btn_is_act (GtkWidget *toggle_button)
+{
+  return gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (toggle_button));
+}
+
 /*----------------------------------------------------------------------*/
 /* Connection dialog. */
 
@@ -360,7 +370,7 @@ username_changed_callback (GtkWidget *w, gpointer data)
 
   str = gtk_entry_get_text (GTK_ENTRY (username_entry));
   username_is_root = str != NULL && STREQ (str, "root");
-  sudo_is_set = gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (sudo_button));
+  sudo_is_set = tgl_btn_is_act (sudo_button);
 
   /* The sudo button is sensitive if:
    * - The username is not "root", or
@@ -470,7 +480,7 @@ test_connection_clicked (GtkWidget *w, gpointer data)
     config->auth.identity.url = NULL;
   config->auth.identity.file_needs_update = 1;
 
-  config->auth.sudo = gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (sudo_button));
+  config->auth.sudo = tgl_btn_is_act (sudo_button);
 
   if (errors)
     return;
@@ -1713,7 +1723,7 @@ vcpus_or_memory_check_callback (GtkWidget *w, gpointer data)
 static bool
 get_phys_topo_from_conv_dlg (void)
 {
-  return gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (vcpu_topo));
+  return tgl_btn_is_act (vcpu_topo);
 }
 
 static int



More information about the Libguestfs mailing list