Refactor GtkStatusStack update event sources.gtk4

2 files changed, 41 insertions, 21 deletions

diff --git a/src/gtkext/statusstack.c b/src/gtkext/statusstack.c
index fb65ddd..92c296a 100644
--- a/src/gtkext/statusstack.c
+++ b/src/gtkext/statusstack.c
@@ -56,6 +56,10 @@ typedef enum _StatusStackProperty {
 static GParamSpec *_status_stack_properties[N_PROPERTIES] = { NULL, };
 
 
+/* Source d'affichage par défaut */
+#define gtk_status_stack_default_source gtk_status_stack_show_simple_message
+
+
 /* Initialise la classe des barres de statut améliorées. */
 static void gtk_status_stack_class_init(GtkStatusStackClass *);
 
@@ -123,7 +127,7 @@ static void init_navigation_info(navigation_info_t *);
 static void fini_navigation_info(navigation_info_t *);
 
 /* S'assure de l'affichage à jour de la partie "navigation". */
-static gboolean gtk_status_stack_show_current_location(GtkStatusStack *);
+static gboolean gtk_status_stack_show_current_location(gpointer);
 
 /* Réagit à un clic sur l'icône de zoom. */
 static void gtk_status_stack_on_zoom_icon_press(GtkEntry *, GtkEntryIconPosition, GtkStatusStack *);
@@ -173,7 +177,7 @@ static void fini_activity_info(activity_info_t *);
 static activity_status_t *find_activity_status_by_id(activity_info_t *, activity_id_t);
 
 /* S'assure de l'affichage à jour de la partie "activité". */
-static gboolean gtk_status_stack_show_current_activity(GtkStatusStack *);
+static gboolean gtk_status_stack_show_current_activity(gpointer);
 
 
 
@@ -264,7 +268,7 @@ static void gtk_status_stack_init(GtkStatusStack *stack)
 {
     gtk_widget_init_template(GTK_WIDGET(stack));
 
-    stack->def_source = NULL;
+    stack->def_source = gtk_status_stack_default_source;
 
     stack->msg = NULL;
 
@@ -290,6 +294,10 @@ static void gtk_status_stack_init(GtkStatusStack *stack)
     stack->network_update_tag = g_timeout_add(NETWORK_UPDATE_INTERVAL,
                                               G_SOURCE_FUNC(gtk_status_stack_update_network_stats), stack);
 
+    /* Premier affichage... */
+
+    gtk_status_stack_reset_to_default(stack);
+
 }
 
 
@@ -385,11 +393,15 @@ GtkStatusStack *gtk_status_stack_new(void)
 *                                                                             *
 ******************************************************************************/
 
-void gtk_status_stack_reset(GtkStatusStack *stack)
+void gtk_status_stack_reset_to_default(GtkStatusStack *stack)
 {
-    gtk_stack_set_visible_child_name(stack->main, "default");
+    /**
+     * Une amélioration possible serait de passer à g_idle_add_once(),
+     * et de s'affranchir des retours G_SOURCE_REMOVE systématiques,
+     * mais la fonction n'est disponible qu'à partir de la GLib 2.74.
+     */
 
-    stack->def_source = NULL;
+    g_idle_add(stack->def_source, stack);
 
 }
 
@@ -822,7 +834,7 @@ void gtk_status_stack_update_current_location(GtkStatusStack *stack, const mrang
 
 /******************************************************************************
 *                                                                             *
-*  Paramètres  : stack = pile de statuts à manipuler.                         *
+*  Paramètres  : data = pile de statuts à manipuler.                          *
 *                                                                             *
 *  Description : S'assure de l'affichage à jour de la partie "navigation".    *
 *                                                                             *
@@ -832,14 +844,15 @@ void gtk_status_stack_update_current_location(GtkStatusStack *stack, const mrang
 *                                                                             *
 ******************************************************************************/
 
-static gboolean gtk_status_stack_show_current_location(GtkStatusStack *stack)
+static gboolean gtk_status_stack_show_current_location(gpointer data)
 {
+    GtkStatusStack *stack;                  /* Version spécialisée         */
     navigation_info_t *info;                /* Informations à constituer   */
     char raw_pos[6 + VMPA_MAX_LEN + 1];     /* Formatage final en direct   */
 
-    stack->def_source = (GSourceFunc)gtk_status_stack_show_current_location;
+    stack = GTK_STATUS_STACK(data);
 
-    gtk_stack_set_visible_child_name(stack->main, "navigation");
+    stack->def_source = gtk_status_stack_show_current_location;
 
     info = stack->nav_info;
 
@@ -863,6 +876,10 @@ static gboolean gtk_status_stack_show_current_location(GtkStatusStack *stack)
 
     gtk_label_set_text(stack->nav_details, info->details != NULL ? info->details : "");
 
+    /* Conclusion */
+
+    gtk_stack_set_visible_child_name(stack->main, "navigation");
+
     return G_SOURCE_REMOVE;
 
 }
@@ -1063,7 +1080,7 @@ activity_id_t gtk_status_stack_add_activity(GtkStatusStack *stack, const char *m
     if (info->tag != 0)
         g_source_remove(info->tag);
 
-    info->tag = g_idle_add((GSourceFunc)gtk_status_stack_show_current_activity, stack);
+    info->tag = g_idle_add(gtk_status_stack_show_current_activity, stack);
 
     g_mutex_unlock(&info->access);
 
@@ -1130,7 +1147,7 @@ void gtk_status_stack_update_activity_message(GtkStatusStack *stack, activity_id
         if (info->tag != 0)
             g_source_remove(info->tag);
 
-        info->tag = g_idle_add((GSourceFunc)gtk_status_stack_show_current_activity, stack);
+        info->tag = g_idle_add(gtk_status_stack_show_current_activity, stack);
 
     }
 
@@ -1187,7 +1204,7 @@ void gtk_status_stack_update_activity_value(GtkStatusStack *stack, activity_id_t
         if (info->tag != 0)
             g_source_remove(info->tag);
 
-        info->tag = g_idle_add((GSourceFunc)gtk_status_stack_show_current_activity, stack);
+        info->tag = g_idle_add(gtk_status_stack_show_current_activity, stack);
 
     }
 
@@ -1239,7 +1256,7 @@ void gtk_status_stack_extend_activity_max(GtkStatusStack *stack, activity_id_t i
         if (info->tag != 0)
             g_source_remove(info->tag);
 
-        info->tag = g_idle_add((GSourceFunc)gtk_status_stack_show_current_activity, stack);
+        info->tag = g_idle_add(gtk_status_stack_show_current_activity, stack);
 
     }
 
@@ -1305,10 +1322,10 @@ void gtk_status_stack_remove_activity(GtkStatusStack *stack, activity_id_t id)
     if (info->count == 0)
     {
         info->tag = 0;
-        g_idle_add(stack->def_source, stack);
+        gtk_status_stack_reset_to_default(stack);
     }
     else if (is_last)
-        info->tag = g_idle_add((GSourceFunc)gtk_status_stack_show_current_activity, stack);
+        info->tag = g_idle_add(gtk_status_stack_show_current_activity, stack);
 
  exit:
 
@@ -1319,7 +1336,7 @@ void gtk_status_stack_remove_activity(GtkStatusStack *stack, activity_id_t id)
 
 /******************************************************************************
 *                                                                             *
-*  Paramètres  : stack = pile de statuts à manipuler.                         *
+*  Paramètres  : data = pile de statuts à manipuler.                          *
 *                                                                             *
 *  Description : S'assure de l'affichage à jour de la partie "activité".      *
 *                                                                             *
@@ -1329,11 +1346,14 @@ void gtk_status_stack_remove_activity(GtkStatusStack *stack, activity_id_t id)
 *                                                                             *
 ******************************************************************************/
 
-static gboolean gtk_status_stack_show_current_activity(GtkStatusStack *stack)
+static gboolean gtk_status_stack_show_current_activity(gpointer data)
 {
+    GtkStatusStack *stack;                  /* Version spécialisée         */
     activity_info_t *info;                  /* Informations à consulter    */
     activity_status_t *last;                /* Dernier statut à traiter    */
 
+    stack = GTK_STATUS_STACK(data);
+
     info = stack->activity_info;
 
     g_mutex_lock(&info->access);
@@ -1342,14 +1362,14 @@ static gboolean gtk_status_stack_show_current_activity(GtkStatusStack *stack)
     {
         if (info->count > 0)
         {
-            gtk_stack_set_visible_child_name(stack->main, "activity");
-
             last = &info->statuses[info->count - 1];
 
             gtk_label_set_text(stack->activity_message, last->message);
 
             gtk_progress_bar_set_fraction(stack->activity_progress, (last->current * 1.0) / last->max);
 
+            gtk_stack_set_visible_child_name(stack->main, "activity");
+
         }
 
         info->tag = 0;
diff --git a/src/gtkext/statusstack.h b/src/gtkext/statusstack.h
index 4995596..96d008c 100644
--- a/src/gtkext/statusstack.h
+++ b/src/gtkext/statusstack.h
@@ -45,7 +45,7 @@ DECLARE_GTYPE(GtkStatusStack, gtk_status_stack, GTK, STATUS_STACK);
 GtkStatusStack *gtk_status_stack_new(void);
 
 /* Réinitialise la barre de statut à son stade par défaut. */
-void gtk_status_stack_reset(GtkStatusStack *);
+void gtk_status_stack_reset_to_default(GtkStatusStack *);