[gtk/wip/otte/listview: 6/145] expression: Add the ability to watch an expression



commit 8fd9f00783799871f5acd7c71fb34d83480c22fd
Author: Benjamin Otte <otte redhat com>
Date:   Thu Nov 21 05:53:56 2019 +0100

    expression: Add the ability to watch an expression

 gtk/gtkexpression.c | 333 ++++++++++++++++++++++++++++++++++++++++++++++++++++
 gtk/gtkexpression.h |  20 ++++
 2 files changed, 353 insertions(+)
---
diff --git a/gtk/gtkexpression.c b/gtk/gtkexpression.c
index e311dbe07a..20f8a53324 100644
--- a/gtk/gtkexpression.c
+++ b/gtk/gtkexpression.c
@@ -99,15 +99,32 @@ struct _GtkExpression
   GtkExpression *owner;
 };
 
+struct _GtkExpressionWatch
+{
+  GtkExpression *expression;
+  GObject *this;
+  GtkExpressionNotify notify;
+  gpointer user_data;
+  GDestroyNotify user_destroy;
+};
+
 struct _GtkExpressionClass
 {
   gsize struct_size;
   const char *type_name;
 
   void                  (* finalize)            (GtkExpression          *expr);
+  gboolean              (* is_static)           (GtkExpression          *expr);
   gboolean              (* evaluate)            (GtkExpression          *expr,
                                                  gpointer                this,
                                                  GValue                 *value);
+
+  gsize watch_size;
+  void                  (* watch)               (GtkExpression          *self,
+                                                 gpointer                this_,
+                                                 GtkExpressionWatch     *watch);
+  void                  (* unwatch)             (GtkExpression          *self,
+                                                 GtkExpressionWatch     *watch);
 };
 
 /**
@@ -143,6 +160,25 @@ gtk_expression_alloc (const GtkExpressionClass *expression_class,
   return self;
 }
 
+static void
+gtk_expression_watch_static (GtkExpression      *self,
+                             gpointer            this_,
+                             GtkExpressionWatch *watch)
+{
+}
+
+static void
+gtk_expression_unwatch_static (GtkExpression      *self,
+                               GtkExpressionWatch *watch)
+{
+}
+
+static void
+gtk_expression_watch_notify (GtkExpressionWatch *watch)
+{
+  watch->notify (watch->user_data);
+}
+
 /*** CONSTANT ***/
 
 typedef struct _GtkConstantExpression GtkConstantExpression;
@@ -162,6 +198,12 @@ gtk_constant_expression_finalize (GtkExpression *expr)
   g_value_unset (&self->value);
 }
 
