[gtk+] adjustment: Privateize sealed members
- From: Benjamin Otte <otte src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gtk+] adjustment: Privateize sealed members
- Date: Wed, 5 Jan 2011 22:56:52 +0000 (UTC)
commit 96091a4d9c20ddea7fa07b26a34b735a2b751234
Author: Benjamin Otte <otte redhat com>
Date: Wed Jan 5 23:03:57 2011 +0100
adjustment: Privateize sealed members
gtk/gtkadjustment.c | 100 ++++++++++++++++++++++++++++++--------------------
gtk/gtkadjustment.h | 12 ++----
2 files changed, 64 insertions(+), 48 deletions(-)
---
diff --git a/gtk/gtkadjustment.c b/gtk/gtkadjustment.c
index ae5e292..40329a4 100644
--- a/gtk/gtkadjustment.c
+++ b/gtk/gtkadjustment.c
@@ -52,6 +52,15 @@
*/
+struct _GtkAdjustmentPrivate {
+ gdouble lower;
+ gdouble upper;
+ gdouble value;
+ gdouble step_increment;
+ gdouble page_increment;
+ gdouble page_size;
+};
+
enum
{
PROP_0,
@@ -239,17 +248,16 @@ gtk_adjustment_class_init (GtkAdjustmentClass *class)
NULL, NULL,
_gtk_marshal_VOID__VOID,
G_TYPE_NONE, 0);
+
+ g_type_class_add_private (class, sizeof (GtkAdjustmentPrivate));
}
static void
gtk_adjustment_init (GtkAdjustment *adjustment)
{
- adjustment->value = 0.0;
- adjustment->lower = 0.0;
- adjustment->upper = 0.0;
- adjustment->step_increment = 0.0;
- adjustment->page_increment = 0.0;
- adjustment->page_size = 0.0;
+ adjustment->priv = G_TYPE_INSTANCE_GET_PRIVATE (adjustment,
+ GTK_TYPE_ADJUSTMENT,
+ GtkAdjustmentPrivate);
}
static void
@@ -259,26 +267,27 @@ gtk_adjustment_get_property (GObject *object,
GParamSpec *pspec)
{
GtkAdjustment *adjustment = GTK_ADJUSTMENT (object);
+ GtkAdjustmentPrivate *priv = adjustment->priv;
switch (prop_id)
{
case PROP_VALUE:
- g_value_set_double (value, adjustment->value);
+ g_value_set_double (value, priv->value);
break;
case PROP_LOWER:
- g_value_set_double (value, adjustment->lower);
+ g_value_set_double (value, priv->lower);
break;
case PROP_UPPER:
- g_value_set_double (value, adjustment->upper);
+ g_value_set_double (value, priv->upper);
break;
case PROP_STEP_INCREMENT:
- g_value_set_double (value, adjustment->step_increment);
+ g_value_set_double (value, priv->step_increment);
break;
case PROP_PAGE_INCREMENT:
- g_value_set_double (value, adjustment->page_increment);
+ g_value_set_double (value, priv->page_increment);
break;
case PROP_PAGE_SIZE:
- g_value_set_double (value, adjustment->page_size);
+ g_value_set_double (value, priv->page_size);
break;
default:
G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
@@ -294,6 +303,7 @@ gtk_adjustment_set_property (GObject *object,
{
GtkAdjustment *adjustment = GTK_ADJUSTMENT (object);
gdouble double_value = g_value_get_double (value);
+ GtkAdjustmentPrivate *priv = adjustment->priv;
switch (prop_id)
{
@@ -301,19 +311,19 @@ gtk_adjustment_set_property (GObject *object,
gtk_adjustment_set_value (adjustment, double_value);
break;
case PROP_LOWER:
- adjustment->lower = double_value;
+ priv->lower = double_value;
break;
case PROP_UPPER:
- adjustment->upper = double_value;
+ priv->upper = double_value;
break;
case PROP_STEP_INCREMENT:
- adjustment->step_increment = double_value;
+ priv->step_increment = double_value;
break;
case PROP_PAGE_INCREMENT:
- adjustment->page_increment = double_value;
+ priv->page_increment = double_value;
break;
case PROP_PAGE_SIZE:
- adjustment->page_size = double_value;
+ priv->page_size = double_value;
break;
default:
G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
@@ -397,7 +407,7 @@ gtk_adjustment_get_value (GtkAdjustment *adjustment)
{
g_return_val_if_fail (GTK_IS_ADJUSTMENT (adjustment), 0.0);
- return adjustment->value;
+ return adjustment->priv->value;
}
/**
@@ -416,17 +426,21 @@ void
gtk_adjustment_set_value (GtkAdjustment *adjustment,
gdouble value)
{
+ GtkAdjustmentPrivate *priv;
+
g_return_if_fail (GTK_IS_ADJUSTMENT (adjustment));
+ priv = adjustment->priv;
+
/* don't use CLAMP() so we don't end up below lower if upper - page_size
* is smaller than lower
*/
- value = MIN (value, adjustment->upper - adjustment->page_size);
- value = MAX (value, adjustment->lower);
+ value = MIN (value, priv->upper - priv->page_size);
+ value = MAX (value, priv->lower);
- if (value != adjustment->value)
+ if (value != priv->value)
{
- adjustment->value = value;
+ priv->value = value;
gtk_adjustment_value_changed (adjustment);
}
@@ -447,7 +461,7 @@ gtk_adjustment_get_lower (GtkAdjustment *adjustment)
{
g_return_val_if_fail (GTK_IS_ADJUSTMENT (adjustment), 0.0);
- return adjustment->lower;
+ return adjustment->priv->lower;
}
/**
@@ -477,7 +491,7 @@ gtk_adjustment_set_lower (GtkAdjustment *adjustment,
{
g_return_if_fail (GTK_IS_ADJUSTMENT (adjustment));
- if (lower != adjustment->lower)
+ if (lower != adjustment->priv->lower)
g_object_set (adjustment, "lower", lower, NULL);
}
@@ -496,7 +510,7 @@ gtk_adjustment_get_upper (GtkAdjustment *adjustment)
{
g_return_val_if_fail (GTK_IS_ADJUSTMENT (adjustment), 0.0);
- return adjustment->upper;
+ return adjustment->priv->upper;
}
/**
@@ -522,7 +536,7 @@ gtk_adjustment_set_upper (GtkAdjustment *adjustment,
{
g_return_if_fail (GTK_IS_ADJUSTMENT (adjustment));
- if (upper != adjustment->upper)
+ if (upper != adjustment->priv->upper)
g_object_set (adjustment, "upper", upper, NULL);
}
@@ -541,7 +555,7 @@ gtk_adjustment_get_step_increment (GtkAdjustment *adjustment)
{
g_return_val_if_fail (GTK_IS_ADJUSTMENT (adjustment), 0.0);
- return adjustment->step_increment;
+ return adjustment->priv->step_increment;
}
/**
@@ -563,7 +577,7 @@ gtk_adjustment_set_step_increment (GtkAdjustment *adjustment,
{
g_return_if_fail (GTK_IS_ADJUSTMENT (adjustment));
- if (step_increment != adjustment->step_increment)
+ if (step_increment != adjustment->priv->step_increment)
g_object_set (adjustment, "step-increment", step_increment, NULL);
}
@@ -582,7 +596,7 @@ gtk_adjustment_get_page_increment (GtkAdjustment *adjustment)
{
g_return_val_if_fail (GTK_IS_ADJUSTMENT (adjustment), 0.0);
- return adjustment->page_increment;
+ return adjustment->priv->page_increment;
}
/**
@@ -604,7 +618,7 @@ gtk_adjustment_set_page_increment (GtkAdjustment *adjustment,
{
g_return_if_fail (GTK_IS_ADJUSTMENT (adjustment));
- if (page_increment != adjustment->page_increment)
+ if (page_increment != adjustment->priv->page_increment)
g_object_set (adjustment, "page-increment", page_increment, NULL);
}
@@ -623,7 +637,7 @@ gtk_adjustment_get_page_size (GtkAdjustment *adjustment)
{
g_return_val_if_fail (GTK_IS_ADJUSTMENT (adjustment), 0.0);
- return adjustment->page_size;
+ return adjustment->priv->page_size;
}
/**
@@ -645,7 +659,7 @@ gtk_adjustment_set_page_size (GtkAdjustment *adjustment,
{
g_return_if_fail (GTK_IS_ADJUSTMENT (adjustment));
- if (page_size != adjustment->page_size)
+ if (page_size != adjustment->priv->page_size)
g_object_set (adjustment, "page-size", page_size, NULL);
}
@@ -676,11 +690,14 @@ gtk_adjustment_configure (GtkAdjustment *adjustment,
gdouble page_increment,
gdouble page_size)
{
+ GtkAdjustmentPrivate *priv;
gboolean value_changed = FALSE;
guint64 old_stamp = adjustment_changed_stamp;
g_return_if_fail (GTK_IS_ADJUSTMENT (adjustment));
+ priv = adjustment->priv;
+
g_object_freeze_notify (G_OBJECT (adjustment));
g_object_set (adjustment,
@@ -697,12 +714,12 @@ gtk_adjustment_configure (GtkAdjustment *adjustment,
value = MIN (value, upper - page_size);
value = MAX (value, lower);
- if (value != adjustment->value)
+ if (value != priv->value)
{
/* set value manually to make sure "changed" is emitted with the
* new value in place and is emitted before "value-changed"
*/
- adjustment->value = value;
+ priv->value = value;
value_changed = TRUE;
}
@@ -766,23 +783,26 @@ gtk_adjustment_clamp_page (GtkAdjustment *adjustment,
gdouble lower,
gdouble upper)
{
+ GtkAdjustmentPrivate *priv;
gboolean need_emission;
g_return_if_fail (GTK_IS_ADJUSTMENT (adjustment));
- lower = CLAMP (lower, adjustment->lower, adjustment->upper);
- upper = CLAMP (upper, adjustment->lower, adjustment->upper);
+ priv = adjustment->priv;
+
+ lower = CLAMP (lower, priv->lower, priv->upper);
+ upper = CLAMP (upper, priv->lower, priv->upper);
need_emission = FALSE;
- if (adjustment->value + adjustment->page_size < upper)
+ if (priv->value + priv->page_size < upper)
{
- adjustment->value = upper - adjustment->page_size;
+ priv->value = upper - priv->page_size;
need_emission = TRUE;
}
- if (adjustment->value > lower)
+ if (priv->value > lower)
{
- adjustment->value = lower;
+ priv->value = lower;
need_emission = TRUE;
}
diff --git a/gtk/gtkadjustment.h b/gtk/gtkadjustment.h
index 319953b..dafd8ce 100644
--- a/gtk/gtkadjustment.h
+++ b/gtk/gtkadjustment.h
@@ -43,8 +43,9 @@ G_BEGIN_DECLS
#define GTK_ADJUSTMENT_GET_CLASS(obj) (G_TYPE_INSTANCE_GET_CLASS ((obj), GTK_TYPE_ADJUSTMENT, GtkAdjustmentClass))
-typedef struct _GtkAdjustment GtkAdjustment;
-typedef struct _GtkAdjustmentClass GtkAdjustmentClass;
+typedef struct _GtkAdjustment GtkAdjustment;
+typedef struct _GtkAdjustmentPrivate GtkAdjustmentPrivate;
+typedef struct _GtkAdjustmentClass GtkAdjustmentClass;
/**
* GtkAdjustment:
@@ -66,12 +67,7 @@ struct _GtkAdjustment
{
GInitiallyUnowned parent_instance;
- gdouble GSEAL (lower);
- gdouble GSEAL (upper);
- gdouble GSEAL (value);
- gdouble GSEAL (step_increment);
- gdouble GSEAL (page_increment);
- gdouble GSEAL (page_size);
+ GtkAdjustmentPrivate *priv;
};
struct _GtkAdjustmentClass
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]