2007-11-20 Emmanuele Bassi <ebassi@openedhand.com>
* clutter/clutter-actor.c:: * clutter/clutter-behaviour-opacity.c: * clutter/clutter-clone-texture.c: * clutter/clutter-effect.c: * clutter/clutter-label.c: Remove the usage of G_PARAM_CONSTRUCT without an override for the ::construct virtual function; using G_PARAM_CONSTRUCT to set the default value of a property is not the recommended approach; default values are set inside the instance init function.
This commit is contained in:
parent
6cb35639e7
commit
9955b06542
12
ChangeLog
12
ChangeLog
@ -1,3 +1,15 @@
|
||||
2007-11-20 Emmanuele Bassi <ebassi@openedhand.com>
|
||||
|
||||
* clutter/clutter-actor.c::
|
||||
* clutter/clutter-behaviour-opacity.c:
|
||||
* clutter/clutter-clone-texture.c:
|
||||
* clutter/clutter-effect.c:
|
||||
* clutter/clutter-label.c: Remove the usage of G_PARAM_CONSTRUCT
|
||||
without an override for the ::construct virtual function; using
|
||||
G_PARAM_CONSTRUCT to set the default value of a property is not
|
||||
the recommended approach; default values are set inside the
|
||||
instance init function.
|
||||
|
||||
2007-11-20 Emmanuele Bassi <ebassi@openedhand.com>
|
||||
|
||||
* clutter/clutter-texture.c (clutter_texture_init): Set filter-quality
|
||||
|
@ -1169,7 +1169,7 @@ clutter_actor_class_init (ClutterActorClass *klass)
|
||||
"Opacity of actor",
|
||||
0, 0xff,
|
||||
0xff,
|
||||
G_PARAM_CONSTRUCT | CLUTTER_PARAM_READWRITE));
|
||||
CLUTTER_PARAM_READWRITE));
|
||||
/**
|
||||
* ClutterActor:visible:
|
||||
*
|
||||
|
@ -175,9 +175,7 @@ clutter_behaviour_opacity_class_init (ClutterBehaviourOpacityClass *klass)
|
||||
"Initial opacity level",
|
||||
0, 255,
|
||||
0,
|
||||
CLUTTER_PARAM_READWRITE |
|
||||
G_PARAM_CONSTRUCT));
|
||||
|
||||
CLUTTER_PARAM_READWRITE));
|
||||
/**
|
||||
* ClutterBehaviourOpacity:opacity-end:
|
||||
*
|
||||
@ -192,8 +190,7 @@ clutter_behaviour_opacity_class_init (ClutterBehaviourOpacityClass *klass)
|
||||
"Final opacity level",
|
||||
0, 255,
|
||||
0,
|
||||
CLUTTER_PARAM_READWRITE |
|
||||
G_PARAM_CONSTRUCT));
|
||||
CLUTTER_PARAM_READWRITE));
|
||||
|
||||
behave_class->alpha_notify = clutter_behaviour_alpha_notify;
|
||||
|
||||
@ -204,6 +201,9 @@ static void
|
||||
clutter_behaviour_opacity_init (ClutterBehaviourOpacity *self)
|
||||
{
|
||||
self->priv = CLUTTER_BEHAVIOUR_OPACITY_GET_PRIVATE (self);
|
||||
|
||||
self->priv->opacity_start = 0;
|
||||
self->priv->opacity_end = 0;
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -345,7 +345,7 @@ clutter_clone_texture_class_init (ClutterCloneTextureClass *klass)
|
||||
"Parent Texture",
|
||||
"The parent texture to clone",
|
||||
CLUTTER_TYPE_TEXTURE,
|
||||
(G_PARAM_CONSTRUCT | CLUTTER_PARAM_READWRITE)));
|
||||
CLUTTER_PARAM_READWRITE));
|
||||
|
||||
g_type_class_add_private (gobject_class, sizeof (ClutterCloneTexturePrivate));
|
||||
}
|
||||
|
@ -252,7 +252,6 @@ clutter_effect_template_class_init (ClutterEffectTemplateClass *klass)
|
||||
"Clone",
|
||||
"controls if effects should clone or reference the templated timeline",
|
||||
TRUE,
|
||||
G_PARAM_CONSTRUCT |
|
||||
CLUTTER_PARAM_READWRITE));
|
||||
|
||||
}
|
||||
|
@ -627,14 +627,14 @@ clutter_label_class_init (ClutterLabelClass *klass)
|
||||
"Font Name",
|
||||
"Pango font description",
|
||||
NULL,
|
||||
G_PARAM_CONSTRUCT | CLUTTER_PARAM_READWRITE));
|
||||
CLUTTER_PARAM_READWRITE));
|
||||
g_object_class_install_property
|
||||
(gobject_class, PROP_TEXT,
|
||||
g_param_spec_string ("text",
|
||||
"Text",
|
||||
"Text to render",
|
||||
NULL,
|
||||
G_PARAM_CONSTRUCT | CLUTTER_PARAM_READWRITE));
|
||||
CLUTTER_PARAM_READWRITE));
|
||||
g_object_class_install_property
|
||||
(gobject_class, PROP_COLOR,
|
||||
g_param_spec_boxed ("color",
|
||||
|
Loading…
Reference in New Issue
Block a user