+static gboolean
+gtk_constant_expression_is_static (GtkExpression *expr)
+{
+  return TRUE;
+}
+
 static gboolean
 gtk_constant_expression_evaluate (GtkExpression *expr,
                                   gpointer       this,
@@ -179,7 +221,11 @@ static const GtkExpressionClass GTK_CONSTANT_EXPRESSION_CLASS =
   sizeof (GtkConstantExpression),
   "GtkConstantExpression",
   gtk_constant_expression_finalize,
+  gtk_constant_expression_is_static,
   gtk_constant_expression_evaluate,
+  sizeof (GtkExpressionWatch),
+  gtk_expression_watch_static,
+  gtk_expression_unwatch_static
 };
 
 /**
@@ -379,6 +425,12 @@ gtk_property_expression_finalize (GtkExpression *expr)
 {
 }
 
+static gboolean
+gtk_property_expression_is_static (GtkExpression *expr)
+{
+  return FALSE;
+}
+
 static gboolean
 gtk_property_expression_evaluate (GtkExpression *expr,
                                   gpointer       this,
@@ -393,12 +445,60 @@ gtk_property_expression_evaluate (GtkExpression *expr,
   return TRUE;
 }
 
+typedef struct _GtkPropertyExpressionWatch GtkPropertyExpressionWatch;
+struct _GtkPropertyExpressionWatch
+{
+  GtkExpressionWatch watch;
+
+  GClosure *closure;
+};
+
+static void
+gtk_property_expression_watch (GtkExpression      *expr,
+                               gpointer            this_,
+                               GtkExpressionWatch *watch)
+{
+  GtkPropertyExpressionWatch *pwatch = (GtkPropertyExpressionWatch *) watch;
+  GtkPropertyExpression *self = (GtkPropertyExpression *) expr;
+  GObject *object = this_;
+
+  if (object == NULL)
+    return;
+
+  pwatch->closure = g_cclosure_new_swap (G_CALLBACK (gtk_expression_watch_notify), pwatch, NULL);
+  if (!g_signal_connect_closure_by_id (object,
+                                       g_signal_lookup ("notify", G_OBJECT_TYPE (object)),
+                                       g_quark_from_string (self->pspec->name),
+                                       g_closure_ref (pwatch->closure),
+                                       FALSE))
+    {
+      g_assert_not_reached ();
+    }
+}
+
+static void
+gtk_property_expression_unwatch (GtkExpression      *expr,
+                                 GtkExpressionWatch *watch)
+{
+  GtkPropertyExpressionWatch *pwatch = (GtkPropertyExpressionWatch *) watch;
+
+  if (pwatch->closure == NULL)
+    return;
+
+  g_closure_invalidate (pwatch->closure);
+  g_closure_unref (pwatch->closure);
+}
+
 static const GtkExpressionClass GTK_PROPERTY_EXPRESSION_CLASS =
 {
   sizeof (GtkPropertyExpression),
   "GtkPropertyExpression",
   gtk_property_expression_finalize,
+  gtk_property_expression_is_static,
   gtk_property_expression_evaluate,
+  sizeof (GtkPropertyExpressionWatch),
+  gtk_property_expression_watch,
+  gtk_property_expression_unwatch
 };
 
 /**
@@ -477,6 +577,21 @@ gtk_closure_expression_finalize (GtkExpression *expr)
   g_closure_unref (self->closure);
 }
 
+static gboolean
+gtk_closure_expression_is_static (GtkExpression *expr)
+{
+  GtkClosureExpression *self = (GtkClosureExpression *) expr;
+  guint i;
+
+  for (i = 0; i < self->n_params; i++)
+    {
+      if (!gtk_expression_is_static (self->params[i]))
+        return FALSE;
+    }
+
+  return TRUE;
+}
+
 static gboolean
 gtk_closure_expression_evaluate (GtkExpression *expr,
                                  gpointer       this,
@@ -514,12 +629,56 @@ out:
   return result;
 }
 
+typedef struct _GtkClosureExpressionWatch GtkClosureExpressionWatch;
+struct _GtkClosureExpressionWatch
+{
+  GtkExpressionWatch watch;
+
+  GSList *watches;
+};
+
+static void
+gtk_closure_expression_watch (GtkExpression      *expr,
+                              gpointer            this_,
+                              GtkExpressionWatch *watch)
+{
+  GtkClosureExpressionWatch *cwatch = (GtkClosureExpressionWatch *) watch;
+  GtkClosureExpression *self = (GtkClosureExpression *) expr;
+  guint i;
+
+  for (i = 0; i < self->n_params; i++)
+    {
+      if (gtk_expression_is_static (self->params[i]))
+        continue;
+
+      cwatch->watches = g_slist_prepend (cwatch->watches,
+                                         gtk_expression_watch (self->params[i],
+                                                               this_,
+                                                               (GtkExpressionNotify) 
gtk_expression_watch_notify,
+                                                               watch,
+                                                               NULL));
+    }
+}
+
+static void
+gtk_closure_expression_unwatch (GtkExpression      *expr,
+                                GtkExpressionWatch *watch)
+{
+  GtkClosureExpressionWatch *cwatch = (GtkClosureExpressionWatch *) watch;
+
+  g_slist_free_full (cwatch->watches, (GDestroyNotify) gtk_expression_watch_unwatch);
+}
+
 static const GtkExpressionClass GTK_CLOSURE_EXPRESSION_CLASS =
 {
   sizeof (GtkClosureExpression),
   "GtkClosureExpression",
   gtk_closure_expression_finalize,
+  gtk_closure_expression_is_static,
   gtk_closure_expression_evaluate,
+  sizeof (GtkClosureExpressionWatch),
+  gtk_closure_expression_watch,
+  gtk_closure_expression_unwatch
 };
 
 /**
@@ -678,3 +837,177 @@ gtk_expression_evaluate (GtkExpression *self,
   return self->expression_class->evaluate (self, this_, value);
 }
 
+/**
+ * gtk_expression_is_static:
+ * @self: a #GtkExpression
+ *
+ * Checks if the expression is static.
+ *
+ * A static expression will never change its result when 
+ * gtk_expression_evaluate() is called on it with the same arguments.
+ *
+ * That means a call to gtk_expression_watch() is not necessary because
+ * it will never trigger a notify.
+ *
+ * Returns: %TRUE if the expression is static
+ **/
+gboolean
+gtk_expression_is_static (GtkExpression *self)
+{
+  return self->expression_class->is_static (self);
+}
+
+static gboolean
+gtk_expression_watch_is_watching (GtkExpressionWatch *watch)
+{
+  return watch->expression != NULL;
+}
+
+static void
+gtk_expression_watch_this_cb (gpointer data,
+                              GObject *this)
+{
+  GtkExpressionWatch *watch = data;
+
+  watch->this = NULL;
+
+  gtk_expression_watch_notify (watch);
+}
+
+/**
+ * gtk_expression_watch:
+ * @self: a #GtkExpression
+ * @this_: (transfer none) (type GObject) (nullable): the this argument to
+ *     watch
+ * @notify: (closure user_data): callback to invoke when the
+ *     expression changes
+ * @user_data: user data to pass to @notify callback
+ * @user_destroy: destroy notify for @user_data
+ *
+ * Installs a watch for the given @expression that calls the @notify function
+ * whenever the evaluation of @self may have changed.
+ *
+ * GTK cannot guarantee that the evaluation did indeed change when the @notify
+ * gets invoked, but it guarantees the opposite: When it did in fact change,
+ * the @notify will be invoked.
+ *
+ * Returns: (transfer none): The newly installed watch. Note that the only
+ *     reference held to the watch will be released when the watch is unwatched
+ *     which can happen automatically, and not just via
+ *     gtk_expression_watch_unwatch(). You should call gtk_expression_watch_ref()
+ *     if you want to keep the watch around.
+ **/
+GtkExpressionWatch *
+gtk_expression_watch (GtkExpression       *self,
+                      gpointer             this_,
+                      GtkExpressionNotify  notify,
+                      gpointer             user_data,
+                      GDestroyNotify       user_destroy)
+{
+  GtkExpressionWatch *watch;
+
+  g_return_val_if_fail (self != NULL, NULL);
+  g_return_val_if_fail (this_ == NULL || G_IS_OBJECT (this_), NULL);
+  g_return_val_if_fail (notify != NULL, NULL);
+
+  watch = g_atomic_rc_box_alloc0 (self->expression_class->watch_size);
+
+  watch->expression = gtk_expression_ref (self);
+  watch->this = this_;
+  if (this_)
+    g_object_weak_ref (this_, gtk_expression_watch_this_cb, watch);
+  watch->notify = notify;
+  watch->user_data = user_data;
+  watch->user_destroy = user_destroy;
+
+  self->expression_class->watch (self, this_, watch);
+
+  return watch;
+}
+
+/**
+ * gtk_expression_watch_ref:
+ * @self: (allow-none): a #GtkExpressionWatch
+ *
+ * Acquires a reference on the given #GtkExpressionWatch.
+ *
+ * Returns: (transfer none): the #GtkExpression with an additional reference
+ */
+GtkExpressionWatch *
+gtk_expression_watch_ref (GtkExpressionWatch *self)
+{
+  return g_atomic_rc_box_acquire (self);
+}
+
+static void
+gtk_expression_watch_finalize (gpointer data)
+{
+  GtkExpressionWatch *watch = data;
+
+  g_assert (!gtk_expression_watch_is_watching (watch));
+}
+
+/**
+ * gtk_expression_watch_unref:
+ * @self: (allow-none): a #GtkExpressionWatch
+ *
+ * Releases a reference on the given #GtkExpressionWatch.
+ *
+ * If the reference was the last, the resources associated to @self are
+ * freed.
+ */
+void
+gtk_expression_watch_unref (GtkExpressionWatch *self)
+{
+  g_atomic_rc_box_release_full (self, gtk_expression_watch_finalize);
+}
+
+/**
+ * gtk_expression_watch_unwatch:
+ * @watch: (transfer none): watch to release
+ *
+ * Stops watching an expression that was established via gtk_expression_watch().
+ **/
+void
+gtk_expression_watch_unwatch (GtkExpressionWatch *watch)
+{
+  if (gtk_expression_watch_is_watching (watch))
+    return;
+
+  watch->expression->expression_class->unwatch (watch->expression, watch);
+
+  if (watch->this)
+    g_object_weak_unref (watch->this, gtk_expression_watch_this_cb, watch);
+
+  if (watch->user_destroy)
+    watch->user_destroy (watch->user_data);
+
+  g_clear_pointer (&watch->expression, gtk_expression_unref);
+
+  gtk_expression_watch_unref (watch);
+}
+
+/**
+ * gtk_expression_watch_evaluate:
+ * @watch: a #GtkExpressionWatch
+ * @value: an empty #GValue to be set
+ *
+ * Evaluates the watched expression and on success stores the result
+ * in @value.
+ *
+ * This is equivalent to calling gtk_expression_evaluate() with the
+ * expression and this pointer originally used to create @watch.
+ *
+ * Returns: %TRUE if the expression could be evaluated and @value was set
+ **/
+gboolean
+gtk_expression_watch_evaluate (GtkExpressionWatch *watch,
+                               GValue             *value)
+{
+  g_return_val_if_fail (watch != NULL, FALSE);
+
+  if (!gtk_expression_watch_is_watching (watch))
+    return FALSE;
+
+  return gtk_expression_evaluate (watch->expression, watch->this, value);
+}
diff --git a/gtk/gtkexpression.h b/gtk/gtkexpression.h
index 22d2f9fc22..5c3f1e1a5d 100644
--- a/gtk/gtkexpression.h
+++ b/gtk/gtkexpression.h
@@ -26,6 +26,9 @@
 G_BEGIN_DECLS
 
 typedef struct _GtkExpression GtkExpression;
