From b658023324c98b3ad4d40a9c2804b7f829a48366 Mon Sep 17 00:00:00 2001
From: Emmanuele Bassi <ebassi@linux.intel.com>
Date: Mon, 18 Jun 2012 22:21:14 +0100
Subject: [PATCH] property-transition: Lazily convert the interpolated value

If the Interval used has a different type than the property we are
animating through a PropertyTransition then we should transform the
interpolated value before applying it, to avoid warnings down the
line.
---
 clutter/clutter-property-transition.c | 39 +++++++++++++++++++++++++++++++----
 1 file changed, 35 insertions(+), 4 deletions(-)

diff --git a/clutter/clutter-property-transition.c b/clutter/clutter-property-transition.c
index 9591169..e74490d 100644
--- a/clutter/clutter-property-transition.c
+++ b/clutter/clutter-property-transition.c
@@ -137,6 +137,7 @@ clutter_property_transition_compute_value (ClutterTransition *transition,
   ClutterPropertyTransition *self = CLUTTER_PROPERTY_TRANSITION (transition);
   ClutterPropertyTransitionPrivate *priv = self->priv;
   GValue value = G_VALUE_INIT;
+  GType p_type, i_type;
   gboolean res;
 
   /* if we have a GParamSpec we also have an animatable instance */
@@ -145,7 +146,10 @@ clutter_property_transition_compute_value (ClutterTransition *transition,
 
   clutter_property_transition_ensure_interval (self, animatable, interval);
 
-  g_value_init (&value, clutter_interval_get_value_type (interval));
+  p_type = G_PARAM_SPEC_VALUE_TYPE (priv->pspec);
+  i_type = clutter_interval_get_value_type (interval);
+
+  g_value_init (&value, i_type);
 
   res = clutter_animatable_interpolate_value (animatable,
                                               priv->property_name,
@@ -154,9 +158,36 @@ clutter_property_transition_compute_value (ClutterTransition *transition,
                                               &value);
 
   if (res)
-    clutter_animatable_set_final_state (animatable,
-                                        priv->property_name,
-                                        &value);
+    {
+      if (i_type != p_type || g_type_is_a (i_type, p_type))
+        {
+          if (g_value_type_transformable (i_type, p_type))
+            {
+              GValue transform = G_VALUE_INIT;
+
+              g_value_init (&transform, p_type);
+
+              if (g_value_transform (&value, &transform))
+                {
+                  clutter_animatable_set_final_state (animatable,
+                                                      priv->property_name,
+                                                      &transform);
+                }
+              else
+                g_warning ("%s: Unable to convert a value of type '%s' from "
+                           "the value type '%s' of the interval.",
+                           G_STRLOC,
+                           g_type_name (p_type),
+                           g_type_name (i_type));
+
+              g_value_unset (&transform);
+            }
+        }
+      else
+        clutter_animatable_set_final_state (animatable,
+                                            priv->property_name,
+                                            &value);
+    }
 
   g_value_unset (&value);
 }
-- 
2.7.4