[gnome-screenshot/wip/exalm/cleanups: 4/15] interactive-dialog: Split target_toggled_cb()



commit 08954b025dc1e7ee7ef6b916ba8ba2f6f6d361ba
Author: Alexander Mikhaylenko <alexm gnome org>
Date:   Fri Apr 3 06:17:50 2020 +0500

    interactive-dialog: Split target_toggled_cb()
    
    Extract set_mode() function. This will allow to switch to template later.

 src/screenshot-interactive-dialog.c | 23 +++++++++++++----------
 1 file changed, 13 insertions(+), 10 deletions(-)
---
diff --git a/src/screenshot-interactive-dialog.c b/src/screenshot-interactive-dialog.c
index 6e5f5ac..69f36d4 100644
--- a/src/screenshot-interactive-dialog.c
+++ b/src/screenshot-interactive-dialog.c
@@ -36,23 +36,26 @@ static GtkWidget *pointer_row = NULL;
 #define TARGET_TOGGLE_WINDOW  1
 #define TARGET_TOGGLE_AREA    2
 
+static void
+set_mode (gint mode)
+{
+  gboolean take_window_shot = (mode == TARGET_TOGGLE_WINDOW);
+  gboolean take_area_shot = (mode == TARGET_TOGGLE_AREA);
+
+  gtk_widget_set_sensitive (pointer_row, !take_area_shot);
+
+  screenshot_config->take_window_shot = take_window_shot;
+  screenshot_config->take_area_shot = take_area_shot;
+}
+
 static void
 target_toggled_cb (GtkToggleButton *button,
                    gpointer         data)
 {
   int target_toggle = GPOINTER_TO_INT (data);
-  gboolean take_area_shot, take_window_shot;
 
   if (gtk_toggle_button_get_active (button))
-    {
-      take_window_shot = (target_toggle == TARGET_TOGGLE_WINDOW);
-      take_area_shot = (target_toggle == TARGET_TOGGLE_AREA);
-
-      gtk_widget_set_sensitive (pointer_row, !take_area_shot);
-
-      screenshot_config->take_window_shot = take_window_shot;
-      screenshot_config->take_area_shot = take_area_shot;
-    }
+    set_mode (target_toggle);
 }
 
 static void


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]