+typedef struct _GtkExpressionWatch GtkExpressionWatch;
+
+typedef void            (* GtkExpressionNotify)                 (gpointer                        user_data);
 
 #define GTK_IS_EXPRESSION(expr) ((expr) != NULL)
 
@@ -42,9 +45,26 @@ void                    gtk_expression_unref                    (GtkExpression
 GDK_AVAILABLE_IN_ALL
 GType                   gtk_expression_get_value_type           (GtkExpression                  *self);
 GDK_AVAILABLE_IN_ALL
+gboolean                gtk_expression_is_static                (GtkExpression                  *self);
+GDK_AVAILABLE_IN_ALL
 gboolean                gtk_expression_evaluate                 (GtkExpression                  *self,
                                                                  gpointer                        this_,
                                                                  GValue                         *value);
+GDK_AVAILABLE_IN_ALL
+GtkExpressionWatch *    gtk_expression_watch                    (GtkExpression                  *self,
+                                                                 gpointer                        this_,
+                                                                 GtkExpressionNotify             notify,
+                                                                 gpointer                        user_data,
+                                                                 GDestroyNotify                  
user_destroy);
+GDK_AVAILABLE_IN_ALL
+GtkExpressionWatch *    gtk_expression_watch_ref                (GtkExpressionWatch             *watch);
+GDK_AVAILABLE_IN_ALL
+void                    gtk_expression_watch_unref              (GtkExpressionWatch             *watch);
+GDK_AVAILABLE_IN_ALL
+gboolean                gtk_expression_watch_evaluate           (GtkExpressionWatch             *watch,
+                                                                 GValue                         *value);
+GDK_AVAILABLE_IN_ALL
+void                    gtk_expression_watch_unwatch            (GtkExpressionWatch             *watch);
 
 GDK_AVAILABLE_IN_ALL
 GtkExpression *         gtk_property_expression_new             (GType                           this_type,


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