[gtk+/wip/cssvalue: 6/142] matcher: Use a vtable
- From: Benjamin Otte <otte src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gtk+/wip/cssvalue: 6/142] matcher: Use a vtable
- Date: Sat, 7 Apr 2012 19:46:52 +0000 (UTC)
commit 313d9946d8f22e0ac6d62601dc5c220abb5ed0d2
Author: Benjamin Otte <otte redhat com>
Date: Sat Mar 17 14:28:07 2012 +0100
matcher: Use a vtable
Now we can do lots of fancy matchers, yay!
gtk/gtkcssmatcher.c | 92 ++++++++++++++++++++--------------
gtk/gtkcssmatcherprivate.h | 115 ++++++++++++++++++++++++++++++++++++--------
2 files changed, 148 insertions(+), 59 deletions(-)
---
diff --git a/gtk/gtkcssmatcher.c b/gtk/gtkcssmatcher.c
index c12403e..e796cfc 100644
--- a/gtk/gtkcssmatcher.c
+++ b/gtk/gtkcssmatcher.c
@@ -21,24 +21,14 @@
#include "gtkwidgetpath.h"
-void
-_gtk_css_matcher_init (GtkCssMatcher *matcher,
- const GtkWidgetPath *path,
- GtkStateFlags state)
-{
- matcher->path = path;
- matcher->state_flags = state;
- matcher->index = gtk_widget_path_length (path) - 1;
- matcher->sibling_index = gtk_widget_path_iter_get_sibling_index (path, matcher->index);
-}
-
-gboolean
-_gtk_css_matcher_get_parent (GtkCssMatcher *matcher,
- const GtkCssMatcher *child)
+static gboolean
+gtk_css_matcher_widget_path_get_parent (GtkCssMatcher *matcher,
+ const GtkCssMatcher *child)
{
if (child->index == 0)
return FALSE;
+ matcher->klass = child->klass;
matcher->path = child->path;
matcher->state_flags = 0;
matcher->index = child->index - 1;
@@ -47,13 +37,14 @@ _gtk_css_matcher_get_parent (GtkCssMatcher *matcher,
return TRUE;
}
-gboolean
-_gtk_css_matcher_get_previous (GtkCssMatcher *matcher,
- const GtkCssMatcher *next)
+static gboolean
+gtk_css_matcher_widget_path_get_previous (GtkCssMatcher *matcher,
+ const GtkCssMatcher *next)
{
if (next->sibling_index == 0)
return FALSE;
+ matcher->klass = next->klass;
matcher->path = next->path;
matcher->state_flags = 0;
matcher->index = next->index;
@@ -62,15 +53,15 @@ _gtk_css_matcher_get_previous (GtkCssMatcher *matcher,
return TRUE;
}
-GtkStateFlags
-_gtk_css_matcher_get_state (const GtkCssMatcher *matcher)
+static GtkStateFlags
+gtk_css_matcher_widget_path_get_state (const GtkCssMatcher *matcher)
{
return matcher->state_flags;
}
-gboolean
-_gtk_css_matcher_has_name (const GtkCssMatcher *matcher,
- const char *name)
+static gboolean
+gtk_css_matcher_widget_path_has_name (const GtkCssMatcher *matcher,
+ const char *name)
{
const GtkWidgetPath *siblings;
GType type;
@@ -83,9 +74,9 @@ _gtk_css_matcher_has_name (const GtkCssMatcher *matcher,
return g_type_is_a (gtk_widget_path_iter_get_object_type (matcher->path, matcher->index), type);
}
-gboolean
-_gtk_css_matcher_has_class (const GtkCssMatcher *matcher,
- const char *class_name)
+static gboolean
+gtk_css_matcher_widget_path_has_class (const GtkCssMatcher *matcher,
+ const char *class_name)
{
const GtkWidgetPath *siblings;
@@ -96,9 +87,9 @@ _gtk_css_matcher_has_class (const GtkCssMatcher *matcher,
return gtk_widget_path_iter_has_class (matcher->path, matcher->index, class_name);
}
-gboolean
-_gtk_css_matcher_has_id (const GtkCssMatcher *matcher,
- const char *id)
+static gboolean
+gtk_css_matcher_widget_path_has_id (const GtkCssMatcher *matcher,
+ const char *id)
{
const GtkWidgetPath *siblings;
@@ -109,8 +100,8 @@ _gtk_css_matcher_has_id (const GtkCssMatcher *matcher,
return gtk_widget_path_iter_has_name (matcher->path, matcher->index, id);
}
-gboolean
-_gtk_css_matcher_has_regions (const GtkCssMatcher *matcher)
+static gboolean
+gtk_css_matcher_widget_path_has_regions (const GtkCssMatcher *matcher)
{
const GtkWidgetPath *siblings;
GSList *regions;
@@ -127,10 +118,10 @@ _gtk_css_matcher_has_regions (const GtkCssMatcher *matcher)
return result;
}
-gboolean
-_gtk_css_matcher_has_region (const GtkCssMatcher *matcher,
- const char *region,
- GtkRegionFlags flags)
+static gboolean
+gtk_css_matcher_widget_path_has_region (const GtkCssMatcher *matcher,
+ const char *region,
+ GtkRegionFlags flags)
{
const GtkWidgetPath *siblings;
GtkRegionFlags region_flags;
@@ -153,14 +144,14 @@ _gtk_css_matcher_has_region (const GtkCssMatcher *matcher,
return TRUE;
}
-guint
-_gtk_css_matcher_get_sibling_index (const GtkCssMatcher *matcher)
+static guint
+gtk_css_matcher_widget_path_get_sibling_index (const GtkCssMatcher *matcher)
{
return matcher->sibling_index;
}
-guint
-_gtk_css_matcher_get_n_siblings (const GtkCssMatcher *matcher)
+static guint
+gtk_css_matcher_widget_path_get_n_siblings (const GtkCssMatcher *matcher)
{
const GtkWidgetPath *siblings;
@@ -171,3 +162,28 @@ _gtk_css_matcher_get_n_siblings (const GtkCssMatcher *matcher)
return gtk_widget_path_length (siblings);
}
+static const GtkCssMatcherClass GTK_CSS_MATCHER_WIDGET_PATH = {
+ gtk_css_matcher_widget_path_get_parent,
+ gtk_css_matcher_widget_path_get_previous,
+ gtk_css_matcher_widget_path_get_state,
+ gtk_css_matcher_widget_path_has_name,
+ gtk_css_matcher_widget_path_has_class,
+ gtk_css_matcher_widget_path_has_id,
+ gtk_css_matcher_widget_path_has_regions,
+ gtk_css_matcher_widget_path_has_region,
+ gtk_css_matcher_widget_path_get_sibling_index,
+ gtk_css_matcher_widget_path_get_n_siblings
+};
+
+void
+_gtk_css_matcher_init (GtkCssMatcher *matcher,
+ const GtkWidgetPath *path,
+ GtkStateFlags state)
+{
+ matcher->klass = >K_CSS_MATCHER_WIDGET_PATH;
+ matcher->path = path;
+ matcher->state_flags = state;
+ matcher->index = gtk_widget_path_length (path) - 1;
+ matcher->sibling_index = gtk_widget_path_iter_get_sibling_index (path, matcher->index);
+}
+
diff --git a/gtk/gtkcssmatcherprivate.h b/gtk/gtkcssmatcherprivate.h
index 704de53..83eff0e 100644
--- a/gtk/gtkcssmatcherprivate.h
+++ b/gtk/gtkcssmatcherprivate.h
@@ -24,36 +24,109 @@
G_BEGIN_DECLS
typedef struct _GtkCssMatcher GtkCssMatcher;
+typedef struct _GtkCssMatcherClass GtkCssMatcherClass;
-struct _GtkCssMatcher {
- const GtkWidgetPath *path;
- GtkStateFlags state_flags;
- guint index;
- guint sibling_index;
-};
-
-void _gtk_css_matcher_init (GtkCssMatcher *matcher,
- const GtkWidgetPath *path,
- GtkStateFlags state);
-gboolean _gtk_css_matcher_get_parent (GtkCssMatcher *matcher,
+struct _GtkCssMatcherClass {
+ gboolean (* get_parent) (GtkCssMatcher *matcher,
const GtkCssMatcher *child);
-gboolean _gtk_css_matcher_get_previous (GtkCssMatcher *matcher,
+ gboolean (* get_previous) (GtkCssMatcher *matcher,
const GtkCssMatcher *next);
-GtkStateFlags _gtk_css_matcher_get_state (const GtkCssMatcher *matcher);
-gboolean _gtk_css_matcher_has_name (const GtkCssMatcher *matcher,
+ GtkStateFlags (* get_state) (const GtkCssMatcher *matcher);
+ gboolean (* has_name) (const GtkCssMatcher *matcher,
const char *name);
-gboolean _gtk_css_matcher_has_class (const GtkCssMatcher *matcher,
+ gboolean (* has_class) (const GtkCssMatcher *matcher,
const char *class_name);
-gboolean _gtk_css_matcher_has_id (const GtkCssMatcher *matcher,
+ gboolean (* has_id) (const GtkCssMatcher *matcher,
const char *id);
-gboolean _gtk_css_matcher_has_regions (const GtkCssMatcher *matcher);
-gboolean _gtk_css_matcher_has_region (const GtkCssMatcher *matcher,
+ gboolean (* has_regions) (const GtkCssMatcher *matcher);
+ gboolean (* has_region) (const GtkCssMatcher *matcher,
const char *region,
GtkRegionFlags flags);
-guint _gtk_css_matcher_get_sibling_index
- (const GtkCssMatcher *matcher);
-guint _gtk_css_matcher_get_n_siblings (const GtkCssMatcher *matcher);
+ guint (* get_sibling_index) (const GtkCssMatcher *matcher);
+ guint (* get_n_siblings) (const GtkCssMatcher *matcher);
+};
+
+struct _GtkCssMatcher {
+ const GtkCssMatcherClass *klass;
+ const GtkWidgetPath *path;
+ GtkStateFlags state_flags;
+ guint index;
+ guint sibling_index;
+};
+
+void _gtk_css_matcher_init (GtkCssMatcher *matcher,
+ const GtkWidgetPath *path,
+ GtkStateFlags state);
+
+static inline gboolean
+_gtk_css_matcher_get_parent (GtkCssMatcher *matcher,
+ const GtkCssMatcher *child)
+{
+ return child->klass->get_parent (matcher, child);
+}
+
+static inline gboolean
+_gtk_css_matcher_get_previous (GtkCssMatcher *matcher,
+ const GtkCssMatcher *next)
+{
+ return next->klass->get_previous (matcher, next);
+}
+
+static inline GtkStateFlags
+_gtk_css_matcher_get_state (const GtkCssMatcher *matcher)
+{
+ return matcher->klass->get_state (matcher);
+}
+
+static inline gboolean
+_gtk_css_matcher_has_name (const GtkCssMatcher *matcher,
+ const char *name)
+{
+ return matcher->klass->has_name (matcher, name);
+}
+
+static inline gboolean
+_gtk_css_matcher_has_class (const GtkCssMatcher *matcher,
+ const char *class_name)
+{
+ return matcher->klass->has_class (matcher, class_name);
+}
+
+static inline gboolean
+_gtk_css_matcher_has_id (const GtkCssMatcher *matcher,
+ const char *id)
+{
+ return matcher->klass->has_id (matcher, id);
+}
+
+
+static inline gboolean
+_gtk_css_matcher_has_regions (const GtkCssMatcher *matcher)
+{
+ return matcher->klass->has_regions (matcher);
+}
+
+static inline gboolean
+_gtk_css_matcher_has_region (const GtkCssMatcher *matcher,
+ const char *region,
+ GtkRegionFlags flags)
+{
+ return matcher->klass->has_region (matcher, region, flags);
+}
+
+static inline guint
+_gtk_css_matcher_get_sibling_index (const GtkCssMatcher *matcher)
+{
+ return matcher->klass->get_sibling_index (matcher);
+}
+
+static inline guint
+_gtk_css_matcher_get_n_siblings (const GtkCssMatcher *matcher)
+{
+ return matcher->klass->get_n_siblings (matcher);
+}
+
G_END_DECLS
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]